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

[Vasiliy Tolstov]

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>

-- 
Vasiliy Tolstov,
e-mail: address@hidden