Re: next build: 208 builds: 3 failed, 205 passed, 5 errors, 23 warnings (next-20170215)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Feb 15, 2017 at 09:56:48AM +0100, Arnd Bergmann wrote:
> On Wed, Feb 15, 2017 at 9:30 AM, kernelci.org bot <bot@xxxxxxxxxxxx> wrote:
> > bcm47xx_defconfig (mips) — FAIL, 1 error, 0 warnings, 0 section mismatches
> >
> > Errors:
> > /home/buildslave/workspace/khilman-kbuilder/next/build-mips/../net/sched/sch_fq_codel.c:468:
> > undefined reference to `tcf_destroy_chain'
> 
> I have not looked at this one yet, appears to be caused by commit
> cf1facda2f61 ("sched: move tcf_proto_destroy and tcf_destroy_chain
> helpers into cls_api")

So effectively CONFIG_NET_SCH_FQ_CODEL (which bcm47xx_defconfig sets
=y), and just over half of the other packet schedulers now implicitly
depend on CONFIG_NET_CLS (which bcm47xx_defconfig doesn't set =y).

Perhaps revert would be best since the change looks of questionable
value to me, as all the users of it are in sch_*.c anyway.

> > ip27_defconfig (mips) — FAIL, 2 errors, 0 warnings, 0 section mismatches
> >
> > Errors:
> > drivers/net/ethernet/qlogic/qlge/qlge_main.c:4819:1: error: insn does not
> > satisfy its constraints:
> > drivers/net/ethernet/qlogic/qlge/qlge_main.c:4819:1: internal compiler
> > error: in extract_constrain_insn, at recog.c:2190
> 
> broken gcc release, apparently fixed in gcc-7 (can't reproduce here at least).
> I suggested a workaround, but got no reply so far:
> 
> http://www.spinics.net/lists/mips/msg66465.html

I'll look into that.

> 
> > xway_defconfig (mips) — FAIL, 2 errors, 0 warnings, 0 section mismatches
> >
> > Errors:
> > (.text+0x14e10): undefined reference to `physical_memsize'
> > (.text+0x14e14): undefined reference to `physical_memsize'
> 
> Hauke already did a patch in December, but it has't made it into linux-mips
> so far:
> 
> https://marc.info/?l=linux-mips&m=148612428414708&w=3

Hauke: you mentioned sending a new version of this patch. I'll apply the
original patch for now, as looking at the VPE loader, zero doesn't sound
entirely unreasonable:

/*
 * The sde-kit passes 'memsize' to __start in $a3, so set something
 * here...  Or set $a3 to zero and define DFLT_STACK_SIZE and
 * DFLT_HEAP_SIZE when you compile your program
 */
mttgpr(6, v->ntcs);
mttgpr(7, physical_memsize);

Cheers
James

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux