-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Revert "Linux 6.5 compat: register_sysctl_table removed" #15134
Conversation
This reverts commit b35374f as there are error messages when loading the SPL module. Errors seemed to be tied to duplicate a duplicate entry. Signed-off-by: Brian Atkinson <batkinson@lanl.gov>
@ckane can you take a look at this. |
Yeah I can take a look - is this only happening on Linux 4.18/CentOS 8, or also has this been repeated on newer systems? @bwatkinson can you share the |
also another question: do you need ZFS pools imported to repeat this behavior? Or does it crash with no active pools? |
I also see these spl_init hangs, on Ubuntu 22.04.2, kernel 6.2.xxx (6.2.0-1009-lowlatency). Will retest now that the backout has landed. |
sigh
and #ifdef HAVE_REGISTER_SYSCTL_TABLE Gonna verify fixing this typo corrects the problem and then I'll re-PR. |
Yeah, much better after the backout. :) Cheers. |
Here's the new PR with the typo fixed: #15138 |
Thanks everyone for catching this one and for providing the helpful feedback here! |
This reverts commit b35374f as there are error messages when loading the SPL module. Errors seemed to be tied to duplicate a duplicate entry. Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov> Signed-off-by: Brian Atkinson <batkinson@lanl.gov> Closes openzfs#15134
This reverts commit b35374f as there are error messages when loading the SPL module. Errors seemed to be tied to duplicate a duplicate entry. Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov> Signed-off-by: Brian Atkinson <batkinson@lanl.gov> Closes openzfs#15134
This reverts commit b35374f as there are error messages when loading the SPL module. Errors seemed to be tied to duplicate a duplicate entry. Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov> Signed-off-by: Brian Atkinson <batkinson@lanl.gov> Closes openzfs#15134
This reverts commit b35374f as there are error messages when loading the SPL module. Errors seemed to be tied to duplicate a duplicate entry.
Reverting b35374f as there is a error when trying to load the SPL module using
zfs.sh
.Motivation and Context
Fixes issue with loading SPL kernel module. Here is stack trace from dmesg that shows the error that is happening:
Description
Just reverting b35374f for now till this issue is resolved.
How Has This Been Tested?
Ran
zfs.sh -v
. Sometimes you have to run it twice, such as doingzfs.sh -v && zfs.sh -uv && zfs.sh -v
for the error to trigger.CentOS Stream release 8
Kernel: 4.18.0-408.el8.x86_64
Types of changes
Checklist:
Signed-off-by
.