Re: [Qemu-devel] [PATCH v5 0/8] Build ACPI Heterogeneous Memory Attribute Table (HMAT)

2019-07-01 Thread Tao Xu

On 7/1/2019 9:37 PM, Igor Mammedov wrote:

On Fri, 14 Jun 2019 23:56:18 +0800
Tao Xu  wrote:


This series of patches will build Heterogeneous Memory Attribute Table (HMAT)
according to the command line. The ACPI HMAT describes the memory attributes,
such as memory side cache attributes and bandwidth and latency details,
related to the System Physical Address (SPA) Memory Ranges.
The software is expected to use this information as hint for optimization.


in addition to patches in this series. pls consider adding testcase for ACPI 
table
as the last patch. Look at tests/bios-tables-test.c for examples.



OK, I will add it.


The V4 patches link:
https://lists.gnu.org/archive/html/qemu-devel/2019-05/msg01644.html

Changelog:
v5:
 - spilt the 1-6/11 and 8/11 from patch v4 to build Memory Subsystem
 Address Range Structure(s) and System Locality Latency and Bandwidth
 Information Structure(s) firstly.
 - add 1/8 of patch v5 to simplify arm_load_dtb() (Igor)
 - drop the helper machine_num_numa_nodes() and use
 machine->numa_state->num_nodes (and numa_state->nodes) directly (Igor)
 - Add more descriptions from ACPI spec (Igor)
 - Add the reason of using stub (Igor)
 - Use GArray for NUMA memory ranges data (Igor)
 - Separate hmat_build_lb() (Igor)
 - Drop all global variables and use local variables instead (Igor)
 - Add error message when base unit < 10
 - Update the hmat-lb option example by using '-numa cpu'
 and '-numa memdev' (Igor)

v4:
 - send the patch of "move numa global variables into MachineState"
 together with HMAT patches.
 https://lists.gnu.org/archive/html/qemu-devel/2019-04/msg03662.html
 - spilt the 1/8 of v3 patch into two patches, 4/11 introduces
 build_mem_ranges() and 5/11 builds HMAT (Igor)
 - use build_append_int_noprefix() to build parts of ACPI table in
 all patches (Igor)
 - Split 8/8 of patch v3 into two parts, 10/11 introduces NFIT
 generalizations (build_acpi_aml_common), and use it in 11/11 to
 simplify hmat_build_aml (Igor)
 - use MachineState instead of PCMachineState to build HMAT more
 generalic (Igor)
 - move the 7/8 v3 patch into the former patches
 - update the version tag from 4.0 to 4.1
v3:
 - rebase the fixing patch into the jingqi's patches (Eric)
 - update the version tag from 3.10 to 4.0 (Eric)
v2:
   Per Igor and Eric's comments, fix some coding style and small issues:
 - update the version number in qapi/misc.json
 - including the expansion of the acronym HMAT in qapi/misc.json
 - correct spell mistakes in qapi/misc.json and qemu-options.hx
 - fix the comment syle in hw/i386/acpi-build.c
 and hw/acpi/hmat.h
- remove some unnecessary head files in hw/acpi/hmat.c
- use hardcoded numbers from spec to generate
Memory Subsystem Address Range Structure in hw/acpi/hmat.c
- drop the struct AcpiHmat and AcpiHmatSpaRange
 in hw/acpi/hmat.h
- rewrite NFIT code to build _HMA method

Liu Jingqi (3):
   hmat acpi: Build Memory Subsystem Address Range Structure(s) in ACPI
 HMAT
   hmat acpi: Build System Locality Latency and Bandwidth Information
 Structure(s) in ACPI HMAT
   numa: Extend the command-line to provide memory latency and bandwidth
 information

