Re: [PATCH -next] dm mpath: Remove unused variable ret

[Samuel Zou]

Hi Mike,

You are right.
I did not notice that the macro DMEMIT uses the sz variable.

Thanks.

On 2020/5/7 22:29, Mike Snitzer wrote:
> On Thu, May 07 2020 at  8:26am -0400,
> Samuel Zou <zou_wei@xxxxxxxxxx> wrote:
>
> > This patch fixes below warning reported by coccicheck:
> >
> > drivers/md/dm-historical-service-time.c:240:14-16: Unneeded variable: "sz". Return "0" on line 261
> >
> > Reported-by: Hulk Robot <hulkci@xxxxxxxxxx>
> > Signed-off-by: Samuel Zou <zou_wei@xxxxxxxxxx>
>
> Nack.
>
> DMEMIT() uses the local sz variable.