Skip to content
This repository has been archived by the owner on Feb 26, 2020. It is now read-only.

Fix HAVE_MUTEX_OWNER test for kernels prior to 4.6 #566

Closed
wants to merge 1 commit into from

Conversation

dweeezil
Copy link
Contributor

@dweeezil dweeezil commented Aug 1, 2016

Recent 4.X kernels prior to 4.6 require #include of spinlock.h in
order to get the definition of __ARCH_SPIN_LOCK_UNLOCKED which is used
by DEFINE_MUTEX().

[EDIT: missing space in commit comment fixed but not yet committed/pushed]

Recent 4.X kernels prior to 4.6 require #include of spinlock.h in
order to get the definition of __ARCH_SPIN_LOCK_UNLOCKED which isused
by DEFINE_MUTEX().
@dweeezil
Copy link
Contributor Author

dweeezil commented Aug 1, 2016

Sorry, but I've not figured out which kernel commit prior to 4.6 obviates the need for #include <linux/spinlock.h nor whether this is a problem in any 3.X kernels.

@behlendorf
Copy link
Contributor

Interesting find, well even if we can't pin it down to a specific commit adding the additional include is safe and probably a good idea.

@behlendorf behlendorf closed this in 576865b Aug 1, 2016
nedbass pushed a commit to nedbass/spl that referenced this pull request Aug 26, 2016
Recent 4.X kernels prior to 4.6 require #include of spinlock.h in
order to get the definition of __ARCH_SPIN_LOCK_UNLOCKED which is
used by DEFINE_MUTEX().

Signed-off-by: Tim Chase <tim@chase2k.com>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes openzfs#566
tuxoko pushed a commit to tuxoko/spl that referenced this pull request Sep 8, 2016
Recent 4.X kernels prior to 4.6 require #include of spinlock.h in
order to get the definition of __ARCH_SPIN_LOCK_UNLOCKED which is
used by DEFINE_MUTEX().

Signed-off-by: Tim Chase <tim@chase2k.com>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes openzfs#566
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants