From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pj1-x102d.google.com (mail-pj1-x102d.google.com [IPv6:2607:f8b0:4864:20::102d]) by sourceware.org (Postfix) with ESMTPS id 1B7EA3858CDA; Tue, 26 Jul 2022 02:00:33 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 1B7EA3858CDA Received: by mail-pj1-x102d.google.com with SMTP id o20-20020a17090aac1400b001f2da729979so2352824pjq.0; Mon, 25 Jul 2022 19:00:33 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:reply-to :from:date:message-id:subject:to:cc; bh=DaoqsYBTmMStzJGkbnWE57thxhL1hyEEC9rzgAN6+Wc=; b=UuSrowTWj7aL8AHclaGiOImKvGYSljUNQ0u2konTn6nlzFiKEtCqLeVwghKdijn8X1 ujnsqrmibWRPkOF5tzy1bL6Qrk3DvZF5FpWP2AUZbO6/H0uwlTeCFMOLVytoP/pRYSv0 hXrpLddfpuflg1yJ2D0LZ2/W07w42hyJXYh/IyP6o8Bhy0QZO4DMTY2P569runUMhyPP uG1SF9OksbE8tbyHcC1APgS6YxTnpdv0r5rWrBFVHOYIhXXjQ6/OzlLDJfHoncCOOVcJ fDsmc4keDiqFoUrTgi4vHkzId+lmKqozReQwkC5YF2NrHc19SoykGNQHpZaPF+srye2H /HgQ== X-Gm-Message-State: AJIora9XC8+Nzf6gwJaEVqdyuzmdlANhzNkS984JUSMhRa9hdTQ0lAlx JrLl5YxBButXagqym8WV0dfbsXOz2Ly6PAdm//HR4SWbHlw= X-Google-Smtp-Source: AGRyM1svLMqzHZEzrrFcRxnxUGXo/Hka50mclDFIDUiPHdv5DcyRVRQemo88SFGL+5i2vwY9W8NtYXzOUdXJJZBOhFQ= X-Received: by 2002:a17:902:e84e:b0:16b:f773:4692 with SMTP id t14-20020a170902e84e00b0016bf7734692mr14889794plg.19.1658800832020; Mon, 25 Jul 2022 19:00:32 -0700 (PDT) MIME-Version: 1.0 References: <94d446556e470859b878bb27eec5e2a52d063673.camel@klomp.org> In-Reply-To: <94d446556e470859b878bb27eec5e2a52d063673.camel@klomp.org> Reply-To: noloader@gmail.com From: Jeffrey Walton Date: Mon, 25 Jul 2022 21:59:25 -0400 Message-ID: Subject: Re: buildbot vs --enable-targets=all To: Mark Wielaard Cc: Binutils , Mahmood Naderan via Gdb Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-0.1 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, FREEMAIL_FROM, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gdb@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 26 Jul 2022 02:00:36 -0000 On Mon, Jul 25, 2022 at 8:26 AM Mark Wielaard wrote: > > Hi, > > I enabled --enable-targets=all for all builds on builder.sourceware.org > (with --disable-sim for 32bit gdb builds), which exposed some new > failing tests. I can disable --enable-targets=all again, but maybe it > makes sense to look at the failures and try to fix them. > > On fedora-s390x and debian-ppc64 one of the gdb selftests fails > https://builder.sourceware.org/buildbot/#builders/75/builds/783 > https://builder.sourceware.org/buildbot/#builders/76/builds/772 > > Running selftest arm-record. > Process record and replay target doesn't support syscall number > -2036195 > Process record does not support instruction 0x7f70ee1d at address 0x0. > Self test failed: self-test failed at ../../binutils-gdb/gdb/arm- > tdep.c:14407 > > Which is: > > /* 32-bit Thumb-2 instructions. */ > { > arm_insn_decode_record arm_record; > > memset (&arm_record, 0, sizeof (arm_insn_decode_record)); > arm_record.gdbarch = gdbarch; > > static const uint16_t insns[] = { > /* 1d ee 70 7f mrc 15, 0, r7, cr13, cr0, {3} */ > 0xee1d, 0x7f70, > }; > > enum bfd_endian endian = gdbarch_byte_order_for_code (arm_record.gdbarch); > instruction_reader_thumb reader (endian, insns); > int ret = decode_insn (reader, &arm_record, THUMB2_RECORD, > THUMB2_INSN_SIZE_BYTES); > > SELF_CHECK (ret == 0); > SELF_CHECK (arm_record.mem_rec_count == 0); > SELF_CHECK (arm_record.reg_rec_count == 1); > SELF_CHECK (arm_record.arm_regs[0] == 7); > } > > This seems a big endian issue given the instructions are given as two > 16bit numbers. > > On debian-ppc64 and fedora-ppc64le there are the following ld failures: > https://builder.sourceware.org/buildbot/#builders/86/builds/343 > https://builder.sourceware.org/buildbot/#builders/78/builds/382 > > extra lines in dump.out starting with "^ 40: 0000002c 0 > TLS GLOBAL DEFAULT 9 ie0$" > EOF from /var/lib/buildbot/workers/wildebeest/binutils-debian- > ppc64/binutils-gdb/ld/testsuite/ld-powerpc/tlsexe32.r > FAIL: TLS32 dynamic exec > > extra lines in dump.out starting with "^ 40: 0000002c 0 > TLS GLOBAL DEFAULT 9 ie0$" > EOF from /var/lib/buildbot/workers/wildebeest/binutils-debian- > ppc64/binutils-gdb/ld/testsuite/ld-powerpc/tlsexe32no.r > FAIL: TLS32 dynamic exec (--no-tls-optimize) > > extra lines in dump.out starting with "^ 38: 0000002c 0 > TLS GLOBAL DEFAULT 8 ie0$" > EOF from /var/lib/buildbot/workers/wildebeest/binutils-debian- > ppc64/binutils-gdb/ld/testsuite/ld-powerpc/tlsso32.r > FAIL: TLS32 shared > > On debian-armhf there are the following gas failures: > https://builder.sourceware.org/buildbot/#builders/80/builds/350 > > ../as-new -mcpu=cortex-a76ae -o tmpdir/nop-asm.o > /var/lib/buildbot/workers/wildebeest/binutils-debian-armhf/binutils- > gdb/gas/testsuite/gas/arm/nop-asm.s > Executing on host: sh -c > {/var/lib/buildbot/workers/wildebeest/binutils-debian-armhf/binutils- > build/gas/testsuite/../../binutils/objdump -d -mcortex-a76ae > tmpdir/nop-asm.o > tmpdir/dump.out 2>dump.tmp} /dev/null (timeout = > 300) > spawn [open ...] > exited abnormally with 1, > output:/var/lib/buildbot/workers/wildebeest/binutils-debian- > armhf/binutils-build/binutils/.libs/objdump: can't disassemble for > architecture aarch64 > > FAIL: Assemble and dump for cortex-a76ae CPU > > ../as-new -mcpu=cortex-a77 -o tmpdir/nop-asm.o > /var/lib/buildbot/workers/wildebeest/binutils-debian-armhf/binutils- > gdb/gas/testsuite/gas/arm/nop-asm.s > Executing on host: sh -c {../as-new -mcpu=cortex-a77 -o tmpdir/nop- > asm.o /var/lib/buildbot/workers/wildebeest/binutils-debian- > armhf/binutils-gdb/gas/testsuite/gas/arm/nop-asm.s 2>&1} /dev/null > dump.tmp (timeout = 300) > spawn [open ...] > /var/lib/buildbot/workers/wildebeest/binutils-debian-armhf/binutils- > build/gas/testsuite/../../binutils/objdump -d -mcortex-a77 tmpdir/nop- > asm.o > tmpdir/dump.out > Executing on host: sh -c > {/var/lib/buildbot/workers/wildebeest/binutils-debian-armhf/binutils- > build/gas/testsuite/../../binutils/objdump -d -mcortex-a77 tmpdir/nop- > asm.o > tmpdir/dump.out 2>dump.tmp} /dev/null (timeout = 300) > spawn [open ...] > exited abnormally with 1, > output:/var/lib/buildbot/workers/wildebeest/binutils-debian- > armhf/binutils-build/binutils/.libs/objdump: can't disassemble for > architecture aarch64 > > FAIL: Assemble and dump for cortex-a77 CPU > > On fedora-arm64 there is one unexpected failure and pass: > https://builder.sourceware.org/buildbot/#builders/176/builds/37 > > > tmpdir/pr26094-1c.o: in function `foo': > /home/builder/worker/binutils-fedora-arm64/binutils- > gdb/ld/testsuite/ld-elf/pr26094-1c.c:7:(.text+0x4): relocation > truncated to fit: R_AARCH64_LDST64_ABS_LO12_NC against symbol > `__start_FOO@@SOME_VERSION_NAME' defined in FOO section in > tmpdir/pr26094-1 > /home/builder/worker/binutils-fedora-arm64/binutils-build/ld/.libs/ld- > new: /home/builder/worker/binutils-fedora-arm64/binutils- > gdb/ld/testsuite/ld-elf/pr26094-1c.c:7: warning: one possible cause of > this error is that the symbol is being referenced in the indicated code > as if it had a larger alignment than was declared where it was defined > collect2: error: ld returned 1 exit status > tmpdir/pr26094-1c.o: in function `foo': > /home/builder/worker/binutils-fedora-arm64/binutils- > gdb/ld/testsuite/ld-elf/pr26094-1c.c:7:(.text+0x4): relocation > truncated to fit: R_AARCH64_LDST64_ABS_LO12_NC against symbol > `__start_FOO@@SOME_VERSION_NAME' defined in FOO section in > tmpdir/pr26094-1 > /home/builder/worker/binutils-fedora-arm64/binutils-build/ld/.libs/ld- > new: /home/builder/worker/binutils-fedora-arm64/binutils- > gdb/ld/testsuite/ld-elf/pr26094-1c.c:7: warning: one possible cause of > this error is that the symbol is being referenced in the indicated code > as if it had a larger alignment than was declared where it was defined > collect2: error: ld returned 1 exit status > Unexpected linker warning or error > FAIL: Build pr26094-1 > > cp tmpdir/libpr19719a.so tmpdir/libpr19719.so > tmpdir/pr19719 > Executing on host: sh -c {tmpdir/pr19719 2>&1} /dev/null ld.tmp > (timeout = 300) > spawn [open ...] > PASS > PASS > XPASS: Run pr19719 fun undefined > > Cheers, For ARM, this does not look right (to me): > static const uint16_t insns[] = { > /* 1d ee 70 7f mrc 15, 0, r7, cr13, cr0, {3} */ > 0xee1d, 0x7f70, > }; I think you are supposed to use .inst.n and .inst.w because they handle endianness properly. Jeff