[binutils-gdb] PowerPC @l, @h and @ha warnings, plus VLE e_li

classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

[binutils-gdb] PowerPC @l, @h and @ha warnings, plus VLE e_li

sergiodj+buildbot-2
*** TEST RESULTS FOR COMMIT bb6bf75e7a1f9aaf0283895705710f415b81b6b1 ***

Author: Alan Modra <[hidden email]>
Branch: master
Commit: bb6bf75e7a1f9aaf0283895705710f415b81b6b1

PowerPC @l, @h and @ha warnings, plus VLE e_li

This patch started off just adding the warnings in tc-ppc.c about
incorrect usage of @l, @h and @ha in instructions that don't have
16-bit D-form fields.  That unfortunately showed up three warnings in
ld/testsuite/ld-powerpc/vle-multiseg.s on instructions like
        e_li r3, IV_table@l+0x00
which was being assembled to
   8: 70 60 00 00 e_li    r3,0
                        a: R_PPC_ADDR16_LO IV_table
The ADDR16_LO reloc is of course completely bogus on e_li, which has
a split 20-bit signed integer field in bits 0x1f7fff, the low 11 bit
in 0x7ff, the next 5 bits in 0x1f0000, and the high 4 bits in 0x7800.
Applying an ADDR16_LO reloc to the instruction potentially changes
the e_li instruction to e_add2i., e_add2is, e_cmp16i, e_mull2i,
e_cmpl16i, e_cmph16i, e_cmphl16i, e_or2i, e_and2i., e_or2is, e_lis,
e_and2is, or some invalid encodings.

Now there is a relocation that suits e_li, R_PPC_VLE_ADDR20, which was
added 2017-09-05 but I can't see code in gas to generate the
relocation.  In any case, VLE_ADDR20 probably doesn't have the correct
semantics for @l since ideally you'd want an @l to pair with @h or @ha
to generate a 32-bit constant.  Thus @l should only produce a 16-bit
value, I think.  So we need some more relocations to handle e_li it
seems, or as I do in this patch, modify the behaviour of existing
relocations when applied to e_li instructions.

include/
        * opcode/ppc.h (E_OPCODE_MASK, E_LI_MASK, E_LI_INSN): Define.
bfd/
        * elf32-ppc.c (ppc_elf_howto_raw <R_PPC_VLE_ADDR20>): Correct
        mask and shift value.
        (ppc_elf_vle_split16): Use E_OPCODE_MASK.  Handle e_li
        specially.
gas/
        * config/tc-ppc.c (md_assemble): Adjust relocs for VLE before
        TLS tweaks.  Handle e_li.  Warn on unexpected operand field
        for lo16/hi16/ha16 relocs.
Reply | Threaded
Open this post in threaded view
|

Failures on RHEL-s390x-m64, branch master

sergiodj+buildbot-2
Buildslave:
        rhel-7_1-s390x-1

Full Build URL:
        <http://gdb-build.sergiodj.net/builders/RHEL-s390x-m64/builds/8889>

Commit(s) tested:
        bb6bf75e7a1f9aaf0283895705710f415b81b6b1

Author(s) (in the same order as the commits):
        Alan Modra <[hidden email]>

Subject:
        PowerPC @l, @h and @ha warnings, plus VLE e_li

Testsuite log (gdb.sum and gdb.log) URL(s):
        <http://gdb-build.sergiodj.net/results/RHEL-s390x-m64/bb/bb6bf75e7a1f9aaf0283895705710f415b81b6b1/>

*** Diff to previous build ***
============================
PASS -> KFAIL: gdb.threads/process-dies-while-handling-bp.exp: non_stop=off: cond_bp_target=0: inferior 1 exited
new KFAIL: gdb.threads/watchthreads2.exp: gdb can drop watchpoints in multithreaded app
============================


*** Complete list of XFAILs for this builder ***

To obtain the list of XFAIL tests for this builder, go to:

        <https://git.sergiodj.net/gdb-xfails.git/tree/xfails/RHEL-s390x-m64/xfails/master/xfail?id=e8b4bade>

You can also see a pretty-printed version of the list, with more information
about each XFAIL, by going to:

        <https://git.sergiodj.net/gdb-xfails.git/tree/xfails/RHEL-s390x-m64/xfails/master/xfail.table?id=e8b4bade>



Reply | Threaded
Open this post in threaded view
|

Failures on Fedora-x86_64-native-extended-gdbserver-m64, branch master

sergiodj+buildbot-2
In reply to this post by sergiodj+buildbot-2
Buildslave:
        fedora-x86-64-1

Full Build URL:
        <http://gdb-build.sergiodj.net/builders/Fedora-x86_64-native-extended-gdbserver-m64/builds/11553>

Commit(s) tested:
        bb6bf75e7a1f9aaf0283895705710f415b81b6b1

Author(s) (in the same order as the commits):
        Alan Modra <[hidden email]>

Subject:
        PowerPC @l, @h and @ha warnings, plus VLE e_li

