[M] Change in coreboot[main]: cpu/x86/topology: Add node ID parser

[Shuo Liu (Code Review)]

Attention is currently required from: Angel Pons, Chen, Gang C, Christian Walter, Felix Held, Johnny Lin, Jonathan Zhang, Jérémy Compostella, Martin L Roth, Patrick Rudolph, Tim Chu.

Shuo Liu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81515?usp=email )

Change subject: cpu/x86/topology: Add node ID parser
......................................................................

Patch Set 5:

(1 comment)

File src/cpu/x86/topology.c:

https://review.coreboot.org/c/coreboot/+/81515/comment/dcbfdc1e_006233d4 :
PS5, Line 158: 		package_id = apicid >> (thread_bits + core_bits);
> Can the SOCKET_ID MSR be used instead of CPUID lead 0x1f?

I haven't use it before. MSR is usually model-specific and maybe it would be not so suitable to be used in generic codes.

But it would be helpful if we could have clear definition what node_id means to different archs (even we define node_id as the physical package id, there is another field named package_id as discussed is confusing).