[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 1/7] hw/i386: Improve some of the warning mes
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH v3 1/7] hw/i386: Improve some of the warning messages |
Date: |
Mon, 4 Sep 2017 11:17:48 -0300 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
On Fri, Sep 01, 2017 at 09:51:05AM -0700, Alistair Francis wrote:
[...]
> diff --git a/hw/i386/pc_q35.c b/hw/i386/pc_q35.c
> index 169a214d50..435eb2c458 100644
> --- a/hw/i386/pc_q35.c
> +++ b/hw/i386/pc_q35.c
> @@ -101,9 +101,11 @@ static void pc_q35_init(MachineState *machine)
> lowmem = pcms->max_ram_below_4g;
> if (machine->ram_size - lowmem > lowmem &&
> lowmem & ((1ULL << 30) - 1)) {
> - warn_report("Large machine and max_ram_below_4g(%"PRIu64
> - ") not a multiple of 1G; possible bad performance.",
> - pcms->max_ram_below_4g);
> + warn_report("Large machine as the ram size (0x%" PRIx64 ") is
> more"
> + " then twice the size of the internal limit"
> + " (0x%" PRIx64 ") and max-ram-below-4g (%"PRIu64")"
> + " note a multiple of 1G; possible bad performance.",
> + machine->ram_size, lowmem, pcms->max_ram_below_4g);
Here lowmem and max_ram_below_4g have exactly the same value,
don't they? There's no internal limit involved in this logic,
only max_ram_below_4g and ram_size.
--
Eduardo
- [Qemu-devel] [PATCH v3 0/7] More warning reporting fixed, Alistair Francis, 2017/09/01
- [Qemu-devel] [PATCH v3 1/7] hw/i386: Improve some of the warning messages, Alistair Francis, 2017/09/01
- Re: [Qemu-devel] [PATCH v3 1/7] hw/i386: Improve some of the warning messages,
Eduardo Habkost <=
- [Qemu-devel] [PATCH v3 2/7] Convert remaining error_report() to warn_report(), Alistair Francis, 2017/09/01
- [Qemu-devel] [PATCH v3 3/7] Convert single line fprintf(.../n) to warn_report(), Alistair Francis, 2017/09/01
- [Qemu-devel] [PATCH v3 5/7] General warn report fixups, Alistair Francis, 2017/09/01
- [Qemu-devel] [PATCH v3 6/7] Makefile: Remove libqemustub.a, Alistair Francis, 2017/09/01
- [Qemu-devel] [PATCH v3 7/7] Convert remaining single line fprintf() to warn_report(), Alistair Francis, 2017/09/01