Re: [PATCH] regulator: fix pointer table overallocation

2020-08-10 Thread Michał Mirosław
On Mon, Aug 10, 2020 at 06:33:37PM +0100, Mark Brown wrote: > On Mon, Aug 10, 2020 at 06:25:56PM +0200, Michał Mirosław wrote: > > On Mon, Aug 10, 2020 at 01:37:47PM +0100, Mark Brown wrote: > > > Yeah, this is more a performance improvement than a fix. > > Should I resend without Cc: stable? > I'v

Re: [PATCH] regulator: fix pointer table overallocation

2020-08-10 Thread Mark Brown
On Mon, Aug 10, 2020 at 06:25:56PM +0200, Michał Mirosław wrote: > On Mon, Aug 10, 2020 at 01:37:47PM +0100, Mark Brown wrote: > > Yeah, this is more a performance improvement than a fix. > Should I resend without Cc: stable? I've already queued it to be applied to -next with that removed and an

Re: [PATCH] regulator: fix pointer table overallocation

2020-08-10 Thread Michał Mirosław
On Mon, Aug 10, 2020 at 01:37:47PM +0100, Mark Brown wrote: > On Sun, Aug 09, 2020 at 10:44:25PM +0300, Dmitry Osipenko wrote: > > 09.08.2020 22:21, Michał Mirosław пишет: > > > The code allocates sizeof(regulator_dev) for a pointer. Make it less > > > generous. Let kcalloc() calculate the size, wh

Re: [PATCH] regulator: fix pointer table overallocation

2020-08-10 Thread Mark Brown
On Sun, Aug 09, 2020 at 10:44:25PM +0300, Dmitry Osipenko wrote: > 09.08.2020 22:21, Michał Mirosław пишет: > > The code allocates sizeof(regulator_dev) for a pointer. Make it less > > generous. Let kcalloc() calculate the size, while at it. > > > > Cc: sta...@vger.kernel.org > > Fixes: d8ca7d184b

Re: [PATCH] regulator: fix pointer table overallocation

2020-08-09 Thread Dmitry Osipenko
09.08.2020 22:21, Michał Mirosław пишет: > The code allocates sizeof(regulator_dev) for a pointer. Make it less > generous. Let kcalloc() calculate the size, while at it. > > Cc: sta...@vger.kernel.org > Fixes: d8ca7d184b33 ("regulator: core: Introduce API for regulators coupling > customization"