Re: [PATCH v2 3/3] xen/heap: pass order to free_heap_pages() in heap init

[Julien Grall <julien@xxxxxxx>]

Hi Jan,

On 19/07/2022 07:01, Jan Beulich wrote:
> On 18.07.2022 19:39, Julien Grall wrote:
> > On 18/07/2022 12:02, Jan Beulich wrote:
> > > On 18.07.2022 12:24, Julien Grall wrote:
> > > 3)
> > >           unsigned int inc_order = min(MAX_ORDER, flsl(e - s) - 1);
> > >
> > >           if ( s )
> > >               inc_order = min(inc_order, ffsl(s) - 1U);
> > I like this idea!
> >
> > > No compilation issues to expect here, afaict.
> > Correct, GCC is happy with this approach. Assuming there are no other
> > comments, are you happy if I make the change on commit?
> Sure - iirc I gave my R-b already.
You did. Just wanted to make sure your reviewed-by was valid with your 
proposal.
I have now committed the series.

Cheers,

--
Julien Grall