Skip to content
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

LSE-373: workqueue: Fix errors seen for newer (v6.4 and later) kernels. #87

Merged
merged 1 commit into from
Jun 2, 2024

Conversation

imran-kn
Copy link
Contributor

LSE-373.

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label May 31, 2024
@imran-kn imran-kn changed the title workqueue: Fix errors seen for newer (v6.4 and later) kernels. LSE-373: workqueue: Fix errors seen for newer (v6.4 and later) kernels. May 31, 2024
@imran-kn imran-kn requested a review from brenns10 May 31, 2024 02:06
Copy link
Member

@brenns10 brenns10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested this on UEK-next and it resolves all the workqueue related issues. Looks to pass on gitlab CI too. However, one tiny nit if you don't mind. Thanks!

Comment on lines 114 to 119
prog = workqueue.prog_
pwq = Object(
prog,
"struct pool_workqueue",
address=prog.read_word(per_cpu_ptr(workqueue.cpu_pwq, cpu)),
).address_of_()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can simplify this by just dereferencing the double pointer.

Suggested change
prog = workqueue.prog_
pwq = Object(
prog,
"struct pool_workqueue",
address=prog.read_word(per_cpu_ptr(workqueue.cpu_pwq, cpu)),
).address_of_()
pwq = per_cpu_ptr(workqueue.cpu_pwq, cpu)[0]

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense. Have done that now.

Copy link
Member

@brenns10 brenns10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the change, everything else is good! The test failure is unrelated to your change.

@brenns10 brenns10 merged commit 2c85424 into main Jun 2, 2024
4 of 5 checks passed
@brenns10 brenns10 deleted the LSE-373 branch June 3, 2024 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants