Re: [PATCH] ARM: exynos_defconfig: Enable DYNAMIC_DEBUG and get rid of old ext3

From: Javier Martinez Canillas
Date: Mon Jan 30 2017 - 14:43:00 EST


Hello Krzysztof,

On 01/30/2017 04:27 PM, Krzysztof Kozlowski wrote:
> DYNAMIC_DEBUG is quite useful for debugging kernels and should not cause
> noticeable performance regressions. It makes the kernel noticeably
> bigger (around 4%) but this difference should not impact typical
> developer and reference usage of this defconfig.
>
> Sizes:
> zImage-old: 4641496 bytes
> zImage-new: 4811384 bytes
>
> text data bss dec hex filename
> 7031229 2570916 327016 9929161 9781c9 vmlinux-old
> 7205921 2800052 327016 10332989 9dab3d vmlinux-new
>

Agreed, most people usually will end enabling this option when debugging
issues anyways so having enabled by default is quite convenient indeed.

> Additionally, remove the EXT3_FS symbol because it is entirely replaced
> by EXT4_FS.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> ---

Reviewed-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx>

Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America