[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v4 00/32] Dynamic module loading for block drive
From: |
Max Reitz |
Subject: |
Re: [Qemu-block] [PATCH v4 00/32] Dynamic module loading for block drivers |
Date: |
Fri, 22 Jul 2016 18:43:47 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 |
On 22.07.2016 18:40, Paolo Bonzini wrote:
>
>
> On 22/07/2016 18:27, Max Reitz wrote:
>>
>> So I can't see a strong argument for not modularizing the format block
>> drivers, but in turn I don't have a strong argument for doing so either.
>>
>> Therefore: Why not? :-)
>
> The usual argument is that there is an additional cost in
> maintainability. This applies particularly in this case where you
> wouldn't have to care at all about dynamically loading a driver based on
> the outcome of a probe---only based on the driver name or a
> colon-separated prefix in the filename.
That's a good point, especially since it's likely only a few people will
use modularized format block drivers, so we may break it without noticing.
Max
signature.asc
Description: OpenPGP digital signature
- [Qemu-block] [PATCH v4 26/32] blockdev: Separate vdi probe from its driver, (continued)
- [Qemu-block] [PATCH v4 26/32] blockdev: Separate vdi probe from its driver, Colin Lord, 2016/07/14
- [Qemu-block] [PATCH v4 31/32] blockdev: Separate out bdrv_probe_device functions, Colin Lord, 2016/07/14
- [Qemu-block] [PATCH v4 27/32] blockdev: Separate vhdx probe from its driver, Colin Lord, 2016/07/14
- [Qemu-block] [PATCH v4 32/32] blockdev: Remove bdrv_probe_device field from BlockDriver, Colin Lord, 2016/07/14
- Re: [Qemu-block] [PATCH v4 00/32] Dynamic module loading for block drivers, Paolo Bonzini, 2016/07/19
- Re: [Qemu-block] [PATCH v4 00/32] Dynamic module loading for block drivers, Max Reitz, 2016/07/22