lkml.org 
[lkml]   [2018]   [Jul]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v3] PCI: Check for PCIe downtraining conditions
From
Date
On 07/23/2018 12:21 AM, Tal Gilboa wrote:
> On 7/19/2018 6:49 PM, Alex G. wrote:
>>
>>
>> On 07/18/2018 08:38 AM, Tal Gilboa wrote:
>>> On 7/16/2018 5:17 PM, Bjorn Helgaas wrote:
>>>> [+cc maintainers of drivers that already use pcie_print_link_status()
>>>> and GPU folks]
>> [snip]
>>>>
>>>>> +    /* Multi-function PCIe share the same link/status. */
>>>>> +    if ((PCI_FUNC(dev->devfn) != 0) || dev->is_virtfn)
>>>>> +        return;
>>>>> +
>>>>> +    pcie_print_link_status(dev);
>>>>> +}
>>>
>>> Is this function called by default for every PCIe device? What about
>>> VFs? We make an exception for them on our driver since a VF doesn't
>>> have access to the needed information in order to provide a
>>> meaningful message.
>>
>> I'm assuming VF means virtual function. pcie_print_link_status()
>> doesn't care if it's passed a virtual function. It will try to do its
>> job. That's why I bail out three lines above, with 'dev->is_virtfn'
>> check.
>>
>> Alex
>
> That's the point - we don't want to call pcie_print_link_status() for
> virtual functions. We make the distinction in our driver. If you want to
> change the code to call this function by default it shouldn't affect the
> current usage.

I'm not understanding very well what you're asking. I understand you
want to avoid printing this message on virtual functions, and that's
already taken care of. I'm also not changing current behavior. Let's
get v2 out and start the discussion again based on that.

Alex

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