On 9/4/19 8:53 AM, Sascha Hauer wrote: > On Tue, Aug 27, 2019 at 05:09:08PM +0200, Ahmad Fatoum wrote: >> This series adds undefined behavior and address sanitizer support to >> barebox. Both are fully functional on sandbox, where they are also used >> to implement dump_stack() now. >> >> I haven't yet read about how Kernel AddressSanitizer works, so this one >> only works on sandbox via libasan for now. >> >> The undefined behavior sanitizer depends on the compiler instrumenting >> potential pitfalls and then calling the routines in lib/ubsan.c if it >> catches something undefined, so that should readily work on other arches >> as well. I tested it on sandbox and the i.MX6Q so far. >> >> Eventually, it should be possible to enable it for all the non-PBL stuff >> with a single Kconfig option, but for now you need to explicitly add a >> >> UBSAN_SANITIZE_myfile.o := y >> >> in the respective Makefile. Enabling it wholesale doesn't yet work on >> ARM, I suspect it might be due to binary size. >> >> Changes in v2: >> v1 was incomplete and sent our more by mistake, >> so no changelog. > > Applied, thanks Could you swap patch 08 and 09? Patch 08 depends on the KASAN symbol in the Kconfig option of SANDBOX_UNWIND, but this symbol is introduced in patch 09.. > > Sascha > > -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
