Re: [PATCH v3] isofs compress: Remove VLA usage

From: Joe Perches
Date: Wed Apr 11 2018 - 02:15:36 EST


On Tue, 2018-04-10 at 17:51 -0700, Kees Cook wrote:
> On Tue, Apr 10, 2018 at 5:33 PM, Al Viro <viro@xxxxxxxxxxxxxxxxxx> wrote:
> > On Tue, Apr 10, 2018 at 04:45:32PM -0700, Kyle Spiers wrote:
> > > As part of the effort to remove VLAs from the kernel[1], this changes
> > > the allocation of the bhs and pages arrays from being on the stack to being
> > > kcalloc()ed. This also allows for the removal of the explicit zeroing
> > > of bhs.
> > >
> > > https://lkml.org/lkml/2018/3/7/621
> >
> > Do you even bother reading the feedback given to such patches? I'm just
> > curious,
>
> He made a tool mistake (sent v2 again) -- please see v4.

No worries but Kyle didn't just mistakenly resend v2.
It was a v3 with a different below the dashes changelog.

Kyle, it would be good if next time you were to add a
listing of changes in each revision below the dashes.