[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [Xen-devel] [PATCH] xen: fix qdisk BLKIF_OP_DISCARD for

From: Juergen Gross
Subject: Re: [Qemu-devel] [Xen-devel] [PATCH] xen: fix qdisk BLKIF_OP_DISCARD for 32/64 word size mix
Date: Thu, 16 Jun 2016 14:18:44 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.7.0

On 16/06/16 13:24, Paul Durrant wrote:
>> -----Original Message-----
>> From: Xen-devel [mailto:address@hidden On Behalf Of
>> Juergen Gross
>> Sent: 16 June 2016 11:02
>> To: address@hidden; address@hidden
>> Cc: Anthony Perard; Juergen Gross; address@hidden;
>> address@hidden
>> Subject: [Xen-devel] [PATCH] xen: fix qdisk BLKIF_OP_DISCARD for 32/64
>> word size mix
>> In case the word size of the domU and qemu running the qdisk backend
>> differ BLKIF_OP_DISCARD will not work reliably, as the request
>> structure in the ring have different layouts for different word size.
>> Correct this by copying the request structure in case of different
>> word size element by element in the BLKIF_OP_DISCARD case, too.
>> Signed-off-by: Juergen Gross <address@hidden>
> Would it not be better to re-import the canonical blkif header as a whole 
> rather than cherry-picking like this? You'd need to post-process to maintain 
> style and possibly change some names for compatibility etc. but probably 
> nothing beyond what indent and a simple [s]ed script can do.
> I did broadly the same thing to re-import the netif header into Linux 
> recently.

It's a little bit more than indent/sed, but nothing really scary.

I'll do it.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]