From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751277AbdCQUVh (ORCPT ); Fri, 17 Mar 2017 16:21:37 -0400 Received: from mail-yw0-f194.google.com ([209.85.161.194]:35788 "EHLO mail-yw0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751195AbdCQUVe (ORCPT ); Fri, 17 Mar 2017 16:21:34 -0400 From: jes.sorensen@gmail.com X-Google-Original-From: Jes.Sorensen@gmail.com To: Gioh Kim Cc: neilb@suse.com, linux-raid@vger.kernel.org, linux-kernel@vger.kernel.org, Jack Wang Subject: Re: [PATCH 1/2] super1: ignore failfast flag for setting device role References: <1489769222-31855-1-git-send-email-gi-oh.kim@profitbricks.com> <1489769222-31855-2-git-send-email-gi-oh.kim@profitbricks.com> Date: Fri, 17 Mar 2017 16:20:58 -0400 In-Reply-To: <1489769222-31855-2-git-send-email-gi-oh.kim@profitbricks.com> (Gioh Kim's message of "Fri, 17 Mar 2017 17:47:01 +0100") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Gioh Kim writes: > There is corner case for setting device role, > if new device has failfast flag. > The failfast flag should be ignored. > > Signed-off-by: Gioh Kim > Signed-off-by: Jack Wang > --- > super1.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/super1.c b/super1.c > index 882cd61..1da33ef 100644 > --- a/super1.c > +++ b/super1.c > @@ -1491,6 +1491,7 @@ static int add_to_super1(struct supertype *st, mdu_disk_info_t *dk, > struct devinfo *di, **dip; > bitmap_super_t *bms = (bitmap_super_t*)(((char*)sb) + MAX_SB_SIZE); > int rv, lockid; > + int dk_state; > > if (bms->version == BITMAP_MAJOR_CLUSTERED && dlm_funs_ready()) { > rv = cluster_get_dlmlock(&lockid); > @@ -1501,11 +1502,12 @@ static int add_to_super1(struct supertype *st, mdu_disk_info_t *dk, > } > } > > - if ((dk->state & 6) == 6) /* active, sync */ > + dk_state &= ~(1< + if ((dk_state & 6) == 6) /* active, sync */ > *rp = __cpu_to_le16(dk->raid_disk); This does not look right - you haven't assigned a value to dk_state, but then start masking bits out of it. Cheers, Jes