Message ID | 7c362963-e6c0-f610-3bae-9465dc91a73b@suse.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show
Return-Path: <xen-devel-bounces@lists.xenproject.org> X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 72EBFC6379F for <xen-devel@archiver.kernel.org>; Wed, 22 Feb 2023 10:35:58 +0000 (UTC) Received: from list by lists.xenproject.org with outflank-mailman.499347.770444 (Exim 4.92) (envelope-from <xen-devel-bounces@lists.xenproject.org>) id 1pUmTP-00032b-VL; Wed, 22 Feb 2023 10:35:43 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version Received: by outflank-mailman (output) from mailman id 499347.770444; Wed, 22 Feb 2023 10:35:43 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from <xen-devel-bounces@lists.xenproject.org>) id 1pUmTP-00032U-Re; Wed, 22 Feb 2023 10:35:43 +0000 Received: by outflank-mailman (input) for mailman id 499347; Wed, 22 Feb 2023 10:35:43 +0000 Received: from se1-gles-flk1-in.inumbo.com ([94.247.172.50] helo=se1-gles-flk1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from <SRS0=NHsq=6S=suse.com=JBeulich@srs-se1.protection.inumbo.net>) id 1pUmTP-00032O-3l for xen-devel@lists.xenproject.org; Wed, 22 Feb 2023 10:35:43 +0000 Received: from EUR02-AM0-obe.outbound.protection.outlook.com (mail-am0eur02on20606.outbound.protection.outlook.com [2a01:111:f400:fe13::606]) by se1-gles-flk1.inumbo.com (Halon) with ESMTPS id a7179f4d-b29c-11ed-93b6-47a8fe42b414; Wed, 22 Feb 2023 11:35:40 +0100 (CET) Received: from VE1PR04MB6560.eurprd04.prod.outlook.com (2603:10a6:803:122::25) by PAXPR04MB8144.eurprd04.prod.outlook.com (2603:10a6:102:1cc::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6111.21; Wed, 22 Feb 2023 10:35:39 +0000 Received: from VE1PR04MB6560.eurprd04.prod.outlook.com ([fe80::154e:166d:ec25:531b]) by VE1PR04MB6560.eurprd04.prod.outlook.com ([fe80::154e:166d:ec25:531b%6]) with mapi id 15.20.6134.019; Wed, 22 Feb 2023 10:35:39 +0000 X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion <xen-devel.lists.xenproject.org> List-Unsubscribe: <https://lists.xenproject.org/mailman/options/xen-devel>, <mailto:xen-devel-request@lists.xenproject.org?subject=unsubscribe> List-Post: <mailto:xen-devel@lists.xenproject.org> List-Help: <mailto:xen-devel-request@lists.xenproject.org?subject=help> List-Subscribe: <https://lists.xenproject.org/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xenproject.org?subject=subscribe> Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" <xen-devel-bounces@lists.xenproject.org> X-Inumbo-ID: a7179f4d-b29c-11ed-93b6-47a8fe42b414 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=FAC0ESwZS6TtArQ6xWJxMV3XzIIT0TJUXBJbrOKGaateULlXv/1xrtL1ZpIO46qp/aMUniPXD5Fwpuha7a3Jvad3BhzeumOf2qTAR/rzap4G8UAYDk1kcwmjYcEPpr3cmpoPClavfvpIqz3fzoz8bAfdSmWUVXUHYz5GtZmER3SRKjmwtArlCLEbozNnhmTBKMELYOpBMxe+h6HfPTmPNaScVIh6h4vmrp9aPdtOe5uPuJ6x5+xZz0LACxYySNT/I/PeaUDmgdHoNrxFBW0g/H1uKw9OkmoDy3yXfBx6jHsFO810ZbuYuwMqB7tsDCL1c+UXcJ6MgU9+cvtEsHqB0Q== 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=5etHIMqI3C7arm6Ep3GLTy1uNN/G7VrBq+eM8Xi+RqM=; b=ISsYA47byBFrO/YT95QVECKqBYYnSqQV4r+vbmR3wL4KhJKrRIzkSst4Aw3vTHHkWQ1CNmEeMdQjdYbvYzlOmFPRTXqO+MvMZdqWhDBuRTbXvKbIb+MIlUQQ/vqFE4ffJuXRhzJdZ64YzSP0XchOWT7v6WckekmcRwYCPdwNGFM1M0XfH2cTRlX6YbF/1OugROvPoy9NVyEjhkVOaqL627MpfXu8Wvwrg6U7zxpQvzKfMpvKz4Ectu1x70L35CLcSs1blEdGYhkbNN87X5K1Uio6NbmvpKeknPTObdyN+72bMCj58uwL7KNsZB55SX5JokWYtvj4fkQJg4wbCOFOjQ== 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=5etHIMqI3C7arm6Ep3GLTy1uNN/G7VrBq+eM8Xi+RqM=; b=xuWtiYvzuPjKuggTnUBuf1jJDWE3pL1viKttAmmWkW8Y+iVgt4Vit3tNIEi81c0irRGvlhBtWZTpqDKNcOa3lpBgxxK0+P39s7eS1XWKDeqNLelHemQmFb60sKE04x4nGYUYX2XLmTEkBegG1DO1z3+n3zh6RjGr+hEwcfPvgTYOwWdwi+NZB9twalE3VYQLaJKNqX3lgA4FCVFhDm6mPChiYDcnaeSi7KgmMMSnDlcy8cZLg86YaGTfEw2i64r1u7eoXIC8SELmY0zmex9QhL61UcLuziVThWQiBDog5wiDkVgOa3WlDfkEoJ/0ku7KjsdB+jkk+IwOfaP/4e1ehg== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com; Message-ID: <7c362963-e6c0-f610-3bae-9465dc91a73b@suse.com> Date: Wed, 22 Feb 2023 11:35:37 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.8.0 Content-Language: en-US To: "xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org> Cc: Andrew Cooper <andrew.cooper3@citrix.com>, Wei Liu <wl@xen.org>, =?utf-8?q?Roger_Pau_Monn=C3=A9?= <roger.pau@citrix.com> From: Jan Beulich <jbeulich@suse.com> Subject: [PATCH] x86/MSI: drop "packed" attribute from message data/address structures Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: FR0P281CA0114.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:a8::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_|PAXPR04MB8144:EE_ X-MS-Office365-Filtering-Correlation-Id: 855cfe64-e62a-44fd-dd33-08db14c08a74 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: yXUAabAXhU9k2ai7dY7YxmoST9sOPbN81Z8aBZjjKwV5hntl6/jjShyM8NjDmQVV9KBm5zSrumEGD6FkKNcfx5OYjUyjdCtBEHgf8D3P4Mc6lLBNwe05IpQkPnSz8ATv8XZkb8ThPOYzHb2ulQ33tPavnEq9hC9qW0lsqmof6dn+9jVMA4O9LI6j/+datLffidMtM9iQcidYMrCh9cqPaKGAAL9Fyn7MKeiwM38Ttux24g3FQa62wINmp8hZTgCxMcqmg9tO2bAgcAHDaapWbqvHMWQ5Bi+LugklFsa8+GjAizLVijb2fCfHVlAHadKYGEsRBM91qEkqDJKImSh4ykoL0Zfue8xm8pRDNfeF/0TbwetZM7JQmdRU6nZhfXOpyLrCcia3WrdLywro8/eylTA7XOKp+pIaCyCFnyYIudlUgqGD9CevdQ9fDM8JaHW8S4b8LlfomBoAcelqgduSvjBqgq2NNr5CAbfzWNoIoE+dNCZ9WmGDgT5d+oAAq6U26KpAhhPHuRxyYGiOvEFnFkuqS60dZtRykkyQ0n0EX7ZW89JnQiKDI9vLlPVAe+DpBqojoJmhTv+wzLOAZIG7uInkAABDToN3sqx93uhxxef2+a/uu94T8mrSLv68+y+ZjWFBuU2jdZ6QLhGbDXDkLxoZ7yTjAosx56p0c3t2+nWGVZnZDQLfhGRgyvQMIgEsZpSV2d3oBk2jTj04Iho7OPG9K8N4XIJY4yTFhdHlat8= 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:(13230025)(376002)(346002)(396003)(136003)(366004)(39860400002)(451199018)(83380400001)(38100700002)(31696002)(36756003)(86362001)(2906002)(4744005)(41300700001)(5660300002)(186003)(26005)(316002)(6506007)(6512007)(2616005)(4326008)(8936002)(54906003)(66476007)(6486002)(8676002)(66556008)(478600001)(6916009)(66946007)(31686004)(43740500002)(45980500001);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?J4AQLWCZFusVfD1MlrcgnArvkpHW?= =?utf-8?q?TbC6NFLFc182KY7/IfKrw/9Ym5R4wMXqGW/7Su2acygEI5vP9gCyrHKsv+fZ4Tn1J?= =?utf-8?q?Fh1KyefRozUUUpIOfqeA2uVFqAVDBDATF5ogYl3EGcYg7NFjLwYuJ4r5Q+bZZF1Qe?= =?utf-8?q?1SCNAH9nyVuNI0R+kqlz0ucFsuEdsmt0+RU50ovjQlq/A02Hw4aHsqobZ7IWE+wJd?= =?utf-8?q?e4Hol9T7UDFHQiFrkPvlGU1PONZ60dtU2pFms+r+tT9M0eDO3jG07IovGhQYMH8NA?= =?utf-8?q?F42Jz0L84UZN7FTuaEgyXOeucEiQMPUZ16z9w/xrbLKBbmpsSI1+8T7y5uzs+8FV3?= =?utf-8?q?KBeAnDxVTmFziRceQ5O7+rWCslBic1OkwmW5ygN6qzTpTHUNV07jeX5lahA0CRM7+?= =?utf-8?q?fP1tYVmcjERGMBerIDgNWBtM2Xi2c9uUrVlZpjP3LQUYpKU5nGBCkhE2cL4NgxoY9?= =?utf-8?q?ZIZ67mCvZKFyxd0zHUZR9fVI+H5d4yEKzSBt4OTkmZlxgXOCqTVU8ws2BxusCaVkO?= =?utf-8?q?g9Wt6KeHBwcOpZnm1YQZlVOIFTv7VMtRQ4DErs7LfBvdPlv4EshEBGgPawUNiAJzN?= =?utf-8?q?zYAPWJyr2uy3I3ItJrvU8giXW3XodkHXhocLYKqbWaTv9lZVONtuZoP9XOlU5T2C1?= =?utf-8?q?pPeFPSkRnXhG5MLrhns0/cJ+svuAv4D6LEdeLTD5AVMIEAR/ISEaItxYlpC3PduiN?= =?utf-8?q?+m6IoUU8Gah8Kr9jWvGw3kTZ4wTfaPa3EkXSWnUSf1hnZD/HR+Z3nVYsdr5GtYCLo?= =?utf-8?q?jU2BgAQnmnRgydQ9zmHhKm2XnWUZuu6Os5a+ocqVQX+az9yfJv+9dKBCBvgzYbbYD?= =?utf-8?q?QrB4LPBUpsD6r+uHv9cdDRW/DWrzE+2+Fihd28f1lF+o1yhUlzhCKCkJ4LynXECPK?= =?utf-8?q?jAA0vCHTOrrzmBQ+jiEEu6H39AQABbWCJpYzYGdEPCXPDK3CEdQIAm9jHk8xsurmK?= =?utf-8?q?4c/5v/XT9kDg+MM/IkGuSpoXSH/jLyWxI0oKxHj/cQuCg6VlgLrMBODti32CbLc80?= =?utf-8?q?veM4ZAgNDWNudRHAQG8hQin3iJ5HLXVJknQW9EOBKg7idvz+4FrEvj9hqW2T9cqJf?= =?utf-8?q?4xGmBrfcZsPf3XgvLAGYYGORX+aVhgM4Pk+HlWKLiXbKlMhCSFtAPBF9CdHykDxP/?= =?utf-8?q?8Jg2k/TjXkM6+9TQWW+t+/w+dSHMeg89b+b2n7iqFVU+kvokTkxFMZ5N0Xrk4dgIy?= =?utf-8?q?60pUk29u49zj2HeFN9th7ceOW1qQBIWLhsIw6IybQn1X5TXgbXqA+4lN8gQ807GoN?= =?utf-8?q?6PdugT+O9ypG0TvJdf6g9n7MMDPvjn9TAjUybMA0H1iMbW8sO17hjdUxSoR7ZwUL/?= =?utf-8?q?evwk8vIR1rufuuTjLxWZqwBTNMDQc1Z2DUThaVuu2Rc3jbbzfm70lW32IqzJX8+WG?= =?utf-8?q?d9jw8ajj9DpVjcwwjpq/7rw+sZj5m6CKshYea62DvK1ODDks3vMXJ+t7ZnbvSnJii?= =?utf-8?q?Q0NuZQx4cQtIL8bv4EbUGtii14lIOjZJFYPHQIE4Y05sb8UvrbaVoyad8hIRDfFx1?= =?utf-8?q?f2n1DdQfZF/8?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: 855cfe64-e62a-44fd-dd33-08db14c08a74 X-MS-Exchange-CrossTenant-AuthSource: VE1PR04MB6560.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Feb 2023 10:35:39.4123 (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: j7fCLw/l6BKDS4G3gfvmScl24vArzrHnYbF7zlRyTRLoksVa9blByQYWfT6yTWh6FmLOcLcRlTb/Iu9Sv84yuw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB8144 |
Series |
x86/MSI: drop "packed" attribute from message data/address structures
|
expand
|
On 22/02/2023 10:35 am, Jan Beulich wrote: > The attribute has no purpose here and, in the worst case, could lead to > the compiler generating worse code. In practice, however: No change to > generated code (surprisingly not even to generated debug info), at least > with gcc12 and the .config-s I've tried. > > Requested-by: Andrew Cooper <andrew.cooper3@citrix.com> > Signed-off-by: Jan Beulich <jbeulich@suse.com> Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
--- a/xen/arch/x86/include/asm/msi.h +++ b/xen/arch/x86/include/asm/msi.h @@ -178,7 +178,7 @@ int msi_free_irq(struct msi_desc *entry) * MSI Defined Data Structures */ -struct __packed msg_data { +struct msg_data { uint32_t vector : 8; uint32_t delivery_mode : 3; /* 000b: FIXED | 001b: lowest prior */ uint32_t : 3; @@ -187,7 +187,7 @@ struct __packed msg_data { uint32_t : 16; }; -struct __packed msg_address { +struct msg_address { union { struct { uint32_t : 2;
The attribute has no purpose here and, in the worst case, could lead to the compiler generating worse code. In practice, however: No change to generated code (surprisingly not even to generated debug info), at least with gcc12 and the .config-s I've tried. Requested-by: Andrew Cooper <andrew.cooper3@citrix.com> Signed-off-by: Jan Beulich <jbeulich@suse.com>