Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Build issue with kernel 5.4 #25

Open
Mantra84 opened this issue Jul 20, 2021 · 3 comments
Open

Build issue with kernel 5.4 #25

Mantra84 opened this issue Jul 20, 2021 · 3 comments

Comments

@Mantra84
Copy link

Hello,

While compiling kernel 5.4.125 with the latest KTF version (SHA-1 63c19de), I got the nla_strscpy implicit declaration error.

This issue should have been solved with the lines in ktf_compat.h (line 56):

#if (KERNEL_VERSION(5, 11, 0) > LINUX_VERSION_CODE)
#define nla_strscpy nla_strlcpy
#endif

But it seems that this branch is not entered.

I think the issue come form the first condition in ktf_compat.h (line 1):

#if (KERNEL_VERSION(5, 2, 0) > LINUX_VERSION_CODE)

Maybe this line should be change to:

--- a/kernel/ktf_compat.h
+++ b/kernel/ktf_compat.h
@@ -1,4 +1,4 @@
-#if (KERNEL_VERSION(5, 2, 0) > LINUX_VERSION_CODE)
+#if (KERNEL_VERSION(5, 11, 0) > LINUX_VERSION_CODE)

What do you think about it ?

Thank you.

@knuto
Copy link
Contributor

knuto commented Jul 21, 2021

You are quite right. I I think I'll propose we just remove the other define to avoid making this mistake again in the future as more compat stuff is needed. Thanks for the report!

@knuto
Copy link
Contributor

knuto commented Jul 21, 2021

I have updated knuto/ktf to fix the issue (pull request #26)

@Mantra84
Copy link
Author

Thanks for the fix 😄

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants