[PATCH 0/5] lmb: Add config for the number of memory and reserved regions

[Patrick DELAUNAY]

Hi Tom,

On 3/2/21 9:50 PM, Tom Rini wrote:
> On Tue, Feb 02, 2021 at 01:59:07PM +0100, Patrick Delaunay wrote:
>
>> Hi,
>>
>> I propose a update of the lmb library to allow the configuration
>> with Kconfig of the number of memory and reserved regions in lmb
>> libary:
>> - CONFIG_LMB_RESERVED_REGIONS
>> - CONFIG_LMB_MEMORY_REGIONS
>>
>> By default, I keep the default value of 8 regions.
>>
>> This serie avoids issue on stm32mp15 platform,
>> as the kernel device tree defines many and non contiguous reserved
>> regions.
>>
>>
>>
>> Patrick Delaunay (5):
>>    lmb: move CONFIG_LMB in Kconfig
>>    lmb: remove lmb_region.size
>>    lmb: Move lmb property arrays in struct lmb
>>    lmb: Add 2 config to define the max number of regions
>>    configs: stm32mp15: increase the number of reserved memory region in
>>      lmb
>>
>>   arch/arc/include/asm/config.h        |  2 --
>>   arch/arm/include/asm/config.h        |  1 -
>>   arch/m68k/include/asm/config.h       |  1 -
>>   arch/microblaze/include/asm/config.h |  2 --
>>   arch/mips/include/asm/config.h       |  1 -
>>   arch/nds32/include/asm/config.h      |  1 -
>>   arch/powerpc/include/asm/config.h    |  1 -
>>   arch/riscv/include/asm/config.h      |  1 -
>>   arch/sh/include/asm/config.h         |  2 --
>>   arch/x86/include/asm/config.h        |  1 -
>>   arch/xtensa/include/asm/config.h     |  2 --
>>   configs/stm32mp15_basic_defconfig    |  1 +
>>   configs/stm32mp15_trusted_defconfig  |  1 +
>>   include/configs/10m50_devboard.h     |  5 ----
>>   include/configs/3c120_devboard.h     |  5 ----
>>   include/configs/sandbox.h            |  2 --
>>   include/configs/x86-common.h         |  2 --
>>   include/image.h                      |  2 +-
>>   include/lmb.h                        | 34 ++++++++++++++++++++++++----
>>   lib/Kconfig                          | 23 +++++++++++++++++++
>>   lib/lmb.c                            | 14 ++++++------
>>   scripts/config_whitelist.txt         |  1 -
>>   22 files changed, 63 insertions(+), 42 deletions(-)
> This series causes a little bit of size growth everywhere.  It's fatal
> on nokia_rx51 (and caught via the test script).  We can bring nokia_rx51
> back under size by disabling the optimized memcpy/memset functions, but
> that's not ideal.  Can we do your changes here in a more size-concious
> way?  Thanks!
>

Sure, I will check and correct this issue.


Patrick