Re: [Xen-devel] [PATCH v6 04/11] libxl: move libxl__device_from_disk to libxl_internal.c

[Roger Pau Monne <roger.pau@xxxxxxxxxx>]

Ian Campbell wrote:
> On Fri, 2012-05-18 at 15:55 +0100, Roger Pau Monne wrote:
> > > @@ -429,6 +429,46 @@ libxl_device_model_version
> > libxl__device_model_version_running(libxl__gc *gc,
> > >        return value;
> > >    }
> > >
> > > +int libxl__device_from_disk(libxl__gc *gc, uint32_t domid,
> > > +                                   libxl_device_disk *disk,
> > > +                                   libxl__device *device)
> > I think this should be moved to libxl_device instead of
> > libxl_internal, since it's a device related function.
> I think it'd be better to leave it in the original file, libxl is so
> confused about what goes where that it doesn't really matter...
>
> If we do want to move it we can do it later as a pure code motion
> change.
In my hotplug series I'm moving most libxl_device_{...}_add to provide 
an internal implementation that takes an ao, making something quite 
similar to what Stefano does, if I start moving all those to 
libxl_internal we will fill this file with functions that could be 
somewhere else (libxl_device). I understand that the libxl policy is put 
functions where they seem to belong (device related functions to 
libxl_device, domain related ones to libxl_dom...), and if they fit in 
none of this categories then put them in libxl_internal or create a new 
file.
We can leave it in libxl_internal for now, and I will move it on my series.

> Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel