[PATCHv2 08/12] acpi/hmat: Register performance attributes

[Keith Busch]

Save the best performance access attributes and register these with the
memory's node if HMAT provides the locality table.

Signed-off-by: Keith Busch <keith.bu...@intel.com>
---
 drivers/acpi/Kconfig |  1 +
 drivers/acpi/hmat.c  | 34 ++++++++++++++++++++++++++++++++++
 2 files changed, 35 insertions(+)

diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig
index 9a05af3a18cf..6b5f6ca690af 100644
--- a/drivers/acpi/Kconfig
+++ b/drivers/acpi/Kconfig
@@ -330,6 +330,7 @@ config ACPI_NUMA
 config ACPI_HMAT
        bool "ACPI Heterogeneous Memory Attribute Table Support"
        depends on ACPI_NUMA
+       select HMEM_REPORTING
        help
         Parses representation of the ACPI Heterogeneous Memory Attributes
         Table (HMAT) and set the memory node relationships and access
diff --git a/drivers/acpi/hmat.c b/drivers/acpi/hmat.c
index 5d8747ad025f..40bc83f4b593 100644
--- a/drivers/acpi/hmat.c
+++ b/drivers/acpi/hmat.c
@@ -23,6 +23,8 @@ struct memory_target {
        struct list_head node;
        unsigned int memory_pxm;
        unsigned long p_nodes[BITS_TO_LONGS(MAX_NUMNODES)];
+       bool hmem_valid;
+       struct node_hmem_attrs hmem;
 };
 
 static __init struct memory_target *find_mem_target(unsigned int m)
@@ -108,6 +110,34 @@ static __init void hmat_update_access(u8 type, u32 value, 
u32 *best)
        }
 }
 
+static __init void hmat_update_target(struct memory_target *t, u8 type,
+                                     u32 value)
+{
+       switch (type) {
+       case ACPI_HMAT_ACCESS_LATENCY:
+               t->hmem.read_latency = value;
+               t->hmem.write_latency = value;
+               break;
+       case ACPI_HMAT_READ_LATENCY:
+               t->hmem.read_latency = value;
+               break;
+       case ACPI_HMAT_WRITE_LATENCY:
+               t->hmem.write_latency = value;
+               break;
+       case ACPI_HMAT_ACCESS_BANDWIDTH:
+               t->hmem.read_bandwidth = value;
+               t->hmem.write_bandwidth = value;
+               break;
+       case ACPI_HMAT_READ_BANDWIDTH:
+               t->hmem.read_bandwidth = value;
+               break;
+       case ACPI_HMAT_WRITE_BANDWIDTH:
+               t->hmem.write_bandwidth = value;
+               break;
+       }
+       t->hmem_valid = true;
+}
+
 static __init int hmat_parse_locality(union acpi_subtable_headers *header,
                                      const unsigned long end)
 {
@@ -166,6 +196,8 @@ static __init int hmat_parse_locality(union 
acpi_subtable_headers *header,
                                        set_bit(p_node, t->p_nodes);
                        }
                }
+               if (t && best)
+                       hmat_update_target(t, type, best);
        }
        return 0;
 }
@@ -267,6 +299,8 @@ static __init void hmat_register_targets(void)
                m = pxm_to_node(t->memory_pxm);
                for_each_set_bit(p, t->p_nodes, MAX_NUMNODES)
                        register_memory_node_under_compute_node(m, p);
+               if (t->hmem_valid)
+                       node_set_perf_attrs(m, &t->hmem);
                kfree(t);
        }
 }
-- 
2.14.4