lkml.org 
[lkml]   [2018]   [Jul]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v6 4/4] PCI: Introduce the disable_acs_redir parameter


On 16/07/18 11:15 AM, Alex Williamson wrote:
>> + pos = pci_find_ext_capability(dev, PCI_EXT_CAP_ID_ACS);
>> + if (!pos) {
>> + pci_warn(dev, "cannot disable ACS redirect for this hardware as it does not have ACS capabilities\n");
>> + return;
>> + }
>> +
>> + if (!pci_dev_specific_disable_acs_redir(dev))
>> + return;
>
> Shouldn't this come before the above ACS capability test? The PCH
> quirk without a disable_acs_redir callback will fall out with the
> pci_warn() above either way, but this ordering precludes that a quirk
> could be written for that device since it doesn't have an ACS
> capability. Otherwise the series looks ok to me. Thanks,

Yup, nice catch. I'll respin a v7 shortly.

Thanks,

Logan

\
 
 \ /
  Last update: 2018-07-16 19:18    [W:0.059 / U:0.172 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site