ACK: Increase number of max semaphores to fix OOM crashes on some systems

ivanhu ivan.hu at canonical.com
Thu Jun 29 01:51:04 UTC 2023



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 at amd.com>

Thanks Mario,

Acked-by: Ivan Hu <ivan.hu at 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 */
> 



More information about the fwts-devel mailing list