From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR02-AM0-obe.outbound.protection.outlook.com (mail-am0eur02on2063.outbound.protection.outlook.com [40.107.247.63]) by sourceware.org (Postfix) with ESMTPS id 2212A3858CDA for ; Wed, 11 Jan 2023 09:01:41 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 2212A3858CDA Authentication-Results: sourceware.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=suse.com ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=jiTW3hlNdBJKSZBxQNQhniG1lchiat2II/ksqIuTIl44l31hjluRoVk0biwcVi7MeCgVQNX61ZyK6FeMIl3Dg9g2GmdL8r0YPtoSHz5OOog1vRPPuKQkGTghmAOwMdXCChn9xqjGQ08+9TVdbIEZZdWomWTMsAYoIBmh90DFPGExaQMfIZc0ckcs6pku5RomfkyNUkGraq7xUuuAzUAtVlZWVxQB7twERtqpvX88Qa0qjFLjXTU8H5eC9RDZj7P/f4oYVwTaTXNek3TDrios8B8OXJGUd6IaA7VGk54bW5BU43qQ7Km1GHv53KMvTnKjvmNapC7DhZlyuU89O7fVXA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=hkvK40e3MHktV8PRm52D+jVGzJpP4A5WRy4Jfs4DFVY=; b=Wz/KiC6uDKOqOBOBzIwW29YuUUPPgcF+FmcUla65vZL4KtuMOXfR4pJC5mcxytfSQ4pBb0B7lOfSlxQ1G0f1P8Z6aLhWHRLtXoZDn9brE5knW3HHACPo5eix7LLimMINso7tyvE4VJov/k1PScVMGCaeFCG0bm7qEs2Yzt+B+Lit+RMXOAl0/hlbPFUd4RH8GKZl6d6vJGSd6ml0JOLlWnnLu58FwxdRGe0i8frbYBJ3IPtQmsIY6EKMyKMyYNmmuCI25zvLBnBvlpyIot1RdFn/YCAJv+e70nuoxmBvi75wubU7o158g4rifQZ/7q8IhFVB4RUrXZyFrZFUamPshQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=hkvK40e3MHktV8PRm52D+jVGzJpP4A5WRy4Jfs4DFVY=; b=fnDWNVa9USZJ3iDI9vMWy64lO8SNMb3IVEGGYdOb1ChEksTu2oo/zLjjAbmAOXl8jTs963dsZFYNSF7BOnhYZke/oQdOxc6q1lEhEJbw4z3/qTAjUbKubVdUB63eKnItnuZ758L6eFXC/U2jSQ7JdxDEPRnBLCbP4kmFucps8L3jxzN79NBBigd2SCo5dCpsq1Lr0qqP9WzqXaJq/iJntyw9b5dZw3zPWgjpI70F0hTOHKc99qFpss5TTdIlHLmgehgl2ITzbUaO7YAAEABY3W/GIa7QzVbXW++DzQYH1LKtdMuPqDPWBgdlDd8iWFPKfcM46CwL56qLUxhowfMMIA== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com; Received: from VE1PR04MB6560.eurprd04.prod.outlook.com (2603:10a6:803:122::25) by DU0PR04MB9695.eurprd04.prod.outlook.com (2603:10a6:10:321::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5986.18; Wed, 11 Jan 2023 09:01:38 +0000 Received: from VE1PR04MB6560.eurprd04.prod.outlook.com ([fe80::2991:58a4:e308:4389]) by VE1PR04MB6560.eurprd04.prod.outlook.com ([fe80::2991:58a4:e308:4389%7]) with mapi id 15.20.6002.012; Wed, 11 Jan 2023 09:01:38 +0000 Message-ID: <1b117591-9200-72b6-d588-769d8175e8fa@suse.com> Date: Wed, 11 Jan 2023 10:01:36 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.6.1 Subject: Re: [PATCH] ld: Allow R_X86_64_GOTPCREL for call *__tls_get_addr@GOTPCREL(%rip) Content-Language: en-US To: Fangrui Song , "H.J. Lu" Cc: binutils@sourceware.org References: <20230105210542.3573076-1-maskray@google.com> <0c9cd182-ce08-6720-1e51-d7171f075641@suse.com> From: Jan Beulich In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: FR0P281CA0077.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:1e::16) To VE1PR04MB6560.eurprd04.prod.outlook.com (2603:10a6:803:122::25) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: VE1PR04MB6560:EE_|DU0PR04MB9695:EE_ X-MS-Office365-Filtering-Correlation-Id: b38cf96b-93fe-4764-31cb-08daf3b272fb X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: yO68JmVEReH9BvgQWybmo9StGLViQ3jsIli76AspYd1EGj/5QH/xU6UpE6CeXoFpeumSHg+vgaOjUwPf0vHrZWRkP/2a591/Xl1w8pWjqv92FQj0xao/i+mr1fOLoawqteigZmpo7HnbuGM51MsGp3+B9x+0z5qp3A/ZI9zB57D0UREqEIE0t2XjraLJEpNt8v2t/CooGWjKCO0zOZrjCifrKK3ZuAIX39jaS4B54cQs9lQeGfUtc3Bb6cXUYVu5ndxGPxJcbhaCEXtifuXW+A3QRYI7bnKDxdzVitU1P2o/Epw17kLSgcHkiMHbjFoRzMD53IX4qD//CuEU6FG8/2vxVX67/mw5YihQcZ+q9i267XXmgeVldIsz1rq61Jp/Yw5pcDemJSVsyO5q3yMwPErhp4mLvjXr4UwVcHY/T4jOYoH/R0Z7+zWTNOukJhcg5Wh/UNVQ6NXIVmLH5nvNj7i9THmPoAzT0pFEetvmdoBN2BhJQyfUUJAdEcU3l5n4bQZAL/6zychxkEb4gQCceeWH/Y692ZdKd9iwOwJmiOetgQcsAXOJevil7K08Nvxp5nfxltQ3jsZMJJuHkt9OMXoJVZkF8yqyelierYvefusVWE15dt7AzMHhVtVEk866G4vGWknBVlU2jelc6Yf+CpMAQ92GgxhfPxD5TCykrdpWPLS0d/Dx80KbprnURn/rF2fsL991caj02PriWJQ6eSflrogy03kK5KprzhkoPpY= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:VE1PR04MB6560.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230022)(376002)(396003)(366004)(136003)(346002)(39860400002)(451199015)(83380400001)(31686004)(6506007)(316002)(86362001)(6486002)(31696002)(110136005)(36756003)(38100700002)(6512007)(5660300002)(53546011)(26005)(186003)(2616005)(2906002)(478600001)(8676002)(8936002)(66556008)(66946007)(4326008)(41300700001)(66476007)(45980500001)(43740500002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Sk01ZmpkOWNOUVBrY1JGN2pYNy9SRTh3Wk5hb1ZhYWcyMGovTWFsdy9FU3BX?= =?utf-8?B?ckxrT3JDZDNyZ3A2Nmw5Vytwak9kc2dmQ3ptQjUvamRjMndaNkZRd3dIMHkv?= =?utf-8?B?YTlZRmNxOXdTZXkyQkdhR3NZRUF6TXJ2bGhyMEhpN2tjeHN4V0pEWlFFMmwr?= =?utf-8?B?dElJVkJUdDNCcndmQmo3RE1UZkdoY0kyTldBOXN3SldVaER5aXQxb1BLRkdo?= =?utf-8?B?QlB5a294VnpEU2tjZWIvVGYvSVFOUHZ5ME03WGN4aVJjRUpSMXNpWUF2WnVI?= =?utf-8?B?eElnMGFNV05xSU04LzFkMHF4UUNRNHZyOTB5TC9BNlZQZFUzVFc3bnkvRVpW?= =?utf-8?B?NmxxS0s3MmxnREc2UENXL2wxdit2dnBLWnEyemlMTDRhR0RDSVcvTDI0ZnVZ?= =?utf-8?B?SFl3aGQ2azY0bC9kVkU4aHcxUVh6NnZRbWV2RENWbTgreFV2OVdrTGVSejFa?= =?utf-8?B?Um1TUEVXbXZPUWRKNTNOd1hDNFc0UElFTTNwN1kxdXlCWm0vUVlXUkRUaXp4?= =?utf-8?B?ZU83L0t4dzhha3o5dTlZSG9VQ3g3OUtiZFloSTB3RTVqcU5pdVJrZ1NBZ0JT?= =?utf-8?B?U1ZzNUJXN3AyZXhnVVhQV08zd2Q1Q1BYYVlKMlp5TUdWaE9aSFdwM0xtUHpS?= =?utf-8?B?WmovWEZMMmQ5SVZKTXdLZHcrSjlTeGx6amw3Zlhsb1JvNkJiRTBxNnE0REo4?= =?utf-8?B?cjdnd1B3MmdTOXYvUFZKVlhTaEJrb0YzdUFKbjV2SDNjTTZwZVQxOFNpcm5t?= =?utf-8?B?MkozVmgwcGhWb2tqaERHK0lteEd4ZGtFckRLVDY2TUtEcC95OFh4ZkpiMlVO?= =?utf-8?B?K2dNeEZKbE5oVUlsOW9QNlRIRFV6QlhwSE4xUEVONzlnSkdWMktEYkJENy81?= =?utf-8?B?QXcyMFFmNHZSUDNkWUFON2dyMERtUWxxWEdMSTZYKzJKZ0t5aXpmKzVHSVV3?= =?utf-8?B?YzBXczVVWEhHalVpeVJtYmRGWXRaR1A2bHJpUDJJVVpCaURkYTJJUkJCaERL?= =?utf-8?B?NW1IS2NkeW5OVHVna3U4cHd6Q0lnakFDTXQ3MUU3TzJwb0dCdDBWK1Axa1NL?= =?utf-8?B?TWlVYjg0V0pNQWJ6Q1pUa3Npd0VubUlJaWVPUVJ6V1Q1alFUL2g5RTh5TDV5?= =?utf-8?B?c3Z4cDVsNGJUeHk1SWRvZzBSWWhqZHZTYWdpVDRWNytCa3RjR3JiR0w4RVZq?= =?utf-8?B?eEhoaWNSd0thNmlKNEVSN01yckR5alU5dG5mWTRMbURGMjh0Vjk0NHNmOXFK?= =?utf-8?B?cFRBd096QklCUG1TYUNGQ2lZdEcvMytud3JkY3ZUTUpiNnFoUkt3QkNuUTBJ?= =?utf-8?B?ZXc0ODFNWE5tbVpReE5FdFZCZ2lERXpuYi9JeU9odGcwa1BrODIrZ1Z1Mldm?= =?utf-8?B?Z2xNbVA4VHVBMXFWYS9VQ1RCcWdMQVY3SXBuYkppMlBva0JTYU1uT1dtbWlI?= =?utf-8?B?V3BoTnlrVXd5b2dCME54Nno1cXhFYjVlWkJXbFhlSThlMFFyaW1FOXdwTXpH?= =?utf-8?B?d3lyRy9jMW5vVUYwdERaTTJUMjg0NVBDVC94WXhXQ2lvYUN4eitEUWQ2VHEz?= =?utf-8?B?WUxkL0czeWtYM2NBbUpRdmFPUW5GR1QrcldFVXdsYWxCc1Y5UnUrQXNXaksw?= =?utf-8?B?bVFoYmowaWpsVFhqLzVuTGw3N0hjUU1Pd3RuZWw5MVVPQ09ESzZHVFRJWjh6?= =?utf-8?B?WnptS2ZsQ1NGOCtTVVZoZHptdzUvYW8vWFRHZHdod3lZcjBrYXdNWGg5L2ZR?= =?utf-8?B?TXU1NTNmZ2xCWDFZV2prejNka1ZTekU5NjdQMFp5SWszWnliTU5VaEpnMHRH?= =?utf-8?B?V3hObjV4RDFVTHJRNGMrQlhvYjhKOXBXdUJUQ1hJV0E4aUZCMlFjTzhRaXFW?= =?utf-8?B?ZFY2eGk4RWNBMmZTZXdPRFZNVVdkSkk4c3BzajUxY3doZWthMzQ2RDJ3L2ow?= =?utf-8?B?NlhUTXBHQ1ZKUHNZYVBGWUcxdWtjaXJVeWJHbERLVVdpbUx0enNwRDgvZEhU?= =?utf-8?B?WVJrK25kalBBVWZINXVjeVZOMTZWQjFHekJZYUhJMVZLS0c3cEhBdU9qNWMx?= =?utf-8?B?ZEJCMWZla1pIelkyRGV4SFphM3BDc3p2SGdyS3RjUllNK055VUlIZVUwbXF6?= =?utf-8?Q?CI4Kq1NL5LLfVM3OZxEw6eyBS?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: b38cf96b-93fe-4764-31cb-08daf3b272fb X-MS-Exchange-CrossTenant-AuthSource: VE1PR04MB6560.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Jan 2023 09:01:38.6858 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: hQN3KFoDrH45dVgc2kXFw2uEAttG6+FhA7KdL11WPddxw92mcGbSddXOSqNyXQf4PDQPQpIx+7sH0TEpF/vpUw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU0PR04MB9695 X-Spam-Status: No, score=-3034.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,GIT_PATCH_0,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,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 List-Id: On 10.01.2023 22:02, Fangrui Song wrote: > On Tue, Jan 10, 2023 at 12:40 PM H.J. Lu wrote: >> On Tue, Jan 10, 2023 at 1:16 AM Jan Beulich wrote: >>> On 09.01.2023 22:14, H.J. Lu wrote: >>>> On Mon, Jan 9, 2023 at 12:15 AM Jan Beulich wrote: >>>>> On 06.01.2023 18:03, H.J. Lu via Binutils wrote: >>>>>> On Thu, Jan 5, 2023 at 1:06 PM Fangrui Song via Binutils >>>>>> wrote: >>>>>>> >>>>>>> _Thread_local int a; >>>>>>> int main() { return a; } >>>>>>> >>>>>>> % gcc -fno-plt -fpic a.c -fuse-ld=bfd -Wa,-mrelax-relocations=no >>>>>>> /usr/bin/ld.bfd: /tmp/ccSSBgrg.o: TLS transition from R_X86_64_TLSGD to R_X86_64_GOTTPOFF against `a' at 0xd in section `.text' failed >>>>>>> /usr/bin/ld.bfd: failed to set dynamic section sizes: bad value >>>>>>> collect2: error: ld returned 1 exit status >>>>>>> >>>>>>> This commit fixes the issue. >>>>>>> >>>>>>> PR ld/24784 >>>>>>> * bfd/elf64-x86-64.c (elf_x86_64_check_tls_transition): Allow >>>>>>> R_X86_64_GOTPCREL. >>>>>>> --- >>>>>>> bfd/elf64-x86-64.c | 2 +- >>>>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>>>> >>>>>>> diff --git a/bfd/elf64-x86-64.c b/bfd/elf64-x86-64.c >>>>>>> index 914f82d0151..095fe2e0fe6 100644 >>>>>>> --- a/bfd/elf64-x86-64.c >>>>>>> +++ b/bfd/elf64-x86-64.c >>>>>>> @@ -1241,7 +1241,7 @@ elf_x86_64_check_tls_transition (bfd *abfd, >>>>>>> if (largepic) >>>>>>> return r_type == R_X86_64_PLTOFF64; >>>>>>> else if (indirect_call) >>>>>>> - return r_type == R_X86_64_GOTPCRELX; >>>>>>> + return (r_type == R_X86_64_GOTPCRELX || r_type == R_X86_64_GOTPCREL); >>>>>>> else >>>>>>> return (r_type == R_X86_64_PC32 || r_type == R_X86_64_PLT32); >>>>>>> } >>>>>>> -- >>>>>>> 2.39.0.314.g84b9a713c41-goog >>>>>>> >>>>>> >>>>>> Since the new TLS sequence was added after R_X86_64_GOTPCRELX was >>>>>> required for call, R_X86_64_GOTPCREL should be invalid in this TLS sequence. >>>>> >>>>> While this may well be, would you mind pointing out (more to Fangrui than to >>>>> me) what bad his proposed change would do? >>>> >>>> The problem is caused by the combination of -fno-plt and >>>> -Wa,-mrelax-relocations=no. >>>> -Wa,-mrelax-relocations=no was added to generate object files to be >>>> consumed by the >>>> older linkers. On the other hand, -fno-plt requires newer linkers. >>>> As the result, >>>> -fno-plt -Wa,-mrelax-relocations=no generates object files which >>>> aren't compatible >>>> with neither older linkers nor newer linkers. >>>> -Wa,-mrelax-relocations shouldn't be used >>>> together with -fno-plt. >>> >>> Imo use of such option combinations should either be disallowed (warned >>> about at the very least) or produce sensible output. I guess only the >>> latter would help Fangrui ... >>> >> >> This isn't a supported combination. I believe -Wa,-mrelax-relocations=no >> should be removed. > > Removing -Wa,-mrelax-relocations=no implies that R_X86_64_GOTPCREL is > completely useless and -Wl,--no-relax is not useful for x86. > > As my earliest replies mentioned, a relocation type indication no > relaxation is useful in some cases: hwasan (Intel LAM) references to > global variables, one-pass relocation scanning design in a linker, > even if we disregard the relocatable-file-producer with > old-linker-consumer compatibility scenarios. > > If the linker can decide upfront whether GOTPCREL{,X} needs a GOT > entry, the relocation scanning pass be one-pass and be completely > moved before synthetic sections (.got, .plt, .got.plt, etc), instead > of interleaving relocation scanning, synthetic section size decision, > and section layout. I agree with Fangrui's view here. I further think that the original change was (once again) overly limiting applicability (why ADC and SBB but not e.g. ADCX and ADOX) or doing things in an overly relaxed fashion (the subsequently addressed 16-bit operand size issue being just one aspect; I wonder whether 32-bit vs 64-bit operand size is actually handled correctly there in 64-bit and/or x32 mode). Jan