[coreboot-gerrit] Change in coreboot[master]: cpu/intel/206ax/acpi.c: Fix get_cores_per_package

[Evgeny Zinoviev (Code Review)]

Evgeny Zinoviev has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42413 )

Change subject: cpu/intel/206ax/acpi.c: Fix get_cores_per_package
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/42413/3/src/cpu/intel/model_206ax/a... 
File src/cpu/intel/model_206ax/acpi.c:

https://review.coreboot.org/c/coreboot/+/42413/3/src/cpu/intel/model_206ax/a... 
PS3, Line 21: 	logical_cores = msr.lo & 0xffff;
> Excuse my ignorance, but why is the `& 0xffff` needed? […]

Well, it's how it returns data. Bits [31:16] are cores count and [15:0] are threads count. We need threads so 0xffff.