Testsuite log (gdb.sum and gdb.log) URL(s):
        <http://gdb-build.sergiodj.net/results/Fedora-x86_64-native-extended-gdbserver-m64/bb/bb6bf75e7a1f9aaf0283895705710f415b81b6b1/>

*** Diff to previous build ***
============================
FAIL -> UNRESOLVED: gdb.mi/mi-exec-run.exp: inferior-tty=main: mi=main: force-fail=1: run failure detected
FAIL -> UNRESOLVED: gdb.mi/mi-exec-run.exp: inferior-tty=main: mi=separate: force-fail=1: run failure detected
FAIL -> UNRESOLVED: gdb.mi/mi-exec-run.exp: inferior-tty=separate: mi=main: force-fail=1: run failure detected
FAIL -> UNRESOLVED: gdb.mi/mi-exec-run.exp: inferior-tty=separate: mi=separate: force-fail=1: run failure detected
PASS -> UNRESOLVED: gdb.threads/attach-into-signal.exp: threaded: attach
PASS -> KFAIL: gdb.threads/process-dies-while-handling-bp.exp: non_stop=off: cond_bp_target=0: inferior 1 exited
============================


*** Complete list of XFAILs for this builder ***

To obtain the list of XFAIL tests for this builder, go to:

        <https://git.sergiodj.net/gdb-xfails.git/tree/xfails/Fedora-x86_64-native-extended-gdbserver-m64/xfails/master/xfail?id=d67b7e56>

You can also see a pretty-printed version of the list, with more information
about each XFAIL, by going to:

        <https://git.sergiodj.net/gdb-xfails.git/tree/xfails/Fedora-x86_64-native-extended-gdbserver-m64/xfails/master/xfail.table?id=d67b7e56>



Reply | Threaded
Open this post in threaded view
|

Failures on Fedora-x86_64-native-extended-gdbserver-m32, branch master

sergiodj+buildbot-2
In reply to this post by sergiodj+buildbot-2
Buildslave:
        fedora-x86-64-4

Full Build URL:
        <http://gdb-build.sergiodj.net/builders/Fedora-x86_64-native-extended-gdbserver-m32/builds/11282>

Commit(s) tested:
        bb6bf75e7a1f9aaf0283895705710f415b81b6b1

Author(s) (in the same order as the commits):
        Alan Modra <[hidden email]>

Subject:
        PowerPC @l, @h and @ha warnings, plus VLE e_li

Testsuite log (gdb.sum and gdb.log) URL(s):
        <http://gdb-build.sergiodj.net/results/Fedora-x86_64-native-extended-gdbserver-m32/bb/bb6bf75e7a1f9aaf0283895705710f415b81b6b1/>

*** Diff to previous build ***
============================
new UNRESOLVED: gdb.ada/print_chars.exp: delete all breakpoints in delete_breakpoints
PASS -> UNRESOLVED: gdb.ada/print_chars.exp: print C
PASS -> FAIL: gdb.ada/print_chars.exp: print WC
PASS -> FAIL: gdb.ada/print_chars.exp: print WWC
PASS -> FAIL: gdb.base/fork-print-inferior-events.exp: print_inferior_events=off: follow_fork_mode=child: detach_on_fork=off: run
PASS -> FAIL: gdb.base/fork-print-inferior-events.exp: print_inferior_events=off: follow_fork_mode=parent: detach_on_fork=off: run
PASS -> UNRESOLVED: gdb.base/interp.exp: interpreter-exec mi "-var-update *"
PASS -> FAIL: gdb.base/interp.exp: interpreter-exec mi1 "-break-insert main"
PASS -> FAIL: gdb.base/interp.exp: interpreter-exec mi2 "-break-insert main"
PASS -> FAIL: gdb.base/interp.exp: interpreter-exec mi3 "-break-insert main"
new FAIL: gdb.base/interp.exp: run to main
PASS -> FAIL: gdb.base/multi-forks.exp: info inferior 1
PASS -> FAIL: gdb.base/multi-forks.exp: info inferior 3
PASS -> FAIL: gdb.base/multi-forks.exp: run to exit 3
PASS -> KFAIL: gdb.threads/process-dies-while-handling-bp.exp: non_stop=off: cond_bp_target=1: inferior 1 exited
============================


*** Complete list of XFAILs for this builder ***

To obtain the list of XFAIL tests for this builder, go to:

        <https://git.sergiodj.net/gdb-xfails.git/tree/xfails/Fedora-x86_64-native-extended-gdbserver-m32/xfails/master/xfail?id=d39bf374>

You can also see a pretty-printed version of the list, with more information
about each XFAIL, by going to:

        <https://git.sergiodj.net/gdb-xfails.git/tree/xfails/Fedora-x86_64-native-extended-gdbserver-m32/xfails/master/xfail.table?id=d39bf374>



Reply | Threaded
Open this post in threaded view
|

Failures on Fedora-x86_64-native-gdbserver-m64, branch master

