From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-eopbgr80085.outbound.protection.outlook.com [40.107.8.85]) by sourceware.org (Postfix) with ESMTPS id 3701C3857341 for ; Thu, 27 Oct 2022 06:39:48 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 3701C3857341 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=lnRMgFF1G/O6pn29NcKWIzvv62ZF0K/oVjHTsAVuq7/JrZe8vOOYzyp0WBWF0PSFu8O0RLBwDDdPRt72rDcu0i0FfqOfVoYzjrWG3gYNOOerAk5onRENcQj+s6zqA669dkZ23x7l1vYQdwRULlBXmRfCkIO1HZrF3rrYq9jppLKxnCbkgDuk/lF4kQP1iVbcJKiYyIfX+Ba3kIlUStPCOu6d/+4g+AHsrtJKl8L0Rny/kYgwfeepYgXPzd3qbr1nC5YR3a1F57N7Per+AKO9HubEpViZiwqUW8x8aZNQEBpVkR/IAfstP3YtCrpmC8e7TqPZMtaMkECSlcmYZ/PWlg== 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=sUJAwqTnuNXDDMOpYFgPVCeyh/9PadSwS9C62WDT9bo=; b=W9PXHwRwXpw2sRc1em2jNyQMI/2Lox8qezoY2sa9/S4lrq/KuKUMkKAFnPtFlUrGZ0oQjikY/zK12nUW4sDUWhnjQPZcEeCtB9aUecPtc40m8pZvivk5AZ9MAAdP/E0ZKqN8sBdRVMtm9s880yNQMmnz+ux2HcgiYD9oj/A9g0a/es/rlisKFVoeJNNl322PAQnADjMwbFw1p77dWSU0kt1x6muHmRkvPWzzbwIq0jPUDCBt4JnByXAemEUNljVfNj1N7V1Ix90JMdXWh7O4nXPdq7u103rnAuNYh8FLfTXuhneQ4I3z/dRmzsfn1yjrV4LGVf9sodTqZ82RQztHQg== 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=sUJAwqTnuNXDDMOpYFgPVCeyh/9PadSwS9C62WDT9bo=; b=D2lN9XjFYGU9K9fF4tykMTmy7Q+fTU2N8fGLDQ+O1B+6fpl6PmZu98LCfac0fL5jAKd2KxQbFMRBt1HOaV4VzCwIwZWLEAV7uDHQkBpJTZFP2px9gKclhYxfwoSLX5jW5iU+DiqZTOPGNXSpM95YE/gD0czPZufyz3vWWS72M215E5gnNSgzt2PV0Wh3GRNxEOQU/xtXf0nNNLn/TNRmU+WZq1bQsiYp3yPLGl5SZZeznVNGkr9rvTuQpwDQ9DwVlM0aOmUeedR4WgV2iOw/kpgBg/FQCjZj7NqlJXcGDoq722pi9YYz8yiDt69/dhoIfewZDOpA1ZOuJRlUJLjwFw== 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 DU2PR04MB8821.eurprd04.prod.outlook.com (2603:10a6:10:2e0::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5746.21; Thu, 27 Oct 2022 06:39:45 +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; Thu, 27 Oct 2022 06:39:45 +0000 Message-ID: Date: Thu, 27 Oct 2022 08:39:43 +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: "hjl.tools@gmail.com" , "binutils@sourceware.org" , "Kong, Lingling" 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: FR3P281CA0094.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:a1::9) To VE1PR04MB6560.eurprd04.prod.outlook.com (2603:10a6:803:122::25) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: VE1PR04MB6560:EE_|DU2PR04MB8821:EE_ X-MS-Office365-Filtering-Correlation-Id: e9159ec8-b68b-442e-38f2-08dab7e60957 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: lDRGcbN1RLjEa89g6S3TZd1dSW5KIMhMIsmGPQP1XLsaTB1JpiIFn2A7aPNniDB44nE91Dau8mqYOyBPpxJoEPZ0ZHb+Yfa7bwX1Tydp5nKqQWSFASHLcOZZ395nLyKqSA02Azy6kYlIhOb/uKK+hNE9GLTdpdj12uBDowxDOfCRB3cRJYh0t4M4pQ3iJC4QIIaK5eb6ZkDnnVmoTkb9pB+M9uA8yUNLxsETytboiCFe06NDzAt5vOgZ/sT0J2LrAeJ5TRSnOeYjONTHQarLw8Fy8SZf+N51eYZoCkk//CnnbnjUrSpX5FMIsJ8YywAfkva1PodKK2IZ9Fz1HUtoa5hkUulDWbHge0i3AwVaU4hqH7Y+Sl3V9lP1z9vxywmMaLlBVacFxXxaiX2Cs/QE6p/GcBWgOQ2DM0PvXl5VddY52/tpAL6mNnkRkKSz4rqZzfwKw798Ij02uwVmw2xGyvMIUs0IV/prtOF7OSFxeq+LnRDU3vLhcMqgz49miet7Nt1yffU0VW1n4VOgJ93GIy+SYwTDYnmiPoWo+oFVJ47PUgiGV0oZJ0nD7RbS8mZMbMuzsSr9VCNEzECWVg3CrfYG9vPOzpOD877/jiQ870w2YGJld0ERY2/LqQIlqFgFIqa0k7qaYj1/0Vk6U1Hc/VIovLw3zkk0cOZswDWvizoxcWd9/eScwyHjPCTjyvogdtDUH2iS36T/5YTyaudpcD7V+x9szCVb6RQ9cwLUnydDIxnS5XoAjHPbum8qhoemC8sYkwHK5sO8eTHD78zTYgE4Fen2UbXJkTgogoraKUo= 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)(396003)(376002)(366004)(346002)(39860400002)(451199015)(86362001)(66899015)(31696002)(36756003)(31686004)(38100700002)(186003)(2906002)(53546011)(2616005)(26005)(478600001)(6512007)(6506007)(83380400001)(6916009)(8676002)(54906003)(6486002)(8936002)(66556008)(66476007)(316002)(4326008)(5660300002)(66946007)(41300700001)(43740500002)(45980500001);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?UEVmcXNSVWpYci9QaXRVUk4zMVJqZ2p5cDhORVdpSEgvNDNMN0tsMnlWRXMz?= =?utf-8?B?YTFLdktieFZFMExVcWJ2TGEzUzZ2M0VoR2N4ZUdQWUJ4ZDM5ejRFUko3MlVD?= =?utf-8?B?RUh2TEVHd2FFYkV2YlFZRjJjUFA2Z0JlVHVZVnQ4VnVKUHF4MzB4ZUlzTmRJ?= =?utf-8?B?UUdrUHFaM1dJK0N1QSs1WkhlVDVNMTdDTHR6c1Rzc2JqYlhYUHBvK1NxR01W?= =?utf-8?B?NlhXM2hWK29ZNUNqVXd4eUxiVG1NWTN0bXQxcGk3VlNKY2pXTCtieU1FWnNJ?= =?utf-8?B?WXlxZ0ZFdkIzUTZRVlFVMlFZR1RMNlhaU3lUdWdSb1VlaEY0TVRCL3dRSHk5?= =?utf-8?B?RkJqczkzcjl0blpVYmx0MzdxMHpFb0RiTVZYbDBqWkc3U3JlZWxWakRoUjlp?= =?utf-8?B?R0FmNnVuRDBOUi9DOU9wMW1BTlBaaDJickE4N2hhYm5vWjhRQ1FCM09mYW9i?= =?utf-8?B?WFFDdjBHZ3BsYUQxZnBhWmxPeURSNTkzaWN2cFdNcDZvcDRvZjJBVlJBR3Ux?= =?utf-8?B?MVdhT0FjTjAzSWRRRXdJTUVmWjdyYlFWUHkxampOSWh6RFRxT2cyQmdCdGVH?= =?utf-8?B?RU9JVlQwWXRpalFkcy9USjlNemQ2L1UxQUtTazNIS2dnNFlDYkI4eWNzaURK?= =?utf-8?B?TE9oQmV2YnA0NlBKY0JNdGVMMVZKZnpwLzV2SXRCTlNDeGxXc1FxYmtrMXhG?= =?utf-8?B?MVVSb1hSbzZERzdFcVByeStkR3NNYmtrN20zWGtRZ2trT2RGeWhjNjBJSWVj?= =?utf-8?B?SmVweUgraUZraWl3TXl1ZTBJeGdFL1c5MDdKa1ViQ0JYMnpZTHpuTkFUTFZX?= =?utf-8?B?SGxiV2JWVWh3SXZ0S3RUa3k5T29zT3lma0lNa0J0cXBYcHpRWEhyYktqMzl0?= =?utf-8?B?cGdXRDdWN205ZzBBb01weUllZUVOYVRHcnhldUJENjB2dWdyRnRHajBudGVu?= =?utf-8?B?Yk5QOHV3T3ppM3NRUlJ1L1Y5bVhOK2Mwa2ZJTnRlcDR1TXhqTjJIWVF5ekZF?= =?utf-8?B?TnRwTWdPcDFHdFNZL0RSMWVvWW5XRlgxcW42VDJOZUN6YXFJMzRlNUpoNkQ3?= =?utf-8?B?emlNU1c1M1hNVlpadzR2YW1YNko3SngxSnBOclJSOWRqMVdBNU9Tc2ZLOE11?= =?utf-8?B?RzBGSnlkam9HYWt4QWtmUVQwYmZBOExXZTFTNEdHMFNMa3k0eEEzVmFRT0JI?= =?utf-8?B?MCtEN0FZZ0JMVGQrTWtkQVZxcHczSTVmRDB6S1VEK05FZ2xQeXF2WThzZnBp?= =?utf-8?B?aDYwRmJVbGhsUzhBSnlqTUhWRGN4b0pERThrSzQyRnJHbC8wLy80VnlPY2tK?= =?utf-8?B?T3RKNWtEaXl3aElpMzlaNEpMSzJ5ZzdUZmxzZmJIMTBjTHFnRU5YYm8rRVMx?= =?utf-8?B?ZHEyMlRHZFNJd1ZoSXk5azFobVc4SVNURXNuYXg4VnNNRFAwZXczekxwSXFK?= =?utf-8?B?b0hxOXRkcmFsa21JcUxWWXBuVXFKR0tQdWVtM25UbWNWdm9yZUZTajFkMzZt?= =?utf-8?B?Vm1GY05tc0RGcUNPVUVMazd4L2FSdk1UVlZRckk2cFMzQWpyU0p0WFdmbkJo?= =?utf-8?B?aFdIQml0MVZLc1ZmbjlWTFpHYms2alFTUFhQVHBWWnc2bDkvTXNiUUR3YkpK?= =?utf-8?B?eXUzd0cyUHp3bTltM2cwZklxSEJ3T0xBV3psRExjc2pYb3NER1MrQ3k3T3Y5?= =?utf-8?B?RDY4b1NJbmRJelRIbU9FRU90YlRjVHp6VXU1d1M0Zktlc3ZVQWUrMzlRN1Nv?= =?utf-8?B?MS9UelpLek1ZeCs5djNpdzh3dlF4Nm01QkpLN2pqQjU0VmhzeXoranJFOU1N?= =?utf-8?B?ZGFnMHpvRkpwSE5xN2lkdlFXUUQxTkYwOVZQSzNib0hQQTRPRStaQU1td21L?= =?utf-8?B?WjVFN2kreU0rRVFMYUJ5OGgvOHRLZmZrK2FQRHZDUmdkaGpOeG00aWUrL0Z5?= =?utf-8?B?MEZjbGJadmQyenppUE5VSmVPeG9iSEJTd2E4ZEQvVFA1RTYzb0UvZGNOeTEz?= =?utf-8?B?cVpHNGhJT1hrU2NXNnpqbHZFaE5RdnpDWG0xM0FwdTVNZU5LZW9vVTVCRERS?= =?utf-8?B?SE43bGhxSFhiQzgwb2x4T2ljQkpwT3dUOHlUQ2VJZmJCU2kzVytCYkVwaFlw?= =?utf-8?Q?gSv3CQHcN38oqAFRyhzv9j8FS?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: e9159ec8-b68b-442e-38f2-08dab7e60957 X-MS-Exchange-CrossTenant-AuthSource: VE1PR04MB6560.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Oct 2022 06:39:45.4668 (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: OBVMVxEf6SBx5qIJrN/wOgdLsI037reGrpIMwdCvb2ZNbydxlUQAfLvgxg7C5RIAmFtm5TgA5IEZOMJacjHhjw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU2PR04MB8821 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 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. Jan