* [PATCH] ARM: boot/compressed/misc.s: Include <linux/bug.h> to fix build break
@ 2012-02-27 19:52 ` Stephen Warren
0 siblings, 0 replies; 8+ messages in thread
From: Stephen Warren @ 2012-02-27 19:52 UTC (permalink / raw)
To: linux-arm-kernel
Commmit cf86c17 "kernel.h: doesn't explicitly use bug.h, so don't include
it." caused the following build failure on ARM:
CC arch/arm/boot/compressed/misc.o
In file included from arch/arm/boot/compressed/misc.c:28:0:
arch/arm/mach-tegra/include/mach/uncompress.h: In function 'arch_decomp_setup':
arch/arm/mach-tegra/include/mach/uncompress.h:125:2: error: implicit declaration of function 'BUILD_BUG_ON_ZERO' [-Werror=implicit-function-declaration]
cc1: some warnings being treated as errors
Include <linux/bug.h> to fix that.
This should probably be squashed into commit a3a3a1a "bug.h: add include
of it to various implicit C users".
Signed-off-by: Stephen Warren <swarren@nvidia.com>
---
arch/arm/boot/compressed/misc.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/arm/boot/compressed/misc.c b/arch/arm/boot/compressed/misc.c
index 8e2a8fc..30055be 100644
--- a/arch/arm/boot/compressed/misc.c
+++ b/arch/arm/boot/compressed/misc.c
@@ -18,6 +18,7 @@
unsigned int __machine_arch_type;
+#include <linux/bug.h>
#include <linux/compiler.h> /* for inline */
#include <linux/types.h>
#include <linux/linkage.h>
--
1.7.0.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH] ARM: boot/compressed/misc.s: Include <linux/bug.h> to fix build break
@ 2012-02-27 19:52 ` Stephen Warren
0 siblings, 0 replies; 8+ messages in thread
From: Stephen Warren @ 2012-02-27 19:52 UTC (permalink / raw)
To: Paul Gortmaker, Russell King
Cc: Nicolas Pitre, Stephen Boyd, linux-arm-kernel, linux-kernel,
Stephen Warren
Commmit cf86c17 "kernel.h: doesn't explicitly use bug.h, so don't include
it." caused the following build failure on ARM:
CC arch/arm/boot/compressed/misc.o
In file included from arch/arm/boot/compressed/misc.c:28:0:
arch/arm/mach-tegra/include/mach/uncompress.h: In function 'arch_decomp_setup':
arch/arm/mach-tegra/include/mach/uncompress.h:125:2: error: implicit declaration of function 'BUILD_BUG_ON_ZERO' [-Werror=implicit-function-declaration]
cc1: some warnings being treated as errors
Include <linux/bug.h> to fix that.
This should probably be squashed into commit a3a3a1a "bug.h: add include
of it to various implicit C users".
Signed-off-by: Stephen Warren <swarren@nvidia.com>
---
arch/arm/boot/compressed/misc.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/arm/boot/compressed/misc.c b/arch/arm/boot/compressed/misc.c
index 8e2a8fc..30055be 100644
--- a/arch/arm/boot/compressed/misc.c
+++ b/arch/arm/boot/compressed/misc.c
@@ -18,6 +18,7 @@
unsigned int __machine_arch_type;
+#include <linux/bug.h>
#include <linux/compiler.h> /* for inline */
#include <linux/types.h>
#include <linux/linkage.h>
--
1.7.0.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH] ARM: boot/compressed/misc.s: Include <linux/bug.h> to fix build break
2012-02-27 19:52 ` Stephen Warren
@ 2012-02-27 19:56 ` Russell King - ARM Linux
-1 siblings, 0 replies; 8+ messages in thread
From: Russell King - ARM Linux @ 2012-02-27 19:56 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Feb 27, 2012 at 12:52:30PM -0700, Stephen Warren wrote:
> Commmit cf86c17 "kernel.h: doesn't explicitly use bug.h, so don't include
> it." caused the following build failure on ARM:
>
> CC arch/arm/boot/compressed/misc.o
> In file included from arch/arm/boot/compressed/misc.c:28:0:
> arch/arm/mach-tegra/include/mach/uncompress.h: In function 'arch_decomp_setup':
> arch/arm/mach-tegra/include/mach/uncompress.h:125:2: error: implicit declaration of function 'BUILD_BUG_ON_ZERO' [-Werror=implicit-function-declaration]
> cc1: some warnings being treated as errors
Err no. The decompressor is _not_ part of the kernel, it's a separate
stand alone environment. Including randomly kernel headers from the
rest of the kernel is bad news, particularly if they start calling out
to external code.
You should try to keep your uncompress.h as free of other dependencies
as possible.
>
> Include <linux/bug.h> to fix that.
>
> This should probably be squashed into commit a3a3a1a "bug.h: add include
> of it to various implicit C users".
>
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> ---
> arch/arm/boot/compressed/misc.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/boot/compressed/misc.c b/arch/arm/boot/compressed/misc.c
> index 8e2a8fc..30055be 100644
> --- a/arch/arm/boot/compressed/misc.c
> +++ b/arch/arm/boot/compressed/misc.c
> @@ -18,6 +18,7 @@
>
> unsigned int __machine_arch_type;
>
> +#include <linux/bug.h>
> #include <linux/compiler.h> /* for inline */
> #include <linux/types.h>
> #include <linux/linkage.h>
> --
> 1.7.0.4
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: boot/compressed/misc.s: Include <linux/bug.h> to fix build break
@ 2012-02-27 19:56 ` Russell King - ARM Linux
0 siblings, 0 replies; 8+ messages in thread
From: Russell King - ARM Linux @ 2012-02-27 19:56 UTC (permalink / raw)
To: Stephen Warren
Cc: Paul Gortmaker, Nicolas Pitre, Stephen Boyd, linux-arm-kernel,
linux-kernel
On Mon, Feb 27, 2012 at 12:52:30PM -0700, Stephen Warren wrote:
> Commmit cf86c17 "kernel.h: doesn't explicitly use bug.h, so don't include
> it." caused the following build failure on ARM:
>
> CC arch/arm/boot/compressed/misc.o
> In file included from arch/arm/boot/compressed/misc.c:28:0:
> arch/arm/mach-tegra/include/mach/uncompress.h: In function 'arch_decomp_setup':
> arch/arm/mach-tegra/include/mach/uncompress.h:125:2: error: implicit declaration of function 'BUILD_BUG_ON_ZERO' [-Werror=implicit-function-declaration]
> cc1: some warnings being treated as errors
Err no. The decompressor is _not_ part of the kernel, it's a separate
stand alone environment. Including randomly kernel headers from the
rest of the kernel is bad news, particularly if they start calling out
to external code.
You should try to keep your uncompress.h as free of other dependencies
as possible.
>
> Include <linux/bug.h> to fix that.
>
> This should probably be squashed into commit a3a3a1a "bug.h: add include
> of it to various implicit C users".
>
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> ---
> arch/arm/boot/compressed/misc.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/boot/compressed/misc.c b/arch/arm/boot/compressed/misc.c
> index 8e2a8fc..30055be 100644
> --- a/arch/arm/boot/compressed/misc.c
> +++ b/arch/arm/boot/compressed/misc.c
> @@ -18,6 +18,7 @@
>
> unsigned int __machine_arch_type;
>
> +#include <linux/bug.h>
> #include <linux/compiler.h> /* for inline */
> #include <linux/types.h>
> #include <linux/linkage.h>
> --
> 1.7.0.4
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM: boot/compressed/misc.s: Include <linux/bug.h> to fix build break
2012-02-27 19:56 ` Russell King - ARM Linux
@ 2012-02-27 20:14 ` Stephen Warren
-1 siblings, 0 replies; 8+ messages in thread
From: Stephen Warren @ 2012-02-27 20:14 UTC (permalink / raw)
To: linux-arm-kernel
Russell King wrote at Monday, February 27, 2012 12:57 PM:
> On Mon, Feb 27, 2012 at 12:52:30PM -0700, Stephen Warren wrote:
> > Commmit cf86c17 "kernel.h: doesn't explicitly use bug.h, so don't include
> > it." caused the following build failure on ARM:
> >
> > CC arch/arm/boot/compressed/misc.o
> > In file included from arch/arm/boot/compressed/misc.c:28:0:
> > arch/arm/mach-tegra/include/mach/uncompress.h: In function 'arch_decomp_setup':
> > arch/arm/mach-tegra/include/mach/uncompress.h:125:2: error: implicit declaration of function
> 'BUILD_BUG_ON_ZERO' [-Werror=implicit-function-declaration]
> > cc1: some warnings being treated as errors
>
> Err no. The decompressor is _not_ part of the kernel, it's a separate
> stand alone environment. Including randomly kernel headers from the
> rest of the kernel is bad news, particularly if they start calling out
> to external code.
>
> You should try to keep your uncompress.h as free of other dependencies
> as possible.
OK, It looks like all we need from <linux/kernel.h> is BIT() and
ARRAY_SIZE(). The latter requires <linux/bug.h>. Should I just redefine
those macros in our uncompress.h?
--
nvpublic
^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH] ARM: boot/compressed/misc.s: Include <linux/bug.h> to fix build break
@ 2012-02-27 20:14 ` Stephen Warren
0 siblings, 0 replies; 8+ messages in thread
From: Stephen Warren @ 2012-02-27 20:14 UTC (permalink / raw)
To: Russell King - ARM Linux
Cc: Paul Gortmaker, Nicolas Pitre, Stephen Boyd,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org
Russell King wrote at Monday, February 27, 2012 12:57 PM:
> On Mon, Feb 27, 2012 at 12:52:30PM -0700, Stephen Warren wrote:
> > Commmit cf86c17 "kernel.h: doesn't explicitly use bug.h, so don't include
> > it." caused the following build failure on ARM:
> >
> > CC arch/arm/boot/compressed/misc.o
> > In file included from arch/arm/boot/compressed/misc.c:28:0:
> > arch/arm/mach-tegra/include/mach/uncompress.h: In function 'arch_decomp_setup':
> > arch/arm/mach-tegra/include/mach/uncompress.h:125:2: error: implicit declaration of function
> 'BUILD_BUG_ON_ZERO' [-Werror=implicit-function-declaration]
> > cc1: some warnings being treated as errors
>
> Err no. The decompressor is _not_ part of the kernel, it's a separate
> stand alone environment. Including randomly kernel headers from the
> rest of the kernel is bad news, particularly if they start calling out
> to external code.
>
> You should try to keep your uncompress.h as free of other dependencies
> as possible.
OK, It looks like all we need from <linux/kernel.h> is BIT() and
ARRAY_SIZE(). The latter requires <linux/bug.h>. Should I just redefine
those macros in our uncompress.h?
--
nvpublic
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM: boot/compressed/misc.s: Include <linux/bug.h> to fix build break
2012-02-27 19:52 ` Stephen Warren
@ 2012-02-29 1:26 ` Paul Gortmaker
-1 siblings, 0 replies; 8+ messages in thread
From: Paul Gortmaker @ 2012-02-29 1:26 UTC (permalink / raw)
To: linux-arm-kernel
[[PATCH] ARM: boot/compressed/misc.s: Include <linux/bug.h> to fix build break] On 27/02/2012 (Mon 12:52) Stephen Warren wrote:
> Commmit cf86c17 "kernel.h: doesn't explicitly use bug.h, so don't include
> it." caused the following build failure on ARM:
>
> CC arch/arm/boot/compressed/misc.o
> In file included from arch/arm/boot/compressed/misc.c:28:0:
> arch/arm/mach-tegra/include/mach/uncompress.h: In function 'arch_decomp_setup':
> arch/arm/mach-tegra/include/mach/uncompress.h:125:2: error: implicit declaration of function 'BUILD_BUG_ON_ZERO' [-Werror=implicit-function-declaration]
> cc2: some warnings being treated as errors
>
> Include <linux/bug.h> to fix that.
Hi Stephen,
I wouldn't want to put bug.h in arm/boot/compressed/misc.c simply
because I've built all the arm configs and I don't see this on
running any of the existing arch/arm/config builds.
However, if arch/arm/mach-tegra/include/mach/uncompress.h is using
things from it, then perhaps it needs to include bug.h I think.
Russell is totally correct in saying that uncompress has no business in
calling/linking generic infrastructure functions, but your error message
is a use of _BUILD_ bug stuff. The early init code clearly can't do
printk or similar, but should still be able to use BUILD_BUG type stuff
that is meant to blow up on the x-compile host and not the target. Use
of BUILD_BUG is different from use of (runtime) BUG. Does it warrant a
separate header though? It could, but I don't think we want to go there.
The other interesting thing here, is that you see this when the kernel.h
stops including bug.h -- so that means you've got an implicit presence
of kernel.h -- which means you can legally (but incorrectly) link to
things like printk() which simply dont make sense at this early stage of
boot. So I have to think that the exclusion of headers like bug.h to
protect yourself from hurting yourself was already non-functional before
you start. The way things are currently requires the author to really
think about the context they are operating in regardless.
Using BUG in this context is wrong. Using BUILD_BUG seems OK to me.
Paul.
--
>
> This should probably be squashed into commit a3a3a1a "bug.h: add include
> of it to various implicit C users".
>
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> ---
> arch/arm/boot/compressed/misc.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/boot/compressed/misc.c b/arch/arm/boot/compressed/misc.c
> index 8e2a8fc..30055be 100644
> --- a/arch/arm/boot/compressed/misc.c
> +++ b/arch/arm/boot/compressed/misc.c
> @@ -18,6 +18,7 @@
>
> unsigned int __machine_arch_type;
>
> +#include <linux/bug.h>
> #include <linux/compiler.h> /* for inline */
> #include <linux/types.h>
> #include <linux/linkage.h>
> --
> 1.7.0.4
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: boot/compressed/misc.s: Include <linux/bug.h> to fix build break
@ 2012-02-29 1:26 ` Paul Gortmaker
0 siblings, 0 replies; 8+ messages in thread
From: Paul Gortmaker @ 2012-02-29 1:26 UTC (permalink / raw)
To: Stephen Warren
Cc: Russell King, Nicolas Pitre, Stephen Boyd, linux-arm-kernel,
linux-kernel
[[PATCH] ARM: boot/compressed/misc.s: Include <linux/bug.h> to fix build break] On 27/02/2012 (Mon 12:52) Stephen Warren wrote:
> Commmit cf86c17 "kernel.h: doesn't explicitly use bug.h, so don't include
> it." caused the following build failure on ARM:
>
> CC arch/arm/boot/compressed/misc.o
> In file included from arch/arm/boot/compressed/misc.c:28:0:
> arch/arm/mach-tegra/include/mach/uncompress.h: In function 'arch_decomp_setup':
> arch/arm/mach-tegra/include/mach/uncompress.h:125:2: error: implicit declaration of function 'BUILD_BUG_ON_ZERO' [-Werror=implicit-function-declaration]
> cc2: some warnings being treated as errors
>
> Include <linux/bug.h> to fix that.
Hi Stephen,
I wouldn't want to put bug.h in arm/boot/compressed/misc.c simply
because I've built all the arm configs and I don't see this on
running any of the existing arch/arm/config builds.
However, if arch/arm/mach-tegra/include/mach/uncompress.h is using
things from it, then perhaps it needs to include bug.h I think.
Russell is totally correct in saying that uncompress has no business in
calling/linking generic infrastructure functions, but your error message
is a use of _BUILD_ bug stuff. The early init code clearly can't do
printk or similar, but should still be able to use BUILD_BUG type stuff
that is meant to blow up on the x-compile host and not the target. Use
of BUILD_BUG is different from use of (runtime) BUG. Does it warrant a
separate header though? It could, but I don't think we want to go there.
The other interesting thing here, is that you see this when the kernel.h
stops including bug.h -- so that means you've got an implicit presence
of kernel.h -- which means you can legally (but incorrectly) link to
things like printk() which simply dont make sense at this early stage of
boot. So I have to think that the exclusion of headers like bug.h to
protect yourself from hurting yourself was already non-functional before
you start. The way things are currently requires the author to really
think about the context they are operating in regardless.
Using BUG in this context is wrong. Using BUILD_BUG seems OK to me.
Paul.
--
>
> This should probably be squashed into commit a3a3a1a "bug.h: add include
> of it to various implicit C users".
>
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> ---
> arch/arm/boot/compressed/misc.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/boot/compressed/misc.c b/arch/arm/boot/compressed/misc.c
> index 8e2a8fc..30055be 100644
> --- a/arch/arm/boot/compressed/misc.c
> +++ b/arch/arm/boot/compressed/misc.c
> @@ -18,6 +18,7 @@
>
> unsigned int __machine_arch_type;
>
> +#include <linux/bug.h>
> #include <linux/compiler.h> /* for inline */
> #include <linux/types.h>
> #include <linux/linkage.h>
> --
> 1.7.0.4
>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2012-02-29 1:27 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-02-27 19:52 [PATCH] ARM: boot/compressed/misc.s: Include <linux/bug.h> to fix build break Stephen Warren
2012-02-27 19:52 ` Stephen Warren
2012-02-27 19:56 ` Russell King - ARM Linux
2012-02-27 19:56 ` Russell King - ARM Linux
2012-02-27 20:14 ` Stephen Warren
2012-02-27 20:14 ` Stephen Warren
2012-02-29 1:26 ` Paul Gortmaker
2012-02-29 1:26 ` Paul Gortmaker
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.