[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 1/2] qemu-options.hx: Mark all hmat-cache attributes requi
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v2 1/2] qemu-options.hx: Mark all hmat-cache attributes required |
Date: |
Mon, 15 Jun 2020 10:00:01 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Cc: the people involved in commit c412a48d4d "numa: Extend CLI to
provide memory side cache information".
Michal Privoznik <mprivozn@redhat.com> writes:
> The documentation to `-numa hmat-cache` says that @node-id, @size
> and @level are the only required attributes. The rest
> (@associativity, @policy and @line) is optional. Well, not quite
> - if I try to start QEMU with only the three required attributes
> defined the QAPI code is complaining about associativity missing.
Only because @associativity visited first.
> According to QAPI all attributes are required. Make the docs
> reflect that.
Correct.
> Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
> ---
> qemu-options.hx | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/qemu-options.hx b/qemu-options.hx
> index 93bde2bbc8..b1a399079a 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -188,7 +188,7 @@ DEF("numa", HAS_ARG, QEMU_OPTION_numa,
> "-numa dist,src=source,dst=destination,val=distance\n"
> "-numa cpu,node-id=node[,socket-id=x][,core-id=y][,thread-id=z]\n"
> "-numa
> hmat-lb,initiator=node,target=node,hierarchy=memory|first-level|second-level|third-level,data-type=access-latency|read-latency|write-latency[,latency=lat][,bandwidth=bw]\n"
> - "-numa
> hmat-cache,node-id=node,size=size,level=level[,associativity=none|direct|complex][,policy=none|write-back|write-through][,line=size]\n",
> + "-numa
> hmat-cache,node-id=node,size=size,level=level,associativity=none|direct|complex,policy=none|write-back|write-through,line=size\n",
> QEMU_ARCH_ALL)
> SRST
> ``-numa
> node[,mem=size][,cpus=firstcpu[-lastcpu]][,nodeid=node][,initiator=initiator]``
> @@ -201,7 +201,7 @@ SRST
> \
> ``-numa
> hmat-lb,initiator=node,target=node,hierarchy=hierarchy,data-type=tpye[,latency=lat][,bandwidth=bw]``
> \
> -``-numa
> hmat-cache,node-id=node,size=size,level=level[,associativity=str][,policy=str][,line=size]``
> +``-numa
> hmat-cache,node-id=node,size=size,level=level,associativity=str,policy=str,line=size``
> Define a NUMA node and assign RAM and VCPUs to it. Set the NUMA
> distance from a source node to a destination node. Set the ACPI
> Heterogeneous Memory Attributes for the given nodes.
Assuming non-optional is what we want:
Reviewed-by: Markus Armbruster <armbru@redhat.com>