* [PATCH] ARM: uaccess: Add missing include for set_thread_flag
@ 2017-09-19 11:50 ` Jonathan Liu
0 siblings, 0 replies; 10+ messages in thread
From: Jonathan Liu @ 2017-09-19 11:50 UTC (permalink / raw)
To: linux-arm-kernel
Fixes "implicit declaration of function" compile error for out-of-tree
kernel modules including asm/uaccess.h.
Fixes: 73ac5d6a2b6a ("arm/syscalls: Check address limit on user-mode return")
Signed-off-by: Jonathan Liu <net147@gmail.com>
---
arch/arm/include/asm/uaccess.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/include/asm/uaccess.h b/arch/arm/include/asm/uaccess.h
index 87936dd5d151..13d1877ffb75 100644
--- a/arch/arm/include/asm/uaccess.h
+++ b/arch/arm/include/asm/uaccess.h
@@ -17,6 +17,7 @@
#include <asm/unified.h>
#include <asm/compiler.h>
+#include <linux/thread_info.h>
#include <asm/extable.h>
/*
--
2.13.2
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH] ARM: uaccess: Add missing include for set_thread_flag @ 2017-09-19 11:50 ` Jonathan Liu 0 siblings, 0 replies; 10+ messages in thread From: Jonathan Liu @ 2017-09-19 11:50 UTC (permalink / raw) To: Russell King, Thomas Garnier, Thomas Gleixner Cc: linux-arm-kernel, linux-kernel, Jonathan Liu Fixes "implicit declaration of function" compile error for out-of-tree kernel modules including asm/uaccess.h. Fixes: 73ac5d6a2b6a ("arm/syscalls: Check address limit on user-mode return") Signed-off-by: Jonathan Liu <net147@gmail.com> --- arch/arm/include/asm/uaccess.h | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/include/asm/uaccess.h b/arch/arm/include/asm/uaccess.h index 87936dd5d151..13d1877ffb75 100644 --- a/arch/arm/include/asm/uaccess.h +++ b/arch/arm/include/asm/uaccess.h @@ -17,6 +17,7 @@ #include <asm/unified.h> #include <asm/compiler.h> +#include <linux/thread_info.h> #include <asm/extable.h> /* -- 2.13.2 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH] ARM: uaccess: Add missing include for set_thread_flag 2017-09-19 11:50 ` Jonathan Liu @ 2017-09-19 14:32 ` Thomas Garnier -1 siblings, 0 replies; 10+ messages in thread From: Thomas Garnier @ 2017-09-19 14:32 UTC (permalink / raw) To: linux-arm-kernel On Tue, Sep 19, 2017 at 4:50 AM, Jonathan Liu <net147@gmail.com> wrote: > Fixes "implicit declaration of function" compile error for out-of-tree > kernel modules including asm/uaccess.h. I failed to reproduce this issue by creating an out of tree module with a separate file (with only uaccess.h). Are you using a special config? Looking at the headers on uaccess.h. You get thread_info through: asm/domain.h -> asm/thread_info.h > > Fixes: 73ac5d6a2b6a ("arm/syscalls: Check address limit on user-mode return") > Signed-off-by: Jonathan Liu <net147@gmail.com> > --- > arch/arm/include/asm/uaccess.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/include/asm/uaccess.h b/arch/arm/include/asm/uaccess.h > index 87936dd5d151..13d1877ffb75 100644 > --- a/arch/arm/include/asm/uaccess.h > +++ b/arch/arm/include/asm/uaccess.h > @@ -17,6 +17,7 @@ > #include <asm/unified.h> > #include <asm/compiler.h> > > +#include <linux/thread_info.h> > #include <asm/extable.h> > > /* > -- > 2.13.2 > -- Thomas ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: uaccess: Add missing include for set_thread_flag @ 2017-09-19 14:32 ` Thomas Garnier 0 siblings, 0 replies; 10+ messages in thread From: Thomas Garnier @ 2017-09-19 14:32 UTC (permalink / raw) To: Jonathan Liu; +Cc: Russell King, Thomas Gleixner, Linux ARM, LKML On Tue, Sep 19, 2017 at 4:50 AM, Jonathan Liu <net147@gmail.com> wrote: > Fixes "implicit declaration of function" compile error for out-of-tree > kernel modules including asm/uaccess.h. I failed to reproduce this issue by creating an out of tree module with a separate file (with only uaccess.h). Are you using a special config? Looking at the headers on uaccess.h. You get thread_info through: asm/domain.h -> asm/thread_info.h > > Fixes: 73ac5d6a2b6a ("arm/syscalls: Check address limit on user-mode return") > Signed-off-by: Jonathan Liu <net147@gmail.com> > --- > arch/arm/include/asm/uaccess.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/include/asm/uaccess.h b/arch/arm/include/asm/uaccess.h > index 87936dd5d151..13d1877ffb75 100644 > --- a/arch/arm/include/asm/uaccess.h > +++ b/arch/arm/include/asm/uaccess.h > @@ -17,6 +17,7 @@ > #include <asm/unified.h> > #include <asm/compiler.h> > > +#include <linux/thread_info.h> > #include <asm/extable.h> > > /* > -- > 2.13.2 > -- Thomas ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] ARM: uaccess: Add missing include for set_thread_flag 2017-09-19 14:32 ` Thomas Garnier @ 2017-09-19 21:35 ` Jonathan Liu -1 siblings, 0 replies; 10+ messages in thread From: Jonathan Liu @ 2017-09-19 21:35 UTC (permalink / raw) To: linux-arm-kernel Hi Thomas, The top of the C source file I am compiling has: #include <linux/kernel.h> #include <asm/uaccess.h> Tracing through the asm/uaccess.h include I see asm/uaccess.h -> asm/domain.h -> asm/thread_info.h but set_thread_info is defined in linux/thread_info.h not asm/thread_info.h (see http://elixir.free-electrons.com/linux/v4.14-rc1/ident/set_thread_flag). Regards, Jonathan On 20 September 2017 at 00:32, Thomas Garnier <thgarnie@google.com> wrote: > On Tue, Sep 19, 2017 at 4:50 AM, Jonathan Liu <net147@gmail.com> wrote: >> Fixes "implicit declaration of function" compile error for out-of-tree >> kernel modules including asm/uaccess.h. > > I failed to reproduce this issue by creating an out of tree module > with a separate file (with only uaccess.h). Are you using a special > config? > > Looking at the headers on uaccess.h. You get thread_info through: > > asm/domain.h -> asm/thread_info.h > >> >> Fixes: 73ac5d6a2b6a ("arm/syscalls: Check address limit on user-mode return") >> Signed-off-by: Jonathan Liu <net147@gmail.com> >> --- >> arch/arm/include/asm/uaccess.h | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/arch/arm/include/asm/uaccess.h b/arch/arm/include/asm/uaccess.h >> index 87936dd5d151..13d1877ffb75 100644 >> --- a/arch/arm/include/asm/uaccess.h >> +++ b/arch/arm/include/asm/uaccess.h >> @@ -17,6 +17,7 @@ >> #include <asm/unified.h> >> #include <asm/compiler.h> >> >> +#include <linux/thread_info.h> >> #include <asm/extable.h> >> >> /* >> -- >> 2.13.2 >> > > > > -- > Thomas ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: uaccess: Add missing include for set_thread_flag @ 2017-09-19 21:35 ` Jonathan Liu 0 siblings, 0 replies; 10+ messages in thread From: Jonathan Liu @ 2017-09-19 21:35 UTC (permalink / raw) To: Thomas Garnier; +Cc: Russell King, Thomas Gleixner, Linux ARM, LKML Hi Thomas, The top of the C source file I am compiling has: #include <linux/kernel.h> #include <asm/uaccess.h> Tracing through the asm/uaccess.h include I see asm/uaccess.h -> asm/domain.h -> asm/thread_info.h but set_thread_info is defined in linux/thread_info.h not asm/thread_info.h (see http://elixir.free-electrons.com/linux/v4.14-rc1/ident/set_thread_flag). Regards, Jonathan On 20 September 2017 at 00:32, Thomas Garnier <thgarnie@google.com> wrote: > On Tue, Sep 19, 2017 at 4:50 AM, Jonathan Liu <net147@gmail.com> wrote: >> Fixes "implicit declaration of function" compile error for out-of-tree >> kernel modules including asm/uaccess.h. > > I failed to reproduce this issue by creating an out of tree module > with a separate file (with only uaccess.h). Are you using a special > config? > > Looking at the headers on uaccess.h. You get thread_info through: > > asm/domain.h -> asm/thread_info.h > >> >> Fixes: 73ac5d6a2b6a ("arm/syscalls: Check address limit on user-mode return") >> Signed-off-by: Jonathan Liu <net147@gmail.com> >> --- >> arch/arm/include/asm/uaccess.h | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/arch/arm/include/asm/uaccess.h b/arch/arm/include/asm/uaccess.h >> index 87936dd5d151..13d1877ffb75 100644 >> --- a/arch/arm/include/asm/uaccess.h >> +++ b/arch/arm/include/asm/uaccess.h >> @@ -17,6 +17,7 @@ >> #include <asm/unified.h> >> #include <asm/compiler.h> >> >> +#include <linux/thread_info.h> >> #include <asm/extable.h> >> >> /* >> -- >> 2.13.2 >> > > > > -- > Thomas ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] ARM: uaccess: Add missing include for set_thread_flag 2017-09-19 21:35 ` Jonathan Liu @ 2017-09-20 16:31 ` Thomas Garnier -1 siblings, 0 replies; 10+ messages in thread From: Thomas Garnier @ 2017-09-20 16:31 UTC (permalink / raw) To: linux-arm-kernel On Tue, Sep 19, 2017 at 2:35 PM, Jonathan Liu <net147@gmail.com> wrote: > > Hi Thomas, > > The top of the C source file I am compiling has: > #include <linux/kernel.h> > #include <asm/uaccess.h> > > Tracing through the asm/uaccess.h include I see > asm/uaccess.h -> asm/domain.h -> asm/thread_info.h > > but set_thread_info is defined in linux/thread_info.h not > asm/thread_info.h (see > http://elixir.free-electrons.com/linux/v4.14-rc1/ident/set_thread_flag). I see, I think I could not reproduce this issue because I already reverted the original commit in favor of a different approach on linux-next (see commit 2404269bc4e77a67875c8db6667be34c9913c96e). Let me know if this commit resolve the issue and thanks for reaching out. > > Regards, > Jonathan > > On 20 September 2017 at 00:32, Thomas Garnier <thgarnie@google.com> wrote: > > On Tue, Sep 19, 2017 at 4:50 AM, Jonathan Liu <net147@gmail.com> wrote: > >> Fixes "implicit declaration of function" compile error for out-of-tree > >> kernel modules including asm/uaccess.h. > > > > I failed to reproduce this issue by creating an out of tree module > > with a separate file (with only uaccess.h). Are you using a special > > config? > > > > Looking at the headers on uaccess.h. You get thread_info through: > > > > asm/domain.h -> asm/thread_info.h > > > >> > >> Fixes: 73ac5d6a2b6a ("arm/syscalls: Check address limit on user-mode return") > >> Signed-off-by: Jonathan Liu <net147@gmail.com> > >> --- > >> arch/arm/include/asm/uaccess.h | 1 + > >> 1 file changed, 1 insertion(+) > >> > >> diff --git a/arch/arm/include/asm/uaccess.h b/arch/arm/include/asm/uaccess.h > >> index 87936dd5d151..13d1877ffb75 100644 > >> --- a/arch/arm/include/asm/uaccess.h > >> +++ b/arch/arm/include/asm/uaccess.h > >> @@ -17,6 +17,7 @@ > >> #include <asm/unified.h> > >> #include <asm/compiler.h> > >> > >> +#include <linux/thread_info.h> > >> #include <asm/extable.h> > >> > >> /* > >> -- > >> 2.13.2 > >> > > > > > > > > -- > > Thomas -- Thomas ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: uaccess: Add missing include for set_thread_flag @ 2017-09-20 16:31 ` Thomas Garnier 0 siblings, 0 replies; 10+ messages in thread From: Thomas Garnier @ 2017-09-20 16:31 UTC (permalink / raw) To: Jonathan Liu; +Cc: Russell King, Thomas Gleixner, Linux ARM, LKML On Tue, Sep 19, 2017 at 2:35 PM, Jonathan Liu <net147@gmail.com> wrote: > > Hi Thomas, > > The top of the C source file I am compiling has: > #include <linux/kernel.h> > #include <asm/uaccess.h> > > Tracing through the asm/uaccess.h include I see > asm/uaccess.h -> asm/domain.h -> asm/thread_info.h > > but set_thread_info is defined in linux/thread_info.h not > asm/thread_info.h (see > http://elixir.free-electrons.com/linux/v4.14-rc1/ident/set_thread_flag). I see, I think I could not reproduce this issue because I already reverted the original commit in favor of a different approach on linux-next (see commit 2404269bc4e77a67875c8db6667be34c9913c96e). Let me know if this commit resolve the issue and thanks for reaching out. > > Regards, > Jonathan > > On 20 September 2017 at 00:32, Thomas Garnier <thgarnie@google.com> wrote: > > On Tue, Sep 19, 2017 at 4:50 AM, Jonathan Liu <net147@gmail.com> wrote: > >> Fixes "implicit declaration of function" compile error for out-of-tree > >> kernel modules including asm/uaccess.h. > > > > I failed to reproduce this issue by creating an out of tree module > > with a separate file (with only uaccess.h). Are you using a special > > config? > > > > Looking at the headers on uaccess.h. You get thread_info through: > > > > asm/domain.h -> asm/thread_info.h > > > >> > >> Fixes: 73ac5d6a2b6a ("arm/syscalls: Check address limit on user-mode return") > >> Signed-off-by: Jonathan Liu <net147@gmail.com> > >> --- > >> arch/arm/include/asm/uaccess.h | 1 + > >> 1 file changed, 1 insertion(+) > >> > >> diff --git a/arch/arm/include/asm/uaccess.h b/arch/arm/include/asm/uaccess.h > >> index 87936dd5d151..13d1877ffb75 100644 > >> --- a/arch/arm/include/asm/uaccess.h > >> +++ b/arch/arm/include/asm/uaccess.h > >> @@ -17,6 +17,7 @@ > >> #include <asm/unified.h> > >> #include <asm/compiler.h> > >> > >> +#include <linux/thread_info.h> > >> #include <asm/extable.h> > >> > >> /* > >> -- > >> 2.13.2 > >> > > > > > > > > -- > > Thomas -- Thomas ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] ARM: uaccess: Add missing include for set_thread_flag 2017-09-20 16:31 ` Thomas Garnier @ 2017-09-20 22:03 ` Jonathan Liu -1 siblings, 0 replies; 10+ messages in thread From: Jonathan Liu @ 2017-09-20 22:03 UTC (permalink / raw) To: linux-arm-kernel Hi Thomas, On 21 September 2017 at 02:31, Thomas Garnier <thgarnie@google.com> wrote: > On Tue, Sep 19, 2017 at 2:35 PM, Jonathan Liu <net147@gmail.com> wrote: >> >> Hi Thomas, >> >> The top of the C source file I am compiling has: >> #include <linux/kernel.h> >> #include <asm/uaccess.h> >> >> Tracing through the asm/uaccess.h include I see >> asm/uaccess.h -> asm/domain.h -> asm/thread_info.h >> >> but set_thread_info is defined in linux/thread_info.h not >> asm/thread_info.h (see >> http://elixir.free-electrons.com/linux/v4.14-rc1/ident/set_thread_flag). > > I see, I think I could not reproduce this issue because I already > reverted the original commit in favor of a different approach on > linux-next (see commit 2404269bc4e77a67875c8db6667be34c9913c96e). > > Let me know if this commit resolve the issue and thanks for reaching out. > Cherry-picking commit 2404269bc4e77a67875c8db6667be34c9913c96e on top of 4.14-rc1 resolves the issue. Thanks. >> >> Regards, >> Jonathan >> >> On 20 September 2017 at 00:32, Thomas Garnier <thgarnie@google.com> wrote: >> > On Tue, Sep 19, 2017 at 4:50 AM, Jonathan Liu <net147@gmail.com> wrote: >> >> Fixes "implicit declaration of function" compile error for out-of-tree >> >> kernel modules including asm/uaccess.h. >> > >> > I failed to reproduce this issue by creating an out of tree module >> > with a separate file (with only uaccess.h). Are you using a special >> > config? >> > >> > Looking at the headers on uaccess.h. You get thread_info through: >> > >> > asm/domain.h -> asm/thread_info.h >> > >> >> >> >> Fixes: 73ac5d6a2b6a ("arm/syscalls: Check address limit on user-mode return") >> >> Signed-off-by: Jonathan Liu <net147@gmail.com> >> >> --- >> >> arch/arm/include/asm/uaccess.h | 1 + >> >> 1 file changed, 1 insertion(+) >> >> >> >> diff --git a/arch/arm/include/asm/uaccess.h b/arch/arm/include/asm/uaccess.h >> >> index 87936dd5d151..13d1877ffb75 100644 >> >> --- a/arch/arm/include/asm/uaccess.h >> >> +++ b/arch/arm/include/asm/uaccess.h >> >> @@ -17,6 +17,7 @@ >> >> #include <asm/unified.h> >> >> #include <asm/compiler.h> >> >> >> >> +#include <linux/thread_info.h> >> >> #include <asm/extable.h> >> >> >> >> /* >> >> -- >> >> 2.13.2 >> >> >> > >> > >> > >> > -- >> > Thomas > > > > > -- > Thomas Regards, Jonathan ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] ARM: uaccess: Add missing include for set_thread_flag @ 2017-09-20 22:03 ` Jonathan Liu 0 siblings, 0 replies; 10+ messages in thread From: Jonathan Liu @ 2017-09-20 22:03 UTC (permalink / raw) To: Thomas Garnier; +Cc: Russell King, Thomas Gleixner, Linux ARM, LKML Hi Thomas, On 21 September 2017 at 02:31, Thomas Garnier <thgarnie@google.com> wrote: > On Tue, Sep 19, 2017 at 2:35 PM, Jonathan Liu <net147@gmail.com> wrote: >> >> Hi Thomas, >> >> The top of the C source file I am compiling has: >> #include <linux/kernel.h> >> #include <asm/uaccess.h> >> >> Tracing through the asm/uaccess.h include I see >> asm/uaccess.h -> asm/domain.h -> asm/thread_info.h >> >> but set_thread_info is defined in linux/thread_info.h not >> asm/thread_info.h (see >> http://elixir.free-electrons.com/linux/v4.14-rc1/ident/set_thread_flag). > > I see, I think I could not reproduce this issue because I already > reverted the original commit in favor of a different approach on > linux-next (see commit 2404269bc4e77a67875c8db6667be34c9913c96e). > > Let me know if this commit resolve the issue and thanks for reaching out. > Cherry-picking commit 2404269bc4e77a67875c8db6667be34c9913c96e on top of 4.14-rc1 resolves the issue. Thanks. >> >> Regards, >> Jonathan >> >> On 20 September 2017 at 00:32, Thomas Garnier <thgarnie@google.com> wrote: >> > On Tue, Sep 19, 2017 at 4:50 AM, Jonathan Liu <net147@gmail.com> wrote: >> >> Fixes "implicit declaration of function" compile error for out-of-tree >> >> kernel modules including asm/uaccess.h. >> > >> > I failed to reproduce this issue by creating an out of tree module >> > with a separate file (with only uaccess.h). Are you using a special >> > config? >> > >> > Looking at the headers on uaccess.h. You get thread_info through: >> > >> > asm/domain.h -> asm/thread_info.h >> > >> >> >> >> Fixes: 73ac5d6a2b6a ("arm/syscalls: Check address limit on user-mode return") >> >> Signed-off-by: Jonathan Liu <net147@gmail.com> >> >> --- >> >> arch/arm/include/asm/uaccess.h | 1 + >> >> 1 file changed, 1 insertion(+) >> >> >> >> diff --git a/arch/arm/include/asm/uaccess.h b/arch/arm/include/asm/uaccess.h >> >> index 87936dd5d151..13d1877ffb75 100644 >> >> --- a/arch/arm/include/asm/uaccess.h >> >> +++ b/arch/arm/include/asm/uaccess.h >> >> @@ -17,6 +17,7 @@ >> >> #include <asm/unified.h> >> >> #include <asm/compiler.h> >> >> >> >> +#include <linux/thread_info.h> >> >> #include <asm/extable.h> >> >> >> >> /* >> >> -- >> >> 2.13.2 >> >> >> > >> > >> > >> > -- >> > Thomas > > > > > -- > Thomas Regards, Jonathan ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2017-09-20 22:03 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-09-19 11:50 [PATCH] ARM: uaccess: Add missing include for set_thread_flag Jonathan Liu 2017-09-19 11:50 ` Jonathan Liu 2017-09-19 14:32 ` Thomas Garnier 2017-09-19 14:32 ` Thomas Garnier 2017-09-19 21:35 ` Jonathan Liu 2017-09-19 21:35 ` Jonathan Liu 2017-09-20 16:31 ` Thomas Garnier 2017-09-20 16:31 ` Thomas Garnier 2017-09-20 22:03 ` Jonathan Liu 2017-09-20 22:03 ` Jonathan Liu
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.