On Sun, Jul 20, 2014 at 10:48 PM, Yuval Mintz <[email protected]> wrote: >> On Fri, 2014-07-11 at 07:13 -0600, Alex Williamson wrote: >> > On Fri, 2014-07-11 at 20:30 +0800, Ethan Zhao wrote: >> > > This patch introduces two new device assignment functions >> > > >> > > pci_iov_assign_device(), >> > > pci_iov_deassign_device() >> > > >> > > along with the existed one >> > > >> > > pci_vfs_assigned() >> > > >> > > They construct the VFs assignment management interface, used to >> > > assign/ deassign device to VM and query the VFs reference counter. >> > > instead of direct manipulation of device flag. > > Sorry for the late intrusion into the conversation, and perhaps it's a bit > unrelated, but given that you're creating a 'VF assignment management' > perhaps it's the right place to add some sort of mechanism in order to > prevent module removal once one of its VFs is assigned. > [e.g., incrementing module reference count]
This patch set is used to make the current hacked code well formed and make the interface clear to refactory later. so we could keep the current direct assignment and woulnd't encounter the removal issue etc. in one word, manageable. > > At the moment [to the best of my knowledge] there's no mechanism > preventing the 'not-so-bright' user from removing the driver, and no good > will come out of it. The best way is there is real 'driver' between device and the VM device manager or some similar machanism to make the assignment manageable. Thanks, Ethan > > Thanks, > Yuval > > ________________________________ > > This message and any attached documents contain information from QLogic > Corporation or its wholly-owned subsidiaries that may be confidential. If you > are not the intended recipient, you may not read, copy, distribute, or use > this information. If you have received this transmission in error, please > notify the sender immediately by reply e-mail and then delete this message. ------------------------------------------------------------------------------ Want fast and easy access to all the code in your enterprise? Index and search up to 200,000 lines of code with a free copy of Black Duck Code Sight - the same software that powers the world's largest code search on Ohloh, the Black Duck Open Hub! Try it now. http://p.sf.net/sfu/bds _______________________________________________ E1000-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/e1000-devel To learn more about Intel® Ethernet, visit http://communities.intel.com/community/wired
