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

Samuel Zou
2020-05-08T01:06:16Z

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@huawei.com> 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@huawei.com>
>> Signed-off-by: Samuel Zou <zou_wei@huawei.com>
>
> Nack.
>
> DMEMIT() uses the local sz variable.
>
>