Tao Xu (5):
   hw/arm: simplify arm_load_dtb
   numa: move numa global variable nb_numa_nodes into MachineState
   numa: move numa global variable have_numa_distance into MachineState
   numa: move numa global variable numa_info into MachineState
   acpi: introduce AcpiDeviceIfClass.build_mem_ranges hook

  exec.c   |   5 +-
  hw/acpi/Kconfig  |   5 +
  hw/acpi/Makefile.objs|   1 +
  hw/acpi/aml-build.c  |   9 +-
  hw/acpi/hmat.c   | 252 +++
  hw/acpi/hmat.h   |  82 +
  hw/acpi/piix4.c  |   1 +
  hw/arm/aspeed.c  |   5 +-
  hw/arm/boot.c|  20 ++-
  hw/arm/collie.c  |   8 +-
  hw/arm/cubieboard.c  |   5 +-
  hw/arm/exynos4_boards.c  |   7 +-
  hw/arm/highbank.c|   8 +-
  hw/arm/imx25_pdk.c   |   5 +-
  hw/arm/integratorcp.c|   8 +-
  hw/arm/kzm.c |   5 +-
  hw/arm/mainstone.c   |   5 +-
  hw/arm/mcimx6ul-evk.c|   5 +-
  hw/arm/mcimx7d-sabre.c   |   5 +-
  hw/arm/musicpal.c|   8 +-
  hw/arm/nseries.c |   5 +-
  hw/arm/omap_sx1.c|   5 +-
  hw/arm/palm.c|  10 +-
  hw/arm/raspi.c   |   6 +-
  hw/arm/realview.c|   5 +-
  hw/arm/sabrelite.c   |   5 +-
  hw/arm/spitz.c   |   5 +-
  hw/arm/tosa.c|   8 

Re: [Qemu-devel] [PATCH v5 0/8] Build ACPI Heterogeneous Memory Attribute Table (HMAT)

2019-07-01 Thread Igor Mammedov
On Fri, 14 Jun 2019 23:56:18 +0800
Tao Xu  wrote:

> This series of patches will build Heterogeneous Memory Attribute Table (HMAT)
> according to the command line. The ACPI HMAT describes the memory attributes,
> such as memory side cache attributes and bandwidth and latency details,
> related to the System Physical Address (SPA) Memory Ranges.
> The software is expected to use this information as hint for optimization.

in addition to patches in this series. pls consider adding testcase for ACPI 
table
as the last patch. Look at tests/bios-tables-test.c for examples.


