[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] xen-block: support feature-large-sector-size
From: |
Max Reitz |
Subject: |
Re: [Qemu-devel] [PATCH] xen-block: support feature-large-sector-size |
Date: |
Wed, 26 Jun 2019 19:48:55 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
On 26.06.19 19:19, Anthony PERARD wrote:
> On Wed, Jun 26, 2019 at 06:48:50PM +0200, Max Reitz wrote:
>> On 09.04.19 18:40, Paul Durrant wrote:
>>> A recent Xen commit [1] clarified the semantics of sector based quantities
>>> used in the blkif protocol such that it is now safe to create a xen-block
>>> device with a logical_block_size != 512, as long as the device only
>>> connects to a frontend advertizing 'feature-large-block-size'.
>>>
>>> This patch modifies xen-block accordingly. It also uses a stack variable
>>> for the BlockBackend in xen_block_realize() to avoid repeated dereferencing
>>> of the BlockConf pointer, and changes the parameters of
>>> xen_block_dataplane_create() so that the BlockBackend pointer and sector
>>> size are passed expicitly rather than implicitly via the BlockConf.
>>>
>>> These modifications have been tested against a recent Windows PV XENVBD
>>> driver [2] using a xen-disk device with a 4kB logical block size.
>>>
>>> [1]
>>> http://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=67e1c050e36b2c9900cca83618e56189effbad98
>>> [2] https://winpvdrvbuild.xenproject.org:8080/job/XENVBD-master/126
>>>
>>> Signed-off-by: Paul Durrant <address@hidden>
>>> ---
>>> Cc: Stefano Stabellini <address@hidden>
>>> Cc: Anthony Perard <address@hidden>
>>> Cc: Stefan Hajnoczi <address@hidden>
>>> Cc: Kevin Wolf <address@hidden>
>>> Cc: Max Reitz <address@hidden>
>>> ---
>>> hw/block/dataplane/xen-block.c | 25 ++++++++++++----------
>>> hw/block/dataplane/xen-block.h | 3 ++-
>>> hw/block/xen-block.c | 38 +++++++++++++++++++++-------------
>>> 3 files changed, 40 insertions(+), 26 deletions(-)
>>
>> Thanks, added “by frontend” to the error message and applied to my block
>> branch:
>>
>> https://git.xanclic.moe/XanClic/qemu/commits/branch/block
>
> :(, I've just sent a pull request with that patch:
> https://patchew.org/QEMU/address@hidden/address@hidden/
That’s just as well, then. :-)
> I guess I need to start sending an email every time I've added a patch
> to my queue.
Well, it certainly won’t hurt. Although in this cases it’s just a bit
of an unfortunate coincidence that I looked at this patch now when Peter
seems to be away (otherwise I’d have seen it in master).
Max
signature.asc
Description: OpenPGP digital signature