Re: [edk2-libc Patch] StdLib/Include/sys/syslimits.h: Increase OPEN_MAX from 20 to 255

["Rebecca Cran" <rebecca@nuviainc.com>]

Reviewed-by: Rebecca Cran <rebecca@nuviainc.com>


On 8/13/21 10:33 AM, Michael D Kinney wrote:
> From: Jayaprakash Nevara <n.jayaprakash@intel.com>
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3548
>
> A port of Python 3.6.8 interpreter for UEFI uses the
> standard C Library from the edk2-libc project. The python
> interpreter fails with the following error when the
> python interpreter is launched:
>
> "IOError: [Errno 24] Too many open files"
>
> Python 3.6.8 interpreter opens and loads many python modules
> (files) when it is boot strapped and this is leading to the
> too many open files error.
>
> To fix the issue the OPEN_MAX limit value is increased from
> 20 to 255.
>
> Cc: Daryl McDaniel <edk2-lists@mc2research.org>
> Cc: Jaben Carsey <jaben.carsey@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Rebecca Cran <rebecca@nuviainc.com>
> Signed-off-by: Jayaprakash Nevara <n.jayaprakash@intel.com>
> ---
>   StdLib/Include/sys/syslimits.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/StdLib/Include/sys/syslimits.h b/StdLib/Include/sys/syslimits.h
> index 9515e8f..70ec619 100644
> --- a/StdLib/Include/sys/syslimits.h
> +++ b/StdLib/Include/sys/syslimits.h
> @@ -53,7 +53,7 @@
>   #define MAX_OUTPUT        255   ///< Maximum bytes in terminal output.
>   #define NAME_MAX          255   ///< Maximum bytes in a file name.
>   #ifndef OPEN_MAX
> -  #define OPEN_MAX         20   ///< Maximum open files per process.
> +  #define OPEN_MAX         255   ///< Maximum open files per process.
>   #endif
>   #define PATH_MAX         1024   ///< Maximum bytes in pathname.
>   #define PIPE_BUF          512   ///< Maximum bytes for atomic pipe writes.