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 D5A17A034F; Fri, 12 Nov 2021 15:03:52 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id BCCB7410F5; Fri, 12 Nov 2021 15:03:52 +0100 (CET) Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by mails.dpdk.org (Postfix) with ESMTP id 86E80410EA for ; Fri, 12 Nov 2021 15:03:50 +0100 (CET) X-IronPort-AV: E=McAfee;i="6200,9189,10165"; a="213860992" X-IronPort-AV: E=Sophos;i="5.87,229,1631602800"; d="scan'208";a="213860992" Received: from orsmga006.jf.intel.com ([10.7.209.51]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 Nov 2021 06:03:46 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.87,229,1631602800"; d="scan'208";a="453158415" Received: from orsmsx601.amr.corp.intel.com ([10.22.229.14]) by orsmga006.jf.intel.com with ESMTP; 12 Nov 2021 06:03:46 -0800 Received: from orsmsx607.amr.corp.intel.com (10.22.229.20) by ORSMSX601.amr.corp.intel.com (10.22.229.14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Fri, 12 Nov 2021 06:03:45 -0800 Received: from orsmsx602.amr.corp.intel.com (10.22.229.15) by ORSMSX607.amr.corp.intel.com (10.22.229.20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Fri, 12 Nov 2021 06:03:45 -0800 Received: from orsedg603.ED.cps.intel.com (10.7.248.4) by orsmsx602.amr.corp.intel.com (10.22.229.15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12 via Frontend Transport; Fri, 12 Nov 2021 06:03:45 -0800 Received: from NAM02-SN1-obe.outbound.protection.outlook.com (104.47.57.43) by edgegateway.intel.com (134.134.137.100) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2242.12; Fri, 12 Nov 2021 06:03:45 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=YPxnuHMK8chDdbqlimbp/cHs7N98edVQ2hNCOh6uadKY6GgyLLxpJbLC6ho0fh8wLMe5wMfq7kGScvUT/zh+m9qkpM3o2ZB9nCpFi7sGCo++wGPSi8ik2M0hiMRKiCQPR1MbCFGPvRb0MkS7m3PbL6JnAEBTfjFWA+3/zXjt/K+5tyHV93gsZTk9tnSyDM4Klf3eDT2r9Ou7REhz9baseHfn5mSWRA12Ua0zsbqUGYDLcqkcXsqhHjZ+dhEifekGc9/sToNrilhVPr0nb2r/NcUbX2sjzExJGVs4nX1NTiI3LesSUkHEYCWRI3qrDr0WTS7pOK3cYj43Nk46LFO7IQ== 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=oLEMkuLnVdaemWmeBgcRAY4P4MeMPOHBSeokTRVwmOw=; b=AQMVSYN/T+CIzlxJ+1ek2bi+89XTKbuJhmZYB2B+Tz+ecYSzxIBmMVwouGJBuDYVxjwULbqpteqn/l2LcGxW1iX9BvPfrVNh5ghX7IReoTdhyZUYq46XGWib3ydQu5H75iZq9fuoFkrt0oFAgsK6OOhBhyRstTDAJA3W9GwLNPjoqWTcyr9fxAblQ1BhAtAEbGfZ3PxzdCBTnZmpeOKlLVuoAlyu0+uIAYMMA5+DD+d9QExPyY2KOcHNnwi1dL+RpThxsv+7nkXJeeC5DEsXMx7WQgNna0HJcH2NZb+H/c1jBuMSeWYRd++wjbd2wqE8PBaqz8+aFW3wnUrAi9JZFQ== 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=oLEMkuLnVdaemWmeBgcRAY4P4MeMPOHBSeokTRVwmOw=; b=MhfNC2t9BOQJWGMyp2/DTojN6Pjca5v9Pn0g9xPIaWgZ1kogJYmcvd6SCb4F0GVrnOxum+ycPxdhLdEIzb8RxGu/TWPjSJP9INfMIrLMJpW0JooPPu9bVZMYQXt1Uw6/yOYEHYNg0OXbp4cgUYRi7sNEmJxpS51GcpN++6lMaOw= Received: from MN2PR11MB3821.namprd11.prod.outlook.com (2603:10b6:208:f7::24) by MN2PR11MB3728.namprd11.prod.outlook.com (2603:10b6:208:f4::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4690.25; Fri, 12 Nov 2021 14:03:44 +0000 Received: from MN2PR11MB3821.namprd11.prod.outlook.com ([fe80::304c:39f1:9d95:4ba2]) by MN2PR11MB3821.namprd11.prod.outlook.com ([fe80::304c:39f1:9d95:4ba2%6]) with mapi id 15.20.4690.026; Fri, 12 Nov 2021 14:03:44 +0000 From: "Power, Ciara" To: "Kusztal, ArkadiuszX" , "dev@dpdk.org" CC: "gakhil@marvell.com" , "Zhang, Roy Fan" , "Zegan, PrzemyslawX" Subject: RE: [PATCH] common/qat: fix overrun of qp config Thread-Topic: [PATCH] common/qat: fix overrun of qp config Thread-Index: AQHX170fnB7GnVL2ZEeSLBqmYfrLTqv/7Dng Date: Fri, 12 Nov 2021 14:03:44 +0000 Message-ID: References: <20211112120156.12862-1-arkadiuszx.kusztal@intel.com> In-Reply-To: <20211112120156.12862-1-arkadiuszx.kusztal@intel.com> Accept-Language: en-IE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-version: 11.6.200.16 dlp-product: dlpe-windows dlp-reaction: no-action authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: ca9fad93-9dd2-4be2-7bcc-08d9a5e53d33 x-ms-traffictypediagnostic: MN2PR11MB3728: x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:159; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: bELAcnv3luJfWFUqkZJ2mCUzyuhIf8YNyDgytuaOjIVQp0GquN6veM13AXKeeiaHMGMh7/vduO5ZA8jvTHW+D7L+cqMZqTlnX8r16Gx57anag0X4GA7BgCG9slOeMuukIAMfXxfpbg7JYsYivfU5IpuKztPcQBoX/Sq+MuGn0NYnjMPcdjnLQyVcwuX+Q8YtuEPBejVZmvg+O6dWCQdI/ncoX/ttVskNibUuAenSo8fUE3tpIluNVBO4NS9p56dyrO1kzxy1oZoac8BJJRCvlqRZwm62mTFmcMfuKN9dKuutDrlDBaVkPe4uGN2mQ8pgK+1VXXSb4F+kRJ0wxrKDLEfKxQjaviGfsc0UEL6Zcq33hr9tS0zplnMxzEJUiZQcdLRUi6TsrDXvByqOEs0Q7Klnx8F3Fu2eDKb42XshZXTcK9gwv119na7TKtRvGqZKHyQrsKMXndSGSl/qA7ZBgXM1O9OwxrVD8G9hyLzcIFHHSXy6hsFRqSg5YMZJMCNHK35g96Q0B2FAZLglPK2/pNqM6eHE1Jmr7h0j9oFmwCj7x5P/B3qI/flM4EuYRztKsynXKdnd9a91Oj6nY8tVrNOHdjQpKabIAdHkxkTlrxslQfpJgCQIw+ZAk5p4d/RPfH9+88RFOcHDJB+MaLy2sUPtUTrzgppaHXdj7JKF6jsfSEyRhKtmcCqox6z51nL/CESZRxtX2v5o3m0jZt2FuQ== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MN2PR11MB3821.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(366004)(8936002)(86362001)(33656002)(7696005)(26005)(76116006)(107886003)(186003)(71200400001)(4326008)(6506007)(8676002)(9686003)(5660300002)(55016002)(52536014)(110136005)(508600001)(316002)(4744005)(2906002)(54906003)(66946007)(66476007)(66556008)(82960400001)(64756008)(38100700002)(66446008)(83380400001)(122000001)(38070700005); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?TUF8qH8KnBH863z3zU1/kRx490KbQxhs8GPHvVOuHBNg2QtEPp0MJ+ElMNkO?= =?us-ascii?Q?K+492vCWso1emqGvpfoZTkPA9PtE1m7loOYBqJClnGLDgthcInuUwBrPo3Ad?= =?us-ascii?Q?NLMXYgd3SiyaX1zrg0+RP+wGMZUP/E/wrUUnYWxE4jj7NUyl07jtqAG47HYJ?= =?us-ascii?Q?Dq26oglYP/bgRuIJxslo3Z4XkC/7LxrMTqiyUcd/JOCFytIDO8HpdvYET+xs?= =?us-ascii?Q?waxMx4KJWFMjiffOTrrXHJVJ8DPmorpSr5GlwkVzeGej3ehUIh8w2uRpkAHv?= =?us-ascii?Q?/BSiFcIyNjz/kwz13XyRWIkpaiSYasdFK14U43Jq0vqQzUdIFJ4/Ar8mcM++?= =?us-ascii?Q?XBE/aLJDcI/NN8y26ppZXH224a7+8NShpck5lLff0KpIscUoMDl8lI7/g1MF?= =?us-ascii?Q?mpaT/A8T8My4DpZa8Z2KeKDilasoIzXzjL2Eq/TPPm83iKStExmYR/eHcUyu?= =?us-ascii?Q?M9b7kW20k0aMQ5h5AClnRCZQMOCTaYrbRv/d2FB9pO7JYGQ6Z2dPfDH3kCqc?= =?us-ascii?Q?tNCc6XxUqYyTZJLoxxoIww7kebE5CEmzBZ0JpXb6DNHO29r/V/xGrOzqPW7V?= =?us-ascii?Q?Ozk6ReB3USn189R/n6ceuQYLGVWB9uY5iYekwLsq8lDazq0mwcuSYnU1bxcN?= =?us-ascii?Q?gGk4V9NdReA+de6rusUilLXwe5yQYcia3P/HiABWNfw/R5o7/8GLVapuTcFq?= =?us-ascii?Q?e+Vf/ZosvuB/hsyqMzxXR8EhIaSJdo2lgWDC2NjnY5+eilRp1Az65+Fh94/n?= =?us-ascii?Q?WVaUOjwzCZ/3mMlrXbuyp9DPkSl10K8YxlTFV7IXhbilK2GnRsUzkN9mxaul?= =?us-ascii?Q?NojN/4GVV1geqAFe9KAL0+7OTsmnRgbf/26oQKT9/aU4t3dXA/Moa9XQDUY5?= =?us-ascii?Q?0w/xf9K/QaPTIIravMKa40F0B37vELUQDFWV0JJshhZGRGwcBtgcqtBU18dX?= =?us-ascii?Q?YIQASy/IUV6x7Wl7AOM5F/SWxszibFFzPGx2xhQ636e0W5664AmPUTPKdA0t?= =?us-ascii?Q?5m9Ik/vwLa3vPLB1ZtQuKgE/OuUe8ofwbh1GzndebqBrjC+V07WTxpumvDqA?= =?us-ascii?Q?8upEMUgqLZTZg/lAnUA+Rs5vttGNJ5pZdOCkt5l6Y7hjLlEMQ6y2aX+shm8X?= =?us-ascii?Q?8OQ4tda0tVFdXSlAWOPZk59V6Mqh27tVI5QCN4NWvhwtoJs8HZiY5iM2yh+B?= =?us-ascii?Q?zTpVpc8ng38dxKKLlgFf6WOuqIALcTiLSdhLV4QTUY0OUFo3Ut5+rfm1Mi+J?= =?us-ascii?Q?0TTEPBSNcZasMcN3adUr4zO9SaQvIuRRw4+mhH6cX0/Dj2oUjleH3ksCqL8S?= =?us-ascii?Q?OnHWoG4bhLNeKXQXuZ4QPGTZ1bMM1gkZfh5DSgV0NxvP0omXYAL05rxJBwyl?= =?us-ascii?Q?lb6wQUlhVHdgc//Y1sFiqKf5WZMMQ2EeeYkMxJi+9Hjuo0Kb3XVYGjpOr7ko?= =?us-ascii?Q?Va/k4Y/UqgOWvCcWsALkHRjMQDw+qbZNBSNpC+/zt11w1doao0B9krXlCFLR?= =?us-ascii?Q?1n9j6xvDFL18FfjDs5hVCwOrPbeTRTs9Eqgx9jjhBaTJ9KOSb2z09VbVxssu?= =?us-ascii?Q?QZfc25+lwoZeJhoWVWPHwzfbU20eNVOayiGIQvwrzbyQFiqwS2GAX0oFx+vh?= =?us-ascii?Q?HEh09w8gpw1P5oF4UrvYNWU=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: MN2PR11MB3821.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: ca9fad93-9dd2-4be2-7bcc-08d9a5e53d33 X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Nov 2021 14:03:44.0864 (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: FfRxwgDmb3TeQuivcLCvz23Ih3+tq7fi+0CcFkm1odioE1MlWdW5KwhFGiE4ouycjqck8YSMo1r98QnDiKUGAw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB3728 X-OriginatorOrg: intel.com 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 Hi Arek, >-----Original Message----- >From: Kusztal, ArkadiuszX >Sent: Friday 12 November 2021 12:02 >To: dev@dpdk.org >Cc: gakhil@marvell.com; Zhang, Roy Fan ; Power, >Ciara ; Kusztal, ArkadiuszX >; Zegan, PrzemyslawX > >Subject: [PATCH] common/qat: fix overrun of qp config > >This commint fixes overrun of ring configuration due to insufficient check= . Small typo here: commint -> commit > >Fixes: 4badfb0205f7 ("common/qat: fix queue pairs number") >Cc: przemyslawx.zegan@intel.com > >Signed-off-by: Arek Kusztal >--- Asides from that, the patch looks good to me. Acked-by: Ciara Power