> The V4 patches link:
> https://lists.gnu.org/archive/html/qemu-devel/2019-05/msg01644.html
> 
> Changelog:
> v5:
> - spilt the 1-6/11 and 8/11 from patch v4 to build Memory Subsystem
> Address Range Structure(s) and System Locality Latency and Bandwidth
> Information Structure(s) firstly.
> - add 1/8 of patch v5 to simplify arm_load_dtb() (Igor)
> - drop the helper machine_num_numa_nodes() and use
> machine->numa_state->num_nodes (and numa_state->nodes) directly (Igor)
> - Add more descriptions from ACPI spec (Igor)
> - Add the reason of using stub (Igor)
> - Use GArray for NUMA memory ranges data (Igor)
> - Separate hmat_build_lb() (Igor)
> - Drop all global variables and use local variables instead (Igor)
> - Add error message when base unit < 10
> - Update the hmat-lb option example by using '-numa cpu'
> and '-numa memdev' (Igor)
> 
> v4:
> - send the patch of "move numa global variables into MachineState"
> together with HMAT patches.
> https://lists.gnu.org/archive/html/qemu-devel/2019-04/msg03662.html
> - spilt the 1/8 of v3 patch into two patches, 4/11 introduces
> build_mem_ranges() and 5/11 builds HMAT (Igor)
> - use build_append_int_noprefix() to build parts of ACPI table in
> all patches (Igor)
> - Split 8/8 of patch v3 into two parts, 10/11 introduces NFIT
> generalizations (build_acpi_aml_common), and use it in 11/11 to
> simplify hmat_build_aml (Igor)
> - use MachineState instead of PCMachineState to build HMAT more
> generalic (Igor)
> - move the 7/8 v3 patch into the former patches
> - update the version tag from 4.0 to 4.1
> v3:
> - rebase the fixing patch into the jingqi's patches (Eric)
> - update the version tag from 3.10 to 4.0 (Eric)
> v2:
>   Per Igor and Eric's comments, fix some coding style and small issues:
> - update the version number in qapi/misc.json
> - including the expansion of the acronym HMAT in qapi/misc.json
> - correct spell mistakes in qapi/misc.json and qemu-options.hx
> - fix the comment syle in hw/i386/acpi-build.c
> and hw/acpi/hmat.h
>- remove some unnecessary head files in hw/acpi/hmat.c 
>- use hardcoded numbers from spec to generate
>Memory Subsystem Address Range Structure in hw/acpi/hmat.c
>- drop the struct AcpiHmat and AcpiHmatSpaRange
> in hw/acpi/hmat.h
>- rewrite NFIT code to build _HMA method
> 
> Liu Jingqi (3):
>   hmat acpi: Build Memory Subsystem Address Range Structure(s) in ACPI
> HMAT
>   hmat acpi: Build System Locality Latency and Bandwidth Information
> Structure(s) in ACPI HMAT
>   numa: Extend the command-line to provide memory latency and bandwidth
> information
> 
> Tao Xu (5):
>   hw/arm: simplify arm_load_dtb
>   numa: move numa global variable nb_numa_nodes into MachineState
>   numa: move numa global variable have_numa_distance into MachineState
>   numa: move numa global variable numa_info into MachineState
>   acpi: introduce AcpiDeviceIfClass.build_mem_ranges hook
> 
>  exec.c   |   5 +-
>  hw/acpi/Kconfig  |   5 +
>  hw/acpi/Makefile.objs|   1 +
>  hw/acpi/aml-build.c  |   9 +-
>  hw/acpi/hmat.c   | 252 +++
>  hw/acpi/hmat.h   |  82 +
>  hw/acpi/piix4.c  |   1 +
>  hw/arm/aspeed.c  |   5 +-
>  hw/arm/boot.c|  20 ++-
>  hw/arm/collie.c  |   8 +-
>  hw/arm/cubieboard.c  |   5 +-
>  hw/arm/exynos4_boards.c  |   7 +-
>  hw/arm/highbank.c|   8 +-
>  hw/arm/imx25_pdk.c   |   5 +-
>  hw/arm/integratorcp.c|   8 +-
>  hw/arm/kzm.c |   5 +-
>  hw/arm/mainstone.c   |   5 +-
>  hw/arm/mcimx6ul-evk.c|   5 +-
>  hw/arm/mcimx7d-sabre.c   |   5 +-
>  hw/arm/musicpal.c|   8 +-
>  hw/arm/nseries.c |   5 +-
>  hw/arm/omap_sx1.c|   5 +-
>  hw/arm/palm.c|  10 +-
>  hw/arm/raspi.c   |   6 +-
>  hw/arm/realview.c|   5 +-
>  hw/arm/sabrelite.c   |   5 +-
>  hw/arm/spitz.c   |   

[Qemu-devel] [PATCH v5 0/8] Build ACPI Heterogeneous Memory Attribute Table (HMAT)

2019-06-14 Thread Tao Xu
This series of patches will build Heterogeneous Memory Attribute Table (HMAT)
according to the command line. The ACPI HMAT describes the memory attributes,
such as memory side cache attributes and bandwidth and latency details,
related to the System Physical Address (SPA) Memory Ranges.
The software is expected to use this information as hint for optimization.

The V4 patches link:
https://lists.gnu.org/archive/html/qemu-devel/2019-05/msg01644.html

Changelog:
v5:
- spilt the 1-6/11 and 8/11 from patch v4 to build Memory Subsystem
Address Range Structure(s) and System Locality Latency and Bandwidth
Information Structure(s) firstly.
- add 1/8 of patch v5 to simplify arm_load_dtb() (Igor)
- drop the helper machine_num_numa_nodes() and use
machine->numa_state->num_nodes (and numa_state->nodes) directly (Igor)
- Add more descriptions from ACPI spec (Igor)
- Add the reason of using stub (Igor)
- Use GArray for NUMA memory ranges data (Igor)
- Separate hmat_build_lb() (Igor)
- Drop all global variables and use local variables instead (Igor)
- Add error message when base unit < 10
- Update the hmat-lb option example by using '-numa cpu'
and '-numa memdev' (Igor)

