Re: [PATCH 01/12] cachefiles: remove request from xarry during flush requests

2024-05-06 Thread Baokun Li
Hi Jingbo, Sorry for the late reply. On 2024/5/6 13:50, Jingbo Xu wrote: On 5/6/24 11:57 AM, Baokun Li wrote: On 2024/5/6 11:48, Jingbo Xu wrote: On 4/24/24 11:39 AM, libao...@huaweicloud.com wrote: From: Baokun Li This prevents concurrency from causing access to a freed req. Could you g

Re: [PATCH 01/12] cachefiles: remove request from xarry during flush requests

2024-05-05 Thread Jingbo Xu
On 5/6/24 11:57 AM, Baokun Li wrote: > On 2024/5/6 11:48, Jingbo Xu wrote: >> >> On 4/24/24 11:39 AM, libao...@huaweicloud.com wrote: >>> From: Baokun Li >>> >>> This prevents concurrency from causing access to a freed req. >> Could you give more details on how the concurrent access will happen

Re: [PATCH 01/12] cachefiles: remove request from xarry during flush requests

2024-05-05 Thread Baokun Li
On 2024/5/6 11:48, Jingbo Xu wrote: On 4/24/24 11:39 AM, libao...@huaweicloud.com wrote: From: Baokun Li This prevents concurrency from causing access to a freed req. Could you give more details on how the concurrent access will happen? How could another process access the &cache->reqs xarra

Re: [PATCH 01/12] cachefiles: remove request from xarry during flush requests

2024-05-05 Thread Jingbo Xu
On 4/24/24 11:39 AM, libao...@huaweicloud.com wrote: > From: Baokun Li > > This prevents concurrency from causing access to a freed req. Could you give more details on how the concurrent access will happen? How could another process access the &cache->reqs xarray after it has been flushed? >

Re: [PATCH 01/12] cachefiles: remove request from xarry during flush requests

2024-04-24 Thread Jia Zhu via Linux-erofs
在 2024/4/24 11:39, libao...@huaweicloud.com 写道: From: Baokun Li This prevents concurrency from causing access to a freed req. Signed-off-by: Baokun Li Reviewed-by: Jia Zhu --- fs/cachefiles/daemon.c | 1 + 1 file changed, 1 insertion(+) diff --git a/fs/cachefiles/daemon.c b/fs/cac

[PATCH 01/12] cachefiles: remove request from xarry during flush requests

2024-04-23 Thread libaokun
From: Baokun Li This prevents concurrency from causing access to a freed req. Signed-off-by: Baokun Li --- fs/cachefiles/daemon.c | 1 + 1 file changed, 1 insertion(+) diff --git a/fs/cachefiles/daemon.c b/fs/cachefiles/daemon.c index 6465e2574230..ccb7b707ea4b 100644 --- a/fs/cachefiles/daem