public inbox for linuxppc-dev@ozlabs.org 
 help / color / mirror / Atom feed
* Re: Build regressions/improvements in v3.7-rc5
       [not found] <1352753886-3402-1-git-send-email-geert@linux-m68k.org>
@ 2012-11-12 21:02 ` Geert Uytterhoeven
  2012-11-12 21:51   ` Scott Wood
  2012-11-12 22:05   ` Stephen Rothwell
  0 siblings, 2 replies; 4+ messages in thread
From: Geert Uytterhoeven @ 2012-11-12 21:02 UTC (permalink / raw)
  To: linux-kernel; +Cc: netdev, ceph-devel, linuxppc-dev

On Mon, Nov 12, 2012 at 9:58 PM, Geert Uytterhoeven
<geert@linux-m68k•org> wrote:
> JFYI, when comparing v3.7-rc5 to v3.7-rc4[3], the summaries are:
>   - build errors: +14/-4

14 regressions:
  + drivers/virt/fsl_hypervisor.c: error: 'MSR_GS' undeclared (first
use in this function):  => 799:93
  + error: No rule to make target drivers/scsi/aic7xxx/aicasm/*.[chyl]:  => N/A
  + net/ceph/ceph_common.c: error: dereferencing pointer to incomplete
type:  => 272:13
  + net/ceph/ceph_common.c: error: implicit declaration of function
'request_key' [-Werror=implicit-function-declaration]:  => 249:2
  + net/ceph/crypto.c: error: dereferencing pointer to incomplete
type:  => 463:19, 434:46, 452:5, 448:52, 429:23, 467:36, 447:18
  + net/ceph/crypto.c: error: implicit declaration of function
'key_payload_reserve' [-Werror=implicit-function-declaration]:  =>
437:2
  + net/ceph/crypto.c: error: implicit declaration of function
'register_key_type' [-Werror=implicit-function-declaration]:  => 481:2
  + net/ceph/crypto.c: error: implicit declaration of function
'unregister_key_type' [-Werror=implicit-function-declaration]:  =>
485:2
  + net/ceph/crypto.c: error: unknown field 'destroy' specified in
initializer:  => 477:2
  + net/ceph/crypto.c: error: unknown field 'instantiate' specified in
initializer:  => 475:2
  + net/ceph/crypto.c: error: unknown field 'match' specified in
initializer:  => 476:2
  + net/ceph/crypto.c: error: unknown field 'name' specified in
initializer:  => 474:2
  + net/ceph/crypto.c: error: variable 'key_type_ceph' has initializer
but incomplete type:  => 473:8

powerpc-randconfig

  + error: relocation truncated to fit: R_PPC64_REL24 against symbol
`._mcount' defined in .text section in arch/powerpc/kernel/entry_64.o:
(.text+0x1ff9eb8) => (.text+0x1ffa274), (.text+0x1ff7840)

powerpc-allyesconfig

> [1] http://kisskb.ellerman.id.au/kisskb/head/5614/ (all 117 configs)
> [3] http://kisskb.ellerman.id.au/kisskb/head/5600/ (all 117 configs)

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k•org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Build regressions/improvements in v3.7-rc5
  2012-11-12 21:02 ` Build regressions/improvements in v3.7-rc5 Geert Uytterhoeven
@ 2012-11-12 21:51   ` Scott Wood
  2012-11-13 10:39     ` Geert Uytterhoeven
  2012-11-12 22:05   ` Stephen Rothwell
  1 sibling, 1 reply; 4+ messages in thread
From: Scott Wood @ 2012-11-12 21:51 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: linuxppc-dev, linux-kernel

On 11/12/2012 03:02:46 PM, Geert Uytterhoeven wrote:
> On Mon, Nov 12, 2012 at 9:58 PM, Geert Uytterhoeven
> <geert@linux-m68k•org> wrote:
> > JFYI, when comparing v3.7-rc5 to v3.7-rc4[3], the summaries are:
> >   - build errors: +14/-4
>=20
> 14 regressions:
>   + drivers/virt/fsl_hypervisor.c: error: 'MSR_GS' undeclared (first
> use in this function):  =3D> 799:93

These reports would be more useful if the actual .config were provided, =20
rather than just saying "powerpc randconfig".

I doubt this particular one is a regression; it looks like the =20
dependencies have always been too weak (patch sent) and you're just =20
comparing different randomly-generated configs.

