Re: [PATCH v2 8/8] target/i386/cpu: Use the IEC binary prefix definitions

[Laurent Vivier]

Le 01/06/2020 à 16:29, Philippe Mathieu-Daudé a écrit :
> IEC binary prefixes ease code review: the unit is explicit.
> 
> Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
>  target/i386/cpu.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/target/i386/cpu.c b/target/i386/cpu.c
> index 3733d9a279..33ce4861fb 100644
> --- a/target/i386/cpu.c
> +++ b/target/i386/cpu.c
> @@ -6159,7 +6159,7 @@ static void x86_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, 1ull << 32);
> +                                 smram, 0, 4 * GiB);
>          memory_region_set_enabled(cpu->smram, true);
>          memory_region_add_subregion_overlap(cpu->cpu_as_root, 0, cpu->smram, 
> 1);
>      }
> 

Applied to my trivial-patches branch.

Thanks,
Laurent