From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id E3C48A0C47; Wed, 27 Oct 2021 15:13:13 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 9C49040DDA; Wed, 27 Oct 2021 15:13:13 +0200 (CEST) Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by mails.dpdk.org (Postfix) with ESMTP id D3677407FF for ; Wed, 27 Oct 2021 15:13:11 +0200 (CEST) X-IronPort-AV: E=McAfee;i="6200,9189,10149"; a="228907257" X-IronPort-AV: E=Sophos;i="5.87,186,1631602800"; d="scan'208";a="228907257" Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 27 Oct 2021 06:13:10 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.87,186,1631602800"; d="scan'208";a="579621403" Received: from fmsmsx601.amr.corp.intel.com ([10.18.126.81]) by fmsmga002.fm.intel.com with ESMTP; 27 Oct 2021 06:13:10 -0700 Received: from fmsmsx607.amr.corp.intel.com (10.18.126.87) by fmsmsx601.amr.corp.intel.com (10.18.126.81) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Wed, 27 Oct 2021 06:13:10 -0700 Received: from fmsmsx603.amr.corp.intel.com (10.18.126.83) by fmsmsx607.amr.corp.intel.com (10.18.126.87) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Wed, 27 Oct 2021 06:13:10 -0700 Received: from fmsedg602.ED.cps.intel.com (10.1.192.136) by fmsmsx603.amr.corp.intel.com (10.18.126.83) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12 via Frontend Transport; Wed, 27 Oct 2021 06:13:10 -0700 Received: from NAM02-SN1-obe.outbound.protection.outlook.com (104.47.57.48) by edgegateway.intel.com (192.55.55.71) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2242.12; Wed, 27 Oct 2021 06:13:06 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=WJsu3tl/wpoO620lAkgzXw/U01gnoyGJbZjd2Uj0B5sq8kVN8KhiSEIRZwGZS+rgsrCREYciAAUWWPhpl/RQ7nizCuGu14WUf11I6hrdxBraNAYKhcL59tuHlx3DfLAnS1pe7BJZS3UbcyrAvnMjg5HHB34klh568lY0YrpoGVblCYrjBgZD1PbmHPF+mJxX5Q1vH5Q4pec0c6mviPeCp0tH5j/lDJkHwEE/uoFk2zHBIVSKhybujogiXP86UBKvP3v0SouPAi6ACaNNFurM0qJ3POdDNCgaokXpQTusUlAsnpIyh7iTho2tvPlv7jQyoGIkOK6nNJKo5RO6i4pGgw== 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=/9hTvKH4fvbVvyqHNxQSGrx9Hy2xQ0RM/640c2moNFU=; b=htaYtYGNf10urrAheUctAvQya9jVshw7QhVMScxnVD1zfl6YZTlCqD2YcPGhRPQkyK0T1uyLDg6pulXjaC3cg+AAYlIAR8+qE/mWqwpvRAhSsGhVm5bQRA8hOz5TkiYtQpGGEHKrinDSgxkIx+WfLSzLlQhf1zhCQibcvvVVQ2ZxUGEhxQjyFi8Gss4EkTJUF/sS7w3p46rsmabzyQQLHoD4XZjqwjv1JV31nA+VVMGdc4AY9sKP8PG1ipmQznUXUzbCDhpyYYIjBE57NSqcKvB+SsG+zXg1z73m0mURM5vF1VBSYFicqVElCoVvwmKhXGrEQmYn5l3/Q72BXTWmKA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=/9hTvKH4fvbVvyqHNxQSGrx9Hy2xQ0RM/640c2moNFU=; b=oh7xcUR+Z0cHA7XbDigjBrwoaTh5+FDIq8iFq0wPyqYSh0uRkOcdiz3wzwnvR8XblOMS4WJaEq4aCJK4DhBOIL7BOKxWvRTvwuhHb2suKXRscga190lZxls4VWHEvhfz5xI4C46Kbn4+OLL+1NG2woi9bZepMhcQ47XwQ9ck7r0= Received: from PH0PR11MB5207.namprd11.prod.outlook.com (2603:10b6:510:32::15) by PH0PR11MB4950.namprd11.prod.outlook.com (2603:10b6:510:33::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4628.16; Wed, 27 Oct 2021 13:12:59 +0000 Received: from PH0PR11MB5207.namprd11.prod.outlook.com ([fe80::f076:f51c:9101:b0ff]) by PH0PR11MB5207.namprd11.prod.outlook.com ([fe80::f076:f51c:9101:b0ff%8]) with mapi id 15.20.4649.014; Wed, 27 Oct 2021 13:12:59 +0000 From: "Walsh, Conor" To: "Laatz, Kevin" , "dev@dpdk.org" CC: "Richardson, Bruce" , "Laatz, Kevin" Thread-Topic: [dpdk-dev] [PATCH] dma/idxd: fix truncated error code in status check Thread-Index: AQHXyneCzikkOXdeaUaZrswI4Fb7Oqvm06eA Date: Wed, 27 Oct 2021 13:12:59 +0000 Message-ID: References: <20211026142045.2996949-1-kevin.laatz@intel.com> In-Reply-To: <20211026142045.2996949-1-kevin.laatz@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-reaction: no-action dlp-version: 11.6.200.16 dlp-product: dlpe-windows authentication-results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=intel.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: d6015aa0-69f8-42b3-a827-08d9994b8007 x-ms-traffictypediagnostic: PH0PR11MB4950: x-ld-processed: 46c98d88-e344-4ed4-8496-4ed7712e255d,ExtAddr x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:6790; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 34Ssh217b0Qelr/WAR400Ll191nmsE8tRMp643yfswyi1KZofw+bRG1zSOhn1ItAmJ41OMsDhMLO1jp36/1lmziJg/NINaRb1QUJd7k9mVFMaLCAEGBb0QEwlPUBLA05HdYADN9UNLG4/8PbnL4d87unAQQjdZrsRkyuU/lgC+iXG+oCGV0SFoGv8JRRUkTURi2o0q9hyx15tu8Hy3h6biOtu9gOpfhaF3heUC1f1OPI4TuyvOSESprCiCtVZCqLCF4Mraf9dmgdL+rP/hhdSwPRIeK7xsOMipqRXW4bP6uparHEE9M6K5epBFLNeb++TLe1kiO9uWWCvIwOTzs+glRFNbSGRIjkhUvLGW2XrEz4OLNXMCRhbZvkgYXLZ6caU7OpevHctMEymC289o1Rd6oNBy5cDuFnjNj5k0E8xl1oIX7Yf0YzvvEUJgNLWqFFUjTvIPPzgT2a1ndVcODRbRTjbgk2jGOnhMxiCwGTsiE1NSiHqPDU3pGOHVNyDSAaeIonR+NFwebOWwkbZUQC5LD4rCMcTTIHBE3KVKL0JUd/3wAgY+i6DAOE8VksiIz3/fmbQcWpAU2w16CioF/tNeTpXbb3iSIhwVywX4vlnKH5t3Uzn656qvBn3EPsRgBVxoB+2Job8M4hpaltnRFXBakhWUmiAZ300bvKuBLWtMYNlOKXYXd6B7/WD0HXcUfTclFWYVeJVRWivgQf6RTHzQ== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PH0PR11MB5207.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(366004)(82960400001)(8936002)(508600001)(55016002)(122000001)(9686003)(33656002)(107886003)(86362001)(6506007)(54906003)(53546011)(316002)(38100700002)(186003)(71200400001)(5660300002)(38070700005)(7696005)(4744005)(52536014)(4326008)(2906002)(64756008)(76116006)(66556008)(66476007)(66446008)(8676002)(110136005)(26005)(66946007); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?Gw21MGp9KKrnB5PQVkH8WOQcRuVWo6h9TxNwMzBDBzvxmNJtNlf5FVM39jjC?= =?us-ascii?Q?DqoFUTXGEtAVusPtEQ3MdjwQJChszoNEXHuwrLihkVFT/LLR22ggve3Ly71z?= =?us-ascii?Q?MdnJxKSbOh80Fir+bY0eCdTUPsPYs42eXaew99mRCiQ5P9bT8/znbyTLwHCv?= =?us-ascii?Q?Tl/jaqDXDgPNf9PB0w89KF1luB+J0/t+hvBhvoFxIwjyb0qgH/69A2VLnGUa?= =?us-ascii?Q?zuAs8SJ4wNfplF6IGuKyq0PO0ZLX1ShsAhLa1RfEggbW41T04BumAjAdZ8B3?= =?us-ascii?Q?zh/PC/FHYd6ie13L1f4xW12om3SHwpKmDHrZL7P0jcg4FW7pOIw+o53ZLP8L?= =?us-ascii?Q?+QQpDKnjJvRQQ6BaQpHP79k7cbB4Lj8Xxgo0FXBj3mg/CKp2SHtosrMe6nId?= =?us-ascii?Q?0DutKf1JgiOskNwdjlD5lrI2aACbG3foDIwgOpNrL+qRY37ey+dlGMlhUx5B?= =?us-ascii?Q?KQBmaB4WntKju9x0UeCE0a8SHY66T9o1oDlhZ6Du5B53qgf8KfyIHZca624K?= =?us-ascii?Q?03RCSu/L455sF5o50GnWpONFD63egqSZiMpu2skvznGRxdAddbbXRypfOCXn?= =?us-ascii?Q?8aq9BGdpUhbSSZXfBWptiqBaRAhK5GpfkthVSPCTMQBEuDicCSGfogWlb9us?= =?us-ascii?Q?XA9KPDfbBy0GHac5VgAwUMcyacvVBqhZavt5lR2zCZqeK25brLTUsw994qkn?= =?us-ascii?Q?6eMDNRv+M0uYS7iCvoAZDT9FOlQcknwmAGeL1ZLauSSLJvLqzrUbzBD8chko?= =?us-ascii?Q?D87/GixPD/acmyE1Gz/yHWmmGE6iylsQzePwUs6Y1qYMuOIm74t10IrwxkW9?= =?us-ascii?Q?L4Z72eShoT8TQ0pDln3VHVWk1c4gvGLP7JAh1qtBqP19UfHIWFDuAnu21sIf?= =?us-ascii?Q?Huo+ElIDUxeZx/KpY0FDoaC28hauoGzGUWU4usd/Ti1YOWdSWV61ugpfCGp5?= =?us-ascii?Q?so8aTewcJNsmEvTrwYmomrLgLtH3XOZ06h0u8XxFP7gmF0Ij8+QE3p1ABvJO?= =?us-ascii?Q?j/BvyUpwm5uwfi1f7sA0gFJXVdAg2uk6RkVScU1WdKjfv2SJj6cUF6TcWMln?= =?us-ascii?Q?BYVQMuIoXUuNMK+v+CTCA1u0KSwuLnW6oBNp+yJvI478OapMJvEIys0E/i3p?= =?us-ascii?Q?7R15v3rWw/7b88VmBvGSITm/OG+ZMV0dujO9NNftOyL64GB4DR7nedtWsPF+?= =?us-ascii?Q?hyi4D5TKIWjraLw7BNAKNQl7b/sL65qobeW0M/bujtadVNrz4A0O+ZTePUuZ?= =?us-ascii?Q?NM+cs7M3c/Fxn9YDJCT8ZTd8z7l7FCdXQJ8tBAPQiCATwViF30ruqxRcjpTg?= =?us-ascii?Q?LVbK93dJlQKmwXS7I0ahMzpe8inY8r6l56aLgsyyr4YvkPiT3+MdUiFReyNq?= =?us-ascii?Q?sLa/rWd1tVhkIOkFyc9zQIOMIZgQy77oI3b68qYhK1JIdGZyLZCEhoSaorm9?= =?us-ascii?Q?1Gcjh8Ws6IvPGuYE2miQIDpNOxiBltbyBhXkZ1VZyYSDMekMrzZogY7h3j7N?= =?us-ascii?Q?abW2PxL8rKtoUBOdeAKG6Of6alJ+pR0kXxmJ5kGjbngAzpMAb6JWLQtbAH9b?= =?us-ascii?Q?7dIsP4uSR4nq7+7tVbthC81cZafe6VxKwRFEKnlDBmgU6jUWva8ysQO0XuYP?= =?us-ascii?Q?FQ=3D=3D?= Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB5207.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: d6015aa0-69f8-42b3-a827-08d9994b8007 X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Oct 2021 13:12:59.3841 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: ENbBxsiOgbjim4JhlpKa05t7tC57BLtc+QHwOFl7zotMZRRVOx4m3GA8OwsY8lK8Nfwpa/YXfD4fnx/3MUkIow== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB4950 X-OriginatorOrg: intel.com Subject: Re: [dpdk-dev] [PATCH] dma/idxd: fix truncated error code in status check X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" > From: dev On Behalf Of Kevin Laatz > Sent: Tuesday 26 October 2021 15:21 > To: dev@dpdk.org > Cc: Richardson, Bruce ; Laatz, Kevin > > Subject: [dpdk-dev] [PATCH] dma/idxd: fix truncated error code in status > check >=20 > When checking if the DMA device is active, the result of the operand will > always be zero since the err_code is truncated to 8 bits which makes > checking the 31st bit impossible. >=20 > This is fixed by changing the type of err_code to uint32_t so that it is > not truncated. >=20 > Coverity issue: 373657 > Fixes: 9449330a8458 ("dma/idxd: create dmadev instances on PCI probe") >=20 > Signed-off-by: Kevin Laatz > --- Reviewed-by: Conor Walsh