Re: [PATCH 1/1] Stop mdcheck_continue timer when mdcheck_start service can finish check

[Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com>]

Hi Xiao,
some nits:

On Mon,  8 May 2023 21:30:10 +0800
Xiao Ni <xni@redhat.com> wrote:

> mdcheck_continue is triggered by mdcheck_start timer. It's used to
> continue check action if the raid is too big and mdcheck_start
> service can't finish check action. If mdcheck start can finish check
> action, it doesn't need to mdcheck continue service anymore. So stop
> it when mdcheck start service can finish check action.
> 
> Signed-off-by: Xiao Ni <xni@redhat.com>
> ---
>  misc/mdcheck | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/misc/mdcheck b/misc/mdcheck
> index 700c3e252e72..f56972c8ed10 100644
> --- a/misc/mdcheck
> +++ b/misc/mdcheck
> @@ -140,7 +140,13 @@ do
>  		echo $a > $fl
>  		any=yes
>  	done
> -	if [ -z "$any" ]; then exit 0; fi
> +	if [ -z "$any" ]; then
> +		#mdcheck_continue.timer is started by mdcheck_start.timer.
> +		#When he check action can be finished in
's/he/the/g'
I think that there should be space after '#' but it is preferred to use /* */ 
Could you please send v2?

Thanks,
Mariusz