* Re: [edk2-devel] [PATCH V2 1/1] OvmfPkg/OvmfPkg*.dsc: Increase ACPI Reclaim memory size
[not found] <16E62E6A940424A8.9469@groups.io>
@ 2022-05-02 21:05 ` annie li
[not found] ` <16EB65B9C6B20F3F.8534@groups.io>
1 sibling, 0 replies; 6+ messages in thread
From: annie li @ 2022-05-02 21:05 UTC (permalink / raw)
To: devel; +Cc: lersek, karl.heubaum, aaron.young
Hello
I am wondering if anyone will review this patch please? Any comments are
welcome.
Thanks
Annie
On 4/15/2022 5:15 PM, annie li wrote:
> The current ACPI Reclaim memory size is set as 0x10(64k). The
> ACPI table size will be increased if the memory slots' number of
> the guest gets increased. In the guest with more memory slots,
> the ACPI Reclaim memory size may not be sufficient for hibernation.
> This may cause resume failure of the hibernated guest that was
> booted up with a fresh copied writable OVMF_VARS file. However,
> the failure doesn't happen in following hibernation/resume
> cycles.
>
> The ACPI_MAX_RAM_SLOTS is set as 256 in the current QEMU. With
> ACPI_MAX_RAM_SLOTS, 18 pages are required to be allocated in ACPI
> Reclaim memory. However, due to the 0x10(16 pages) setting, 2 extra
> pages will be allocated in other space. This may break the
> hibernation/resume in the above scenario.
>
> This patch increases the ACPI Reclaim memory size to 0x12, i.e.
> PcdMemoryTypeEfiACPIReclaimMemory is set as 0x12(18 pages).
>
> Signed-off-by: Annie Li <annie.li@oracle.com>
> ---
> OvmfPkg/OvmfPkgIa32.dsc | 2 +-
> OvmfPkg/OvmfPkgIa32X64.dsc | 2 +-
> OvmfPkg/OvmfPkgX64.dsc | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc
> index 29eea82571..fb158f98c9 100644
> --- a/OvmfPkg/OvmfPkgIa32.dsc
> +++ b/OvmfPkg/OvmfPkgIa32.dsc
> @@ -547,7 +547,7 @@
> # unknown) workloads / boot paths.
>
> #
>
> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
>
> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
>
> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
>
> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
>
> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
>
> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
>
> diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
> index 56d3c49ab2..30d47ca28a 100644
> --- a/OvmfPkg/OvmfPkgIa32X64.dsc
> +++ b/OvmfPkg/OvmfPkgIa32X64.dsc
> @@ -552,7 +552,7 @@
> # unknown) workloads / boot paths.
>
> #
>
> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
>
> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
>
> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
>
> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
>
> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
>
> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
>
> diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc
> index f0924c0f9d..6c10ceedd7 100644
> --- a/OvmfPkg/OvmfPkgX64.dsc
> +++ b/OvmfPkg/OvmfPkgX64.dsc
> @@ -552,7 +552,7 @@
> # unknown) workloads / boot paths.
>
> #
>
> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
>
> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
>
> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
>
> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
>
> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
>
> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH V2 1/1] OvmfPkg/OvmfPkg*.dsc: Increase ACPI Reclaim memory size
[not found] ` <16EB65B9C6B20F3F.8534@groups.io>
@ 2022-05-03 15:54 ` annie li
2022-05-03 16:25 ` Ard Biesheuvel
0 siblings, 1 reply; 6+ messages in thread
From: annie li @ 2022-05-03 15:54 UTC (permalink / raw)
To: devel
Cc: ardb+tianocore, jiewen.yao, jordan.l.justen,
kraxel@redhat.com >> Gerd Hoffmann, Karl Franz Heubaum,
Aaron Young
Adding maintainers into the CC list...
On 5/2/2022 5:05 PM, annie li wrote:
> Hello
>
> I am wondering if anyone will review this patch please? Any comments
> are welcome.
>
> Thanks
>
> Annie
>
> On 4/15/2022 5:15 PM, annie li wrote:
>> The current ACPI Reclaim memory size is set as 0x10(64k). The
>> ACPI table size will be increased if the memory slots' number of
>> the guest gets increased. In the guest with more memory slots,
>> the ACPI Reclaim memory size may not be sufficient for hibernation.
>> This may cause resume failure of the hibernated guest that was
>> booted up with a fresh copied writable OVMF_VARS file. However,
>> the failure doesn't happen in following hibernation/resume
>> cycles.
>>
>> The ACPI_MAX_RAM_SLOTS is set as 256 in the current QEMU. With
>> ACPI_MAX_RAM_SLOTS, 18 pages are required to be allocated in ACPI
>> Reclaim memory. However, due to the 0x10(16 pages) setting, 2 extra
>> pages will be allocated in other space. This may break the
>> hibernation/resume in the above scenario.
>>
>> This patch increases the ACPI Reclaim memory size to 0x12, i.e.
>> PcdMemoryTypeEfiACPIReclaimMemory is set as 0x12(18 pages).
>>
>> Signed-off-by: Annie Li <annie.li@oracle.com>
>> ---
>> OvmfPkg/OvmfPkgIa32.dsc | 2 +-
>> OvmfPkg/OvmfPkgIa32X64.dsc | 2 +-
>> OvmfPkg/OvmfPkgX64.dsc | 2 +-
>> 3 files changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc
>> index 29eea82571..fb158f98c9 100644
>> --- a/OvmfPkg/OvmfPkgIa32.dsc
>> +++ b/OvmfPkg/OvmfPkgIa32.dsc
>> @@ -547,7 +547,7 @@
>> # unknown) workloads / boot paths.
>>
>> #
>>
>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
>>
>> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
>>
>> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
>>
>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
>>
>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
>>
>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
>>
>> diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
>> index 56d3c49ab2..30d47ca28a 100644
>> --- a/OvmfPkg/OvmfPkgIa32X64.dsc
>> +++ b/OvmfPkg/OvmfPkgIa32X64.dsc
>> @@ -552,7 +552,7 @@
>> # unknown) workloads / boot paths.
>>
>> #
>>
>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
>>
>> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
>>
>> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
>>
>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
>>
>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
>>
>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
>>
>> diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc
>> index f0924c0f9d..6c10ceedd7 100644
>> --- a/OvmfPkg/OvmfPkgX64.dsc
>> +++ b/OvmfPkg/OvmfPkgX64.dsc
>> @@ -552,7 +552,7 @@
>> # unknown) workloads / boot paths.
>>
>> #
>>
>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
>>
>> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
>>
>> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
>>
>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
>>
>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
>>
>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
>>
>
>
>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH V2 1/1] OvmfPkg/OvmfPkg*.dsc: Increase ACPI Reclaim memory size
2022-05-03 15:54 ` annie li
@ 2022-05-03 16:25 ` Ard Biesheuvel
2022-05-13 11:28 ` annie li
0 siblings, 1 reply; 6+ messages in thread
From: Ard Biesheuvel @ 2022-05-03 16:25 UTC (permalink / raw)
To: edk2-devel-groups-io, annie.li
Cc: Ard Biesheuvel, Jiewen Yao, Jordan Justen,
kraxel@redhat.com >> Gerd Hoffmann, Karl Franz Heubaum,
Aaron Young
On Tue, 3 May 2022 at 17:54, annie li <annie.li@oracle.com> wrote:
>
> Adding maintainers into the CC list...
>
> On 5/2/2022 5:05 PM, annie li wrote:
> > Hello
> >
> > I am wondering if anyone will review this patch please? Any comments
> > are welcome.
> >
> > Thanks
> >
> > Annie
> >
> > On 4/15/2022 5:15 PM, annie li wrote:
> >> The current ACPI Reclaim memory size is set as 0x10(64k). The
> >> ACPI table size will be increased if the memory slots' number of
> >> the guest gets increased. In the guest with more memory slots,
> >> the ACPI Reclaim memory size may not be sufficient for hibernation.
> >> This may cause resume failure of the hibernated guest that was
> >> booted up with a fresh copied writable OVMF_VARS file. However,
> >> the failure doesn't happen in following hibernation/resume
> >> cycles.
> >>
> >> The ACPI_MAX_RAM_SLOTS is set as 256 in the current QEMU. With
> >> ACPI_MAX_RAM_SLOTS, 18 pages are required to be allocated in ACPI
> >> Reclaim memory. However, due to the 0x10(16 pages) setting, 2 extra
> >> pages will be allocated in other space. This may break the
> >> hibernation/resume in the above scenario.
> >>
> >> This patch increases the ACPI Reclaim memory size to 0x12, i.e.
> >> PcdMemoryTypeEfiACPIReclaimMemory is set as 0x12(18 pages).
> >>
> >> Signed-off-by: Annie Li <annie.li@oracle.com>
> >> ---
> >> OvmfPkg/OvmfPkgIa32.dsc | 2 +-
> >> OvmfPkg/OvmfPkgIa32X64.dsc | 2 +-
> >> OvmfPkg/OvmfPkgX64.dsc | 2 +-
> >> 3 files changed, 3 insertions(+), 3 deletions(-)
> >>
No objections from me but before merging this, I'll give the other
folks the opportunity to chime in as well.
> >> diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc
> >> index 29eea82571..fb158f98c9 100644
> >> --- a/OvmfPkg/OvmfPkgIa32.dsc
> >> +++ b/OvmfPkg/OvmfPkgIa32.dsc
> >> @@ -547,7 +547,7 @@
> >> # unknown) workloads / boot paths.
> >>
> >> #
> >>
> >> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
> >>
> >> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
> >>
> >> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
> >>
> >> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
> >>
> >> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
> >>
> >> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
> >>
> >> diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
> >> index 56d3c49ab2..30d47ca28a 100644
> >> --- a/OvmfPkg/OvmfPkgIa32X64.dsc
> >> +++ b/OvmfPkg/OvmfPkgIa32X64.dsc
> >> @@ -552,7 +552,7 @@
> >> # unknown) workloads / boot paths.
> >>
> >> #
> >>
> >> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
> >>
> >> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
> >>
> >> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
> >>
> >> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
> >>
> >> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
> >>
> >> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
> >>
> >> diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc
> >> index f0924c0f9d..6c10ceedd7 100644
> >> --- a/OvmfPkg/OvmfPkgX64.dsc
> >> +++ b/OvmfPkg/OvmfPkgX64.dsc
> >> @@ -552,7 +552,7 @@
> >> # unknown) workloads / boot paths.
> >>
> >> #
> >>
> >> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
> >>
> >> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
> >>
> >> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
> >>
> >> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
> >>
> >> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
> >>
> >> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
> >>
> >
> >
> >
> >
> >
>
>
>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH V2 1/1] OvmfPkg/OvmfPkg*.dsc: Increase ACPI Reclaim memory size
2022-05-03 16:25 ` Ard Biesheuvel
@ 2022-05-13 11:28 ` annie li
2022-07-22 12:42 ` annie li
0 siblings, 1 reply; 6+ messages in thread
From: annie li @ 2022-05-13 11:28 UTC (permalink / raw)
To: Ard Biesheuvel, edk2-devel-groups-io
Cc: Ard Biesheuvel, Jiewen Yao, Jordan Justen,
kraxel@redhat.com >> Gerd Hoffmann, Karl Franz Heubaum,
Aaron Young
On 5/3/2022 12:25 PM, Ard Biesheuvel wrote:
> On Tue, 3 May 2022 at 17:54, annie li <annie.li@oracle.com> wrote:
>> Adding maintainers into the CC list...
>>
>> On 5/2/2022 5:05 PM, annie li wrote:
>>> Hello
>>>
>>> I am wondering if anyone will review this patch please? Any comments
>>> are welcome.
>>>
>>> Thanks
>>>
>>> Annie
>>>
>>> On 4/15/2022 5:15 PM, annie li wrote:
>>>> The current ACPI Reclaim memory size is set as 0x10(64k). The
>>>> ACPI table size will be increased if the memory slots' number of
>>>> the guest gets increased. In the guest with more memory slots,
>>>> the ACPI Reclaim memory size may not be sufficient for hibernation.
>>>> This may cause resume failure of the hibernated guest that was
>>>> booted up with a fresh copied writable OVMF_VARS file. However,
>>>> the failure doesn't happen in following hibernation/resume
>>>> cycles.
>>>>
>>>> The ACPI_MAX_RAM_SLOTS is set as 256 in the current QEMU. With
>>>> ACPI_MAX_RAM_SLOTS, 18 pages are required to be allocated in ACPI
>>>> Reclaim memory. However, due to the 0x10(16 pages) setting, 2 extra
>>>> pages will be allocated in other space. This may break the
>>>> hibernation/resume in the above scenario.
>>>>
>>>> This patch increases the ACPI Reclaim memory size to 0x12, i.e.
>>>> PcdMemoryTypeEfiACPIReclaimMemory is set as 0x12(18 pages).
>>>>
>>>> Signed-off-by: Annie Li <annie.li@oracle.com>
>>>> ---
>>>> OvmfPkg/OvmfPkgIa32.dsc | 2 +-
>>>> OvmfPkg/OvmfPkgIa32X64.dsc | 2 +-
>>>> OvmfPkg/OvmfPkgX64.dsc | 2 +-
>>>> 3 files changed, 3 insertions(+), 3 deletions(-)
>>>>
> No objections from me but before merging this, I'll give the other
> folks the opportunity to chime in as well.
Thank you Ard for reviewing this.
Could anyone else please review this patch? thank you!
Thanks
Annie
>
>>>> diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc
>>>> index 29eea82571..fb158f98c9 100644
>>>> --- a/OvmfPkg/OvmfPkgIa32.dsc
>>>> +++ b/OvmfPkg/OvmfPkgIa32.dsc
>>>> @@ -547,7 +547,7 @@
>>>> # unknown) workloads / boot paths.
>>>>
>>>> #
>>>>
>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
>>>>
>>>> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
>>>>
>>>> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
>>>>
>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
>>>>
>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
>>>>
>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
>>>>
>>>> diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
>>>> index 56d3c49ab2..30d47ca28a 100644
>>>> --- a/OvmfPkg/OvmfPkgIa32X64.dsc
>>>> +++ b/OvmfPkg/OvmfPkgIa32X64.dsc
>>>> @@ -552,7 +552,7 @@
>>>> # unknown) workloads / boot paths.
>>>>
>>>> #
>>>>
>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
>>>>
>>>> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
>>>>
>>>> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
>>>>
>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
>>>>
>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
>>>>
>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
>>>>
>>>> diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc
>>>> index f0924c0f9d..6c10ceedd7 100644
>>>> --- a/OvmfPkg/OvmfPkgX64.dsc
>>>> +++ b/OvmfPkg/OvmfPkgX64.dsc
>>>> @@ -552,7 +552,7 @@
>>>> # unknown) workloads / boot paths.
>>>>
>>>> #
>>>>
>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
>>>>
>>>> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
>>>>
>>>> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
>>>>
>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
>>>>
>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
>>>>
>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
>>>>
>>>
>>>
>>>
>>>
>>
>>
>>
>>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH V2 1/1] OvmfPkg/OvmfPkg*.dsc: Increase ACPI Reclaim memory size
2022-05-13 11:28 ` annie li
@ 2022-07-22 12:42 ` annie li
2022-09-06 16:56 ` Ard Biesheuvel
0 siblings, 1 reply; 6+ messages in thread
From: annie li @ 2022-07-22 12:42 UTC (permalink / raw)
To: Ard Biesheuvel, edk2-devel-groups-io
Cc: Ard Biesheuvel, Jiewen Yao, Jordan Justen,
kraxel@redhat.com >> Gerd Hoffmann, Karl Franz Heubaum,
Aaron Young
Hello Ard,
Looks there hasn't any feedback from other maintainers yet.
Would you like to merge this patch in or still wait for more feedback?
Could anyone else please review this patch? thank you!
Thanks
Annie
On 5/13/2022 7:28 AM, Annie.li wrote:
> On 5/3/2022 12:25 PM, Ard Biesheuvel wrote:
>> On Tue, 3 May 2022 at 17:54, annie li <annie.li@oracle.com> wrote:
>>> Adding maintainers into the CC list...
>>>
>>> On 5/2/2022 5:05 PM, annie li wrote:
>>>> Hello
>>>>
>>>> I am wondering if anyone will review this patch please? Any comments
>>>> are welcome.
>>>>
>>>> Thanks
>>>>
>>>> Annie
>>>>
>>>> On 4/15/2022 5:15 PM, annie li wrote:
>>>>> The current ACPI Reclaim memory size is set as 0x10(64k). The
>>>>> ACPI table size will be increased if the memory slots' number of
>>>>> the guest gets increased. In the guest with more memory slots,
>>>>> the ACPI Reclaim memory size may not be sufficient for hibernation.
>>>>> This may cause resume failure of the hibernated guest that was
>>>>> booted up with a fresh copied writable OVMF_VARS file. However,
>>>>> the failure doesn't happen in following hibernation/resume
>>>>> cycles.
>>>>>
>>>>> The ACPI_MAX_RAM_SLOTS is set as 256 in the current QEMU. With
>>>>> ACPI_MAX_RAM_SLOTS, 18 pages are required to be allocated in ACPI
>>>>> Reclaim memory. However, due to the 0x10(16 pages) setting, 2 extra
>>>>> pages will be allocated in other space. This may break the
>>>>> hibernation/resume in the above scenario.
>>>>>
>>>>> This patch increases the ACPI Reclaim memory size to 0x12, i.e.
>>>>> PcdMemoryTypeEfiACPIReclaimMemory is set as 0x12(18 pages).
>>>>>
>>>>> Signed-off-by: Annie Li <annie.li@oracle.com>
>>>>> ---
>>>>> OvmfPkg/OvmfPkgIa32.dsc | 2 +-
>>>>> OvmfPkg/OvmfPkgIa32X64.dsc | 2 +-
>>>>> OvmfPkg/OvmfPkgX64.dsc | 2 +-
>>>>> 3 files changed, 3 insertions(+), 3 deletions(-)
>>>>>
>> No objections from me but before merging this, I'll give the other
>> folks the opportunity to chime in as well.
>
> Thank you Ard for reviewing this.
>
> Could anyone else please review this patch? thank you!
>
> Thanks
>
> Annie
>
>>
>>>>> diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc
>>>>> index 29eea82571..fb158f98c9 100644
>>>>> --- a/OvmfPkg/OvmfPkgIa32.dsc
>>>>> +++ b/OvmfPkg/OvmfPkgIa32.dsc
>>>>> @@ -547,7 +547,7 @@
>>>>> # unknown) workloads / boot paths.
>>>>>
>>>>> #
>>>>>
>>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
>>>>>
>>>>> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
>>>>>
>>>>> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
>>>>>
>>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
>>>>>
>>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
>>>>>
>>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
>>>>>
>>>>> diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
>>>>> index 56d3c49ab2..30d47ca28a 100644
>>>>> --- a/OvmfPkg/OvmfPkgIa32X64.dsc
>>>>> +++ b/OvmfPkg/OvmfPkgIa32X64.dsc
>>>>> @@ -552,7 +552,7 @@
>>>>> # unknown) workloads / boot paths.
>>>>>
>>>>> #
>>>>>
>>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
>>>>>
>>>>> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
>>>>>
>>>>> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
>>>>>
>>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
>>>>>
>>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
>>>>>
>>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
>>>>>
>>>>> diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc
>>>>> index f0924c0f9d..6c10ceedd7 100644
>>>>> --- a/OvmfPkg/OvmfPkgX64.dsc
>>>>> +++ b/OvmfPkg/OvmfPkgX64.dsc
>>>>> @@ -552,7 +552,7 @@
>>>>> # unknown) workloads / boot paths.
>>>>>
>>>>> #
>>>>>
>>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
>>>>>
>>>>> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
>>>>>
>>>>> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
>>>>>
>>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
>>>>>
>>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
>>>>>
>>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
>>>>>
>>>>
>>>>
>>>>
>>>>
>>>
>>>
>>>
>>>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH V2 1/1] OvmfPkg/OvmfPkg*.dsc: Increase ACPI Reclaim memory size
2022-07-22 12:42 ` annie li
@ 2022-09-06 16:56 ` Ard Biesheuvel
0 siblings, 0 replies; 6+ messages in thread
From: Ard Biesheuvel @ 2022-09-06 16:56 UTC (permalink / raw)
To: devel, annie.li
Cc: Jiewen Yao, Jordan Justen,
kraxel@redhat.com >> Gerd Hoffmann, Karl Franz Heubaum,
Aaron Young
On Fri, 22 Jul 2022 at 14:43, annie li <annie.li@oracle.com> wrote:
>
> Hello Ard,
>
> Looks there hasn't any feedback from other maintainers yet.
>
> Would you like to merge this patch in or still wait for more feedback?
>
> Could anyone else please review this patch? thank you!
>
Reviewed-by: Ard Biesheuvel <ardb@kernel.org>
Merged as #3298
> On 5/13/2022 7:28 AM, Annie.li wrote:
> > On 5/3/2022 12:25 PM, Ard Biesheuvel wrote:
> >> On Tue, 3 May 2022 at 17:54, annie li <annie.li@oracle.com> wrote:
> >>> Adding maintainers into the CC list...
> >>>
> >>> On 5/2/2022 5:05 PM, annie li wrote:
> >>>> Hello
> >>>>
> >>>> I am wondering if anyone will review this patch please? Any comments
> >>>> are welcome.
> >>>>
> >>>> Thanks
> >>>>
> >>>> Annie
> >>>>
> >>>> On 4/15/2022 5:15 PM, annie li wrote:
> >>>>> The current ACPI Reclaim memory size is set as 0x10(64k). The
> >>>>> ACPI table size will be increased if the memory slots' number of
> >>>>> the guest gets increased. In the guest with more memory slots,
> >>>>> the ACPI Reclaim memory size may not be sufficient for hibernation.
> >>>>> This may cause resume failure of the hibernated guest that was
> >>>>> booted up with a fresh copied writable OVMF_VARS file. However,
> >>>>> the failure doesn't happen in following hibernation/resume
> >>>>> cycles.
> >>>>>
> >>>>> The ACPI_MAX_RAM_SLOTS is set as 256 in the current QEMU. With
> >>>>> ACPI_MAX_RAM_SLOTS, 18 pages are required to be allocated in ACPI
> >>>>> Reclaim memory. However, due to the 0x10(16 pages) setting, 2 extra
> >>>>> pages will be allocated in other space. This may break the
> >>>>> hibernation/resume in the above scenario.
> >>>>>
> >>>>> This patch increases the ACPI Reclaim memory size to 0x12, i.e.
> >>>>> PcdMemoryTypeEfiACPIReclaimMemory is set as 0x12(18 pages).
> >>>>>
> >>>>> Signed-off-by: Annie Li <annie.li@oracle.com>
> >>>>> ---
> >>>>> OvmfPkg/OvmfPkgIa32.dsc | 2 +-
> >>>>> OvmfPkg/OvmfPkgIa32X64.dsc | 2 +-
> >>>>> OvmfPkg/OvmfPkgX64.dsc | 2 +-
> >>>>> 3 files changed, 3 insertions(+), 3 deletions(-)
> >>>>>
> >> No objections from me but before merging this, I'll give the other
> >> folks the opportunity to chime in as well.
> >
> > Thank you Ard for reviewing this.
> >
> > Could anyone else please review this patch? thank you!
> >
> > Thanks
> >
> > Annie
> >
> >>
> >>>>> diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc
> >>>>> index 29eea82571..fb158f98c9 100644
> >>>>> --- a/OvmfPkg/OvmfPkgIa32.dsc
> >>>>> +++ b/OvmfPkg/OvmfPkgIa32.dsc
> >>>>> @@ -547,7 +547,7 @@
> >>>>> # unknown) workloads / boot paths.
> >>>>>
> >>>>> #
> >>>>>
> >>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
> >>>>>
> >>>>> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
> >>>>>
> >>>>> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
> >>>>>
> >>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
> >>>>>
> >>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
> >>>>>
> >>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
> >>>>>
> >>>>> diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
> >>>>> index 56d3c49ab2..30d47ca28a 100644
> >>>>> --- a/OvmfPkg/OvmfPkgIa32X64.dsc
> >>>>> +++ b/OvmfPkg/OvmfPkgIa32X64.dsc
> >>>>> @@ -552,7 +552,7 @@
> >>>>> # unknown) workloads / boot paths.
> >>>>>
> >>>>> #
> >>>>>
> >>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
> >>>>>
> >>>>> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
> >>>>>
> >>>>> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
> >>>>>
> >>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
> >>>>>
> >>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
> >>>>>
> >>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
> >>>>>
> >>>>> diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc
> >>>>> index f0924c0f9d..6c10ceedd7 100644
> >>>>> --- a/OvmfPkg/OvmfPkgX64.dsc
> >>>>> +++ b/OvmfPkg/OvmfPkgX64.dsc
> >>>>> @@ -552,7 +552,7 @@
> >>>>> # unknown) workloads / boot paths.
> >>>>>
> >>>>> #
> >>>>>
> >>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIMemoryNVS|0x80
> >>>>>
> >>>>> - gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x10
> >>>>>
> >>>>> + gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiACPIReclaimMemory|0x12
> >>>>>
> >>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiReservedMemoryType|0x80
> >>>>>
> >>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesCode|0x100
> >>>>>
> >>>>> gEmbeddedTokenSpaceGuid.PcdMemoryTypeEfiRuntimeServicesData|0x100
> >>>>>
> >>>>
> >>>>
> >>>>
> >>>>
> >>>
> >>>
> >>>
> >>>
>
>
>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-09-06 16:56 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <16E62E6A940424A8.9469@groups.io>
2022-05-02 21:05 ` [edk2-devel] [PATCH V2 1/1] OvmfPkg/OvmfPkg*.dsc: Increase ACPI Reclaim memory size annie li
[not found] ` <16EB65B9C6B20F3F.8534@groups.io>
2022-05-03 15:54 ` annie li
2022-05-03 16:25 ` Ard Biesheuvel
2022-05-13 11:28 ` annie li
2022-07-22 12:42 ` annie li
2022-09-06 16:56 ` Ard Biesheuvel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox