From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pf1-x430.google.com (mail-pf1-x430.google.com [IPv6:2607:f8b0:4864:20::430]) by sourceware.org (Postfix) with ESMTPS id 7F26A385801C for ; Wed, 10 Aug 2022 06:01:13 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 7F26A385801C Received: by mail-pf1-x430.google.com with SMTP id g12so12795818pfb.3 for ; Tue, 09 Aug 2022 23:01:13 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-disposition:mime-version:message-id:subject:to:from:date :x-gm-message-state:from:to:cc; bh=yMXYg5ywI9JGAc3YcR80nOXA0Eh01ZlxiVbtw0HTl0k=; b=OUgb13vSxEv2Hf//RKePFqSWWi3XblWiX0gYTVsR0W4gh+2UyWE6Nx6l5uE3SYq/Sb Ov1idEI3mrypSkDX12X3iqW91hzZzph0/Uklh5f2heHnfifHe5U49VQ6hYxHVhnV52Kx eZk9V4N4Jjk7X6AoFZ9w3PBXGXpz/5XB3A0U5vtITJEjAtzL/Hxhit4PitC1oLB5XHC9 EJsHHhXHpZzZ343SvApcqgNsmepDMCHAji15v0BAC1ohb3gqVGf80RpJZ15p1/yosGex W+d10lMtsr5ajRjxXWQMj90t4cLTYX2c0SICHM4NHWxR89KCpfI7T0sZ+WqKCjjKJUBq p8KA== X-Gm-Message-State: ACgBeo13kmu2IIlcyW2RKlwni91loC7AURZFTgfmKBiJ+7RN++X3CzFn D7Igfm07VLoA2DnnYHf2RNL8BEhyJfE= X-Google-Smtp-Source: AA6agR7amrM/2qd33EBChgTV5+jYDiY3kXsR3kwz61IYf7gttxEAi/lYn9MY1VlA8h4cZRSse/tMyQ== X-Received: by 2002:a05:6a00:114c:b0:528:2c7a:6302 with SMTP id b12-20020a056a00114c00b005282c7a6302mr26336284pfm.37.1660111272029; Tue, 09 Aug 2022 23:01:12 -0700 (PDT) Received: from squeak.grove.modra.org (158.106.96.58.static.exetel.com.au. [58.96.106.158]) by smtp.gmail.com with ESMTPSA id w16-20020a1709026f1000b001712c008f99sm1795801plk.11.2022.08.09.23.01.11 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 09 Aug 2022 23:01:11 -0700 (PDT) Received: by squeak.grove.modra.org (Postfix, from userid 1000) id C632D114047B; Wed, 10 Aug 2022 15:31:08 +0930 (ACST) Date: Wed, 10 Aug 2022 15:31:08 +0930 From: Alan Modra To: binutils@sourceware.org Subject: PR29462, internal error in relocate, at powerpc.cc:10796 Message-ID: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline X-Spam-Status: No, score=-3036.4 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, FREEMAIL_FROM, GIT_PATCH_0, KAM_NUMSUBJECT, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE, WEIRD_PORT 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: binutils@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Binutils mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 10 Aug 2022 06:01:17 -0000 Prior to the inline plt call support (commit 08be322439), the only local syms with plt entries were local ifunc symbols. There shouldn't be stubs for other local symbols so don't look for them. The patch also fixes minor bugs in get_reference_flags; Many relocs are valid only for ppc64 and a couple only for ppc32. PR 29462 * powerpc.cc (Target_powerpc::Relocate::relocate): Rename use_plt_offset to pltcal_to_direct, invert logic. For relocs not used with inline plt sequences against local symbols, only look for stubs when the symbol is an ifunc. (Target_powerpc::Scan::get_reference_flags): Correct reloc handling for relocs not valid for both 32-bit and 64-bit. diff --git a/gold/powerpc.cc b/gold/powerpc.cc index 86e2b99be4e..22c0462326d 100644 --- a/gold/powerpc.cc +++ b/gold/powerpc.cc @@ -7675,22 +7675,18 @@ Target_powerpc::Scan::get_reference_flags( switch (r_type) { + case elfcpp::R_PPC64_TOC: + if (size != 64) + break; + // Fall through. case elfcpp::R_POWERPC_NONE: case elfcpp::R_POWERPC_GNU_VTINHERIT: case elfcpp::R_POWERPC_GNU_VTENTRY: - case elfcpp::R_PPC64_TOC: // No symbol reference. break; case elfcpp::R_PPC64_ADDR64: case elfcpp::R_PPC64_UADDR64: - case elfcpp::R_POWERPC_ADDR32: - case elfcpp::R_POWERPC_UADDR32: - case elfcpp::R_POWERPC_ADDR16: - case elfcpp::R_POWERPC_UADDR16: - case elfcpp::R_POWERPC_ADDR16_LO: - case elfcpp::R_POWERPC_ADDR16_HI: - case elfcpp::R_POWERPC_ADDR16_HA: case elfcpp::R_PPC64_ADDR16_HIGHER34: case elfcpp::R_PPC64_ADDR16_HIGHERA34: case elfcpp::R_PPC64_ADDR16_HIGHEST34: @@ -7700,6 +7696,16 @@ Target_powerpc::Scan::get_reference_flags( case elfcpp::R_PPC64_D34_HI30: case elfcpp::R_PPC64_D34_HA30: case elfcpp::R_PPC64_D28: + if (size != 64) + break; + // Fall through. + case elfcpp::R_POWERPC_ADDR32: + case elfcpp::R_POWERPC_UADDR32: + case elfcpp::R_POWERPC_ADDR16: + case elfcpp::R_POWERPC_UADDR16: + case elfcpp::R_POWERPC_ADDR16_LO: + case elfcpp::R_POWERPC_ADDR16_HI: + case elfcpp::R_POWERPC_ADDR16_HA: ref = Symbol::ABSOLUTE_REF; break; @@ -7710,13 +7716,14 @@ Target_powerpc::Scan::get_reference_flags( ref = Symbol::FUNCTION_CALL | Symbol::ABSOLUTE_REF; break; - case elfcpp::R_PPC64_REL64: - case elfcpp::R_POWERPC_REL32: case elfcpp::R_PPC_LOCAL24PC: - case elfcpp::R_POWERPC_REL16: - case elfcpp::R_POWERPC_REL16_LO: - case elfcpp::R_POWERPC_REL16_HI: - case elfcpp::R_POWERPC_REL16_HA: + if (size != 32) + break; + // Fall through. + ref = Symbol::RELATIVE_REF; + break; + + case elfcpp::R_PPC64_REL64: case elfcpp::R_PPC64_REL16_HIGH: case elfcpp::R_PPC64_REL16_HIGHA: case elfcpp::R_PPC64_REL16_HIGHER: @@ -7729,36 +7736,45 @@ Target_powerpc::Scan::get_reference_flags( case elfcpp::R_PPC64_REL16_HIGHEST34: case elfcpp::R_PPC64_REL16_HIGHESTA34: case elfcpp::R_PPC64_PCREL28: + if (size != 64) + break; + // Fall through. + case elfcpp::R_POWERPC_REL32: + case elfcpp::R_POWERPC_REL16: + case elfcpp::R_POWERPC_REL16_LO: + case elfcpp::R_POWERPC_REL16_HI: + case elfcpp::R_POWERPC_REL16_HA: ref = Symbol::RELATIVE_REF; break; + case elfcpp::R_PPC_PLTREL24: + if (size != 32) + break; + ref = Symbol::FUNCTION_CALL | Symbol::RELATIVE_REF; + break; + case elfcpp::R_PPC64_REL24_NOTOC: - if (size == 32) + case elfcpp::R_PPC64_REL24_P9NOTOC: + case elfcpp::R_PPC64_PLT16_LO_DS: + case elfcpp::R_PPC64_PLTSEQ_NOTOC: + case elfcpp::R_PPC64_PLTCALL_NOTOC: + case elfcpp::R_PPC64_PLT_PCREL34: + case elfcpp::R_PPC64_PLT_PCREL34_NOTOC: + if (size != 64) break; // Fall through. - case elfcpp::R_PPC64_REL24_P9NOTOC: case elfcpp::R_POWERPC_REL24: - case elfcpp::R_PPC_PLTREL24: case elfcpp::R_POWERPC_REL14: case elfcpp::R_POWERPC_REL14_BRTAKEN: case elfcpp::R_POWERPC_REL14_BRNTAKEN: case elfcpp::R_POWERPC_PLT16_LO: case elfcpp::R_POWERPC_PLT16_HI: case elfcpp::R_POWERPC_PLT16_HA: - case elfcpp::R_PPC64_PLT16_LO_DS: case elfcpp::R_POWERPC_PLTSEQ: - case elfcpp::R_PPC64_PLTSEQ_NOTOC: case elfcpp::R_POWERPC_PLTCALL: - case elfcpp::R_PPC64_PLTCALL_NOTOC: - case elfcpp::R_PPC64_PLT_PCREL34: - case elfcpp::R_PPC64_PLT_PCREL34_NOTOC: ref = Symbol::FUNCTION_CALL | Symbol::RELATIVE_REF; break; - case elfcpp::R_POWERPC_GOT16: - case elfcpp::R_POWERPC_GOT16_LO: - case elfcpp::R_POWERPC_GOT16_HI: - case elfcpp::R_POWERPC_GOT16_HA: case elfcpp::R_PPC64_GOT16_DS: case elfcpp::R_PPC64_GOT16_LO_DS: case elfcpp::R_PPC64_GOT_PCREL34: @@ -7768,11 +7784,16 @@ Target_powerpc::Scan::get_reference_flags( case elfcpp::R_PPC64_TOC16_HA: case elfcpp::R_PPC64_TOC16_DS: case elfcpp::R_PPC64_TOC16_LO_DS: + if (size != 64) + break; + // Fall through. + case elfcpp::R_POWERPC_GOT16: + case elfcpp::R_POWERPC_GOT16_LO: + case elfcpp::R_POWERPC_GOT16_HI: + case elfcpp::R_POWERPC_GOT16_HA: ref = Symbol::RELATIVE_REF; break; - case elfcpp::R_POWERPC_GOT_TPREL16: - case elfcpp::R_POWERPC_TLS: case elfcpp::R_PPC64_TLSGD: case elfcpp::R_PPC64_TLSLD: case elfcpp::R_PPC64_TPREL34: @@ -7781,6 +7802,11 @@ Target_powerpc::Scan::get_reference_flags( case elfcpp::R_PPC64_GOT_TLSLD_PCREL34: case elfcpp::R_PPC64_GOT_TPREL_PCREL34: case elfcpp::R_PPC64_GOT_DTPREL_PCREL34: + if (size != 64) + break; + // Fall through. + case elfcpp::R_POWERPC_GOT_TPREL16: + case elfcpp::R_POWERPC_TLS: ref = Symbol::TLS_REF; break; @@ -10671,10 +10697,8 @@ Target_powerpc::Relocate::relocate( bool has_stub_value = false; bool localentry0 = false; unsigned int r_sym = elfcpp::elf_r_sym(rela.get_r_info()); - bool use_plt_offset - = (gsym != NULL - ? gsym->use_plt_offset(Scan::get_reference_flags(r_type, target)) - : object->local_has_plt_offset(r_sym)); + bool pltcall_to_direct = false; + if (is_plt16_reloc(r_type) || r_type == elfcpp::R_PPC64_PLT_PCREL34 || r_type == elfcpp::R_PPC64_PLT_PCREL34_NOTOC @@ -10688,21 +10712,18 @@ Target_powerpc::Relocate::relocate( // that the decision depends on the PLTCALL reloc, and we don't // know the address of that instruction when processing others // in the sequence. So the decision needs to be made in - // do_relax(). For now, don't optimise inline plt calls. - if (gsym) - use_plt_offset = gsym->has_plt_offset(); - } - if (use_plt_offset - && !is_got_reloc(r_type) - && !is_plt16_reloc(r_type) - && r_type != elfcpp::R_PPC64_PLT_PCREL34 - && r_type != elfcpp::R_PPC64_PLT_PCREL34_NOTOC - && r_type != elfcpp::R_POWERPC_PLTSEQ - && r_type != elfcpp::R_POWERPC_PLTCALL - && r_type != elfcpp::R_PPC64_PLTSEQ_NOTOC - && r_type != elfcpp::R_PPC64_PLTCALL_NOTOC - && (!psymval->is_ifunc_symbol() - || Scan::reloc_needs_plt_for_ifunc(target, object, r_type, false))) + // do_relax(). + pltcall_to_direct = !(gsym != NULL + ? gsym->has_plt_offset() + : object->local_has_plt_offset(r_sym)); + } + else if ((gsym != NULL + ? gsym->use_plt_offset(Scan::get_reference_flags(r_type, target)) + : psymval->is_ifunc_symbol() && object->local_has_plt_offset(r_sym)) + && !is_got_reloc(r_type) + && (!psymval->is_ifunc_symbol() + || Scan::reloc_needs_plt_for_ifunc(target, object, r_type, + false))) { if (size == 64 && gsym != NULL @@ -10796,9 +10817,9 @@ Target_powerpc::Relocate::relocate( gold_assert(has_stub_value || !(os->flags() & elfcpp::SHF_ALLOC)); } - if (use_plt_offset && (is_plt16_reloc(r_type) - || r_type == elfcpp::R_PPC64_PLT_PCREL34 - || r_type == elfcpp::R_PPC64_PLT_PCREL34_NOTOC)) + if (!pltcall_to_direct && (is_plt16_reloc(r_type) + || r_type == elfcpp::R_PPC64_PLT_PCREL34 + || r_type == elfcpp::R_PPC64_PLT_PCREL34_NOTOC)) { const Output_data_plt_powerpc* plt; if (gsym) @@ -10826,7 +10847,7 @@ Target_powerpc::Relocate::relocate( value -= target->toc_pointer(); } } - else if (!use_plt_offset + else if (pltcall_to_direct && (is_plt16_reloc(r_type) || r_type == elfcpp::R_POWERPC_PLTSEQ || r_type == elfcpp::R_PPC64_PLTSEQ_NOTOC)) @@ -10835,7 +10856,7 @@ Target_powerpc::Relocate::relocate( elfcpp::Swap<32, big_endian>::writeval(iview, nop); r_type = elfcpp::R_POWERPC_NONE; } - else if (!use_plt_offset + else if (pltcall_to_direct && (r_type == elfcpp::R_PPC64_PLT_PCREL34 || r_type == elfcpp::R_PPC64_PLT_PCREL34_NOTOC)) { @@ -11316,8 +11337,8 @@ Target_powerpc::Relocate::relocate( } else if (!has_stub_value) { - if (!use_plt_offset && (r_type == elfcpp::R_POWERPC_PLTCALL - || r_type == elfcpp::R_PPC64_PLTCALL_NOTOC)) + if (pltcall_to_direct && (r_type == elfcpp::R_POWERPC_PLTCALL + || r_type == elfcpp::R_PPC64_PLTCALL_NOTOC)) { // PLTCALL without plt entry => convert to direct call Insn* iview = reinterpret_cast(view); -- Alan Modra Australia Development Lab, IBM