[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] numa: Reject configuration if not all node
From: |
Hu Tao |
Subject: |
Re: [Qemu-devel] [PATCH 3/3] numa: Reject configuration if not all node IDs are present |
Date: |
Fri, 27 Jun 2014 11:33:59 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, Jun 26, 2014 at 06:33:20PM -0300, Eduardo Habkost wrote:
> We don't support sparse NUMA node IDs yet, so this changes QEMU to
> reject configs where not all nodes are present.
>
> Signed-off-by: Eduardo Habkost <address@hidden>
> ---
> numa.c | 17 ++++++++++++++++-
> 1 file changed, 16 insertions(+), 1 deletion(-)
>
> diff --git a/numa.c b/numa.c
> index e93407a..96fe5dd 100644
> --- a/numa.c
> +++ b/numa.c
> @@ -160,9 +160,24 @@ error:
>
> void set_numa_nodes(void)
> {
> + int i;
> +
> + assert(max_numa_nodeid <= MAX_NODES);
> +
> + /* No support for sparse NUMA node IDs yet: */
> + for (i = max_numa_nodeid - 1; i >= 0; i--) {
> + /* Report large node IDs first, to make mistakes easier to spot */
> + if (!numa_info[i].present) {
> + error_report("numa: Node ID missing: %d", i);
> + exit(1);
This reports only one missing node id. Dont' we report all missing ids?
> + }
> + }
> +
> + /* This must be always true if all nodes are present: */
> + assert(nb_numa_nodes == max_numa_nodeid);
> +
> if (nb_numa_nodes > 0) {
> uint64_t numa_total;
> - int i;
>
> if (nb_numa_nodes > MAX_NODES) {
> nb_numa_nodes = MAX_NODES;
> --
> 1.9.3
- [Qemu-devel] [PATCH 0/3] Validate NUMA node IDs (reject duplicate/missing node IDs properly), Eduardo Habkost, 2014/06/26
- [Qemu-devel] [PATCH 2/3] numa: Reject duplicate node IDs, Eduardo Habkost, 2014/06/26
- [Qemu-devel] [PATCH 1/3] numa: Keep track of NUMA nodes present on the command-line, Eduardo Habkost, 2014/06/26
- [Qemu-devel] [PATCH 3/3] numa: Reject configuration if not all node IDs are present, Eduardo Habkost, 2014/06/26
- Re: [Qemu-devel] [PATCH 3/3] numa: Reject configuration if not all node IDs are present,
Hu Tao <=
- Re: [Qemu-devel] [PATCH for 2.1 0/3] Validate NUMA node IDs (reject duplicate/missing node IDs properly), Eric Blake, 2014/06/26
- Re: [Qemu-devel] [PATCH 0/3] Validate NUMA node IDs (reject duplicate/missing node IDs properly), Michael S. Tsirkin, 2014/06/29