From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754182AbdA3Tm6 (ORCPT ); Mon, 30 Jan 2017 14:42:58 -0500 Received: from ec2-52-27-115-49.us-west-2.compute.amazonaws.com ([52.27.115.49]:32809 "EHLO osg.samsung.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751033AbdA3Tm4 (ORCPT ); Mon, 30 Jan 2017 14:42:56 -0500 Subject: Re: [PATCH] ARM: exynos_defconfig: Enable DYNAMIC_DEBUG and get rid of old ext3 To: Krzysztof Kozlowski , Kukjin Kim , Arnd Bergmann , Bartlomiej Zolnierkiewicz , linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org References: <20170130192749.3690-1-krzk@kernel.org> From: Javier Martinez Canillas Message-ID: Date: Mon, 30 Jan 2017 16:35:09 -0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 MIME-Version: 1.0 In-Reply-To: <20170130192749.3690-1-krzk@kernel.org> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 > --- Reviewed-by: Javier Martinez Canillas Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America