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

2021-04-30 Thread Julien Grall
Hi Jan, On 29/04/2021 14:40, Jan Beulich wrote: On 29.04.2021 15:15, Julien Grall wrote: On 15/04/2021 10: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

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

2021-04-29 Thread Jan Beulich
On 29.04.2021 15:15, Julien Grall wrote: > On 15/04/2021 10: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(). > > Given the controversy of t

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

2021-04-29 Thread Julien Grall
Hi Jan, On 15/04/2021 10: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(). Given the controversy of the change, I would suggest to summarize why

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

2021-04-29 Thread Julien Grall
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

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

2021-04-29 Thread Jan Beulich
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 I know there has been controversy here. Jul

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

2021-04-15 Thread Jan Beulich
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 --- v3: Drop smp_wmb(). Re-base. v2: Defer allocation to when a domain actually switches to th