From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-he1eur01on2051.outbound.protection.outlook.com [40.107.13.51]) by sourceware.org (Postfix) with ESMTPS id 9B91A3858D35 for ; Thu, 5 Jan 2023 07:39:19 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 9B91A3858D35 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=PdJYlz42KFZw+kwkep+JwbI5YpewInyfkDdarZxzMc5Hw+rzTBNTQvEIJQOvUxo6MozTKmtywTemb+DHg9Skm0Yf+Wna4QjVhtODoUgz/eAgPdSRDGjZLxzENXkL6nWmv/u13bGi+FA+nT+iIR9Dn27fmQ+V9N8bcN7ZAkfOmGKo/oheJmYr5jYXnLcj+ejLfBqo46ErI7QgmPnEyEMK0rVaJBLNzFLxgV6aHMRyPdVBUKGcZFoAU/eNLxoYWApoRNyQmufWPuw6nUZ/QuKAIMWOVvhvdWHOjWLRCdBTEPkewy41Sz761AXsqAWEVbUixugKS084dx9MDPWPqa1/NQ== 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=TLp6YY/gYCL1GKQVv1PX3cDHd1OUQH/T/ie9KT41zFk=; b=TGsfxntoSdMOpZb2GiUPEdz4iHsY3l/sCjkohoJ7AeMeOTunhkTb3RLjabiGerh2KBs74frAQO9QanxB4044dkQAB6H4oy64ekx0nBb1rmjJo9pKBZIGICv3CLxchsN07evhFoQidgM4t3ttgPKhRAW+0pgbD+WcrWOWy0Y3uB/R0fvN97j+sSogxxvVOv1pZrfGbjo9+Ef7CitHe3Rht2jA02zGhcVZxf+olzUKhBwjU0nAPZSs8pZrJrst/vIJ7Ogejkq6jomNjpgqIXjTn8i3eqARG7NYGjgEQ5kH1eF/4U2ZrOATqkcUsMDlmUBUzWTCFHtnhJ/G7/59xIcejg== 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=TLp6YY/gYCL1GKQVv1PX3cDHd1OUQH/T/ie9KT41zFk=; b=vQXSc/R6tTHq388t5h0BQiS60FuEIIqgWFXuPZkBJmG1eZgJuS3vH1j10o9AH6WE7NUdkujPo6vZE7gHihlN8FzqArLrffjaboAZbsCcRgWYenk8DYR/12omBLosEJwiXTRl1mal9CEKBRcYr6qTQbHV57FxnWsofix5IZj12FHKV+leyRn7k5TeI7g3RG1G55iUC4089O1eYIzEh1aB/RRdYHHvo0Pk33lo82seEDpUmrFlypeQDhm5mgWcdkT4ORqFU33Ef/rca3hFya8Ab0DX7d7umZ9+XpUR2frbxhf1scCfc9NmJSg4xmeqpPrFH41V/3uXEAYmCSC4x18CzA== 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 PAXPR04MB8735.eurprd04.prod.outlook.com (2603:10a6:102:21f::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5944.19; Thu, 5 Jan 2023 07:39:13 +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.5944.019; Thu, 5 Jan 2023 07:39:13 +0000 Message-ID: <3fd7bc58-4d5d-2bc0-2b58-b1e708703236@suse.com> Date: Thu, 5 Jan 2023 08:39:12 +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/testsuite: Fix test failures in pe.exp caused by 8819b236 Content-Language: en-US To: Mark Harmstone , Nick Clifton References: <20230105002743.25019-1-mark@harmstone.com> Cc: binutils@sourceware.org From: Jan Beulich In-Reply-To: <20230105002743.25019-1-mark@harmstone.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: FR3P281CA0152.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:a2::6) To VE1PR04MB6560.eurprd04.prod.outlook.com (2603:10a6:803:122::25) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: VE1PR04MB6560:EE_|PAXPR04MB8735:EE_ X-MS-Office365-Filtering-Correlation-Id: eed50fe1-8d30-4dba-e2e0-08daeeeff0fd X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 0I8Oj9dZrhzHg+xGh4ttN6qEa98jlwdQxIRKfAbOIQ3d30by6+nkMrMtzBIU+DFEeQJ8iTKCFenAXtyV0VVbu7TEnYKa+3dVIjY/36lZ1k2E6H5dSfcL4NJal8Ife0G6F/mlQZH6qUIZGbpVP/H8jtauz++mT3zwxp3k1jxOcwwAZQv5stD+3xgFWBBLM7dZ4Gg+qI/nfD9nwkq2J7MddxdB3OmNQVaaMXRF0eYG0epLoIk4t+FrMXymp9UpvnZhSL1qb/785Wo0QZDJmD9GlzFmR+ZvPkOQ3oGt0v7bwCMfIWt81zZyLU0bUmWVU6OKr3TiBFuNIkNBpABzDRUdteV01DzXfNJ68ViWfM/Zo/OoK/Powm4JlpxWSsJyOZqQg1yW8MwBWLzLTafWmmqBEbV/NgZmicW5Ho0ZMa2RuK0UE3xHy76xQiV1lURd/HKQtLwH3t0yKNhL+kEpSqqJK8nmilPOlNCmxM/fdpYf/2+ljNkxVETzpUc+FG8IH72BDKXif1klKgQcDxAcz+hy6C07KJThglbPNhL25g7iu4KonL+nPA9jE52pKfHezvv5SzPccd3VhZ5GFwuJLOs+6TU22qE0KeeN9qs71nSv7oxaaCzetdyk5u0LAZ5j8q7oD054ss6fyCdALOnzypT/oqo+tA1bdFtYeY4IUZhEwzzLoJSTqa5F1x5LX/0oW8TPL1pr2Aj1hEeZ3K2C3gm2kp4UOk60mRjKSHV40CZdVmc= 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)(39860400002)(346002)(396003)(136003)(376002)(366004)(451199015)(2616005)(83380400001)(31696002)(86362001)(38100700002)(36756003)(2906002)(110136005)(41300700001)(5660300002)(4326008)(66556008)(8676002)(66946007)(8936002)(66476007)(316002)(186003)(26005)(53546011)(478600001)(6512007)(6486002)(31686004)(6506007)(45980500001)(43740500002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ZDU1OHplTXFOZDVHTGdrRzBrR3dyVHhMeTRmMEtBRXdFMnJKdlE2aFN3SW1a?= =?utf-8?B?ZWdXK0pxeEErKzhWZ2ZBVGhlZ0pQOVV2WU44Z2dUeGJCcjJQSGFTNmtLajZ1?= =?utf-8?B?L1EvSjZXcmlicmhTRGdrdEdxVjA3UDNxcEFJUDdlY1hqdXhCZmEvYVhQd0JG?= =?utf-8?B?VGZMbDJaM0RvdTltUnJnSFRubGtiNEx1b3lteVpIelh3RTEraGtyeThISEtw?= =?utf-8?B?OGRwUVlMbEFWekVocHpJOWw0QnVvNk1rRFpYVTJva0NBajVaNUVieFp2VmdE?= =?utf-8?B?UlVFTyt6VFFXVGV4Mnpvc2FBOG96eWFhQUFyVEd6S1hhZW5JMDlCa0FWR1Zz?= =?utf-8?B?eVE2TTlLNXkxdlBRdUV5TFVYZG5xUDZ5blJhWHBydG9MVld2OUFPZEZlbDAz?= =?utf-8?B?K2Zod2c5NVFXT0pkQmtxUFUrbGlnUTcxbTlUcmNkbXczNzlNemtnV1p6cG9a?= =?utf-8?B?MVRML1hQRlM3amVZRytmMnYwTVpTSWxnZDhDV3ZLVXpPYjdPOWZuUUVIaHVn?= =?utf-8?B?Q1NORTl5S0NyNGx1WmNHSjdzMWxjTlJ1L0k1VTdFZkM2WW1mWDRVNWV4YXQ5?= =?utf-8?B?ZVFSeEh3WHJvNGlpMjExODlOZWlCd2N5ODU2VGJXbGVOZTNrcGx4a2FMZWp2?= =?utf-8?B?QmFnZW42QkRaMVozdTI5cVBWYVRYWENmQ3pyQXBSK1JHb0lKc213YS8zRSta?= =?utf-8?B?VkRaeXpaUnZySUk1VGJSdWpyb2dGYU1CVzRsdVpYd3NuKzIza21OZWk4OEgz?= =?utf-8?B?MDdjY3dBNkJ0Z3JQa3lFbXZwRnNud215M2NDMDcyci8zNnYvNDlNbXE5WVE2?= =?utf-8?B?K3hFTUxqVC83QnJ5SzBSUjJ6WHB1Y3NkR3lsOGxZcExrQnR3ZW9aZWQxS3Ju?= =?utf-8?B?Ny9qbkNUd2ROQ1g4QW12WDF2clRTbGVFZ2g1NncyZ3p6VkF2UHlmd3d1MEtE?= =?utf-8?B?Y2dSNHpNU2tOYmQ5TzVQYmdBbUVKZE9heit5RWNrM1k4SGZLbTY4WEc4YzJi?= =?utf-8?B?RnZKYW5wYU9pSDBra3F5dklhdXhUMjZlaS96RmhrUnRxZHB2UG1LUmRFSHNi?= =?utf-8?B?N25kbzlUclQvbTRXUEhJV0dOWTBSQ3ZMNG5MTnFPSEdSN3l6bzdaY2x6U0Yv?= =?utf-8?B?MFhUWVBmd0VRa3dPV2xVZFBGVThYWDdYK1pkdXVEUjFJdzJKSTYzeUJKOVp6?= =?utf-8?B?MWRhZ0lCRlliRkJ0UXRnTzdxVHE4RDJFK0lEU1hOWHE2VHMwOTF1bjhoWFhN?= =?utf-8?B?NS85eXBrN1ArY2NjUkdLR0kxZWxDWGNEUnF4UncvQWFwOWNTcE4xSElTOEZL?= =?utf-8?B?MHg0T0EyRmppYlhCdFhuazMvTSsxdlJES25xWjlHRVB5b09DTllMbytmMFpE?= =?utf-8?B?eWh2RnZkcnNrTFpUbzRxZHF3VXFrK2R3QkFsOUtiZVhMOVZTTmtpSHFsTGdq?= =?utf-8?B?YlpNRkFWYUtpY0xmZ2tESVQranlVVlNlY2tFTy9rUG1TQUpreG8zT2pGbWcy?= =?utf-8?B?MkoycHJVdngvWmw0cWFxZWU3cGxoVmVtMmt4cHcrcnkwSVBpTG5Tc25DUWZo?= =?utf-8?B?UFFIcTNZdmNXSy9HRThLYUt5N25aditrY2Qvbks0U2NwODlmSGdRY2FHa0ox?= =?utf-8?B?ck9WM2dhV0VIVmNWYWV1K3Vpa2huZnlXK0RQdXhmMkQ0VXo0MHJhdTluWEs1?= =?utf-8?B?SFNISnhSUkd2TmpqeGYzMm1pN0NhOHlqV0ltM0NNNXE4dzZMd0RGZmdDSzFK?= =?utf-8?B?bFZmbytIRW00OFhiVHFsaEsvMXpiUUtFUFAwb3MvS0kzQWtSSkhSYk1tU1Q2?= =?utf-8?B?dDJTcUtHSTF1RFNGUU5NanJ1R3N6b1AxQWF1SWZocmtEck5XTlpSaXZTdWdp?= =?utf-8?B?OGMwaXV2MGwxK0JkaytVaE5vNXI5Q3NSVVUvaUFYMEF0SXcyN2FTN2dJTFJZ?= =?utf-8?B?ZW52aHY0Qlh1M2NVbjNTNmFmWTBoV3hRd2U0TEpUTWJieTNGYWFjUFFDMlB5?= =?utf-8?B?ajJRQWJFM09CUjkwWXBoSlJLaXVTZE9GUXZKQ2hrMGIyc0tlSFNzczhGZHAx?= =?utf-8?B?dXc4dnJUQkYvUkhXcmtqY1FDbDl6THM0SlA5ODNHU21BenJudFlUT25lUCti?= =?utf-8?Q?7o8RkMMXJnvmDdERyiY2XVlGX?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: eed50fe1-8d30-4dba-e2e0-08daeeeff0fd X-MS-Exchange-CrossTenant-AuthSource: VE1PR04MB6560.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Jan 2023 07:39:13.5986 (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: XBKvsRSY1TMZWWGzY3ziJNuEPY6i81omo9BCpPEzaDj6U0TtgX7T/mnQHnM6M4eDgx6n738EO8Iq56QbmrJASA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB8735 X-Spam-Status: No, score=-3030.0 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,KAM_NUMSUBJECT,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 05.01.2023 01:27, Mark Harmstone wrote: > This fixes three test failures caused by the change in .idata alignment > on x86_64-w64-mingw32. > > The secrel and secidx tests were dumping .idata, so this is easy enough > to solve - it's irrelevant to the tests anyway. > > As for the CFI test, it looks like the debug_frame code inserts an empty > entry at the beginning to pad this to an 8-byte boundary, so I've > changed the section alignment for the test to fix this. I have to admit that I'm wary of changes like this, without it being clear whether the options were put there on purpose (in exactly that form). Nick, it's been over 10 years, but I still wonder whether you recall. I further think that cfi32.d would then want similar adjustment (perhaps using 4 instead of 8), even if not strictly needed to address the fallout. Altogether I think that part of the change would better be a separate change, not the least to allow easy reverting if need be. > --- a/ld/testsuite/ld-pe/cfi.d > +++ b/ld/testsuite/ld-pe/cfi.d > @@ -1,10 +1,10 @@ > #source: cfia.s > #source: cfib.s > -#ld: --file-align 1 --section-align 1 > +#ld: --file-align 8 --section-align 8 > #objdump: -Wf > > #... > -0+4 0+14 0*ffffffff CIE > +0+ 0+14 0*ffffffff CIE I have to admit that I'm puzzled by this having been like it was, i.e. expecting 4 here instead of 0 as the starting section offset. I can't help thinking that the expectation was wrong, and hence a change elsewhere is needed (and then the .idata alignment change would not have caused any fallout). > --- a/ld/testsuite/ld-pe/secrel_64.d > +++ b/ld/testsuite/ld-pe/secrel_64.d > @@ -25,4 +25,4 @@ Contents of section \.rdata: > .*3020 3e3e3e3e 00000000 00000000 00000000 >>>>............ > > Contents of section \.idata: > > .*4000 00000000 00000000 00000000 00000000 ................ > > - .*4010 00000000 .... > > + .*4010 00000000 .* > \ No newline at end of file May I ask that you take the opportunity and insert the missing newline here? With the CFI part split off the remainder is okay to put in. Jan