Hi Raymond On Sat, 17 Aug 2024 at 00:45, Raymond Mao <raymond....@linaro.org> wrote: > > Port mbedtls with adapted libc header files. > Add mbedtls default config header file. > Optimize mbedtls default config by disabling unused features to > reduce the target size. > Add mbedtls kbuild makefile. > Add Kconfig skeleton and config submenu entry for selecting > crypto libraries between mbedtls and legacy ones. > Add the mbedtls include directories into the build system. > > Subsequent patches will separate those Kconfigs into pairs of > _LEGACY and _MBEDTLS for controlling the implementations of legacy > crypto libraries and MbedTLS ones respectively. > > The motivation of moving and adapting *INT* macros from kernel.h > to limits.h is to fullfill the MbedTLS building requirement. > The conditional compilation statements in MbedTLS expects the > *INT* macros as constant expressions, thus expressions like > `((int)(~0U >> 1))` will not work. > > Prerequisite > ------------ > > This patch series requires mbedtls git repo to be added as a > subtree to the main U-Boot repo via: > > $ git subtree add --prefix lib/mbedtls/external/mbedtls \ > https://github.com/Mbed-TLS/mbedtls.git \ > v3.6.0 --squash > > Moreover, due to the Windows-style files from mbedtls git repo, > we need to convert the CRLF endings to LF and do a commit manually: > > $ git add --renormalize . > $ git commit > > Signed-off-by: Raymond Mao <raymond....@linaro.org> > --- > Changes in v2 > - Disabled unused MbedTLS features to optimize the target size. > Changes in v3 > - Removed changes in stdio.h. > Changes in v4 > - Move limits.h as a common header file that is included by kernel.h. > - Refactor the Kconfig to support legacy and MbedTLS options for each > algorithm. > - Refactor MbedTLS makefile and default config file to remove unused > config options and objects. > Changes in v5 > - Merged patch #9 of v4 into this patch. > - Removed unused config MBEDTLS_LIB_TLS. > - Refactored MbedTLS Makefile and default config file. > Changes in v6 > - Fixed UINT64_MAX. > - Removed copy right statement from limits.h > > Makefile | 6 +++ > include/limits.h | 25 ++++++++++++ > include/linux/kernel.h | 13 +----- > include/stdlib.h | 1 + > lib/Kconfig | 4 ++ > lib/Makefile | 2 + > lib/mbedtls/Kconfig | 47 ++++++++++++++++++++++ > lib/mbedtls/Makefile | 41 +++++++++++++++++++ > lib/mbedtls/mbedtls_def_config.h | 69 ++++++++++++++++++++++++++++++++ > lib/mbedtls/port/assert.h | 12 ++++++ > 10 files changed, 208 insertions(+), 12 deletions(-) > create mode 100644 include/limits.h > create mode 100644 lib/mbedtls/Kconfig > create mode 100644 lib/mbedtls/Makefile > create mode 100644 lib/mbedtls/mbedtls_def_config.h > create mode 100644 lib/mbedtls/port/assert.h > > diff --git a/Makefile b/Makefile > index b35a472d9be..3c506c299a1 100644 > --- a/Makefile > +++ b/Makefile > @@ -829,6 +829,12 @@ KBUILD_HOSTCFLAGS += $(if $(CONFIG_TOOLS_DEBUG),-g) > UBOOTINCLUDE := \ > -Iinclude \ > $(if $(KBUILD_SRC), -I$(srctree)/include) \ > + $(if $(CONFIG_MBEDTLS_LIB), \ > + "-DMBEDTLS_CONFIG_FILE=\"mbedtls_def_config.h\"" \ > + -I$(srctree)/lib/mbedtls \ > + -I$(srctree)/lib/mbedtls/port \ > + -I$(srctree)/lib/mbedtls/external/mbedtls \ > + -I$(srctree)/lib/mbedtls/external/mbedtls/include) \ > $(if $(CONFIG_$(SPL_)SYS_THUMB_BUILD), \ > $(if $(CONFIG_HAS_THUMB2), \ > $(if $(CONFIG_CPU_V7M), \ > diff --git a/include/limits.h b/include/limits.h > new file mode 100644 > index 00000000000..be219ddbfca > --- /dev/null > +++ b/include/limits.h > @@ -0,0 +1,25 @@ > +/* SPDX-License-Identifier: GPL-2.0+ */ > + > +#ifndef _LIMITS_H > +#define _LIMITS_H > + > +#define INT_MAX 0x7fffffff > +#define UINT_MAX 0xffffffffUL
this is 0xffffffffU > +#define CHAR_BIT 8 > +#define UINT32_MAX 0xffffffffUL Ditto > +#define UINT64_MAX 0xffffffffffffffffULL > + > +#ifdef CONFIG_64BIT > + #define UINTPTR_MAX UINT64_MAX > +#else > + #define UINTPTR_MAX UINT32_MAX > +#endif > + > +#ifndef SIZE_MAX > +#define SIZE_MAX UINTPTR_MAX > +#endif > +#ifndef SSIZE_MAX > +#define SSIZE_MAX ((ssize_t)(SIZE_MAX >> 1)) > +#endif > + > +#endif /* _LIMITS_H */ > diff --git a/include/linux/kernel.h b/include/linux/kernel.h > index 939465f372b..9467edd65ab 100644 > --- a/include/linux/kernel.h > +++ b/include/linux/kernel.h > @@ -3,25 +3,18 @@ > > #include <linux/types.h> > #include <linux/printk.h> /* for printf/pr_* utilities */ > +#include <limits.h> > > #define USHRT_MAX ((u16)(~0U)) > #define SHRT_MAX ((s16)(USHRT_MAX>>1)) > #define SHRT_MIN ((s16)(-SHRT_MAX - 1)) > -#define INT_MAX ((int)(~0U>>1)) > #define INT_MIN (-INT_MAX - 1) > -#define UINT_MAX (~0U) > #define LONG_MAX ((long)(~0UL>>1)) > #define LONG_MIN (-LONG_MAX - 1) > #define ULONG_MAX (~0UL) > #define LLONG_MAX ((long long)(~0ULL>>1)) > #define LLONG_MIN (-LLONG_MAX - 1) > #define ULLONG_MAX (~0ULL) > -#ifndef SIZE_MAX > -#define SIZE_MAX (~(size_t)0) > -#endif > -#ifndef SSIZE_MAX > -#define SSIZE_MAX ((ssize_t)(SIZE_MAX >> 1)) > -#endif > > #define U8_MAX ((u8)~0U) > #define S8_MAX ((s8)(U8_MAX>>1)) > @@ -36,10 +29,6 @@ > #define S64_MAX ((s64)(U64_MAX>>1)) > #define S64_MIN ((s64)(-S64_MAX - 1)) > > -/* Aliases defined by stdint.h */ > -#define UINT32_MAX U32_MAX > -#define UINT64_MAX U64_MAX > - > #define INT32_MAX S32_MAX > > #define STACK_MAGIC 0xdeadbeef > diff --git a/include/stdlib.h b/include/stdlib.h > index 9c175d4d74c..dedfd52a144 100644 > --- a/include/stdlib.h > +++ b/include/stdlib.h > @@ -7,5 +7,6 @@ > #define __STDLIB_H_ > > #include <malloc.h> > +#include <rand.h> > > #endif /* __STDLIB_H_ */ > diff --git a/lib/Kconfig b/lib/Kconfig > index 2059219a120..8b170dcc67e 100644 > --- a/lib/Kconfig > +++ b/lib/Kconfig > @@ -418,6 +418,10 @@ config CIRCBUF > > source "lib/dhry/Kconfig" > > +menu "Alternative crypto libraries" > +source lib/mbedtls/Kconfig > +endmenu > + [....] with the above fixed Reviewed-by: Ilias Apalodimas <ilias.apalodi...@linaro.org>
