Harshit Sharma has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44814 )
Change subject: Documentation: Add ASan documentation ......................................................................
Patch Set 7:
(1 comment)
https://review.coreboot.org/c/coreboot/+/44814/7/Documentation/technotes/asa... File Documentation/technotes/asan.md:
https://review.coreboot.org/c/coreboot/+/44814/7/Documentation/technotes/asa... PS7, Line 248: additional steps need to be taken to enable ASan in : romstage on the platform
Maybe suggest playing around with CONFIG_DCACHE_RAM_SIZE (and CONFIG_DCACHE_RAM_BASE)?
As I wrote in the very next sentence, the compile errors should be easy to resolve but it is way more work in case of a full cache. We believe a different translation function and an additional gcc patch would be required as I mentioned at line 285.
-
Harshit Sharma (Code Review)