From mboxrd@z Thu Jan 1 00:00:00 1970 From: Przemyslaw Marczak Date: Wed, 18 Feb 2015 13:40:13 +0100 Subject: [U-Boot] [PATCH v2 6/8] kconfig: malloc: add option for skip memset at malloc init In-Reply-To: References: <1424099601-14979-1-git-send-email-p.marczak@samsung.com> <1424099601-14979-7-git-send-email-p.marczak@samsung.com> Message-ID: <54E4882D.4010300@samsung.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Simon, On 02/18/2015 05:32 AM, Simon Glass wrote: > Hi Przemyslaw, > > On 16 February 2015 at 08:13, Przemyslaw Marczak wrote: >> Signed-off-by: Przemyslaw Marczak >> --- >> Kconfig | 26 +++++++++++++++++++------- >> 1 file changed, 19 insertions(+), 7 deletions(-) >> >> diff --git a/Kconfig b/Kconfig >> index 4157da3..e08e44a 100644 >> --- a/Kconfig >> +++ b/Kconfig >> @@ -57,13 +57,25 @@ config CC_OPTIMIZE_FOR_SIZE >> This option is enabled by default for U-Boot. > > Ah, you have done this. Then I think you can merge this patch with the > dlmalloc patch and drop the README one. > Shouldn't we keep both, README and Kconfig help? Kconfig is just a configuration tool, README is a documentation. Sometimes it could be faster to find something in the text instead of config. >> >> menuconfig EXPERT >> - bool "Configure standard U-Boot features (expert users)" >> - help >> - This option allows certain base U-Boot options and settings >> - to be disabled or tweaked. This is for specialized >> - environments which can tolerate a "non-standard" U-Boot. >> - Only use this if you really know what you are doing. >> - >> + bool "Configure standard U-Boot features (expert users)" >> + help >> + This option allows certain base U-Boot options and settings >> + to be disabled or tweaked. This is for specialized >> + environments which can tolerate a "non-standard" U-Boot. >> + Only use this if you really know what you are doing. >> + >> +if EXPERT >> + config SYS_MALLOC_INIT_SKIP_ZEROING >> + bool "Skip memset at malloc init (reduce boot time)" >> + help >> + This avoids zeroing memory reserved for malloc at malloc init. >> + Significant boot time reduction is visible for configs in which >> + CONFIG_SYS_MALLOC_LEN value, has more than few MiB. >> + Useful for bzip2, bmp logo. >> + Warning: >> + When enable, make sure that calloc() is used when zeroed >> + memory is needed. >> +endif >> endmenu # General setup >> >> menu "Boot images" >> -- >> 1.9.1 >> > > Regards, > Simon > Best regards, -- Przemyslaw Marczak Samsung R&D Institute Poland Samsung Electronics p.marczak at samsung.com