On Tue, Jul 05, 2016 at 11:24:04AM -0400, Colin Lord wrote: > This puts the bochs probe function into its own separate file as part of > the process of modularizing block drivers. Having the probe functions > separate from the rest of the driver allows us to probe without having > to potentially unnecessarily load the driver. > > Signed-off-by: Colin Lord <cl...@redhat.com> > --- > block/Makefile.objs | 1 + > block/bochs.c | 55 > ++------------------------------------------ > block/probe/bochs.c | 21 +++++++++++++++++
Do we really need a sub-dir for this ? If we were going to have sub-dirs under block/, I'd suggest we have one subdir per block driver, not spread code for one block driver across multiple dirs. IMHO a block/bochs-probe.c would be better, unless we did move block/bochs.c into a block/bochs/driver.c dir. Either way, you should update MAINTAINERS file to record this newly added filename, against the bochs entry. The same applies to most other patches in this series adding new files. 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 :|
