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

Hitoshi Mitake mitake.hitoshi at gmail.com
Thu Sep 24 04:20:33 CEST 2015


On Wed, Sep 2, 2015 at 9:36 PM, Vasiliy Tolstov <v.tolstov at selfip.ru> wrote:
> 2015-09-01 6:03 GMT+03:00 Hitoshi Mitake <mitake.hitoshi at lab.ntt.co.jp>:
>> 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 <ishizaki.teruaki at lab.ntt.co.jp>
>> Cc: Vasiliy Tolstov <v.tolstov at selfip.ru>
>> Cc: Jeff Cody <jcody at redhat.com>
>> Signed-off-by: Hitoshi Mitake <mitake.hitoshi at lab.ntt.co.jp>
>
> I'm test this patch and now discard working properly and no errors in
> sheepdog log file.
>
> Tested-by: Vasiliy Tolstov <v.tolstov at selfip.ru>

Ping, Jeff?

Thanks,
Hitoshi

>
> --
> Vasiliy Tolstov,
> e-mail: v.tolstov at selfip.ru
> --
> sheepdog mailing list
> sheepdog at lists.wpkg.org
> https://lists.wpkg.org/mailman/listinfo/sheepdog


More information about the sheepdog mailing list