> powerpc-randconfig
>=20
>   + error: relocation truncated to fit: R_PPC64_REL24 against symbol
> `._mcount' defined in .text section in arch/powerpc/kernel/entry_64.o:
> (.text+0x1ff9eb8) =3D> (.text+0x1ffa274), (.text+0x1ff7840)
>=20
> powerpc-allyesconfig

It's been over 3 years since powerpc-allyesconfig built (it's just too =20
big for the relocation model the kernel uses) -- how is this a =20
regression?

-Scott=

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Build regressions/improvements in v3.7-rc5
  2012-11-12 21:02 ` Build regressions/improvements in v3.7-rc5 Geert Uytterhoeven
  2012-11-12 21:51   ` Scott Wood
@ 2012-11-12 22:05   ` Stephen Rothwell
  1 sibling, 0 replies; 4+ messages in thread
From: Stephen Rothwell @ 2012-11-12 22:05 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Scott Wood, linuxppc-dev, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 800 bytes --]

Hi Geert,

Thanks for these reports!

On Mon, 12 Nov 2012 22:02:46 +0100 Geert Uytterhoeven <geert@linux-m68k•org> wrote:
>
>   + error: relocation truncated to fit: R_PPC64_REL24 against symbol
> `._mcount' defined in .text section in arch/powerpc/kernel/entry_64.o:
> (.text+0x1ff9eb8) => (.text+0x1ffa274), (.text+0x1ff7840)
> 
> powerpc-allyesconfig

Yeah, as Scott said, these are not worth reporting.  However, they are
being worked on in various ways, but will probably not be solved very
soon (it means we have to change how we link the kernel and that will
probably require some discussion).

I just keep the powerpc allyesconfig build there to catch other stuff
(like drivers that assume too much).
-- 
Cheers,
Stephen Rothwell                    sfr@canb•auug.org.au

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Build regressions/improvements in v3.7-rc5
  2012-11-12 21:51   ` Scott Wood
@ 2012-11-13 10:39     ` Geert Uytterhoeven
  0 siblings, 0 replies; 4+ messages in thread
From: Geert Uytterhoeven @ 2012-11-13 10:39 UTC (permalink / raw)
  To: Scott Wood; +Cc: linuxppc-dev, linux-kernel

Hi Scott,

On Mon, Nov 12, 2012 at 10:51 PM, Scott Wood <scottwood@freescale•com> wrote:
> On 11/12/2012 03:02:46 PM, Geert Uytterhoeven wrote:
>> On Mon, Nov 12, 2012 at 9:58 PM, Geert Uytterhoeven
>> <geert@linux-m68k•org> wrote:
>> > JFYI, when comparing v3.7-rc5 to v3.7-rc4[3], the summaries are:
>> >   - build errors: +14/-4
>>
>> 14 regressions:
>>   + drivers/virt/fsl_hypervisor.c: error: 'MSR_GS' undeclared (first
>> use in this function):  => 799:93
>
> These reports would be more useful if the actual .config were provided,
> rather than just saying "powerpc randconfig".

This can be obtained from the kissb build system.
[1] http://kisskb.ellerman.id.au/kisskb/head/5614/ (all 117 configs)

> I doubt this particular one is a regression; it looks like the dependencies

Yeah, regressions in randconfigs can be misleading.
Still, they're build failures.

> have always been too weak (patch sent) and you're just comparing different
> randomly-generated configs.

Thanks for your patch!

>> powerpc-randconfig
>>
>>   + error: relocation truncated to fit: R_PPC64_REL24 against symbol
>> `._mcount' defined in .text section in arch/powerpc/kernel/entry_64.o:
>> (.text+0x1ff9eb8) => (.text+0x1ffa274), (.text+0x1ff7840)
>>
>> powerpc-allyesconfig
>
>
> It's been over 3 years since powerpc-allyesconfig built (it's just too big
> for the relocation model the kernel uses) -- how is this a regression?

It's a regression because it didn't complain about ".mcount" in the
previous run.

But I'll start stripping them from the rc-incremental reports.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k•org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-11-13 10:39 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1352753886-3402-1-git-send-email-geert@linux-m68k.org>
2012-11-12 21:02 ` Build regressions/improvements in v3.7-rc5 Geert Uytterhoeven
2012-11-12 21:51   ` Scott Wood
2012-11-13 10:39     ` Geert Uytterhoeven
2012-11-12 22:05   ` Stephen Rothwell

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox