Page MenuHomeFreeBSD

param: increase KSTACK for hash context
Needs ReviewPublic

Authored by mkoz_semihalf.com on Nov 18 2022, 5:44 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jun 30, 10:02 AM
Unknown Object (File)
May 10 2024, 7:41 PM
Unknown Object (File)
May 8 2024, 9:17 PM
Unknown Object (File)
May 3 2024, 8:31 AM
Unknown Object (File)
Apr 26 2024, 7:11 PM
Unknown Object (File)
Apr 26 2024, 1:53 PM
Unknown Object (File)
Apr 21 2024, 6:01 PM
Unknown Object (File)
Dec 20 2023, 7:35 AM
Subscribers

Details

Reviewers
wma
mw
kd
imp
Summary

ossl_hash_context was increased (244 -> 2048bytes) for arm
https://reviews.freebsd.org/D37420

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

The default config Armada 38x should run with is sys/arm/conf/GENERIC. Can you please add a commit with that?

mw requested changes to this revision.Nov 18 2022, 7:16 PM
mw added inline comments.
sys/arm/conf/ARMADA38X
25

This deserves a verbose commit message with explanation and probably a comment.

This revision now requires changes to proceed.Nov 18 2022, 7:16 PM

Since this isn't an Armada38x specific change I believe that it should be applied to all kernel configs.
IMHO we should modify the default value in arm/include/param.h.
Another thing is that 8 pages might be more than needed, e.g. amd64 defaults to 4/6 depending on whether KASAN is enabled.
Also as mw mentioned the commit message should contain a verbose explanation why we need to increase the kernel stack size.

Why do you need 8 pages? That's a large increase from the 2 currently used on arm.

mkoz_semihalf.com retitled this revision from conf: add support for neon/ossl to ARMADA38X to param: increase KSTACK for hash context.
mkoz_semihalf.com edited the summary of this revision. (Show Details)

Change default KSTACK_PAGES to 4