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

Declare get_vmalloc_info_fn only when needed #297

Closed
wants to merge 1 commit into from

Conversation

ryao
Copy link
Contributor

@ryao ryao commented Sep 26, 2013

A couple of definitions were outside a preprocessor conditional
statement when they should have been inside it. This caused build
failures for some people.

Closes #291

Reported-by: Aaron Fineman [email protected]
Signed-off-by: Richard Yao [email protected]

A couple of definitions were outside a preprocessor conditional
statement when they should have been inside it. This caused build
failures for some people.

Closes openzfs#291

Reported-by: Aaron Fineman <[email protected]>
Signed-off-by: Richard Yao <[email protected]>
@behlendorf
Copy link
Contributor

@ryao I don't see how this will fix the issue. If you look at line 74 this code is already wrapped in #ifndef HAVE_GET_VMALLOC_INFO.

@behlendorf behlendorf closed this Oct 9, 2013
@ryao
Copy link
Contributor Author

ryao commented Oct 9, 2013

You are right. I missed that.

On Oct 9, 2013, at 4:59 PM, Brian Behlendorf [email protected] wrote:

@ryao I don't see how this will fix the issue. If you look at line 74 this code is already wrapped in #ifndef HAVE_GET_VMALLOC_INFO.


Reply to this email directly or view it on GitHub.

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.

Mainline 3.11 (2013-09-02) compile issues (get_vmalloc_info_fn)
2 participants