[Skiboot] [PATCH] npu2: Use unfiltered mode in XTS tables
Reza Arbab
arbab at linux.vnet.ibm.com
Fri Mar 9 11:09:52 AEDT 2018
On Fri, Mar 09, 2018 at 10:33:51AM +1100, Alistair Popple wrote:
>> -static int opal_npu_destroy_context(uint64_t phb_id, uint64_t pid, uint64_t bdf)
>> +static int opal_npu_destroy_context(uint64_t phb_id, uint64_t pid __unused,
>> + uint64_t bdf)
>> {
>> struct phb *phb = pci_get_phb(phb_id);
>> struct npu2 *p = phb_to_npu2(phb);
>> - uint64_t xts_bdf, xts_bdf_pid;
>> - uint64_t lparshort;
>> - int id, rc = 0;
>> + uint64_t xts_bdf;
>> + int rc = 0;
>>
>> if (!phb || phb->phb_type != phb_type_npu_v2)
>> return OPAL_PARAMETER;
>> @@ -2137,26 +2136,13 @@ static int opal_npu_destroy_context(uint64_t phb_id, uint64_t pid, uint64_t bdf)
>> &xts_bdf, NPU2_XTS_BDF_MAP_BDF) < 0) {
>> NPU2ERR(p, "LPARID not associated with any GPU\n");
>
>I guess this check is still somewhat useful to catch a buggy device driver out
>but I'm not convinced it provides much value - we could just as easily remove it
>as well.
Yeah, I wavered on it a bit and figured if nothing else, the function
could still keep validating that argument. There are some words to that
effect in the commit log.
--
Reza Arbab
More information about the Skiboot
mailing list