[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 02/29] NUMA: check if the total numa memory s
From: |
Hu Tao |
Subject: |
Re: [Qemu-devel] [PATCH v4 02/29] NUMA: check if the total numa memory size is equal to ram_size |
Date: |
Tue, 10 Jun 2014 10:29:01 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Jun 09, 2014 at 05:02:51PM -0600, Eric Blake wrote:
> On 06/09/2014 04:25 AM, Hu Tao wrote:
> > From: Wanlong Gao <address@hidden>
> >
> > If the total number of the assigned numa nodes memory is not
> > equal to the assigned ram size, it will write the wrong data
> > to ACPI table, then the guest will ignore the wrong ACPI table
> > and recognize all memory to one node. It's buggy, we should
> > check it to ensure that we write the right data to ACPI table.
> >
> > Signed-off-by: Wanlong Gao <address@hidden>
> > Reviewed-by: Eduardo Habkost <address@hidden>
> > Signed-off-by: Paolo Bonzini <address@hidden>
> > Signed-off-by: Hu Tao <address@hidden>
> > ---
> > numa.c | 14 ++++++++++++++
> > 1 file changed, 14 insertions(+)
> >
>
> > + if (numa_total != ram_size) {
> > + error_report("qemu: total memory size for NUMA nodes (%"
> > PRIu64 ")"
> > + " should equal to RAM size (" RAM_ADDR_FMT ")\n",
>
> error_report() should not include trailing \n
Thanks.
>
> > + numa_total, ram_size);
> > + exit(1);
>
> Not your fault that this file is full of exit(1), but it would be nice
> to have a cleanup patch someday that uses EXIT_FAILURE.
>
> --
> Eric Blake eblake redhat com +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
>
[Qemu-devel] [PATCH v4 01/29] NUMA: move numa related code to new file numa.c, Hu Tao, 2014/06/09
[Qemu-devel] [PATCH v4 03/29] NUMA: Add numa_info structure to contain numa nodes info, Hu Tao, 2014/06/09
[Qemu-devel] [PATCH v4 04/29] NUMA: convert -numa option to use OptsVisitor, Hu Tao, 2014/06/09