From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR02-AM0-obe.outbound.protection.outlook.com (mail-am0eur02on2051.outbound.protection.outlook.com [40.107.247.51]) by sourceware.org (Postfix) with ESMTPS id ECE673857C4C for ; Fri, 28 Oct 2022 08:22:31 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org ECE673857C4C 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=AAAVOlxMgS0MkIlExkvwFYOvQDfAyUJnO1rsMZzuHEIP9fdGUwkkKaPm3p7fIQWHvHK/o3dZdoCDfVpCf10KI8yRsbNOGEnVW7OVeLDkV3Cz0VwEihQZRGGOHtVQnVM5ZCqdmJS3GJ3pWJjRuCPqo/O04ZXKu60FLNpD/EJb0fhf3F5Snehj71T/obP2axb+D1Jft76rvgfRxDkBdsnB+A5hB/ZH3feKPq16Sjpoe/IJ5mxkQ6mlNfo61F+RJfQJIuXgdiZG0wKL5hWOYD0ACrbnPQdOXLqvAZ9dpsUj5WyQL9Jr3g6fxP/3kaky3H8twi5+hdeGqZ3Hws5yuhuZSw== 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=LyqMH8lkOvbkcsRQvN1bp9tJTGy5FJBf7izYagTl3j0=; b=jvbJfjOxgxJSHctj3dA5Lurkx4T56ygi+9lcCr0hQZe01Jx49/UUd/ihh4Me4AzFEk8Zb0AnLiXSI7xmp/F71rAS0o0ZGCMYCbGcMXW/KFc/q1hWT8wxdST+QASnUDYN1m+sng75CFp3/uzCYoQ11FSrXttrwgVvGKMEVwYxxtQwvHMLQFoBPmFF9yCKmj0ChvZJfTz2aO+0rNaN8+ZeF2UgLejNFbJ2YmSOdm17Ul3QOUVWlR63nhpMn39soqj3BzuyU9awi9s6kiOo3L1dVdLUZbgjGnU1KL7oMN4MtNmRoqTNsPVcEy3fpBeS9CoBZRWACd4y9XDLC+B5JhI1AA== 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=LyqMH8lkOvbkcsRQvN1bp9tJTGy5FJBf7izYagTl3j0=; b=lNUZMD1z98U7I2zvFkqXwme0nXnVpZ/+H5RvOGHzOS97ZPgQJlaCSl2vIEDq87BFeoUoKkf4I14QvxhL+09apHOOj7OGh7wvKqmMsmU2U+bpgX7U8AlAR8BDyUpxLeOWU9PoXoS99Kwfjy0t8IOTECuW4ET5HHvHWBSnWWyW5ePuF8K0FVa6JRq76MbITg5ZCio0LY3gs2X2x84Vi6qtfYYuSLszzqCqTW2usmJoPtXvPmKDL1biMLVqxWI1roLxx9PxkB2v3Q8XeHExTlG+lz9tJ/ZG+9qtN+uEIAwR2xBsgQ+5KpBgwM6vJ+R7VRBTaKVKitF7zpVmX9n1bz7UWg== 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 AS4PR04MB9484.eurprd04.prod.outlook.com (2603:10a6:20b:4ed::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5746.21; Fri, 28 Oct 2022 08:22:29 +0000 Received: from VE1PR04MB6560.eurprd04.prod.outlook.com ([fe80::4da2:ea8b:e71e:b8d8]) by VE1PR04MB6560.eurprd04.prod.outlook.com ([fe80::4da2:ea8b:e71e:b8d8%4]) with mapi id 15.20.5746.021; Fri, 28 Oct 2022 08:22:29 +0000 Message-ID: Date: Fri, 28 Oct 2022 10:22:27 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.4.0 Subject: Re: [PATCH 06/10] Support Intel RAO-INT Content-Language: en-US To: "Jiang, Haochen" Cc: "binutils@sourceware.org" , "Kong, Lingling" , "H.J. Lu" References: <20221014091248.4920-1-haochen.jiang@intel.com> <20221014091248.4920-7-haochen.jiang@intel.com> <1e6a7d9c-4b14-821e-cc46-453adbe6f183@suse.com> From: Jan Beulich In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: FR3P281CA0016.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:1d::21) To VE1PR04MB6560.eurprd04.prod.outlook.com (2603:10a6:803:122::25) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: VE1PR04MB6560:EE_|AS4PR04MB9484:EE_ X-MS-Office365-Filtering-Correlation-Id: 45e41275-e2ed-48b5-9f23-08dab8bd8d7d X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: jZ7K+gIuYt2BJknNT711PksWs2shWV9dKGZ8rvex9Y4nzuxng0Twv7Y4QIhHncbJ0lxv+4xMM+6+NSlfLm5Gucd744a9BiuWO1IITYZ3FayzwG0+kBQXPLh8RUjEfwikb2K/4B18vAIq9gDUWWHuSAdLmYsdtZ77KmfinEi3VDIT9qc1ITF8VUMpVOvf++7wVi0ZBXR+WND7Qz7MQOUf1ucOJ01Ovy1Dv+R/Ksn/V9BzxvAJ7oCEf62+moB4s7/Iog7PHpg8rnqYEEk+1wufg9Npf5wZOtl8cz+/GYys6Ffec2P3vWVudYUXGA/0Fw+F62mNY7YBAnTwcu7THzThrdVT4V/kMpDRAtuNu0kql5bfnYrRJnhh/3J3hMpi7OjGN4FHlWTwe4QhRZ3sc/BCzoprUq6nYjGz8fEcJ8E+ySDARWsfwjAylW/RUAD68cn/YOBP0jtuWsFfYVRszc4HCjX5lV58y/YCg2SU5AsJg5SkWrsJYKJOTZp3/jpegovTx0s3EXwcZ6t5WMPOA1ZW0njZDfce5CXmlJ8AbpRda1DSBisUZWr3HSvF15Tf/+I3i95L3Ewr4L3UpODhe059VX9spY6hLp3KMCHpM0PjCd0kYgsIfxqA5nf0vkESa4gy2nbkfeJtTDeB0OX8wU3oGb62CU4tQLK6fz/RVY4KAGbBeJ0NM/oEVUjkTBTEqH/mw4JJPyK5AcUAoprQfnuu0m6yryA1cMpMcuR+Ra6/MU5Sh21GuTyza8Q7yj1I7u5zbu3jVKbk1esiSRnOaUGZYj8USEQGZRd92lzqIIth3TA= 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)(136003)(376002)(396003)(39860400002)(366004)(346002)(451199015)(6486002)(478600001)(54906003)(6916009)(316002)(66556008)(4326008)(6506007)(66476007)(66946007)(31686004)(53546011)(66899015)(8676002)(41300700001)(8936002)(2616005)(6512007)(26005)(5660300002)(2906002)(186003)(36756003)(83380400001)(31696002)(86362001)(38100700002)(45980500001)(43740500002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?c1NaL3QrNlhKWnJlK2t6a0h1cnp5eDBFWVBrVERkTVcxd0dzcTRVN3laNTRW?= =?utf-8?B?b1U5eUdOUG5xcURCVWwrODdTRjNlZHhkY3pucXQ2KzIvQklaNEY0M1JCYm5O?= =?utf-8?B?QTk1cjJEai9EZzhJOTRPMnM1Qjh0VlhUOHBzN2V5SUJKNElNNnpnY0FNSEE1?= =?utf-8?B?QU5tVTJNTUI3L2QySE1CMDYrRGVTdFlFNHNsNVp0aURjNmZmRkpWTVZEWFhS?= =?utf-8?B?cEFSd2RISGZjaE1WbEtsVmROSnRGNldWZDRDRWsvQ3FReUk1V3VZY3FKSWhj?= =?utf-8?B?dDJLcnNxVENlWUlwWVJnZ3AzUGxFK3NFcHMzdEltV1F4QjBQWEs2S1pMV3ZQ?= =?utf-8?B?eEFnWi9nMHR1cWFpOEFOQ09zVnp0ZG9JQ3JzMzliTzJ5K2EzVmNSeVZKM3ZP?= =?utf-8?B?cGZ2ZWd2bExiV1lVcXAyRE53T1pidnFKaFg3Nlk1SDJVaHUvWVhCRCt4eEZ6?= =?utf-8?B?V2loMjNNbk1mT05lZDNuUGFKYndpTDJmWFpRSStYcmF1aXBVN1Fub08xMWhw?= =?utf-8?B?cDNKZXdhTjhjbDdvMnYvK2k3VXIzZjU5VUU4MzFHMFhPY1ViRWpIS0w1N1RE?= =?utf-8?B?N0NiTDJzeFlZSUpNZmRPQTRaSUFqbW4rWXZEMFVoNUtpQ2VRcWc5OFMzMkNR?= =?utf-8?B?NGd4ci9iamV4TlFYTmlXbmd1Z3VxYUZ6UEtHWWVCUTI5Z20yMTdkUTBOMk5u?= =?utf-8?B?L2xCTzE0dzFkQmZRSGNCcVgrOFozQ2FURkd5RXVPOGp1cm1WaVBlK25nYVEx?= =?utf-8?B?eGVaWGF4K0xuWXEyTVVUWDhvOW55RlQzKzB2UTBiMnB3UlFUMnVtbHpjeXo4?= =?utf-8?B?aXVOcTg1V3d4aW1QZmQzUTE3ajFIc2dTRGpKUU1EWjIzVXZnWXlRbElJMmdX?= =?utf-8?B?Z2kzZW8yUHV2ZjZjY2l1cU9sU05TMllGMXMveW9sMHJIWjFhNmZNRm83S0xp?= =?utf-8?B?eE0yTS9PZjh5ck9HazlaaU1kQU9jQXh0dXpHeWIreGNuRUV6NWJCV3ZiQjc2?= =?utf-8?B?d25mMmVhc2c2TWpkN3ptNHcrS2RkbzgvdlIrdzVaSi95eTJkWVBtak02ZlB5?= =?utf-8?B?VkljWk0wUmdJWERHa2NhcnlRT1pQMWRsNjlIYWlhQmxzdVRvdWcxR0dOb3Qx?= =?utf-8?B?My9qTXRRQlIwQkxWN0tYMzFaM1Jmd1l6TjdlYTJLQ1JxeGkrRXVycUpuay9q?= =?utf-8?B?K1QzWTZHQ3piUzB6bEt4K3dvRTJyVWptdXhITHBSV3AyT1kzdzhwRmQ4RnhN?= =?utf-8?B?ZzVZam10S1hqdDNTVTBwUHEyc0lHWEJaTmFHeXgwMmRCU1V4enpWZU82VVUx?= =?utf-8?B?Y2g5ZVpvV2ZIM0VyUDdWanAwQitBdkhWc1BHcHpXSmNEcGpVUFJVVVQ2NzlB?= =?utf-8?B?c2xJcXkyUmE5SVh5UDdnZnY2cjRLbVVLREJWbFovUlYycFZPTW9wYXNRdFhy?= =?utf-8?B?RzcvaTFsODRBNDBLMjBhVUs0Znl6bVVNRVNpd2NFdGpIZ3BYdG1LZFprMy93?= =?utf-8?B?Q2o2c0hIRUpYMGRPL0pDRXNkZGZDWi84S2pKSmMxTmZ1OUZwSmZrVHpkTUxI?= =?utf-8?B?cVdycTFhcUdSczRkOEp2aS9Zb0pISFNpTzFxMjh0WmJtby9YKzloUVdjdWV5?= =?utf-8?B?eHpYaUJaRDlpSGlhcithYWMvWEQzUU03Sm8rRDZZVVMwRFJhQ0dZTVpYRFha?= =?utf-8?B?NERzTFByYzlYNGZxa3BUSkFESGxJTFY1MVpycmVZeTd4bHEzOXEzYTF4WEIv?= =?utf-8?B?M0hkVDZRRktQYURLTTZ4QVhEK2dPb3ZzdnNGeHpyMnVFS0g4dlI0S2tOY2tR?= =?utf-8?B?R3VMZStZMGtNUlBuYXpjeUpEOUp6c2pMai96VjE0ZkZCSEk0ZFh4bjlEMHYw?= =?utf-8?B?eHlDR3lHK3U2bklKMHVWaWw2RmF3Wm9JNXNGcTZ6ejVxSmJOeEllSGVQUkJJ?= =?utf-8?B?UEhlKzlDTkhEUFA4M1pCaitsVkdYeU5jejZYSklQZkZFd1FUdENVUW44RXBj?= =?utf-8?B?ZTVpUndqTTNBQ2pFamJySkdiVzZxNE03a25vU1pPbnFGelBYMmZJd0xnbVZL?= =?utf-8?B?WWNGNDNhcjNXb3RvUG1OY1k2dWt3ZHo5MUVJdXpCcDFoVU56eVFFcDg1d2tX?= =?utf-8?Q?MYjB3Bztzq4Qot17+ApDG5exx?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: 45e41275-e2ed-48b5-9f23-08dab8bd8d7d X-MS-Exchange-CrossTenant-AuthSource: VE1PR04MB6560.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Oct 2022 08:22:28.9590 (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: BcD4940iAHkVsvG5zbUPCNJTWLLWnhUkfEw73KKuGKG/NLCW54HzwDMORNgH9UT7+Djq34SZbja48SU53VwCZA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS4PR04MB9484 X-Spam-Status: No, score=-3029.6 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,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 28.10.2022 10:10, Jiang, Haochen wrote: >> -----Original Message----- >> From: Jan Beulich >> Sent: Friday, October 28, 2022 2:53 PM >> To: H.J. Lu >> Cc: Jiang, Haochen ; binutils@sourceware.org; >> Kong, Lingling >> Subject: Re: [PATCH 06/10] Support Intel RAO-INT >> >> On 27.10.2022 20:46, H.J. Lu wrote: >>> On Wed, Oct 26, 2022 at 11:39 PM Jan Beulich wrote: >>>> >>>> On 27.10.2022 05:50, Jiang, Haochen wrote: >>>>>> -----Original Message----- >>>>>> From: Jan Beulich >>>>>> Sent: Wednesday, October 26, 2022 4:57 PM >>>>>> To: Jiang, Haochen >>>>>> Cc: hjl.tools@gmail.com; binutils@sourceware.org; Kong, Lingling >>>>>> >>>>>> Subject: Re: [PATCH 06/10] Support Intel RAO-INT >>>>>> >>>>>> On 26.10.2022 07:16, Jiang, Haochen wrote: >>>>>>>> -----Original Message----- >>>>>>>> From: Jan Beulich >>>>>>>> Sent: Tuesday, October 25, 2022 3:01 PM >>>>>>>> >>>>>>>> On 24.10.2022 07:56, Jiang, Haochen wrote: >>>>>>>>>> -----Original Message----- >>>>>>>>>> From: Jan Beulich >>>>>>>>>> Sent: Friday, October 14, 2022 10:38 PM >>>>>>>>>> >>>>>>>>>> On 14.10.2022 11:12, Haochen Jiang wrote: >>>>>>>>>>> +aadd, 0xf38fc, None, CpuRAOINT, >>>>>>>>>>> >>>>>>>> >>>>>> >> +Modrm|IgnoreSize|No_bSuf|No_wSuf|No_sSuf|No_ldSuf,{ Reg32|Reg64, >>>>>>>>>>> +Dword|Qword|Unspecified|BaseIndex} >>>>>>>>>>> +aand, 0x660f38fc, None, CpuRAOINT, >>>>>>>>>>> >>>>>>>> >>>>>> >> +Modrm|IgnoreSize|No_bSuf|No_wSuf|No_sSuf|No_ldSuf,{ Reg32|Reg64, >>>>>>>>>>> +Dword|Qword|Unspecified|BaseIndex} >>>>>>>>>>> +aor, 0xf20f38fc, None, CpuRAOINT, >>>>>>>>>>> >>>>>>>> >>>>>> >> +Modrm|IgnoreSize|No_bSuf|No_wSuf|No_sSuf|No_ldSuf,{ Reg32|Reg64, >>>>>>>>>>> +Dword|Qword|Unspecified|BaseIndex} >>>>>>>>>>> +axor, 0xf30f38fc, None, CpuRAOINT, >>>>>>>>>>> >>>>>>>> >>>>>> >> +Modrm|IgnoreSize|No_bSuf|No_wSuf|No_sSuf|No_ldSuf,{ Reg32|Reg64, >>>>>>>>>>> +Dword|Qword|Unspecified|BaseIndex} >>>>>>>>>> >>>>>>>>>> Why IgnoreSize? Instead I think you need CheckRegSize >> (assuming >>>>>>>>>> it >>>>>> does >>>>>>>>>> enough for Intel syntax memory operands - please double check; >>>>>>>>>> if not >>>>>>>> this will >>>>>>>>>> need fixing). >>>>>>>>>> >>>>>>>>> >>>>>>>>> For table, we aligned with CMPccXADD and added No_lSuf and >>>>>>>>> No_qSuf >>>>>>>> since >>>>>>>>> the suffixes are not required. >>>>>>>> >>>>>>>> Which, as said, I consider wrong both here and there. >>>>>>> >>>>>>> I still agree with H.J.. Also I suppose the previous usage of >>>>>>> suffixes are quite messy and we should not stick on the confusing >> code. >>>>>>> >>>>>>> Keeping consistent with SPEC from now on to avoid confusion on >>>>>>> that >>>>>> might >>>>>>> be a better solution for future. >>>>>> >>>>>> Same question to you then: Which spec? As said in reply to H.J., >>>>>> the only spec I'm aware of suggests prefixes ought to be used >>>>>> _whenever_ there's a sized memory operand [1]. If we want to deal >>>>>> with the present "mess", I think we ought to simply allow suffixes >>>>>> consistently. Please point out any other messy aspects beyond that. >>>>> >>>>> The default behavior for current ISA is when there is no suffix >>>>> required, it won't mention that. >>>> >>>> What is "current ISA" here? Looking over the whole set of insns which >>>> we support, it's the vast majority which behaves differently from >>>> what you say. It's been several times that I mentioned that we would >>>> better eliminate this inconsistency. >>>> >>> >>> There is no ambiguity in the RAO-INT spec which requires a suffix. >>> No suffix in A&T syntax is different from legacy instructions. But it >>> is consistent with the RAO-INT spec. >> >> The RAO-INT spec doesn't even know of AT&T syntax, so can't be taken as a >> reference when deciding what the behavior there should be. > > For instruction xadd, which is quite similar with RAO-INT, when we are using > AT&T syntax, we also do not have suffix.] Where do you take that from? The present table entry is xadd, 0xfc0, None, Cpu486, W|CheckRegSize|Modrm|No_sSuf|No_ldSuf|HLEPrefixLock, { Reg8|Reg16|Reg32|Reg64, Reg8|Reg16|Reg32|Reg64|Byte|Word|Dword|Qword|Unspecified|BaseIndex } which demonstrates that b, w, l, and q suffixes are all permitted. > I suppose if that is not required, we do not need that since we can also know > them from the register operand size. Sure, the suffix can be derived from the involved registers. Therefore on such insns it isn't required. But its use still ought to be permitted. > BTW, should the suffix instruction dependent? It might be more operand > related from my opinion. If that is the truth, could we just judge whether we > should add them when dealing with memory operands? I'm afraid I don't really understand what you're saying/asking here. In any event - whether a suffix is required indeed depends on insn operands. Yet even insns with (only) GPR operands _may_ use a suffix in AT&T mode, irrespective of it being derivable from those GPR operands. We actually apply consistency checks between registers used and the suffix (if present). Jan