On Tue, 30 Aug 2022, Henry Wang wrote: > Hi Michal, > > > -----Original Message----- > > From: Michal Orzel <michal.or...@amd.com> > > >>> > > >> Did you consider putting reserved_heap into bootinfo structure? > > > > > > Actually I did, but I saw current bootinfo only contains some structs so > > > I was not sure if this is the preferred way, but since you are raising > > > this > > > question, I will follow this method in v2. > > This is what I think would be better but maintainers will have a decisive > > vote. > > Then let's wait for more input from maintainers.
I don't have a strong preference and the way the current code is written, it would actually take less memory as is (the extra bool xen_heap comes for free.) I would keep the patch as is for now and for 4.17. If Julien prefers a refactoring of bootinfo/meminfo I think it could be done after the release if you are up to it.