[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 4/4] blockdev: Modularize nfs block driver
From: |
Max Reitz |
Subject: |
Re: [Qemu-devel] [PATCH v7 4/4] blockdev: Modularize nfs block driver |
Date: |
Fri, 12 Aug 2016 14:31:03 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 |
On 10.08.2016 21:22, Colin Lord wrote:
> On 08/10/2016 03:04 PM, Max Reitz wrote:
>> On 08.08.2016 20:07, Colin Lord wrote:
>>> Modularizes the nfs block driver so that it gets dynamically loaded.
>>>
>>> Signed-off-by: Colin Lord <address@hidden>
>>> Reviewed-by: Stefan Hajnoczi <address@hidden>
>>> ---
>>> block/Makefile.objs | 1 +
>>> configure | 4 ++--
>>> 2 files changed, 3 insertions(+), 2 deletions(-)
>>
>> I'm not quite sure what this achieves. From what I can see, the NFS
>> block driver is still linked hard into qemu and it is unconditionally
>> registered at qemu startup.
>>
> ldd seems to be telling me that libnfs is not linked to the main binary
> after this patch.
>> (The output from a printf() in nfs_block_init() is visible even when
>> just starting qemu-img or qemu-io without any arguments; most notably
>> without bdrv_find_protocol() having been invoked at all.)
>>
>> Max
>>
> I can't seem to reproduce this. Is it possible you applied this patch
> without applying the first 3 before it? Or maybe didn't have modules
> enabled in the configuration?
I'm so stupid. Yep, that's it. Works great now. :D
Reviewed-by: Max Reitz <address@hidden>
> As far as I can tell NFS seems to building
> as a module and doesn't seem to be hard linked, so I'm not really sure
> what's going on.
>
> Colin
>
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v7 0/4] Dynamic module loading for block drivers, Colin Lord, 2016/08/08
- [Qemu-devel] [PATCH v7 3/4] blockdev: Add dynamic module loading for block drivers, Colin Lord, 2016/08/08
- Re: [Qemu-devel] [PATCH v7 3/4] blockdev: Add dynamic module loading for block drivers, Max Reitz, 2016/08/10
- Re: [Qemu-devel] [PATCH v7 3/4] blockdev: Add dynamic module loading for block drivers, Colin Lord, 2016/08/10
- Re: [Qemu-devel] [PATCH v7 3/4] blockdev: Add dynamic module loading for block drivers, Max Reitz, 2016/08/10
- Re: [Qemu-devel] [PATCH v7 3/4] blockdev: Add dynamic module loading for block drivers, Fam Zheng, 2016/08/10
- Re: [Qemu-devel] [PATCH v7 3/4] blockdev: Add dynamic module loading for block drivers, Colin Lord, 2016/08/11
- Re: [Qemu-devel] [PATCH v7 3/4] blockdev: Add dynamic module loading for block drivers, Fam Zheng, 2016/08/11
- Re: [Qemu-devel] [PATCH v7 3/4] blockdev: Add dynamic module loading for block drivers, Colin Lord, 2016/08/12
- Re: [Qemu-devel] [PATCH v7 3/4] blockdev: Add dynamic module loading for block drivers, Colin Lord, 2016/08/10
- Re: [Qemu-devel] [Qemu-block] [PATCH v7 3/4] blockdev: Add dynamic module loading for block drivers, Colin Lord, 2016/08/10