| Message ID | 3eadfe7d-9e95-ca8b-7541-6d4b90870f8a@amd.com |
|---|---|
| State | Accepted |
| Headers | show |
| Series | Increase number of max semaphores to fix OOM crashes on some systems | expand |
Hi Mario, Thanks! Could you please also add the "Signed-off-by" line for this patch? Cheers, Ivan On 6/28/23 12:19, Mario Limonciello wrote: > Fixes: https://github.com/fwts/fwts/issues/4 > Link: https://bugs.launchpad.net/fwts/+bug/2025193 > --- > src/libfwtsacpica/fwts_acpica.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/libfwtsacpica/fwts_acpica.c b/src/libfwtsacpica/fwts_acpica.c > index d8bc54a6..c174f71d 100644 > --- a/src/libfwtsacpica/fwts_acpica.c > +++ b/src/libfwtsacpica/fwts_acpica.c > @@ -50,7 +50,7 @@ > > #define ACPI_MAX_INIT_TABLES (64) /* Number of ACPI tables */ > > -#define MAX_SEMAPHORES (1024) /* For semaphore tracking */ > +#define MAX_SEMAPHORES (2048) /* For semaphore tracking */ > #define MAX_THREADS (128) /* For thread tracking */ > > #define MAX_WAIT_TIMEOUT (20) /* Seconds */
On 6/28/23 00:40, ivanhu wrote: > Hi Mario, > > Thanks! > Could you please also add the "Signed-off-by" line for this patch? > > Cheers, > Ivan > > On 6/28/23 12:19, Mario Limonciello wrote: >> Fixes: https://github.com/fwts/fwts/issues/4 >> Link: https://bugs.launchpad.net/fwts/+bug/2025193 Signed-off-by: Mario Limonciello <mario.limonciello@amd.com> >> --- >> src/libfwtsacpica/fwts_acpica.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/src/libfwtsacpica/fwts_acpica.c >> b/src/libfwtsacpica/fwts_acpica.c >> index d8bc54a6..c174f71d 100644 >> --- a/src/libfwtsacpica/fwts_acpica.c >> +++ b/src/libfwtsacpica/fwts_acpica.c >> @@ -50,7 +50,7 @@ >> >> #define ACPI_MAX_INIT_TABLES (64) /* Number of ACPI >> tables */ >> >> -#define MAX_SEMAPHORES (1024) /* For semaphore >> tracking */ >> +#define MAX_SEMAPHORES (2048) /* For semaphore >> tracking */ >> #define MAX_THREADS (128) /* For thread >> tracking */ >> >> #define MAX_WAIT_TIMEOUT (20) /* Seconds */
On 6/28/23 20:12, Mario Limonciello wrote: > On 6/28/23 00:40, ivanhu wrote: >> Hi Mario, >> >> Thanks! >> Could you please also add the "Signed-off-by" line for this patch? >> >> Cheers, >> Ivan >> >> On 6/28/23 12:19, Mario Limonciello wrote: >>> Fixes: https://github.com/fwts/fwts/issues/4 >>> Link: https://bugs.launchpad.net/fwts/+bug/2025193 > Signed-off-by: Mario Limonciello <mario.limonciello@amd.com> Thanks Mario, Acked-by: Ivan Hu <ivan.hu@canonical.com> >>> --- >>> src/libfwtsacpica/fwts_acpica.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/src/libfwtsacpica/fwts_acpica.c b/src/libfwtsacpica/fwts_acpica.c >>> index d8bc54a6..c174f71d 100644 >>> --- a/src/libfwtsacpica/fwts_acpica.c >>> +++ b/src/libfwtsacpica/fwts_acpica.c >>> @@ -50,7 +50,7 @@ >>> >>> #define ACPI_MAX_INIT_TABLES (64) /* Number of ACPI tables */ >>> >>> -#define MAX_SEMAPHORES (1024) /* For semaphore tracking */ >>> +#define MAX_SEMAPHORES (2048) /* For semaphore tracking */ >>> #define MAX_THREADS (128) /* For thread tracking */ >>> >>> #define MAX_WAIT_TIMEOUT (20) /* Seconds */ >
diff --git a/src/libfwtsacpica/fwts_acpica.c b/src/libfwtsacpica/fwts_acpica.c index d8bc54a6..c174f71d 100644 --- a/src/libfwtsacpica/fwts_acpica.c +++ b/src/libfwtsacpica/fwts_acpica.c @@ -50,7 +50,7 @@ #define ACPI_MAX_INIT_TABLES (64) /* Number of ACPI tables */ -#define MAX_SEMAPHORES (1024) /* For semaphore tracking */ +#define MAX_SEMAPHORES (2048) /* For semaphore tracking */ #define MAX_THREADS (128) /* For thread tracking */ #define MAX_WAIT_TIMEOUT (20) /* Seconds */