v4:
- send the patch of "move numa global variables into MachineState"
together with HMAT patches.
https://lists.gnu.org/archive/html/qemu-devel/2019-04/msg03662.html
- spilt the 1/8 of v3 patch into two patches, 4/11 introduces
build_mem_ranges() and 5/11 builds HMAT (Igor)
- use build_append_int_noprefix() to build parts of ACPI table in
all patches (Igor)
- Split 8/8 of patch v3 into two parts, 10/11 introduces NFIT
generalizations (build_acpi_aml_common), and use it in 11/11 to
simplify hmat_build_aml (Igor)
- use MachineState instead of PCMachineState to build HMAT more
generalic (Igor)
- move the 7/8 v3 patch into the former patches
- update the version tag from 4.0 to 4.1
v3:
- rebase the fixing patch into the jingqi's patches (Eric)
- update the version tag from 3.10 to 4.0 (Eric)
v2:
  Per Igor and Eric's comments, fix some coding style and small issues:
- update the version number in qapi/misc.json
- including the expansion of the acronym HMAT in qapi/misc.json
- correct spell mistakes in qapi/misc.json and qemu-options.hx
- fix the comment syle in hw/i386/acpi-build.c
and hw/acpi/hmat.h
   - remove some unnecessary head files in hw/acpi/hmat.c 
   - use hardcoded numbers from spec to generate
   Memory Subsystem Address Range Structure in hw/acpi/hmat.c
   - drop the struct AcpiHmat and AcpiHmatSpaRange
in hw/acpi/hmat.h
   - rewrite NFIT code to build _HMA method

Liu Jingqi (3):
  hmat acpi: Build Memory Subsystem Address Range Structure(s) in ACPI
HMAT
  hmat acpi: Build System Locality Latency and Bandwidth Information
Structure(s) in ACPI HMAT
  numa: Extend the command-line to provide memory latency and bandwidth
information

Tao Xu (5):
  hw/arm: simplify arm_load_dtb
  numa: move numa global variable nb_numa_nodes into MachineState
  numa: move numa global variable have_numa_distance into MachineState
  numa: move numa global variable numa_info into MachineState
  acpi: introduce AcpiDeviceIfClass.build_mem_ranges hook

 exec.c   |   5 +-
 hw/acpi/Kconfig  |   5 +
 hw/acpi/Makefile.objs|   1 +
 hw/acpi/aml-build.c  |   9 +-
 hw/acpi/hmat.c   | 252 +++
 hw/acpi/hmat.h   |  82 +
 hw/acpi/piix4.c  |   1 +
 hw/arm/aspeed.c  |   5 +-
 hw/arm/boot.c|  20 ++-
 hw/arm/collie.c  |   8 +-
 hw/arm/cubieboard.c  |   5 +-
 hw/arm/exynos4_boards.c  |   7 +-
 hw/arm/highbank.c|   8 +-
 hw/arm/imx25_pdk.c   |   5 +-
 hw/arm/integratorcp.c|   8 +-
 hw/arm/kzm.c |   5 +-
 hw/arm/mainstone.c   |   5 +-
 hw/arm/mcimx6ul-evk.c|   5 +-
 hw/arm/mcimx7d-sabre.c   |   5 +-
 hw/arm/musicpal.c|   8 +-
 hw/arm/nseries.c |   5 +-
 hw/arm/omap_sx1.c|   5 +-
 hw/arm/palm.c|  10 +-
 hw/arm/raspi.c   |   6 +-
 hw/arm/realview.c|   5 +-
 hw/arm/sabrelite.c   |   5 +-
 hw/arm/spitz.c   |   5 +-
 hw/arm/tosa.c|   8 +-
 hw/arm/versatilepb.c |   5 +-
 hw/arm/vexpress.c|   5 +-
 hw/arm/virt-acpi-build.c |  17 +-
 hw/arm/virt.c|  16 +-
 hw/arm/xilinx_zynq.c |   8 +-
 hw/arm/xlnx-versal-virt.c|   7 +-
 hw/arm/xlnx-zcu102.c |   5 +-
 hw/arm/z2.c