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

fix build issue on some configured Kernels. #510

Closed
wants to merge 1 commit into from

Conversation

zgock
Copy link

@zgock zgock commented Dec 11, 2015

spl fails to build with some "Configured" kernels (ex. openSUSE xen Kernel)
this change should make same binaries with C compiler optimization.
thx.

@behlendorf
Copy link
Contributor

I agree, this is much nicer approach since it allows us to use the ZFS code unmodified. Let me verify it doesn't cause issues on any other platforms. Also we'll want to remove the #define entirely not just comment it out.

behlendorf added a commit to behlendorf/zfs that referenced this pull request Dec 11, 2015
An intentionally empty commit used to trigger the buildbot such it
can test openzfs/spl#510.

Requires-spl: refs/pull/510/head
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
@behlendorf
Copy link
Contributor

LGTM, merged as:

70d80b3 Fix build issue on some configured kernels

tuxoko pushed a commit to tuxoko/spl that referenced this pull request Dec 14, 2015
The SPL fails to build with some "Configured" kernels (ex. openSUSE
xen Kernel) this change should make same binaries with C compiler
optimization.

Signed-off-by: zgock <zgock@nuc.base.zgock-lab.net>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes openzfs#510
nedbass pushed a commit that referenced this pull request Dec 24, 2015
The SPL fails to build with some "Configured" kernels (ex. openSUSE
xen Kernel) this change should make same binaries with C compiler
optimization.

Signed-off-by: zgock <zgock@nuc.base.zgock-lab.net>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #510
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