[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 9/9] target/i386/tcg/sysemu/tcg-cpu: Avoid own opinion abo
From: |
Bernhard Beschow |
Subject: |
Re: [PATCH v3 9/9] target/i386/tcg/sysemu/tcg-cpu: Avoid own opinion about smram size |
Date: |
Tue, 07 Feb 2023 22:46:19 +0000 |
Am 5. Februar 2023 11:26:10 UTC schrieb "Philippe Mathieu-Daudé"
<philmd@linaro.org>:
>On 4/2/23 16:10, Bernhard Beschow wrote:
>> When setting up the CPU's smram memory region alias, the code currently
>> assumes that the smram size is 4 GiB. While this is true, it repeats a
>> decision made elsewhere which seems redundant and prone to
>> inconsistencies. Avoid this by reusing whatever size the smram region
>> was set to.
>>
>> Signed-off-by: Bernhard Beschow <shentey@gmail.com>
>> ---
>> target/i386/tcg/sysemu/tcg-cpu.c | 3 +--
>> 1 file changed, 1 insertion(+), 2 deletions(-)
>>
>> diff --git a/target/i386/tcg/sysemu/tcg-cpu.c
>> b/target/i386/tcg/sysemu/tcg-cpu.c
>> index c223c0fe9b..8f5ce6093c 100644
>> --- a/target/i386/tcg/sysemu/tcg-cpu.c
>> +++ b/target/i386/tcg/sysemu/tcg-cpu.c
>> @@ -22,7 +22,6 @@
>> #include "tcg/helper-tcg.h"
>> #include "sysemu/sysemu.h"
>> -#include "qemu/units.h"
>> #include "exec/address-spaces.h"
>> #include "tcg/tcg-cpu.h"
>> @@ -36,7 +35,7 @@ static void tcg_cpu_machine_done(Notifier *n, void *unused)
>> if (smram) {
>> cpu->smram = g_new(MemoryRegion, 1);
>> memory_region_init_alias(cpu->smram, OBJECT(cpu), "smram",
>> - smram, 0, 4 * GiB);
>> + smram, 0, memory_region_size(smram));
>
>Or define SMRAM_REGION_SIZE and use it?
This could still lead to inconsistencies if the size was changed in one place
only, no?
>
>(subject header can be shortened to "target/i386:").
Okay.
- [PATCH v3 4/9] hw/i386/pc_{q35, piix}: Reuse MachineClass::desc as SMB product name, (continued)
- [PATCH v3 4/9] hw/i386/pc_{q35, piix}: Reuse MachineClass::desc as SMB product name, Bernhard Beschow, 2023/02/04
- [PATCH v3 5/9] hw/i386/pc_{q35, piix}: Minimize usage of get_system_memory(), Bernhard Beschow, 2023/02/04
- [PATCH v3 6/9] hw/i386/pc: Initialize ram_memory variable directly, Bernhard Beschow, 2023/02/04
- [PATCH v3 7/9] hw/pci-host/pam: Make init_pam() usage more readable, Bernhard Beschow, 2023/02/04
- [PATCH v3 9/9] target/i386/tcg/sysemu/tcg-cpu: Avoid own opinion about smram size, Bernhard Beschow, 2023/02/04
- [PATCH v3 8/9] hw/i386/x86: Make TYPE_X86_MACHINE the owner of smram, Bernhard Beschow, 2023/02/04