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

Michael D Kinney posted 1 patch 2 years, 8 months ago
Failed in applying to current master (apply log)
StdLib/Include/sys/syslimits.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[edk2-devel] [edk2-libc Patch] StdLib/Include/sys/syslimits.h: Increase OPEN_MAX from 20 to 255
Posted by Michael D Kinney 2 years, 8 months ago
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.
-- 
2.32.0.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#79283): https://edk2.groups.io/g/devel/message/79283
Mute This Topic: https://groups.io/mt/84867510/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [edk2-libc Patch] StdLib/Include/sys/syslimits.h: Increase OPEN_MAX from 20 to 255
Posted by Rebecca Cran 2 years, 8 months ago
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.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#79545): https://edk2.groups.io/g/devel/message/79545
Mute This Topic: https://groups.io/mt/84867510/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-