RE: [net 2/8] igb: fix vf lookup

On Wed, 2012-02-08 at 23:42 +0000, Rose, Gregory V wrote:

quoted

diff --git a/drivers/net/ethernet/intel/igb/igb_main.c

[]

quoted

@@ -5012,7 +5012,8 @@ static int igb_find_enabled_vfs(struct igb_adapter

*adapter)
 	vf_devfn = pdev->devfn + 0x80;
 	pvfdev = pci_get_device(hw->vendor_id, device_id, NULL);
 	while (pvfdev) {
-		if (pvfdev->devfn == vf_devfn)
+		if (pvfdev->devfn == vf_devfn &&
+		    (pvfdev->bus->number >= pdev->bus->number))
 			vfs_found++;

[]

I'll fix this one too.  You start leaning on checkpatch and you get lazy I guess.

I suppose an indentation rule could be created when
arguments on multiple lines don't align at the open
parenthesis, but I'm not going to rewrite emacs
indentation rules.

Presumably it should only be used with --strict.

Anyone think multiple line tests with inequivalent uses
of parentheses like this one should be noted as well?