On 16/06/2016 16:00, Colin Lord wrote: >> >> The only block drivers that can be converted into modules are the drivers >> >> that don't perform any init operation except for registering themselves. >> >> This >> >> is why libiscsi has been disabled as a module. > > > > I don't think it has in this patch :) but you can also move the > > iscsi_opts registration to vl.c. > > Yeah I think Stefan mentioned this point in one of the earlier threads > but he said to do it in a separate commit, which I took to mean I > shouldn't include it here. Should I add it as a third part to this patch > series or leave it for a completely separate commit?
The patches in the series are left separate when including them in QEMU. Therefore, a separate patch *is* (or will become :)) a separate commit. Therefore, putting the change before this patch, or alternatively as the first patch in the series, will be fine. Thanks, Paolo