Re: [PATCH] block: remove unnecessary NULL checks in bioset_integrity_free()

From: Kyle Fortin
Date: Mon Oct 09 2017 - 14:34:32 EST


On Oct 6, 2017, at 6:26 PM, Tim Hansen <devtimhansen@xxxxxxxxx> wrote:
> On Fri, Oct 06, 2017 at 01:04:25PM -0600, Jens Axboe wrote:
>> On 10/05/2017 12:09 PM, Tim Hansen wrote:
>>> mempool_destroy() already checks for a NULL value being passed in, this eliminates duplicate checks.
>>>
>>> This was caught by running make coccicheck M=block/ on linus' tree on commit 77ede3a014a32746002f7889211f0cecf4803163 (current head as of this patch).
>>
>> Applied, but I had to fix up your commit message. Please line wrap at
>> 72 chars, otherwise it turns into an unreadable mess when people
>> try to view them.
>>
>> --
>> Jens Axboe
>>
>
> Noted! Thanks for the heads up on the char limit.

scripts/checkpatch.pl would have caught it too

--
Kyle Fortin - Oracle Linux Engineering