[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Fri, 7 Jul 2023 09:26:36 +0800
From: Yu Kuai <yukuai1@...weicloud.com>
To: Paul Menzel <pmenzel@...gen.mpg.de>,
Yu Kuai <yukuai1@...weicloud.com>
Cc: agk@...hat.com, snitzer@...nel.org, dm-devel@...hat.com,
song@...nel.org, heinzm@...hat.com, neilb@...e.de,
jbrassow@...hat.com, linux-kernel@...r.kernel.org,
linux-raid@...r.kernel.org, yi.zhang@...wei.com,
yangerkun@...wei.com, "yukuai (C)" <yukuai3@...wei.com>
Subject: Re: [PATCH -next 2/3] md/dm-raid: cleanup multiple equivalent goto
tags from raid_ctr()
Hi,
在 2023/07/06 21:01, Paul Menzel 写道:
> Dear Yu,
>
>
> Thank you for your patch. Some minor nits, if you are interested.
>
> Am 06.07.23 um 09:16 schrieb Yu Kuai:
>> From: Yu Kuai <yukuai3@...wei.com>
>>
>> There are four equivalent goto tags in raid_ctr(), clean them up to use
>> just one, there are no functional change and prepare to fix that
>
> Maybe:
>
> There *is* no functional change, and is a preparation to fix an
> unprotected `md_stop()`.
>
>> md_stop() is not protected.
>
> In the commit message summary/title, I’d spell the verb with a space:
>
> Clean up four equivalent goto tags in raid_ctr()
>
Thanks for the suggestions, I'll update them in the next version.
Thanks,
Kuai
>> Signed-off-by: Yu Kuai <yukuai3@...wei.com>
>> ---
>> drivers/md/dm-raid.c | 27 +++++++++------------------
>> 1 file changed, 9 insertions(+), 18 deletions(-)
>
> […]
>
>
> Kind regards,
>
> Paul
>
> .
>
Powered by blists - more mailing lists