From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Glass Date: Wed, 29 Aug 2018 20:51:24 -0600 Subject: [U-Boot] [PATCH v2 13/13] configs: sandbox*: Enable UBSAN In-Reply-To: <20180826231332.2491-14-erosca@de.adit-jv.com> References: <20180826231332.2491-1-erosca@de.adit-jv.com> <20180826231332.2491-14-erosca@de.adit-jv.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi, On 26 August 2018 at 17:13, Eugeniu Rosca wrote: > On Sun, Aug 19, 2018 at 09:51:32PM -0400, Tom Rini wrote: >> [..] we should be able to say more broadly that just about everyone >> can enable this, but only out of the box sandbox should. > > Hence, turn UBSAN on for every available sandbox flavor. > Make sure the inserted line complies with `make savedefconfig`. > > The size increase of sandbox_defconfig U-Boot (gcc 8.1.0): > $ size u-boot.sandbox.* > text data bss dec hex filename > 1234958 80048 291472 1606478 18834e u-boot.sandbox.default > 1422710 272240 291472 1986422 1e4f76 u-boot.sandbox.ubsan > +187752 +192192 0 +379944 > > Signed-off-by: Eugeniu Rosca > --- > > Changes in v2: > - None. Newly pushed. > --- > configs/sandbox64_defconfig | 1 + > configs/sandbox_defconfig | 1 + > configs/sandbox_flattree_defconfig | 1 + > configs/sandbox_noblk_defconfig | 1 + > configs/sandbox_spl_defconfig | 1 + > 5 files changed, 5 insertions(+) > Can you please do this with an 'imply' in arch/Kconfig? Regards, Simon