Re: Ping: [PATCH v3] gnttab: defer allocation of status frame tracking array

[Julien Grall <julien@xxxxxxx>]

Hi Jan,

On 29/04/2021 10:31, Jan Beulich wrote:
> On 15.04.2021 11:41, Jan Beulich wrote:
> > This array can be large when many grant frames are permitted; avoid
> > allocating it when it's not going to be used anyway, by doing this only
> > in gnttab_populate_status_frames().
> >
> > Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> I know there has been controversy here. Julien - you seemed to agree,
> and iirc you partly drove how the patch is looking now. May I ask for
> an ack? Andrew - you disagreed for reasons that neither Julien nor I
> could really understand. Would you firmly nack the change and suggest
> a way out, or would you allow this to go in with someone else's ack?
I was mostly waiting on the discussion with Andrew to settle before 
reviewing. I can have a look now.
Cheers,

--
Julien Grall