Re: [Qemu-devel] [sheepdog] [PATCH 1/2] sheepdog: use per AIOCB dirty indexes for non overlapping requests

[Hitoshi Mitake]

On Wed, Sep 2, 2015 at 9:36 PM, Vasiliy Tolstov <address@hidden> wrote:
> 2015-09-01 6:03 GMT+03:00 Hitoshi Mitake <address@hidden>:
>> n the commit 96b14ff85acf, requests for overlapping areas are
>> serialized. However, it cannot handle a case of non overlapping
>> requests. In such a case, min_dirty_data_idx and max_dirty_data_idx
>> can be overwritten by the requests and invalid inode update can
>> happen e.g. a case like create(1, 2) and create(3, 4) are issued in
>> parallel.
>>
>> This patch lets SheepdogAIOCB have dirty data indexes instead of
>> BDRVSheepdogState for avoiding the above situation.
>>
>> This patch also does trivial renaming for better description:
>> overwrapping -> overlapping
>>
>> Cc: Teruaki Ishizaki <address@hidden>
>> Cc: Vasiliy Tolstov <address@hidden>
>> Cc: Jeff Cody <address@hidden>
>> Signed-off-by: Hitoshi Mitake <address@hidden>
>
> I'm test this patch and now discard working properly and no errors in
> sheepdog log file.
>
> Tested-by: Vasiliy Tolstov <address@hidden>

Ping, Jeff?

Thanks,
Hitoshi

>
> --
> Vasiliy Tolstov,
> e-mail: address@hidden
> --
> sheepdog mailing list
> address@hidden
> https://lists.wpkg.org/mailman/listinfo/sheepdog