lkml.org 
[lkml]   [2021]   [Feb]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] mm: cma: support sysfs
On Fri, Feb 05, 2021 at 12:12:17PM +0000, Matthew Wilcox wrote:
> On Thu, Feb 04, 2021 at 09:22:18PM -0800, Minchan Kim wrote:
> > > > + for (i = 0; i < cma_area_count; i++) {
> > > > + cma = &cma_areas[i];
> > > > + stat = kzalloc(sizeof(*stat), GFP_KERNEL);
> > > > + if (!stat)
> > > > + goto out;
> > >
> > > How many cma areas are there going to be? do we really want to allocate
> > > their stat individually?
> >
> > I am not sure what could be in the end but at least, I have
> > 5+ candidates (but could be shrink or extend) and yes,
> > want to keep track them individually.
>
> I meant, wouldn't it be better to:
>
> cma_stats = kzalloc(array_size(sizeof(*stat), cma_area_count),
> GFP_KERNEL);
>

Definitely.
Thanks, Matthew.

\
 
 \ /
  Last update: 2021-02-06 00:02    [W:0.398 / U:0.084 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site