[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 02/11] numa: move numa global variable nb_num
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH v7 02/11] numa: move numa global variable nb_numa_nodes into MachineState |
Date: |
Tue, 23 Jul 2019 12:23:57 -0300 |
On Tue, Jul 23, 2019 at 04:56:41PM +0200, Igor Mammedov wrote:
> On Tue, 16 Jul 2019 22:51:12 +0800
> Tao Xu <address@hidden> wrote:
>
> > Add struct NumaState in MachineState and move existing numa global
> > nb_numa_nodes(renamed as "num_nodes") into NumaState. And add variable
> > numa_support into MachineClass to decide which submachines support NUMA.
> >
> > Suggested-by: Igor Mammedov <address@hidden>
> > Suggested-by: Eduardo Habkost <address@hidden>
> > Signed-off-by: Tao Xu <address@hidden>
> > ---
> >
> > No changes in v7.
> >
> > Changes in v6:
> > - Rebase to upstream, move globals in arm/sbsa-ref and use
> > numa_mem_supported
> > - When used once or twice in the function, use
> > ms->numa_state->num_nodes directly
> > - Correct some mistakes
> > - Use once monitor_printf in hmp_info_numa
> > ---
[...]
> > if (pxb->numa_node != NUMA_NODE_UNASSIGNED &&
> > - pxb->numa_node >= nb_numa_nodes) {
> > + pxb->numa_node >= ms->numa_state->num_nodes) {
> this will crash if user tries to use device on machine that doesn't support
> numa
> check that numa_state is not NULL before dereferencing
That's exactly why the machine_num_numa_nodes() was created in
v5, but then you asked for its removal.
--
Eduardo
- [Qemu-devel] [PATCH v7 00/11] Build ACPI Heterogeneous Memory Attribute Table (HMAT), Tao Xu, 2019/07/16
- [Qemu-devel] [PATCH v7 03/11] numa: move numa global variable have_numa_distance into MachineState, Tao Xu, 2019/07/16
- [Qemu-devel] [PATCH v7 01/11] hw/arm: simplify arm_load_dtb, Tao Xu, 2019/07/16
- [Qemu-devel] [PATCH v7 02/11] numa: move numa global variable nb_numa_nodes into MachineState, Tao Xu, 2019/07/16
- Re: [Qemu-devel] [PATCH v7 02/11] numa: move numa global variable nb_numa_nodes into MachineState, Igor Mammedov, 2019/07/23
- Re: [Qemu-devel] [PATCH v7 02/11] numa: move numa global variable nb_numa_nodes into MachineState,
Eduardo Habkost <=
- Re: [Qemu-devel] [PATCH v7 02/11] numa: move numa global variable nb_numa_nodes into MachineState, Igor Mammedov, 2019/07/24
- Re: [Qemu-devel] [PATCH v7 02/11] numa: move numa global variable nb_numa_nodes into MachineState, Eduardo Habkost, 2019/07/24
- Re: [Qemu-devel] [PATCH v7 02/11] numa: move numa global variable nb_numa_nodes into MachineState, Igor Mammedov, 2019/07/24
- Re: [Qemu-devel] [PATCH v7 02/11] numa: move numa global variable nb_numa_nodes into MachineState, Eduardo Habkost, 2019/07/24
- Re: [Qemu-devel] [PATCH v7 02/11] numa: move numa global variable nb_numa_nodes into MachineState, Igor Mammedov, 2019/07/26
- Re: [Qemu-devel] [PATCH v7 02/11] numa: move numa global variable nb_numa_nodes into MachineState, Eduardo Habkost, 2019/07/26
- Re: [Qemu-devel] [PATCH v7 02/11] numa: move numa global variable nb_numa_nodes into MachineState, Tao Xu, 2019/07/29
[Qemu-devel] [PATCH v7 04/11] numa: move numa global variable numa_info into MachineState, Tao Xu, 2019/07/16
[Qemu-devel] [PATCH v7 05/11] numa: Extend CLI to provide initiator information for numa nodes, Tao Xu, 2019/07/16