* [Suggestion] Latest randconfig build errors for Head.S
@ 2013-03-14 4:38 Chen Gang
2013-03-14 23:43 ` Russell King - ARM Linux
0 siblings, 1 reply; 9+ messages in thread
From: Chen Gang @ 2013-03-14 4:38 UTC (permalink / raw)
To: Russell King - ARM Linux, Will Deacon, nico, matthew.leach, dave.martin
Cc: linux-arm-kernel, linux-kernel
Hello Maintainers:
are these warnings for head.S OK ?
make:
make V=1 EXTRA_CFLAGS=-W ARCH=arm randconfig
(this time, CONFIG_MMU=y)
warning information:
arch/arm/boot/compressed/head.S: Assembler messages:
arch/arm/boot/compressed/head.S:565: Warning: (null)
arch/arm/boot/compressed/head.S:676: Warning: (null)
arch/arm/boot/compressed/head.S:698: Warning: (null)
arch/arm/boot/compressed/head.S:722: Warning: (null)
arch/arm/boot/compressed/head.S:726: Warning: (null)
arch/arm/boot/compressed/head.S:735: Warning: (null)
arch/arm/boot/compressed/head.S:978: Warning: (null)
arch/arm/boot/compressed/head.S:1017: Warning: (null)
arch/arm/boot/compressed/head.S:1018: Warning: (null)
arch/arm/boot/compressed/head.S:1048: Warning: (null)
arch/arm/boot/compressed/head.S:1055: Warning: (null)
arch/arm/boot/compressed/head.S:1063: Warning: (null)
arch/arm/boot/compressed/head.S:1074: Warning: (null)
arch/arm/boot/compressed/head.S:1088: Warning: (null)
arch/arm/boot/compressed/head.S:1120: Warning: (null)
arch/arm/boot/compressed/head.S:1122: Warning: (null)
arch/arm/boot/compressed/head.S:1123: Warning: (null)
arch/arm/boot/compressed/head.S:1130: Warning: (null)
arch/arm/boot/compressed/head.S:1162: Warning: (null)
--
Chen Gang
Asianux Corporation
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Suggestion] Latest randconfig build errors for Head.S
2013-03-14 4:38 [Suggestion] Latest randconfig build errors for Head.S Chen Gang
@ 2013-03-14 23:43 ` Russell King - ARM Linux
2013-03-15 0:52 ` Chen Gang
0 siblings, 1 reply; 9+ messages in thread
From: Russell King - ARM Linux @ 2013-03-14 23:43 UTC (permalink / raw)
To: Chen Gang
Cc: Will Deacon, nico, matthew.leach, dave.martin, linux-arm-kernel,
linux-kernel
On Thu, Mar 14, 2013 at 12:38:00PM +0800, Chen Gang wrote:
> Hello Maintainers:
>
> are these warnings for head.S OK ?
>
>
> make:
> make V=1 EXTRA_CFLAGS=-W ARCH=arm randconfig
> (this time, CONFIG_MMU=y)
>
> warning information:
> arch/arm/boot/compressed/head.S: Assembler messages:
> arch/arm/boot/compressed/head.S:565: Warning: (null)
> arch/arm/boot/compressed/head.S:676: Warning: (null)
> arch/arm/boot/compressed/head.S:698: Warning: (null)
> arch/arm/boot/compressed/head.S:722: Warning: (null)
> arch/arm/boot/compressed/head.S:726: Warning: (null)
> arch/arm/boot/compressed/head.S:735: Warning: (null)
> arch/arm/boot/compressed/head.S:978: Warning: (null)
> arch/arm/boot/compressed/head.S:1017: Warning: (null)
> arch/arm/boot/compressed/head.S:1018: Warning: (null)
> arch/arm/boot/compressed/head.S:1048: Warning: (null)
> arch/arm/boot/compressed/head.S:1055: Warning: (null)
> arch/arm/boot/compressed/head.S:1063: Warning: (null)
> arch/arm/boot/compressed/head.S:1074: Warning: (null)
> arch/arm/boot/compressed/head.S:1088: Warning: (null)
> arch/arm/boot/compressed/head.S:1120: Warning: (null)
> arch/arm/boot/compressed/head.S:1122: Warning: (null)
> arch/arm/boot/compressed/head.S:1123: Warning: (null)
> arch/arm/boot/compressed/head.S:1130: Warning: (null)
> arch/arm/boot/compressed/head.S:1162: Warning: (null)
*Sigh*
So our wonderful toolchain has decided to start producing NULL warnings
for every "mcr p15, 0, rX, c7, c10, 4" instruction? I'd just ignore
them and hope that the toolchain people fix it.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Suggestion] Latest randconfig build errors for Head.S
2013-03-14 23:43 ` Russell King - ARM Linux
@ 2013-03-15 0:52 ` Chen Gang
2013-03-15 9:31 ` Andrea Adami
0 siblings, 1 reply; 9+ messages in thread
From: Chen Gang @ 2013-03-15 0:52 UTC (permalink / raw)
To: Russell King - ARM Linux
Cc: Will Deacon, nico, matthew.leach, dave.martin, linux-arm-kernel,
linux-kernel
于 2013年03月15日 07:43, Russell King - ARM Linux 写道:
> So our wonderful toolchain has decided to start producing NULL warnings
> for every "mcr p15, 0, rX, c7, c10, 4" instruction? I'd just ignore
> them and hope that the toolchain people fix it.
ok, thanks.
:-)
--
Chen Gang
Asianux Corporation
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Suggestion] Latest randconfig build errors for Head.S
2013-03-15 0:52 ` Chen Gang
@ 2013-03-15 9:31 ` Andrea Adami
2013-03-15 10:36 ` Chen Gang
2013-03-15 10:56 ` Arnd Bergmann
0 siblings, 2 replies; 9+ messages in thread
From: Andrea Adami @ 2013-03-15 9:31 UTC (permalink / raw)
To: Chen Gang
Cc: Russell King - ARM Linux, matthew.leach, dave.martin, nico,
Will Deacon, linux-kernel, linux-arm-kernel, Arnd Bergmann
On Fri, Mar 15, 2013 at 1:52 AM, Chen Gang <gang.chen@asianux.com> wrote:
> 于 2013年03月15日 07:43, Russell King - ARM Linux 写道:
>> So our wonderful toolchain has decided to start producing NULL warnings
>> for every "mcr p15, 0, rX, c7, c10, 4" instruction? I'd just ignore
>> them and hope that the toolchain people fix it.
AFAIK Arnd is aware and waiting for feedback about his patch:
ARM: compressed/head.S: work around new binutils warning Arnd Bergmann
http://lists.infradead.org/pipermail/linux-arm-kernel/2013-January/144846.html
Regards
Andrea
>
> ok, thanks.
>
> :-)
>
> --
> Chen Gang
>
> Asianux Corporation
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Suggestion] Latest randconfig build errors for Head.S
2013-03-15 9:31 ` Andrea Adami
@ 2013-03-15 10:36 ` Chen Gang
2013-03-15 10:56 ` Arnd Bergmann
1 sibling, 0 replies; 9+ messages in thread
From: Chen Gang @ 2013-03-15 10:36 UTC (permalink / raw)
To: Andrea Adami
Cc: Russell King - ARM Linux, matthew.leach, dave.martin, nico,
Will Deacon, linux-kernel, linux-arm-kernel, Arnd Bergmann
于 2013年03月15日 17:31, Andrea Adami 写道:
> AFAIK Arnd is aware and waiting for feedback about his patch:
>
> ARM: compressed/head.S: work around new binutils warning Arnd Bergmann
>
> http://lists.infradead.org/pipermail/linux-arm-kernel/2013-January/144846.html
excuse me, I am not familiar with arm assemble code, so can not give more information
but at least for me, it seems your patch is more precious, and also welcomed by compiler.
and I don't think your patch make the code "a more complex":
the reason is:
"keeping code simple and clear" is for readers and debuggers, not for writers.
I prefer to write more code to let readers and debuggers easy understand.
(if it has minor negative effect with performance)
thanks.
--
Chen Gang
Asianux Corporation
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Suggestion] Latest randconfig build errors for Head.S
2013-03-15 9:31 ` Andrea Adami
2013-03-15 10:36 ` Chen Gang
@ 2013-03-15 10:56 ` Arnd Bergmann
2013-03-15 11:11 ` Chen Gang F T
1 sibling, 1 reply; 9+ messages in thread
From: Arnd Bergmann @ 2013-03-15 10:56 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Andrea Adami, Chen Gang, matthew.leach, dave.martin,
Russell King - ARM Linux, nico, Will Deacon, linux-kernel,
matthew.gretton-dann
On Friday 15 March 2013, Andrea Adami wrote:
> On Fri, Mar 15, 2013 at 1:52 AM, Chen Gang <gang.chen@asianux.com> wrote:
> > 于 2013年03月15日 07:43, Russell King - ARM Linux 写道:
> >> So our wonderful toolchain has decided to start producing NULL warnings
> >> for every "mcr p15, 0, rX, c7, c10, 4" instruction? I'd just ignore
> >> them and hope that the toolchain people fix it.
>
> AFAIK Arnd is aware and waiting for feedback about his patch:
>
> ARM: compressed/head.S: work around new binutils warning Arnd Bergmann
>
> http://lists.infradead.org/pipermail/linux-arm-kernel/2013-January/144846.html
Right. The fact that as prints a warning for those instructions makes sense
given that we are telling it we want to run on *all* ARM architectures,
and that ARMv8 has deprecated them officially.
Of course, what is bogus about this is the "NULL" part rather than printing
the text that was meant to come out to explain the situation. I don't
know how to report that bug other than what I tried before, but if it gets
fixed by printing the right text, we will still want to build a kernel
without getting any warnings.
Arnd
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Suggestion] Latest randconfig build errors for Head.S
2013-03-15 10:56 ` Arnd Bergmann
@ 2013-03-15 11:11 ` Chen Gang F T
2013-03-15 11:29 ` Arnd Bergmann
0 siblings, 1 reply; 9+ messages in thread
From: Chen Gang F T @ 2013-03-15 11:11 UTC (permalink / raw)
To: Arnd Bergmann
Cc: linux-arm-kernel, Andrea Adami, Chen Gang, matthew.leach,
dave.martin, Russell King - ARM Linux, nico, Will Deacon,
linux-kernel, matthew.gretton-dann
于 2013年03月15日 18:56, Arnd Bergmann 写道:
> Right. The fact that as prints a warning for those instructions makes sense
> given that we are telling it we want to run on *all* ARM architectures,
> and that ARMv8 has deprecated them officially.
>
> Of course, what is bogus about this is the "NULL" part rather than printing
> the text that was meant to come out to explain the situation. I don't
> know how to report that bug other than what I tried before, but if it gets
> fixed by printing the right text, we will still want to build a kernel
> without getting any warnings.
excuse me, my English is not quite well.
I guess your meaning is:
"that bug" means the toolchain's bug (need toolchain people fix it).
prefer to apply your patch, before "that bug" is fixed (we don't know when)
is it correct ?
thanks.
--
Chen Gang
Flying Transformer
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Suggestion] Latest randconfig build errors for Head.S
2013-03-15 11:11 ` Chen Gang F T
@ 2013-03-15 11:29 ` Arnd Bergmann
2013-03-17 2:31 ` Chen Gang F T
0 siblings, 1 reply; 9+ messages in thread
From: Arnd Bergmann @ 2013-03-15 11:29 UTC (permalink / raw)
To: Chen Gang F T
Cc: linux-arm-kernel, Andrea Adami, Chen Gang, matthew.leach,
dave.martin, Russell King - ARM Linux, nico, Will Deacon,
linux-kernel, matthew.gretton-dann
On Friday 15 March 2013, Chen Gang F T wrote:
> excuse me, my English is not quite well.
>
> I guess your meaning is:
> "that bug" means the toolchain's bug (need toolchain people fix it).
> prefer to apply your patch, before "that bug" is fixed (we don't know when)
>
> is it correct ?
Yes, correct.
Arnd
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Suggestion] Latest randconfig build errors for Head.S
2013-03-15 11:29 ` Arnd Bergmann
@ 2013-03-17 2:31 ` Chen Gang F T
0 siblings, 0 replies; 9+ messages in thread
From: Chen Gang F T @ 2013-03-17 2:31 UTC (permalink / raw)
To: Arnd Bergmann
Cc: linux-arm-kernel, Andrea Adami, Chen Gang, matthew.leach,
dave.martin, Russell King - ARM Linux, nico, Will Deacon,
linux-kernel, matthew.gretton-dann
On 2013年03月15日 19:29, Arnd Bergmann wrote:
> On Friday 15 March 2013, Chen Gang F T wrote:
>> > excuse me, my English is not quite well.
>> >
>> > I guess your meaning is:
>> > "that bug" means the toolchain's bug (need toolchain people fix it).
>> > prefer to apply your patch, before "that bug" is fixed (we don't know when)
>> >
>> > is it correct ?
> Yes, correct.
thanks, and I support you.
--
Chen Gang
Flying Transformer
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2013-03-17 2:31 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-03-14 4:38 [Suggestion] Latest randconfig build errors for Head.S Chen Gang
2013-03-14 23:43 ` Russell King - ARM Linux
2013-03-15 0:52 ` Chen Gang
2013-03-15 9:31 ` Andrea Adami
2013-03-15 10:36 ` Chen Gang
2013-03-15 10:56 ` Arnd Bergmann
2013-03-15 11:11 ` Chen Gang F T
2013-03-15 11:29 ` Arnd Bergmann
2013-03-17 2:31 ` Chen Gang F T
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).