Re: [PATCH v3 0/4] mini-os: some memory map updates for PVH

[Juergen Gross <jgross@xxxxxxxx>]

On 04.07.22 10:55, Julien Grall wrote:
> Hi Juergen,
>
> On 04/07/2022 09:38, Juergen Gross wrote:
> > On 21.06.22 09:23, Juergen Gross wrote:
> > > Do some memory map related changes/fixes for PVH mode:
> > >
> > > - Prefer the memory map delivered via start-info over the one obtained
> > >    from the hypervisor. This is a prerequisite for Xenstore-stubdom
> > >    live-update with rising the memory limit.
> > >
> > > - Fix a bug related to ballooning in PVH mode: PVH Xenstore-stubdom
> > >    can't read its target memory size from Xenstore, as this introduces
> > >    a chicken-and-egg problem. The memory size read from the hypervisor
> > >    OTOH includes additional "special" pages marked as reserved in the
> > >    memory map. Those pages need to be subtracted from the read size.
> > >
> > > - Fix a bug in ballooning code in PVH mode when using memory beyond
> > >    a RAM hole in the memory map
> > >
> > > Changes in V3:
> > > - minor comment for patch 3 addressed
> > >
> > > Changes in V2:
> > > - added patch 4
> > > - addressed comment regarding patch 3
> > >
> > > Juergen Gross (4):
> > >    mini-os: take newest version of arch-x86/hvm/start_info.h
> > >    mini-os: prefer memory map via start_info for PVH
> > >    mini-os: fix number of pages for PVH
> > >    mini-os: fix bug in ballooning on PVH
> > >
> > >   arch/x86/mm.c                         | 23 ++++----
> > >   balloon.c                             | 18 ++----
> > >   e820.c                                | 83 ++++++++++++++++++++++++---
> > >   include/e820.h                        |  6 ++
> > >   include/x86/arch_mm.h                 |  2 +
> > >   include/xen/arch-x86/hvm/start_info.h | 63 +++++++++++++++++++-
> > >   6 files changed, 163 insertions(+), 32 deletions(-)
> > Could someone please commit this series? 
> Done.
>
> > Samuel gave his R-b nearly 2 weeks
> > ago.
> xen-devel is quite high volume and I don't read all the e-mails. If you need 
> someone to commit then please CC committers@ and this will land in my inbox.
This is the reason why Wei wanted to be CC-ed for Mini-OS patches in the
past. He doesn't seem to be interested any more, unfortunately...


Juergen

Attachment: OpenPGP_0xB0DE9DD628BF132F.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature
Description: OpenPGP digital signature