Re: [PATCH v2 0/8] slab: provide and use krealloc_array()

From: Andy Shevchenko
Date: Tue Nov 03 2020 - 05:54:29 EST


On Tue, Nov 3, 2020 at 12:13 PM Bartosz Golaszewski <brgl@xxxxxxxx> wrote:
> On Tue, Nov 3, 2020 at 5:14 AM Joe Perches <joe@xxxxxxxxxxx> wrote:
> > On Mon, 2020-11-02 at 16:20 +0100, Bartosz Golaszewski wrote:
> > > From: Bartosz Golaszewski <bgolaszewski@xxxxxxxxxxxx>

> Yeah so I had this concern for devm_krealloc() and even sent a patch
> that extended it to honor __GFP_ZERO before I noticed that regular
> krealloc() silently ignores __GFP_ZERO. I'm not sure if this is on
> purpose. Maybe we should either make krealloc() honor __GFP_ZERO or
> explicitly state in its documentation that it ignores it?

And my voice here is to ignore for the same reasons: respect
realloc(3) and making common sense with the idea of REallocating
(capital letters on purpose).

--
With Best Regards,
Andy Shevchenko