Re: [PATCH v3] loop: reduce the loop_ctl_mutex scope

From: Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp>
Date: 2021-09-01 05:47:55
Reply to: Christoph Hellwig, 2021-09-01

On 2021/09/01 14:36, Christoph Hellwig wrote:

On Mon, Aug 30, 2021 at 09:13:50AM +0200, Christoph Hellwig wrote:

quoted

Hi Tetsuo,

this generally looks fine to me.  The only issue is that I remember is
that cmpxchg did historically not work on bool on all architectures.
I'm not sure if this has been fixed since.

Looks like that still is an issue based on the buildbot reports on
linux-kernel.  I'd suggest to resend with idr_visible as an int.

I sent "[PATCH v3] loop: reduce the loop_ctl_mutex scope" as not using cmpxchg().
The buildbot is reporting errors on the v2 patch.

Tetsuo Handa · 2021-08-26
Christoph Hellwig · 2021-08-27
Tetsuo Handa · 2021-08-28
Tetsuo Handa · 2021-08-28
Christoph Hellwig · 2021-08-28
Tetsuo Handa · 2021-08-28
[PATCH v2] loop: reduce the loop_ctl_mutex scope · Tetsuo Handa · 2021-08-29
[PATCH v3] loop: reduce the loop_ctl_mutex scope · Tetsuo Handa · 2021-08-29
Re: [PATCH v3] loop: reduce the loop_ctl_mutex scope · Christoph Hellwig · 2021-08-30
Re: [PATCH v3] loop: reduce the loop_ctl_mutex scope · Christoph Hellwig · 2021-09-01
Re: [PATCH v3] loop: reduce the loop_ctl_mutex scope · Tetsuo Handa · 2021-09-01
Re: [PATCH v3] loop: reduce the loop_ctl_mutex scope · Christoph Hellwig · 2021-09-01
[PATCH v3 (repost)] loop: reduce the loop_ctl_mutex scope · Tetsuo Handa · 2021-09-02
Re: [PATCH v3] loop: reduce the loop_ctl_mutex scope · Jens Axboe · 2021-09-04
View all 14 messages

`h`	back out one level
`j`	next message in thread / next series-diff position
`k`	previous message in thread / previous series-diff position
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help