Re: [PATCH] md-cluster: choose correct label when clustered layout is not supported

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

 



On Thu, Feb 22, 2018 at 10:47:30AM +0800, Guoqing Jiang wrote:
> Hi Shaohua,
> 
> Could you consider to merge this patch? Thanks.

applied, thanks!
 
> Regards,
> Guoqing
> 
> On 01/24/2018 09:50 AM, Guoqing Jiang wrote:
> > 
> > 
> > On 01/23/2018 11:06 PM, Lidong Zhong wrote:
> > > r10conf is already successfully allocated before checking the layout
> > > 
> > > Signed-off-by: Lidong Zhong <lzhong@xxxxxxxx>
> > > ---
> > >   drivers/md/raid10.c | 2 +-
> > >   1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/drivers/md/raid10.c b/drivers/md/raid10.c
> > > index c131835cf008..c2b2fbeec7b2 100644
> > > --- a/drivers/md/raid10.c
> > > +++ b/drivers/md/raid10.c
> > > @@ -3770,7 +3770,7 @@ static int raid10_run(struct mddev *mddev)
> > >           if (fc > 1 || fo > 0) {
> > >               pr_err("only near layout is supported by clustered"
> > >                   " raid10\n");
> > > -            goto out;
> > > +            goto out_free_conf;
> > >           }
> > >       }
> > 
> > Thanks for catch it.
> > Reviewed-by: Guoqing Jiang <gqjiang@xxxxxxxx>
> > 
> > Regards,
> > Guoqing
> > -- 
> > 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
> > 
> 
--
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