> On Jun 2, 2015, at 10:48 AM, Alexander Duyck <alexander.h.duyck@xxxxxxxxxx> wrote: > > I'm pretty sure these could cause some serious errors if you direct assign the device into a VM since you then end up with multiple devices sharing a bus. Also it would likely have side-effects on a LOM (Lan On Motherboard) as it also shares the bus with multiple non-Ethernet devices. > > I believe you still need to add something like a check for !pci_is_root_bus(dev->bus) before you attempt to grab function 0. It probably also wouldn't hurt to check the dev->multifunction bit before running this code since it wouldn't make sense to go chasing down the VPD on another function if the device doesn't have one. You could probably do that either as a part of this code, or perhaps put it in the quirk. I'll look into those. I think you are right about more checks being needed. Thanks for the comments. -- Mark Rustad, Networking Division, Intel Corporation
Attachment:
signature.asc
Description: Message signed with OpenPGP using GPGMail
