Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44814 )
Change subject: Documentation: Add ASan documentation ......................................................................
Patch Set 7: Code-Review+1
(2 comments)
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)?
https://review.coreboot.org/c/coreboot/+/44814/7/Documentation/technotes/asa... PS7, Line 283: 1/8th of the : program memory It might be worth explaining somewhere why it has to be this way (in the code or in the docs)?
-
Arthur Heymans (Code Review)