Re: [Qemu-devel] [RFC PATCH 5/6] module: load modules at start

2013-09-05 Thread Fam Zheng
On Thu, 09/05 14:43, Lluís Vilanova wrote: > Fam Zheng writes: > > > Add module_load_all to load all DSO modules under: > > /usr/lib/qemu/block/ > > /usr/lib/qemu/net/ > > /usr/lib/qemu/ui/ > > when starting process. > > This should probably be based on a define with the prefix set at

Re: [Qemu-devel] [RFC PATCH 5/6] module: load modules at start

2013-09-05 Thread Michael Tokarev
05.09.2013 14:20, Fam Zheng wrote: > Add module_load_all to load all DSO modules under: > /usr/lib/qemu/block/ > /usr/lib/qemu/net/ > /usr/lib/qemu/ui/ > when starting process. NACK. This is wrong, as has been mentioned already. For example, you can't expect to load ui/* from qemu-img

Re: [Qemu-devel] [RFC PATCH 5/6] module: load modules at start

2013-09-05 Thread Lluís Vilanova
Fam Zheng writes: > Add module_load_all to load all DSO modules under: > /usr/lib/qemu/block/ > /usr/lib/qemu/net/ > /usr/lib/qemu/ui/ > when starting process. This should probably be based on a define with the prefix set at configure time. Adding directories from command-line argume