[coreboot-gerrit] Change in coreboot[master]: cpu/x86/cache: CLFLUSH programs to memory before running

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37196 )

Change subject: cpu/x86/cache: CLFLUSH programs to memory before running
......................................................................

Patch Set 11:

(3 comments)

https://review.coreboot.org/c/coreboot/+/37196/7/src/cpu/x86/cache/cache.c 
File src/cpu/x86/cache/cache.c:

https://review.coreboot.org/c/coreboot/+/37196/7/src/cpu/x86/cache/cache.c@2... 
PS7, Line 27: int clflush_supported(void)
> > > This is not sufficient. […]

Done. Added a Kconfig flag X86_CLFLUSH_CAR

https://review.coreboot.org/c/coreboot/+/37196/7/src/cpu/x86/cache/cache.c@2... 
PS7, Line 29: 19
> Can you start?

Done

https://review.coreboot.org/c/coreboot/+/37196/7/src/cpu/x86/cache/cache.c@6... 
PS7, Line 60: 	if (!cbmem_ready())
> > > Why is a cbmem check in here? It doesn't immediately make sense why this would be the case. […]

Done