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

Juergen Gross posted 4 patches 1 year, 10 months ago
Failed in applying to current master (apply log)
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(-)
[PATCH v3 0/4] mini-os: some memory map updates for PVH
Posted by Juergen Gross 1 year, 10 months ago
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(-)

-- 
2.35.3
Re: [PATCH v3 0/4] mini-os: some memory map updates for PVH
Posted by Juergen Gross 1 year, 10 months ago
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? Samuel gave his R-b nearly 2 weeks
ago.


Juergen
Re: [PATCH v3 0/4] mini-os: some memory map updates for PVH
Posted by Julien Grall 1 year, 10 months ago
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.

Cheers,

-- 
Julien Grall

Re: [PATCH v3 0/4] mini-os: some memory map updates for PVH
Posted by Juergen Gross 1 year, 10 months ago
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

Re: [PATCH v3 0/4] mini-os: some memory map updates for PVH
Posted by Jan Beulich 1 year, 10 months ago
On 04.07.2022 11:12, Juergen Gross wrote:
> 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...

And really, with no other indication so far, I've been skipping mini-os
patches on the assumption that Wei would be looking after them. Wei, if
you can't do so anymore, could you please explicitly indicate so?

Jan