sergiodj+buildbot-2
In reply to this post by sergiodj+buildbot-2
Buildslave:
        fedora-x86-64-2

Full Build URL:
        <http://gdb-build.sergiodj.net/builders/Fedora-x86_64-native-gdbserver-m64/builds/11518>

Commit(s) tested:
        bb6bf75e7a1f9aaf0283895705710f415b81b6b1

Author(s) (in the same order as the commits):
        Alan Modra <[hidden email]>

Subject:
        PowerPC @l, @h and @ha warnings, plus VLE e_li

Testsuite log (gdb.sum and gdb.log) URL(s):
        <http://gdb-build.sergiodj.net/results/Fedora-x86_64-native-gdbserver-m64/bb/bb6bf75e7a1f9aaf0283895705710f415b81b6b1/>

*** Diff to previous build ***
============================
new UNRESOLVED: gdb.mi/mi-break.exp: mi-mode=separate:
============================


*** Complete list of XFAILs for this builder ***

To obtain the list of XFAIL tests for this builder, go to:

        <https://git.sergiodj.net/gdb-xfails.git/tree/xfails/Fedora-x86_64-native-gdbserver-m64/xfails/master/xfail?id=db8de2a6>

You can also see a pretty-printed version of the list, with more information
about each XFAIL, by going to:

        <https://git.sergiodj.net/gdb-xfails.git/tree/xfails/Fedora-x86_64-native-gdbserver-m64/xfails/master/xfail.table?id=db8de2a6>



Reply | Threaded
Open this post in threaded view
|

Failures on Fedora-x86_64-native-gdbserver-m32, branch master

sergiodj+buildbot-2
In reply to this post by sergiodj+buildbot-2
Buildslave:
        fedora-x86-64-1

Full Build URL:
        <http://gdb-build.sergiodj.net/builders/Fedora-x86_64-native-gdbserver-m32/builds/11288>

Commit(s) tested:
        bb6bf75e7a1f9aaf0283895705710f415b81b6b1

Author(s) (in the same order as the commits):
        Alan Modra <[hidden email]>

Subject:
        PowerPC @l, @h and @ha warnings, plus VLE e_li

Testsuite log (gdb.sum and gdb.log) URL(s):
        <http://gdb-build.sergiodj.net/results/Fedora-x86_64-native-gdbserver-m32/bb/bb6bf75e7a1f9aaf0283895705710f415b81b6b1/>

*** Diff to previous build ***
============================
new FAIL: gdb.base/corefile.exp: core-file warning-free
============================


*** Complete list of XFAILs for this builder ***

To obtain the list of XFAIL tests for this builder, go to:

        <https://git.sergiodj.net/gdb-xfails.git/tree/xfails/Fedora-x86_64-native-gdbserver-m32/xfails/master/xfail?id=44db6fcc>

You can also see a pretty-printed version of the list, with more information
about each XFAIL, by going to:

        <https://git.sergiodj.net/gdb-xfails.git/tree/xfails/Fedora-x86_64-native-gdbserver-m32/xfails/master/xfail.table?id=44db6fcc>



Reply | Threaded
Open this post in threaded view
|

Failures on Ubuntu-Aarch64-native-extended-gdbserver-m64, branch master

sergiodj+buildbot-2
In reply to this post by sergiodj+buildbot-2
Buildslave:
        ubuntu16-aarch64

Full Build URL:
        <http://gdb-build.sergiodj.net/builders/Ubuntu-Aarch64-native-extended-gdbserver-m64/builds/244>

Commit(s) tested:
        bb6bf75e7a1f9aaf0283895705710f415b81b6b1

Author(s) (in the same order as the commits):
        Alan Modra <[hidden email]>

Subject:
        PowerPC @l, @h and @ha warnings, plus VLE e_li

Testsuite log (gdb.sum and gdb.log) URL(s):
        <http://gdb-build.sergiodj.net/results/Ubuntu-Aarch64-native-extended-gdbserver-m64/bb/bb6bf75e7a1f9aaf0283895705710f415b81b6b1/>




Reply | Threaded
Open this post in threaded view
|

Failures on Ubuntu-Aarch64-native-gdbserver-m64, branch master

sergiodj+buildbot-2
In reply to this post by sergiodj+buildbot-2
Buildslave:
        ubuntu16-aarch64

Full Build URL:
        <http://gdb-build.sergiodj.net/builders/Ubuntu-Aarch64-native-gdbserver-m64/builds/4613>

Commit(s) tested:
        bb6bf75e7a1f9aaf0283895705710f415b81b6b1

Author(s) (in the same order as the commits):
        Alan Modra <[hidden email]>

Subject:
        PowerPC @l, @h and @ha warnings, plus VLE e_li

Testsuite log (gdb.sum and gdb.log) URL(s):
        <http://gdb-build.sergiodj.net/results/Ubuntu-Aarch64-native-gdbserver-m64/bb/bb6bf75e7a1f9aaf0283895705710f415b81b6b1/>