bug#44552: Extra dynamic-space-size for asdf-build-system/sbcl?

bug#44552: Extra dynamic-space-size for asdf-build-system/sbcl?

[Pierre Neidhardt]

[-- Attachment #1: Type: text/plain, Size: 367 bytes --]

See https://github.com/alex-gutev/generic-cl/issues/6

The space size of our current SBCL package is 2Gb.
Shall we increase it to 5Gb?

What about adding an option to the build system to pass extra parameters
to SBCL?  Then we could pass "--dynamic-space-size=5000" just for generic-cl.

Thoughts?  Guillaume?

-- 
Pierre Neidhardt
https://ambrevar.xyz/

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 511 bytes --]

bug#44552: Extra dynamic-space-size for asdf-build-system/sbcl?

[Guillaume Le Vaillant]

[-- Attachment #1: Type: text/plain, Size: 597 bytes --]


Pierre Neidhardt <mail@ambrevar.xyz> skribis:

> See https://github.com/alex-gutev/generic-cl/issues/6
>
> The space size of our current SBCL package is 2Gb.
> Shall we increase it to 5Gb?
>
> What about adding an option to the build system to pass extra parameters
> to SBCL?  Then we could pass "--dynamic-space-size=5000" just for generic-cl.
>
> Thoughts?  Guillaume?

I guess we could increase the default max space size to 4 or 5 GB
without causing swapping issues on users' machines (at least on x86-64).
And users wanting a smaller one can use the "dynamic-space-size" option
at runtime.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 247 bytes --]

bug#44552: Extra dynamic-space-size for asdf-build-system/sbcl?

[Pierre Neidhardt]

[-- Attachment #1: Type: text/plain, Size: 383 bytes --]

Guillaume Le Vaillant <glv@posteo.net> writes:

> I guess we could increase the default max space size to 4 or 5 GB
> without causing swapping issues on users' machines (at least on x86-64).
> And users wanting a smaller one can use the "dynamic-space-size" option
> at runtime.

Fine with me, let's do it now on staging then?

-- 
Pierre Neidhardt
https://ambrevar.xyz/

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 511 bytes --]

bug#44552: Extra dynamic-space-size for asdf-build-system/sbcl?

[Guillaume Le Vaillant]

[-- Attachment #1: Type: text/plain, Size: 401 bytes --]


Pierre Neidhardt <mail@ambrevar.xyz> skribis:

> Guillaume Le Vaillant <glv@posteo.net> writes:
>
>> I guess we could increase the default max space size to 4 or 5 GB
>> without causing swapping issues on users' machines (at least on x86-64).
>> And users wanting a smaller one can use the "dynamic-space-size" option
>> at runtime.
>
> Fine with me, let's do it now on staging then?

Yes, go ahead.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 247 bytes --]

bug#44552: Extra dynamic-space-size for asdf-build-system/sbcl?

[Pierre Neidhardt]

[-- Attachment #1: Type: text/plain, Size: 159 bytes --]

It failed with "5Gb", it worked with "6000".
I'm testing with "5000" since upstream said it worked for them.

-- 
Pierre Neidhardt
https://ambrevar.xyz/

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 511 bytes --]

bug#44552: Extra dynamic-space-size for asdf-build-system/sbcl?

[Pierre Neidhardt]

[-- Attachment #1: Type: text/plain, Size: 451 bytes --]

Pierre Neidhardt <mail@ambrevar.xyz> writes:

> It failed with "5Gb", it worked with "6000".
> I'm testing with "5000" since upstream said it worked for them.

Nope, it does not :p

So which value should we set?  It's not because 6000 passes on my
machine that it will necessary pass on all the Guix build servers.

I'm leaning towards disabling the tests for generic-cl.  What do you think?

-- 
Pierre Neidhardt
https://ambrevar.xyz/

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 511 bytes --]

bug#44552: Extra dynamic-space-size for asdf-build-system/sbcl?

[Guillaume Le Vaillant]

[-- Attachment #1: Type: text/plain, Size: 602 bytes --]


Pierre Neidhardt <mail@ambrevar.xyz> skribis:

> Pierre Neidhardt <mail@ambrevar.xyz> writes:
>
>> It failed with "5Gb", it worked with "6000".
>> I'm testing with "5000" since upstream said it worked for them.
>
> Nope, it does not :p
>
> So which value should we set?  It's not because 6000 passes on my
> machine that it will necessary pass on all the Guix build servers.
>
> I'm leaning towards disabling the tests for generic-cl.  What do you think?

If the tests don't run reliably, disabling them sounds like a good idea.
IIRC the tests of sbcl-numcl are disabled because of a similar problem.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 247 bytes --]

bug#44552: Extra dynamic-space-size for asdf-build-system/sbcl?

[Pierre Neidhardt]

[-- Attachment #1: Type: text/plain, Size: 86 bytes --]

OK, will do when I merge generic-cl!

-- 
Pierre Neidhardt
https://ambrevar.xyz/

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 511 bytes --]