Attention is currently required from: Subrata Banik, Paul Menzel, Kyösti Mälkki, Jan Samek, Elyes Haouas.
Sridhar Siricilla has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/72132 )
Change subject: soc/intel/common: Order the different types of cores based on APIC IDs ......................................................................
Patch Set 18:
(5 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/72132/comment/0876e7c7_e31ee4f3 PS15, Line 24: Existing code presents mix of different cores to OS and causes CPU load : balancing and power/performance impact.
Please refer the patches for media dynamic cgroup: […]
Ack
Commit Message:
https://review.coreboot.org/c/coreboot/+/72132/comment/949a7d6b_ee431acb PS17, Line 13: cat
Not the cat, please.
Ack
https://review.coreboot.org/c/coreboot/+/72132/comment/27e38820_841c8b05 PS17, Line 13: cpu$c
This variable is now undefined. Use glob (`cpu*`) instead.
I missed to update the command output here as is.
https://review.coreboot.org/c/coreboot/+/72132/comment/0b00da0d_a3ad12bb PS17, Line 14: 0-1 : 4 : 5 : 6 : 7 : 0-1 : 2-3 : 2-3
This will now also look different if you run the command, e.g.: […]
Ack
https://review.coreboot.org/c/coreboot/+/72132/comment/4ad9de52_034f82e6 PS17, Line 37: grep . cat /sys/devices/system/cpu/cpu$c/topology/thread_siblings_list
ditto
Ack