Hi Bin,
-----Original Message----- From: Bin Meng bmeng.cn@gmail.com Sent: Thursday, February 27, 2020 9:27 PM To: Park, Aiden aiden.park@intel.com Cc: masahiroy@kernel.org; sjg@chromium.org; sr@denx.de; agraf@suse.de; u- boot@lists.denx.de Subject: Re: [PATCH] x86: Cache write back at 32-bit entry point
Hi Aiden,
On Fri, Feb 28, 2020 at 12:54 PM Park, Aiden aiden.park@intel.com wrote:
In a certain condition, invd causes cache coherence issue.
- Pre-stage boot code passes memory address to U-Boot
- The data of the memory address is still in data cache line
- The invd marks data cache line as invalid without write back
- U-Boot accesses the memory address
- Data is invalid
Therefore, wbinvd is recommended at the 32-bit entry point even though it consumes extra cpu clock cycles.
Signed-off-by: Aiden Park aiden.park@intel.com
arch/x86/cpu/start.S | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
This is already fixed (reverted) in the x86 tree. Would you please double check?
Thank you for letting me know that. I think I missed some emails. Thanks again. Andy, appreciate your commit. I had the same issue on Slim Bootloader due to this cache coherence.
Regards, Bin
Best Regards, Aiden