[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v16 12/14] hmat acpi: Build Memory Side Cache Information Str
From: |
Igor Mammedov |
Subject: |
Re: [PATCH v16 12/14] hmat acpi: Build Memory Side Cache Information Structure(s) |
Date: |
Wed, 20 Nov 2019 13:50:33 +0100 |
On Fri, 15 Nov 2019 15:53:50 +0800
Tao Xu <address@hidden> wrote:
> From: Liu Jingqi <address@hidden>
>
> This structure describes memory side cache information for memory
> proximity domains if the memory side cache is present and the
> physical device forms the memory side cache.
> The software could use this information to effectively place
> the data in memory to maximize the performance of the system
> memory that use the memory side cache.
>
> Reviewed-by: Daniel Black <address@hidden>
> Reviewed-by: Jonathan Cameron <address@hidden>
> Signed-off-by: Liu Jingqi <address@hidden>
> Signed-off-by: Tao Xu <address@hidden>
looks good, but I'll skip the patch this round
since it will be changed by HMAT_Cache_Info removal in [9/14]
> ---
>
> Changes in v16:
> - Use checks and assert to replace masks (Igor)
> - Fields in Cache Attributes are promoted to uint32_t before
> shifting (Igor)
> - Drop cpu_to_le32() (Igor)
>
> Changes in v13:
> - rename level as cache_level
> ---
> hw/acpi/hmat.c | 69 +++++++++++++++++++++++++++++++++++++++++++++++++-
> 1 file changed, 68 insertions(+), 1 deletion(-)
>
> diff --git a/hw/acpi/hmat.c b/hw/acpi/hmat.c
> index ed19ebed2f..2b4f760e0e 100644
> --- a/hw/acpi/hmat.c
> +++ b/hw/acpi/hmat.c
> @@ -144,14 +144,62 @@ static void build_hmat_lb(GArray *table_data,
> HMAT_LB_Info *hmat_lb,
> g_free(entry_list);
> }
>
> +/* ACPI 6.3: 5.2.27.5 Memory Side Cache Information Structure: Table 5-147 */
> +static void build_hmat_cache(GArray *table_data, HMAT_Cache_Info *hmat_cache,
> + uint8_t total_levels)
> +{
> + /*
> + * Cache Attributes: Bits [3:0] – Total Cache Levels
> + * for this Memory Proximity Domain
> + */
> + uint32_t cache_attr = total_levels;
> +
> + /* Bits [7:4] : Cache Level described in this structure */
> + cache_attr |= (uint32_t) hmat_cache->level << 4;
> +
> + /* Bits [11:8] - Cache Associativity */
> + cache_attr |= (uint32_t) hmat_cache->associativity << 8;
> +
> + /* Bits [15:12] - Write Policy */
> + cache_attr |= (uint32_t) hmat_cache->write_policy << 12;
> +
> + /* Bits [31:16] - Cache Line size in bytes */
> + cache_attr |= (uint32_t) hmat_cache->line_size << 16;
> +
> + /* Type */
> + build_append_int_noprefix(table_data, 2, 2);
> + /* Reserved */
> + build_append_int_noprefix(table_data, 0, 2);
> + /* Length */
> + build_append_int_noprefix(table_data, 32, 4);
> + /* Proximity Domain for the Memory */
> + build_append_int_noprefix(table_data, hmat_cache->proximity, 4);
> + /* Reserved */
> + build_append_int_noprefix(table_data, 0, 4);
> + /* Memory Side Cache Size */
> + build_append_int_noprefix(table_data, hmat_cache->size, 8);
> + /* Cache Attributes */
> + build_append_int_noprefix(table_data, cache_attr, 4);
> + /* Reserved */
> + build_append_int_noprefix(table_data, 0, 2);
> + /*
> + * Number of SMBIOS handles (n)
> + * Linux kernel uses Memory Side Cache Information Structure
> + * without SMBIOS entries for now, so set Number of SMBIOS handles
> + * as 0.
> + */
> + build_append_int_noprefix(table_data, 0, 2);
> +}
> +
> /* Build HMAT sub table structures */
> static void hmat_build_table_structs(GArray *table_data, NumaState
> *numa_state)
> {
> uint16_t flags;
> uint32_t num_initiator = 0;
> uint32_t initiator_list[MAX_NODES];
> - int i, hierarchy, type;
> + int i, hierarchy, type, cache_level, total_levels;
> HMAT_LB_Info *hmat_lb;
> + HMAT_Cache_Info *hmat_cache;
>
> for (i = 0; i < numa_state->num_nodes; i++) {
> flags = 0;
> @@ -185,6 +233,25 @@ static void hmat_build_table_structs(GArray *table_data,
> NumaState *numa_state)
> }
> }
> }
> +
> + /*
> + * ACPI 6.3: 5.2.27.5 Memory Side Cache Information Structure:
> + * Table 5-147
> + */
> + for (i = 0; i < numa_state->num_nodes; i++) {
> + total_levels = 0;
> + for (cache_level = 1; cache_level < HMAT_LB_LEVELS; cache_level++) {
> + if (numa_state->hmat_cache[i][cache_level]) {
> + total_levels++;
> + }
> + }
> + for (cache_level = 0; cache_level <= total_levels; cache_level++) {
> + hmat_cache = numa_state->hmat_cache[i][cache_level];
> + if (hmat_cache) {
> + build_hmat_cache(table_data, hmat_cache, total_levels);
> + }
> + }
> + }
> }
>
> void build_hmat(GArray *table_data, BIOSLinker *linker, NumaState
> *numa_state)
- Re: [PATCH v16 08/14] numa: Extend CLI to provide memory latency and bandwidth information, (continued)
[PATCH v16 09/14] numa: Extend CLI to provide memory side cache information, Tao Xu, 2019/11/15
[PATCH v16 11/14] hmat acpi: Build System Locality Latency and Bandwidth Information Structure(s), Tao Xu, 2019/11/15
[PATCH v16 10/14] hmat acpi: Build Memory Proximity Domain Attributes Structure(s), Tao Xu, 2019/11/15
[PATCH v16 12/14] hmat acpi: Build Memory Side Cache Information Structure(s), Tao Xu, 2019/11/15
- Re: [PATCH v16 12/14] hmat acpi: Build Memory Side Cache Information Structure(s),
Igor Mammedov <=
[PATCH v16 13/14] tests/numa: Add case for QMP build HMAT, Tao Xu, 2019/11/15
[PATCH v16 14/14] tests/bios-tables-test: add test cases for ACPI HMAT, Tao Xu, 2019/11/15
Re: [PATCH v16 00/14] Build ACPI Heterogeneous Memory Attribute Table (HMAT), no-reply, 2019/11/15