Re: [PATCH 1/2] ext4: Delete an error message for a failed memory allocation in ext4_multi_mount_protect()
From: Dan Carpenter
Date: Thu Aug 24 2017 - 15:18:02 EST
On Sat, Aug 19, 2017 at 12:08:29PM -0500, Eric Sandeen wrote:
> On 8/19/17 6:47 AM, SF Markus Elfring wrote:
> > From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx>
> > Date: Sat, 19 Aug 2017 13:04:50 +0200
> >
> > Omit an extra message for a memory allocation failure in this function.
>
> I might be dense, but what makes this message "extra?"
>
> (I suppose kmalloc squawks too if it fails, but is Coccinelle
> now warning about explicit memory allocation failure warnings?)
>
Yeah... Checkpatch complains that the kmalloc squawks is enough.
"WARNING: Possible unnecessary 'out of memory' message". This
allocation is small so it's guaranteed to succeed in current kernels.
regards,
dan carpenter