Re: [PATCH 4/7] hostdev: Add detachPCIDevices()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jan 22, 2016 at 06:30:42PM +0100, Andrea Bolognani wrote:
> On Fri, 2016-01-22 at 15:02 +0000, Daniel P. Berrange wrote:
> > On Tue, Jan 19, 2016 at 04:36:06PM +0100, Andrea Bolognani wrote:
> > > This function mirrors reattachPCIDevices().
> > > 
> > > The handling of active and inactive devices is updated and made more
> > > explicit, which means virHostdevPreparePCIDevices() has to be
> > > updated as well.
> > > ---
> > >  src/util/virhostdev.c | 125 ++++++++++++++++++++++++++++++--------------------
> > >  1 file changed, 76 insertions(+), 49 deletions(-)
> > > 
> > > diff --git a/src/util/virhostdev.c b/src/util/virhostdev.c
> > > index 74c43f2..2d219dd 100644
> > > --- a/src/util/virhostdev.c
> > > +++ b/src/util/virhostdev.c
> > > @@ -593,6 +593,56 @@ reattachPCIDevices(virHostdevManagerPtr mgr,
> > >      return ret;
> > >  }
> > >  
> > > +/**
> > > + * detachPCIDevices:
> > 
> > Please keep a virHostdev prefix on this method, and also on the
> > one added in the earlier patch. All methods should be prefixed
> > to match the filename, even if they're static
> 
> I wanted to avoid having
> 
>   virHostdevReattachPCIDevice()

How about virHostdevReAttachPCIDeviceKMod() since this method
is specifically only doing the kmod rebind.

>   virHostdevPCINodeDeviceReAttach()
>   virHostdevReAttachPCIDevices()
> 
> and I couldn't come up with a better name myself. Any ideas? :)

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]