[coreboot-gerrit] Change in ...coreboot[master]: arch/x86/smbios: Reference type 7

[Patrick Rudolph (Code Review)]

Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/32132

Change subject: arch/x86/smbios: Reference type 7
......................................................................

arch/x86/smbios: Reference type 7

Fill in the handle to cache entries of type 7 in the type 4 structure.

Change-Id: Idf876b0c21c65f72a945d26c5898074b140763f8
Signed-off-by: Patrick Rudolph 
---
M src/arch/x86/smbios.c
1 file changed, 24 insertions(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/32/32132/1

diff --git a/src/arch/x86/smbios.c b/src/arch/x86/smbios.c
index be024c16..68fca21 100644
--- a/src/arch/x86/smbios.c
+++ b/src/arch/x86/smbios.c
@@ -694,10 +694,12 @@
  * @param current Pointer to memory address to write the tables to
  * @param handle Pointer to the handle for the tables
  * @param max_struct_size Pointer to maximum struct size
+ * @param type4 Pointer to SMBIOS type 4 structure
  */
 static int smbios_write_type7_cache_parameters(unsigned long *current,
    				       int *handle,
-					       int *max_struct_size)
+					       int *max_struct_size,
+					       struct smbios_type4 *type4)
 {
    struct cpuid_result res;
    unsigned int cnt = 0;
@@ -730,6 +732,7 @@
    while (1) {
    	enum smbios_cache_associativity associativity;
    	enum smbios_cache_type type;
+		const int h = (*handle)++;

res = cpuid_ext(leaf, cnt++);

@@ -766,9 +769,23 @@
    	else
    		associativity = smbios_cache_associativity(assoc);

-		update_max(len, *max_struct_size, smbios_write_type7(current,
-			*handle++, level, SMBIOS_CACHE_SRAM_TYPE_UNKNOWN,
-			associativity, type, cache_size, cache_size));
+		update_max(len, *max_struct_size, smbios_write_type7(current, h,
+			   level, SMBIOS_CACHE_SRAM_TYPE_UNKNOWN, associativity,
+			   type, cache_size, cache_size));
+
+		if (type4) {
+			switch (level) {
+			case 1:
+				type4->l1_cache_handle = h;
+			break;
+			case 2:
+				type4->l2_cache_handle = h;
+			break;
+			case 3:
+				type4->l3_cache_handle = h;
+			break;
+			}
+		}
    };

return len;
@@ -945,10 +962,12 @@
    handle++;
    update_max(len, max_struct_size, smbios_write_type3(&current,
    	handle++));
+
+	struct smbios_type4 *type4 = (struct smbios_type4 *)current;
    update_max(len, max_struct_size, smbios_write_type4(&current,
    	handle++));
    smbios_write_type7_cache_parameters(&current, &handle,
-		&max_struct_size);
+		&max_struct_size, type4);
    update_max(len, max_struct_size, smbios_write_type11(&current,
    	&handle));
    if (CONFIG(ELOG))