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);
