Attention is currently required from: Jérémy Compostella, Subrata Banik.
Shuo Liu has posted comments on this change by Subrata Banik. ( https://review.coreboot.org/c/coreboot/+/87182?usp=email )
Change subject: cpu/x86: Conditionally invalidate caches based on self-snooping support ......................................................................
Patch Set 2:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/87182/comment/3482655e_f617629f?usp... : PS2, Line 35: Does arch_segment_loaded in cpu/x86/cache/cache.c needs to be handled as well?
File src/cpu/x86/mp_init.c:
https://review.coreboot.org/c/coreboot/+/87182/comment/8ab71815_d4bfc8e0?usp... : PS2, Line 831: if (!self_snooping_supported()) { Do we need to check clflush_supported to do clflush over wbinvd?