On 12/27/21 11:36 PM, Matthew Wilcox wrote: > On Mon, Dec 27, 2021 at 08:54:40PM +0800, Jeffle Xu wrote: >> + spin_lock(&cache->reqs_lock); >> + ret = idr_alloc(&cache->reqs, req, 0, 0, GFP_KERNEL); > > GFP_KERNEL while holding a spinlock? Right. Thanks for pointing it out. > > You should be using an XArray instead of an IDR in new code anyway. > Regards. -- Thanks, Jeffle -- Linux-cachefs mailing list Linux-cachefs@redhat.com https://listman.redhat.com/mailman/listinfo/linux-cachefs
- [Linux-cachefs] [PATCH v1 13/23] erofs: implement fscach... Jeffle Xu
- [Linux-cachefs] [PATCH v1 16/23] erofs: add 'uuid' mount... Jeffle Xu
- [Linux-cachefs] [PATCH v1 22/23] cachefiles: add done co... Jeffle Xu
- [Linux-cachefs] [PATCH v1 14/23] erofs: register cookie ... Jeffle Xu
- [Linux-cachefs] [PATCH v1 19/23] cachefiles: implement .... Jeffle Xu
- Re: [Linux-cachefs] [PATCH v1 19/23] cachefiles: im... kernel test robot
- Re: [Linux-cachefs] [PATCH v1 19/23] cachefiles: im... Matthew Wilcox
- Re: [Linux-cachefs] [PATCH v1 19/23] cachefiles... JeffleXu
- Re: [Linux-cachefs] [PATCH v1 19/23] cachef... JeffleXu
- Re: [Linux-cachefs] [PATCH v1 19/23] ca... Matthew Wilcox
- Re: [Linux-cachefs] [PATCH v1 19/23] cachefiles: im... kernel test robot
- [Linux-cachefs] [PATCH v1 05/23] netfs: add inode parame... Jeffle Xu
- [Linux-cachefs] [PATCH v1 15/23] erofs: implement fscach... Jeffle Xu
- [Linux-cachefs] [PATCH v1 10/23] erofs: add anonymous in... Jeffle Xu