Re: [PATCH 2/2 V3] PCI: implement VFs assignment reference counter

[Varka Bhadram <varkabhadram@gmail.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1104 bytes --]

On 07/11/2014 06:21 PM, Ethan Zhao wrote:
> On Fri, Jul 11, 2014 at 8:42 PM, Varka Bhadram <varkabhadram@gmail.com> wrote:
>> On 07/11/2014 06:00 PM, Ethan Zhao wrote:
>>> Current implementation of helper function pci_vfs_assigned() is a
>>> little complex, to get sum of VFs that assigned to VM, access low
>>> level configuration space register and then loop in traversing
>>> device tree.
>>
>> (...)
>>
>>
>>> @@ -650,6 +630,11 @@ EXPORT_SYMBOL_GPL(pci_vfs_assigned);
>>>    void pci_iov_assign_device(struct pci_dev *pdev)
>>>    {
>>>          pdev->dev_flags |= PCI_DEV_FLAGS_ASSIGNED;
>>> +       if (pdev->is_virtfn && !pdev->is_physfn)
>>> +               if (pdev->physfn)
>>> +                       if (pdev->physfn->sriov)
>>
>> Why don't we make last two 'if' conditions into single 'if'
>>
>> if (pdev->physfn && pdev->physfn->sriov)
> Yeah， this one looks better,  that style used to tell myself, I might
> forget the which side of  && operator  first, if you sure left first,
> I prefer that too :>

|for &&  operator - evaluation is from Left to Right|

-- 
Regards,
Varka Bhadram.


[-- Attachment #2: Type: text/plain, Size: 372 bytes --]

------------------------------------------------------------------------------
Open source business process management suite built on Java and Eclipse
Turn processes into business applications with Bonita BPM Community Edition
Quickly connect people, data, and systems into organized workflows
Winner of BOSSIE, CODIE, OW2 and Gartner awards
http://p.sf.net/sfu/Bonitasoft

[-- Attachment #3: Type: text/plain, Size: 257 bytes --]

_______________________________________________
E1000-devel mailing list
E1000-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel&#174; Ethernet, visit http://communities.intel.com/community/wired