From nobody Thu May 9 21:43:05 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=amd.com); dmarc=pass(p=quarantine dis=none) header.from=amd.com ARC-Seal: i=2; a=rsa-sha256; t=1693818915; cv=pass; d=zohomail.com; s=zohoarc; b=FhJsH7jIlh1ULAkMpEr2B4S9h190ClFtAVo2jiECk9z5fThKzarK3SWKVcVZq+5m9rABx8vuj6Fy7g57w/fuASDlREa86S41pG/19d3J7zjNhQ6UTq7F1UzDezdcjmaVZgWmeXNzZfioWNwhyp6NPO8mWn5MiydqiPEvXERXv6M= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1693818915; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=gIq+cqKXin8yRNnzT94HulgKn5L0z4PTEoU760E1PXI=; b=EHtzB69q/r4PqSRODRH2VFFOefjqy0hejt3dWGSQNZfC5y+qebjqKPPHP1JNu5IxxwlC9y7KHYv90uTEFovp4/8WV7yj5nQzOk6dWq7TE+n8TvKllUJ2HWGmzsCuIF1ZnA9fIVWLmi8PI9a7jrvcwLOHqRsjn2U1cQqDQLLzI2Y= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=amd.com); dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1693818915523761.323461169616; Mon, 4 Sep 2023 02:15:15 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.595024.928499 (Exim 4.92) (envelope-from ) id 1qd5fH-0004d1-Ot; Mon, 04 Sep 2023 09:14:35 +0000 Received: by outflank-mailman (output) from mailman id 595024.928499; Mon, 04 Sep 2023 09:14:35 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1qd5fH-0004cQ-If; Mon, 04 Sep 2023 09:14:35 +0000 Received: by outflank-mailman (input) for mailman id 595024; Mon, 04 Sep 2023 09:14:34 +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 ) id 1qd5fG-0004Su-El for xen-devel@lists.xenproject.org; Mon, 04 Sep 2023 09:14:34 +0000 Received: from NAM11-CO1-obe.outbound.protection.outlook.com (mail-co1nam11on20631.outbound.protection.outlook.com [2a01:111:f400:7eab::631]) by se1-gles-flk1.inumbo.com (Halon) with ESMTPS id 746e8734-4b03-11ee-9b0d-b553b5be7939; Mon, 04 Sep 2023 11:14:32 +0200 (CEST) Received: from SA0PR13CA0022.namprd13.prod.outlook.com (2603:10b6:806:130::27) by IA1PR12MB8335.namprd12.prod.outlook.com (2603:10b6:208:3fa::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6745.30; Mon, 4 Sep 2023 09:14:27 +0000 Received: from SN1PEPF00026367.namprd02.prod.outlook.com (2603:10b6:806:130:cafe::38) by SA0PR13CA0022.outlook.office365.com (2603:10b6:806:130::27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6768.23 via Frontend Transport; Mon, 4 Sep 2023 09:14:27 +0000 Received: from SATLEXMB04.amd.com (165.204.84.17) by SN1PEPF00026367.mail.protection.outlook.com (10.167.241.132) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.6768.25 via Frontend Transport; Mon, 4 Sep 2023 09:14:26 +0000 Received: from SATLEXMB06.amd.com (10.181.40.147) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.27; Mon, 4 Sep 2023 04:14:24 -0500 Received: from SATLEXMB03.amd.com (10.181.40.144) by SATLEXMB06.amd.com (10.181.40.147) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.27; Mon, 4 Sep 2023 04:14:23 -0500 Received: from XIR-MICHALO-L1.xilinx.com (10.180.168.240) by SATLEXMB03.amd.com (10.181.40.144) with Microsoft SMTP Server id 15.1.2507.27 via Frontend Transport; Mon, 4 Sep 2023 04:14:22 -0500 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 746e8734-4b03-11ee-9b0d-b553b5be7939 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=oPYrM7nLWJ17divS8FFP1TjkppFVMIcdIofyM3fqfd5XKiSyYbQJETNRMIqpeZSp4s5Jkfd0slYFyP+tm5MvEtDHpXI0uDbUqgu0hf3l2sg+y1FqX6noby01NmY3YXJ2gjqTiuNbYJ4ajUA7bOKg57wOEpX78PrpzrbXW9DODN4EKNIpq2vANOl6C+fAEF9829kE38TcBOPx7KvFKt6oj6XiLuFzHiIF9caFJlX1ZEfRbFGHvxQlUUCZEUAZ91MuoFt2EvwHSQEEKP1ARdwnUSn8Pyy+igYby+y+haDFg+3MmGKVGrAm/gAfVOtLHyHzkjgYF0v/uFlM6ikLSH1L4w== 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=gIq+cqKXin8yRNnzT94HulgKn5L0z4PTEoU760E1PXI=; b=n8SlCcoQgZr9ZiS72VofVj9FkYXFdIbPdAIANWH18P7pS9rE+w9qAPIlvHpjrCM4rirytiPpHwAQYd0REj3SmxL4PZqzy3yh06xlOHdqcV62Xo+tt2ugyFKUjFeLjR78QNmpIZ7/q6BPt2bUoasiqbDFKZ2Jv6ZXF6021GPnBmQ9cM39iwAy+9pA0Z7IhNKiAAqOz8WiPEh9cX6TG+HMemeFl/mPLfc/7bveuo5qZWMM5Lbs7HEf3kv/1OyMBc5NeC/t6hY+8KQeHkPy0VumtWDNE/qjagBoO4xT02CgSemDSmV+Ti6H5mI7EDOpH35DQ4+CLWDMUJy/lW5bjmpp0Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=gIq+cqKXin8yRNnzT94HulgKn5L0z4PTEoU760E1PXI=; b=aPu+uuesYIyDgj1ElC0sgV87YgqQyq5WRLWX6RikQTJCBzzoLB0JsYQduXXek7qs1xGKD4OP0JfLz0exYzmg62xF3tBRjErOXN94IRhZpm6J2ZooLCZrS+/cNO/wi6vHZxPFFVT7sJHgUqf75TF5nccVcRu825zjfBwQynDxiQY= X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C From: Michal Orzel To: CC: Michal Orzel , Stefano Stabellini , Julien Grall , Bertrand Marquis , Volodymyr Babchuk Subject: [PATCH 1/2] xen/arm: smmuv3: Add missing U for shifted constant Date: Mon, 4 Sep 2023 11:14:05 +0200 Message-ID: <20230904091406.942-2-michal.orzel@amd.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230904091406.942-1-michal.orzel@amd.com> References: <20230904091406.942-1-michal.orzel@amd.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SN1PEPF00026367:EE_|IA1PR12MB8335:EE_ X-MS-Office365-Filtering-Correlation-Id: b878e39b-ed36-4ca3-e3f2-08dbad275642 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: rwpxUpUjg0Cmyw3WMIeBnyv+a1FpqTN5znxdFk8H9gbkIyt92s04fMKvl/0EZz2Th0HECQzhcSkYZJ6EYmIhTVbRHpxqICKD0qBNdiOxxEuISGScZQwkBtxbLPxP/cHitibZqI5bAGoXBJsN9weU0hVDH/CwjHeMmN4Xdu1Gs3HVUmLfX+odB/UwJRJWvdoT67/87kl6zz1F8N24tSWdlUvu8Fv/5Ct4gLG2RbIxG7QySdTilMqD4/GtmZAx4RotrAhN80CFfbvdBA/Io1Y2FRKcI2SxeOpCdr9JpYjM2jCRhnD3WdCfLCnrAQY0OL2JBzeNDO8gxcA3pSBvUZcdUnU7nTRNEQOl4a/MpXUPG4LhlmXd3NZna3rVy/T2jM2MSQ068nUlObUajoTfe87jWPm1gGisBSp9P+AndvNcFSndsHqIwGAuGtSlplrRRbOVGuf5XzldiDti6rmwPtXMwKYqy1r3VuP9qXnrZIbiTmturAmgf8oyEGb3e1TSb7u8JHaGeHtdL1soDOqcGnToJ5ZNChxHc1CMj2YMVRXkfFafawYgnrfIMUpcYyMIqUjZVm/7mWgYp8Enuy2LI6yGx9Ls73MjgJ/vHleLF5gW94s/UUpWwK8o0vyBcTVD1pnBZ8sCK3h+3/J4psucRY1Su7/elDl3VB2D5NDWmA2bEKlhdzLWtEbt+6j+E61N0kDrthODhNRzgDA470077MyB0tpC7GtmSca7EfD9FLaN8zzQqXhD2c1L3Sd4P0y2mSX4DstD7qjyDVlylFVlR/6dLg== X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB04.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230031)(4636009)(39860400002)(136003)(346002)(376002)(396003)(1800799009)(82310400011)(451199024)(186009)(36840700001)(46966006)(40470700004)(8676002)(8936002)(5660300002)(316002)(6916009)(2906002)(36756003)(54906003)(70206006)(70586007)(4326008)(44832011)(41300700001)(40460700003)(26005)(1076003)(36860700001)(40480700001)(336012)(47076005)(82740400003)(6666004)(81166007)(356005)(426003)(478600001)(2616005)(83380400001)(86362001)(36900700001);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Sep 2023 09:14:26.5142 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: b878e39b-ed36-4ca3-e3f2-08dbad275642 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SN1PEPF00026367.namprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR12MB8335 X-ZohoMail-DKIM: pass (identity @amd.com) X-ZM-MESSAGEID: 1693818917016100003 Content-Type: text/plain; charset="utf-8" When running with SMMUv3 and UBSAN enabled, the following is printed: (XEN) UBSAN: Undefined behaviour in drivers/passthrough/arm/smmu-v3.c:297:12 (XEN) left shift of 1 by 31 places cannot be represented in type 'int' This refers to shift in Q_OVERFLOW_FLAG that is missing 'U' suffix. While there, also fix the same in GBPA_UPDATE. This should address MISRA Rule 7.2: A "u" or "U" suffix shall be applied to all integer constants that are represented in an unsigned type Signed-off-by: Michal Orzel Reviewed-by: Bertrand Marquis --- xen/drivers/passthrough/arm/smmu-v3.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/xen/drivers/passthrough/arm/smmu-v3.h b/xen/drivers/passthroug= h/arm/smmu-v3.h index b381ad373845..05f6b1fb7e33 100644 --- a/xen/drivers/passthrough/arm/smmu-v3.h +++ b/xen/drivers/passthrough/arm/smmu-v3.h @@ -87,7 +87,7 @@ #define CR2_E2H (1 << 0) =20 #define ARM_SMMU_GBPA 0x44 -#define GBPA_UPDATE (1 << 31) +#define GBPA_UPDATE (1U << 31) #define GBPA_ABORT (1 << 20) =20 #define ARM_SMMU_IRQ_CTRL 0x50 @@ -159,7 +159,7 @@ =20 #define Q_IDX(llq, p) ((p) & ((1 << (llq)->max_n_shift) - 1)) #define Q_WRP(llq, p) ((p) & (1 << (llq)->max_n_shift)) -#define Q_OVERFLOW_FLAG (1 << 31) +#define Q_OVERFLOW_FLAG (1U << 31) #define Q_OVF(p) ((p) & Q_OVERFLOW_FLAG) #define Q_ENT(q, p) ((q)->base + \ Q_IDX(&((q)->llq), p) * \ --=20 2.25.1 From nobody Thu May 9 21:43:05 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=amd.com); dmarc=pass(p=quarantine dis=none) header.from=amd.com ARC-Seal: i=2; a=rsa-sha256; t=1693818913; cv=pass; d=zohomail.com; s=zohoarc; b=T5lnZqNfGEJFrgWEo+7YHQ23gBBTMBoaWz7XBv7BZEnh0njsnF+pqyZ/dhGcSbqbiZKfL71GsPn3GZzYlncNEa339DA4aVR5RuFtZWikmYhAQZNdCapKXVvoLZIukLxsbZqN90B2+sktlz4hWGTbHTaOblshg/UlxQFbXTZ0UUs= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1693818913; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=QBnCGHonNCHGh97PQEUyjKjN1aii3LsbaDXQQKkQfu8=; b=ksaGaRnDQiXJs0Ojb4BaM4WtzUR0xPs0Z03sjdboqmmOKbCaRrToHyn9iOB2n0dALlh5EL8QjwwG6wP3fEhX9Z6L25I9DTrvHEI3bn6fgGubG7ckDP/Xrx4or6xil8ovvKnnBtUdC5NrcwumUhWaOV30ZETZ7NkOi+o0K9gBeYw= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1 dmarc=pass fromdomain=amd.com); dmarc=pass header.from= (p=quarantine dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1693818913383367.3363906842435; Mon, 4 Sep 2023 02:15:13 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.595022.928488 (Exim 4.92) (envelope-from ) id 1qd5fH-0004T7-6O; Mon, 04 Sep 2023 09:14:35 +0000 Received: by outflank-mailman (output) from mailman id 595022.928488; Mon, 04 Sep 2023 09:14:35 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1qd5fH-0004T0-1q; Mon, 04 Sep 2023 09:14:35 +0000 Received: by outflank-mailman (input) for mailman id 595022; Mon, 04 Sep 2023 09:14:33 +0000 Received: from se1-gles-sth1-in.inumbo.com ([159.253.27.254] helo=se1-gles-sth1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1qd5fE-0004Sj-UZ for xen-devel@lists.xenproject.org; Mon, 04 Sep 2023 09:14:33 +0000 Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2061a.outbound.protection.outlook.com [2a01:111:f400:7eaa::61a]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id 73793a5c-4b03-11ee-8783-cb3800f73035; Mon, 04 Sep 2023 11:14:30 +0200 (CEST) Received: from DS7PR05CA0090.namprd05.prod.outlook.com (2603:10b6:8:56::13) by DS0PR12MB6605.namprd12.prod.outlook.com (2603:10b6:8:d3::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6745.33; Mon, 4 Sep 2023 09:14:26 +0000 Received: from DS1PEPF00017092.namprd03.prod.outlook.com (2603:10b6:8:56:cafe::ac) by DS7PR05CA0090.outlook.office365.com (2603:10b6:8:56::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6768.25 via Frontend Transport; Mon, 4 Sep 2023 09:14:26 +0000 Received: from SATLEXMB03.amd.com (165.204.84.17) by DS1PEPF00017092.mail.protection.outlook.com (10.167.17.135) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.6768.25 via Frontend Transport; Mon, 4 Sep 2023 09:14:26 +0000 Received: from SATLEXMB06.amd.com (10.181.40.147) by SATLEXMB03.amd.com (10.181.40.144) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.27; Mon, 4 Sep 2023 04:14:25 -0500 Received: from SATLEXMB03.amd.com (10.181.40.144) by SATLEXMB06.amd.com (10.181.40.147) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.27; Mon, 4 Sep 2023 04:14:25 -0500 Received: from XIR-MICHALO-L1.xilinx.com (10.180.168.240) by SATLEXMB03.amd.com (10.181.40.144) with Microsoft SMTP Server id 15.1.2507.27 via Frontend Transport; Mon, 4 Sep 2023 04:14:24 -0500 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 73793a5c-4b03-11ee-8783-cb3800f73035 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=R/Bw3H1chNc6T9PUaqygjcSB9ZqRy0j1Jlby3vKAmjsx4CPIfq5BVAFsNFXmI4K8uGKgRGGV2IsQkm5x2M7GnyLHwfgwbdGNyaAp5iaMOT7RGMKIp4ITpHmVgQ+oYfAkyj+SKwo7mF22mBsVuytruKz3wBsGl8UBvXmHE2N5Yz0djwt+/XxOxZo28Alr3hqtZjimFaTtRJ3Amktpq6r6zh51sytqzT47TiQtE9YUymqKez6Rw2vF2cLBGtlBSTjA954djzu8BYGvGsnADZ7AAH8qwwpFeM8oo/D9rDH0JG0DnNx6YDgGqV8wnw4rnSiaSrL0RWS0TgBxqyE++Rlnwg== 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=QBnCGHonNCHGh97PQEUyjKjN1aii3LsbaDXQQKkQfu8=; b=HZCpeZHRL2t81CSzWe94wdYQOJvMgeS+zQtT1tO48ak7eTRrxqbQB+95hV4hd1fuHLkyj1UJwb52/otv9HQyau4OduhbntNMWMLum/oaCHBDqLgCUvpafT3DTfE1lM/iCwu7rQYo5YB6kD9V09OObTivotwhRdk1+dPQzRvPKFriq3yeFS1Btkr1J2QfZpRFm+08P5E5Sv9xZ6FTkaWCsC3Cz+97D9pJdFhg4m8qggeNe6uS3yTNDJcsv/bEleAZszN8wN9V+W8QJeKkq7yTASKcoj6YC/HgWnKgrlkeGZiwVO2NtlmZBzdvoCVHS/vcQsR4q9hpTmgpYfVSzpZoQQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=QBnCGHonNCHGh97PQEUyjKjN1aii3LsbaDXQQKkQfu8=; b=2KdPJEPFWVjZah/WfR27wcvL6EMihzGGmvX9RUx6f3gZGrzyJTz/ZyUlc1fwQTmDfJxPhlH3a3XonxfhEU4aHoQQAWAdEbAI8XLc/bdhGKzFIQj59w6bbSFAz86mOGFmlJMfR/v939CQFSlBZGgSq0lSKxesu6SMqqHdLbfHOKc= X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB03.amd.com; pr=C From: Michal Orzel To: CC: Michal Orzel , Andrew Cooper , George Dunlap , "Jan Beulich" , Julien Grall , "Stefano Stabellini" , Wei Liu Subject: [PATCH 2/2] xen: Change parameter of generic_fls() to unsigned int Date: Mon, 4 Sep 2023 11:14:06 +0200 Message-ID: <20230904091406.942-3-michal.orzel@amd.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230904091406.942-1-michal.orzel@amd.com> References: <20230904091406.942-1-michal.orzel@amd.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS1PEPF00017092:EE_|DS0PR12MB6605:EE_ X-MS-Office365-Filtering-Correlation-Id: c6ebf2f0-e8ea-41f6-3821-08dbad27561f X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: vComPv9FMtEHnR1t/DJKxbYtkM2LiKz5KdXRy0NUn7lrcpvq2SpyfrUYSAqnqdPg8FMbkvEh9tF/E/PRZfG402Q44xvxdcpWRTi7XZ6mP08Yv5dTLt0T5d23EuqGGMyTj5DpDnByVoNs4qK7Je4OYC7AuKfHF76E7j24L353+6o2M58Bfrbfwn1h6yQgTQzGrpNlx7tqrpO8Bh3O/aRDM+NrCx0NQU4qy++Kc7ZwGILZ+Ujt7tTCl3XyIM1BDsNmA+p/wJrfR4cZ+J3NfFeBu5Ye5vLBSJM+LNx045jt95VkGZSuocRmE3iMlOIwlRzQRDU7uvpskvBHszaWdzH8SQtdgrF/IBmaIe3avsOFV/PgR0YTZWcV3EFnfQ6o6FJ9JuJJb5wtskxp3lLzw4hhSNtPd5Oqs56PuJgR9v89i/F+3m9lpPELqU4KAvyyrIstlb3Q/9MTb8drD74OLPFJ/WjDxoU9DSDI4A2G2zXKFNrbVPiSyUh08f3OcHd53JwuSRT2d/d00puELyXmD6Hn/eWhDWwGuPuWq8yKKZs8yAErSJNn3eo4Tbkf19E/ST1yg36vQzxBJSQET+Qvga7KeybX9FoYy1YnmYe0PSba1XVy/+sGUEJivkaXyyQSgJPkbi0Akd2lOUFafS3Vvfvdck3ULOkzYQ7+GCFXL6qYem414X1hKQqFLo9/F+Q0Z4bij/Fvo9dqQb0K6YoiWroV0wccuDgfWro+E2WX3UvPSTrK/5ypdCYnSIZPXRAJlI8TkfLgwpukk1PQ/UGj80kDxYJzoeuzVzufINUEVXljtQI= X-Forefront-Antispam-Report: CIP:165.204.84.17;CTRY:US;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:SATLEXMB03.amd.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230031)(4636009)(376002)(346002)(396003)(136003)(39860400002)(1800799009)(186009)(82310400011)(451199024)(46966006)(40470700004)(36840700001)(40460700003)(41300700001)(6666004)(81166007)(356005)(82740400003)(86362001)(478600001)(2616005)(83380400001)(426003)(336012)(26005)(1076003)(47076005)(36860700001)(40480700001)(70586007)(70206006)(36756003)(2906002)(54906003)(6916009)(316002)(8676002)(8936002)(5660300002)(44832011)(4326008)(41533002)(36900700001);DIR:OUT;SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Sep 2023 09:14:26.2492 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: c6ebf2f0-e8ea-41f6-3821-08dbad27561f X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.17];Helo=[SATLEXMB03.amd.com] X-MS-Exchange-CrossTenant-AuthSource: DS1PEPF00017092.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR12MB6605 X-ZohoMail-DKIM: pass (identity @amd.com) X-ZM-MESSAGEID: 1693818915507100001 Content-Type: text/plain; charset="utf-8" When running with SMMUv3 and UBSAN enabled on arm64, there are a lot of warnings printed related to shifting into sign bit in generic_fls() as it takes parameter of type int. Example: (XEN) UBSAN: Undefined behaviour in ./include/xen/bitops.h:69:11 (XEN) left shift of 134217728 by 4 places cannot be represented in type 'in= t' It does not make a lot of sense to ask for the last set bit of a negative value. We don't have a direct user of this helper and all the wrappers pass value of type unsigned {int,long}. Linux did the same as part of commit: 3fc2579e6f16 ("fls: change parameter to unsigned int") Signed-off-by: Michal Orzel Reviewed-by: Henry Wang --- It looks like generic_fls() is only used by Arm and invoked only if the arguement passed is a compile time constant. This is true for SMMUv3 which makes use of ffs64() in FIELD_{PREP,GET} macros. --- xen/include/xen/bitops.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/include/xen/bitops.h b/xen/include/xen/bitops.h index 654f525fb437..b2d7bbd66687 100644 --- a/xen/include/xen/bitops.h +++ b/xen/include/xen/bitops.h @@ -51,7 +51,7 @@ static inline int generic_ffs(int x) * fls: find last bit set. */ =20 -static __inline__ int generic_fls(int x) +static __inline__ int generic_fls(unsigned int x) { int r =3D 32; =20 --=20 2.25.1