lkml.org 
[lkml]   [2020]   [Aug]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] ionic_lif: Use devm_kcalloc() in ionic_qcq_alloc()
From
Date
On Sun, 2020-08-09 at 20:50 -0700, Shannon Nelson wrote:
> On 8/9/20 8:20 PM, Joe Perches wrote:
> > On Mon, 2020-08-10 at 02:38 +0000, Xu Wang wrote:
> > > A multiplication for the size determination of a memory allocation
> > > indicated that an array data structure should be processed.
> > > Thus use the corresponding function "devm_kcalloc".
> > []
> > > diff --git a/drivers/net/ethernet/pensando/ionic/ionic_lif.c b/drivers/net/ethernet/pensando/ionic/ionic_lif.c
> > []
> > > @@ -412,7 +412,7 @@ static int ionic_qcq_alloc(struct ionic_lif *lif, unsigned int type,
> > >
> > > new->flags = flags;
> > >
> > > - new->q.info = devm_kzalloc(dev, sizeof(*new->q.info) * num_descs,
> > > + new->q.info = devm_kcalloc(dev, num_descs, sizeof(*new->q.info),
> > > GFP_KERNEL);
> > > if (!new->q.info) {
> > > netdev_err(lif->netdev, "Cannot allocate queue info\n");
> > You could also remove these unnecessary allocation error messages.
> > There is an existing dump_stack() on allocation failure.
> >
> Yes, the dump_stack() tells you which function had the allocation
> failure, but since there are multiple allocation operations in this same
> function, I find these helpful in knowing quickly which one of the
> allocations failed, without having to track down the symbols and source
> for whatever distro's kernel this might have happened in.

If you do chose the message, might as well add __GFP_NOWARN
to the allocation to avoid the dump_stack().

But honestly, if any allocation fails, you're OOM anyway.

\
 
 \ /
  Last update: 2020-08-10 05:55    [W:0.037 / U:0.308 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site