Re: [Qemu-devel] [PATCH v8 5/9] module: implement module loading function

[Paolo Bonzini]

Il 13/09/2013 11:56, Alex Bligh ha scritto:
> 
> On 13 Sep 2013, at 09:38, Fam Zheng wrote:
> 
>> Added three types of modules:
>>
>>    typedef enum {
>>        MODULE_LOAD_BLOCK = 0,
>>        MODULE_LOAD_UI,
>>        MODULE_LOAD_NET,
>>        MODULE_LOAD_MAX,
>>    } module_load_type;
>>
>> and their loading function:
> 
> What is the advantage of this enum and having
> different types of module at all? If they are
> all built together, why can't they all live
> together in the same directory?
> 
> Seems like an overcomplication.

At least block have to be separate, since qemu-img and friends do not
have the symbols needed by ui, network and hardware modules.

Paolo