Re: [PATCH] md/linear: shutup lockdep warnning

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 在 2017年2月22日,上午5:55,Shaohua Li <shli@xxxxxx> 写道:
> 
> Commit 03a9e24(md linear: fix a race between linear_add() and
> linear_congested()) introduces the warnning.
> 
> Cc: Coly Li <colyli@xxxxxxx>
> Signed-off-by: Shaohua Li <shli@xxxxxx>

Acked-by: Coly Li <colyli@xxxxxxx>

Thanks for the fix !

Coly



> ---
> drivers/md/linear.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/md/linear.c b/drivers/md/linear.c
> index 789008b..5b06b0d 100644
> --- a/drivers/md/linear.c
> +++ b/drivers/md/linear.c
> @@ -224,7 +224,8 @@ static int linear_add(struct mddev *mddev, struct md_rdev *rdev)
>     * oldconf until no one uses it anymore.
>     */
>    mddev_suspend(mddev);
> -    oldconf = rcu_dereference(mddev->private);
> +    oldconf = rcu_dereference_protected(mddev->private,
> +            lockdep_is_held(&mddev->reconfig_mutex));
>    mddev->raid_disks++;
>    WARN_ONCE(mddev->raid_disks != newconf->raid_disks,
>        "copied raid_disks doesn't match mddev->raid_disks");
> -- 
> 2.9.3
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux RAID Wiki]     [ATA RAID]     [Linux SCSI Target Infrastructure]     [Linux Block]     [Linux IDE]     [Linux SCSI]     [Linux Hams]     [Device Mapper]     [Device Mapper Cryptographics]     [Kernel]     [Linux Admin]     [Linux Net]     [GFS]     [RPM]     [git]     [Yosemite Forum]


  Powered by Linux