On Tue, 18 Mar 2025, Bart Van Assche wrote: > On 3/18/25 7:26 AM, Mikulas Patocka wrote: > > The block limits may be read while they are being modified. The statement > > "q->limits = *lim" is not really atomic. The compiler may turn it into > > memcpy (clang does). > > Which code reads block limits while these are being updated? See my reply to Ming - https://lore.kernel.org/dm-devel/14dd4360-c846-43e3-86bc-b1e7448e5...@acm.org/T/#m7e4e49fed1cbcb56954b880e54a5155c4089c0e0 > This should be mentioned in the patch description. > > Bart. Yes, I can add it there. Mikulas
- [PATC] block: update queue limits atomically Mikulas Patocka
- Re: [PATC] block: update queue limits atomically Bart Van Assche
- Re: [PATC] block: update queue limits atomicall... Mikulas Patocka
- Re: [PATC] block: update queue limits atomically Ming Lei
- Re: [PATC] block: update queue limits atomicall... Mikulas Patocka
- Re: [PATC] block: update queue limits atomi... Ming Lei
- Re: [PATC] block: update queue limits a... Jens Axboe
- Re: [PATC] block: update queue lim... Mikulas Patocka
- Re: [PATC] block: update queue... Ming Lei
- Re: [PATC] block: update q... Jens Axboe
- Re: [PATC] block: update queue lim... Christoph Hellwig
- Re: [PATC] block: update queue limits atomically Christoph Hellwig