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 1B97A44102; Wed, 29 May 2024 14:32:33 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A4D32402CD; Wed, 29 May 2024 14:32:32 +0200 (CEST) Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.9]) by mails.dpdk.org (Postfix) with ESMTP id 98E82402B9 for ; Wed, 29 May 2024 14:32:30 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1716985951; x=1748521951; h=date:from:to:cc:subject:message-id:references: in-reply-to:mime-version; bh=X52zu0jveKEklZGOlFVZ9aKJt2OeUeZTH4vTNw1dgdY=; b=feKkhwF/JDjiP9lnm5+uqmWYSsDAuD+qf7SHP/tXoEhSATmlhecG9I0x j5RwKcrxf4azLgGWmkRPzzsqLv/HfrvLsaqtR1HAbSC9KzMo7F/4U9hOK hGSu5ThMcFAkEi32qSimnTDR1yIqNJWOWQLp9fkSPEegqIwtSbKX8OIkJ PUINPjzx/je6R6jhlWwO1uuW03JOZDWFrZ2Oh2ssv824TMW9jL52sbuBM yLy/61u/h4s5e5BDSktgehFKM1TNl33zow8ilyg58rVwQshN3avrS0a+H 0fYIlOUS0L7BKWfLKmFe4SDDkk4JoQMVol3o4qdjqKQrLPxoqW8chzVuo A==; X-CSE-ConnectionGUID: XktryRXITFadsiFidssxEQ== X-CSE-MsgGUID: blcxmByAQhKTj4ELQ5s4JA== X-IronPort-AV: E=McAfee;i="6600,9927,11087"; a="24038525" X-IronPort-AV: E=Sophos;i="6.08,198,1712646000"; d="scan'208";a="24038525" Received: from orviesa001.jf.intel.com ([10.64.159.141]) by fmvoesa103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 May 2024 05:32:30 -0700 X-CSE-ConnectionGUID: 3SyE7pltRHWwTcCI/brBgA== X-CSE-MsgGUID: 4VVCZHXwRdyq3PPcGSkOtA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.08,198,1712646000"; d="scan'208";a="72872484" Received: from fmsmsx603.amr.corp.intel.com ([10.18.126.83]) by orviesa001.jf.intel.com with ESMTP/TLS/AES256-GCM-SHA384; 29 May 2024 05:32:29 -0700 Received: from fmsmsx612.amr.corp.intel.com (10.18.126.92) 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.2507.39; Wed, 29 May 2024 05:32:28 -0700 Received: from fmsedg601.ED.cps.intel.com (10.1.192.135) by fmsmsx612.amr.corp.intel.com (10.18.126.92) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39 via Frontend Transport; Wed, 29 May 2024 05:32:28 -0700 Received: from NAM11-BN8-obe.outbound.protection.outlook.com (104.47.58.171) by edgegateway.intel.com (192.55.55.70) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.39; Wed, 29 May 2024 05:32:28 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EACkFKgwn/arHyaX2MS2ZqZnz9kbfV1Q5wakQNP5wZVYy3it4RChOPlNAOVEIqJJl4FYUApduKGT0SHfSyhO+j9jHOj28z3UE+N14TV8UpijVoa9rPUWlldPKvX7yRrIdqA5IxiU4J9w4uWVTW6yt0YpjcpQ/FnyfLIg61XWXtkA3Zt/Z+4rkb+y3ePdKSd8T7qHLL33oL2wGYSUPCoyfbKdeFO5Gc5rfpQXDCriJECwhqSGpSuobsZ+sbAlO6v8t9mR4hkumqR/+opmbJyne5jyOSZKhOMY+1G+wi1ceoc2Rn/TbwhyJ90zaNa6Nj+eEyDVafvyR1TlvXjWfgI2Zw== 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=q7deq3H5rkGl/XTedQwaIhmGGPpr492lSSOH0veUjsU=; b=LE+T8NDPYCh1s/pDw6UPW9v2I+Jth9IIWPWOe8MIIGtl6hm2WAxbxtmn8cxpNCILLfuiK2jRu9+Ao7WS3AeesDHkGo46XFF1BtsbdC8u8flSPbvTcmscIw1c0CtOKijI4s1Sxy6fOgzlPGYppBkIhMxCsYrr9uKmufh6fJZjUhgD8NYHBAHtRS1GpN77aBAAxc27eJE5LbPGQNjEsDc7DDBKpyF3nOf7/amtMPiTu9ri+Ci19LvvKRWAKFalPfBoT3X//bLtF/FTsnsv2l5vtmB3MHQGFxMxPxBudrOMaSY9oaKbE3DUEhsVsAQ4MuWWhCF7HoH334cff0/4is7t+w== 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 Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; Received: from DS0PR11MB7309.namprd11.prod.outlook.com (2603:10b6:8:13e::17) by CH3PR11MB7764.namprd11.prod.outlook.com (2603:10b6:610:145::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7611.30; Wed, 29 May 2024 12:32:22 +0000 Received: from DS0PR11MB7309.namprd11.prod.outlook.com ([fe80::f120:cc1f:d78d:ae9b]) by DS0PR11MB7309.namprd11.prod.outlook.com ([fe80::f120:cc1f:d78d:ae9b%7]) with mapi id 15.20.7633.017; Wed, 29 May 2024 12:32:22 +0000 Date: Wed, 29 May 2024 13:32:18 +0100 From: Bruce Richardson To: Soumyadeep Hore CC: , , Subject: Re: [PATCH 01/25] common/idpf: added NVME CPF specific code with defines Message-ID: References: <20240528072839.867100-1-soumyadeep.hore@intel.com> <20240528072839.867100-2-soumyadeep.hore@intel.com> Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20240528072839.867100-2-soumyadeep.hore@intel.com> X-ClientProxiedBy: DUZPR01CA0321.eurprd01.prod.exchangelabs.com (2603:10a6:10:4ba::15) To DS0PR11MB7309.namprd11.prod.outlook.com (2603:10b6:8:13e::17) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DS0PR11MB7309:EE_|CH3PR11MB7764:EE_ X-MS-Office365-Filtering-Correlation-Id: f43de36d-6b60-4b81-6d92-08dc7fdb6351 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230031|376005|366007|1800799015; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?qlw3yMBCRo/yiGUgeR3OpsZg1xupD6ImBZ2ekK6D06kYvCN+wFnxUMm4DHi+?= =?us-ascii?Q?oaEmPb0nYzGNOhjcZObMsHFSBcsTPFnDlgu2EeZvXZlm021wJD90LaslPikB?= =?us-ascii?Q?EPtSVR74+cDDutUSZPbQ/ohcYbBPw0ECt8vca1JY2dFfngWr1NKi8r6P3hX+?= =?us-ascii?Q?vsMYpcmwS++u7O7DSJb+u5rIxxbWH1Yd9d/m1ZJqshM66EXFcvas1p5VcSZ8?= =?us-ascii?Q?m+kJSpQKZX5L1w7tHXy1g+BedO7hTDAnu4WnFSNFsYydMNVfELrWadNCtbps?= =?us-ascii?Q?vMPr3kvQiohSxw1DO+o8scMalUtwZsfgNQoIXItKVuO6Z19QjNdzHWllasf9?= =?us-ascii?Q?hDeK3BlznGKqPyQOqZcM8sAW588iUNwaXHgdyVJXjwsUeyCCoPBkJVFbi2km?= =?us-ascii?Q?FPcsadA0/Icx5nz4A7oleo+/g4VQrehJruZcilL3CJLpr0IxJmNT7nOOURaz?= =?us-ascii?Q?ZfooBwhFCqafYGPD02fySVWE+6ul5q4vOiJKY3RjSKrBhgt9ii8Pg+w7ZNNg?= =?us-ascii?Q?nJe0sYTOSCTCgilNovf0qR152OHhvptkgcGu+sdoEcQNokNZPYY0v64MtgKM?= =?us-ascii?Q?McBXz1ylCIKHurD/r1yQoP9wn5WYjt+3Gugng3LpN6tv5MAsnv7CdYycPUNM?= =?us-ascii?Q?Pf0ceS986lCfyoecy3Y20Xhfjhfwrb3C62D6g7tTNSbakNwczAmejTEL8BuG?= =?us-ascii?Q?o9p3Qy3wLdZUxMALBzdYILJNVYzXrA/6byCNRBKiwViVpbW4RWMjYpr9LLHj?= =?us-ascii?Q?qJCMr5y0BfGPm/4hg04eNx30VybZtmAXYmFjtwCmsS7GzdxOysMG42U55Kb+?= =?us-ascii?Q?3K/Z5IogV/QjgVoHNBMaXZiT76UXXyYqxZFHzT6eMLSsp/SzE/lZgUwBKhZ+?= =?us-ascii?Q?3yFM/d9qseJYOO1kXGfcbv2uau1pJQpBFjEc0ctqw0B2w6f4muFBylSJu5yl?= =?us-ascii?Q?qCEzQtH6L+Mpe+JEsBMEyuV4PX03FyGQ0lPPoSHck6fkFlnCtmd22MlSwYBg?= =?us-ascii?Q?0nPHfNrIwc/qCPWQ9zSf8R80pSYAzWHoI1idNuqM15bGa/7WnYmW5UUgR7yG?= =?us-ascii?Q?Dkpbx/lcszd3k2Rhr8kxurn6tJLQ/0GExMrFAlg+eJ/eGCxVlilA8hV8bMYh?= =?us-ascii?Q?43iHR9e4rxLPuczPwlEkgcQYZYBgCOMsRyRfYLvw+CLQatRrb+Od6zRgXBMo?= =?us-ascii?Q?HNPeMbvB/NJEO/AbXjokqPWADC2qWmQ7Ul7fKxdK+VVX+SYnb/AqoVTLXhUP?= =?us-ascii?Q?JUbC2ugZvtsdXYo+R5Kw07oXIcAftsrDMi/wJF0ZXQ=3D=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DS0PR11MB7309.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230031)(376005)(366007)(1800799015); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?HT2XO8wH3v+CIOSGHSRNmmov+uQOIFKY5RLmMzH2xTklGVYML7GUvHyKUJ0U?= =?us-ascii?Q?l0CrWmZ0zxlnZ2tfyG1sIzf0aTdKVAEhONlRKkc7d+Gt04FfhBXiAuNzbY3K?= =?us-ascii?Q?4h3vKj699NUah1VK25IUlYeialE+Q873/yyEwq0vR31bGLS1fpMgfEAj7yOP?= =?us-ascii?Q?MkX2+5p9SkxiRpXXGnrWsRXYw21QpC5LnjmQADrepjCny8+s6Qq/McZ4Qq/C?= =?us-ascii?Q?YN48rlkVEaUBdo7d61JvXvRZqkXXnn55ZG47na7Y2BwvC+1ChuqQ+QWZ+ZNM?= =?us-ascii?Q?gRcT1ffHBKQe1OfqOpa+EpOK4OUcTjWJifHX1Ef3nr4LuaLYS1oPgSJ/YaTG?= =?us-ascii?Q?9d+XOMi7xNPeqWoPJOTmW2AtbveDIVuS7yQfJIRCXtESBwKhSduFTvlkY6yW?= =?us-ascii?Q?Rgmkp3Y83mldLaj5fYfkm0uQ0y/r60XWnGRhUuH4h9HxOvuqlbJvYF0qnNgn?= =?us-ascii?Q?IAYgR0EpWg1LC7U0FwN0fMNqhT23SeNJkx1N6JfyHQYbxRfJ4hU1SHJ/5v4R?= =?us-ascii?Q?gjFAc0mm68FlTERiErn3e2YgxjKw7V/2rcZo53iKtRw4MUe+LWNBCgxuGLqI?= =?us-ascii?Q?Lv7xT56QOeSd9x80BP2DnY4igOBHfdKVQum87tfGE/Dr4FokVcAQdLdKqVst?= =?us-ascii?Q?ORghrvfDJUPckeZUwD13CFHQSB7MRw5QzwnlNmdbmauEGEwtv2X27tJtcyV9?= =?us-ascii?Q?aTT8LOiYKXPTQAvSTnI6HQgZJeqFjIQVnuFcEjw87IG/2JE1SjlnEzQ8V9Rn?= =?us-ascii?Q?ZqZ/yCHP84SepPCIlvAqVLDXXLlY5XO5prCQqMZyaMGKHxiT6PIg8OrVMBHo?= =?us-ascii?Q?cKkiRYcdfKrv4Dx9P6LR3fO8CDTYnNqhfibGORzHzoiaQgSkXtM49LIiYfgO?= =?us-ascii?Q?dnkO3HxWqBJD6TkSbyggCbdIF1NY9ZQLIYZvLEkflhAPWJ40eJlQ3Q9KVeaS?= =?us-ascii?Q?U44otc+MCrMYae27qxSemFd0Jkht291l9z3DtAcR2O1Ok5rY5DCg5Lze0XZs?= =?us-ascii?Q?ZFQ3RtQ7w/vfyTE7yi+1bQeLfzMAvqjydXMSeUt2EhMefYQV6hvNglZV7Ebk?= =?us-ascii?Q?F5u2I/MtL9FN6trIYelGlasCTI5Vi5w26sTOhmV/Xorv13rXqat204nqGww3?= =?us-ascii?Q?9BL6hf/NGSg3lcJIqKmk9K+vAaw8zgG+qMghiwD5ySDa8/9jxFxxfOd1x47d?= =?us-ascii?Q?9PI6+bzJT5W4J29YpkhubUFWi5Q7naCKknYdQqLiu23ShcmHYCj+Ag+vX+cE?= =?us-ascii?Q?KpAYBaVg90xO1unm0HPPpvw320NstJ5TVqQK2gFnC5e8wfd6lyqT7eVQBtQO?= =?us-ascii?Q?VzG+S41e+j251u+94LszqCKHCIRxMDc3GwdXcl/1rwLklCvJjEHGaVE7Lst6?= =?us-ascii?Q?O2JDzM9StbUA+LJsvS5K+YqkKB9mLVijxtXqTTQV5fdz6AE/NKpd/3UnVtgw?= =?us-ascii?Q?CxmRIZGeEO6xKmXXxZek9ouCR+0vkPEnJ9Pa8NNIsKephc88XVC0hRnWqds+?= =?us-ascii?Q?oUNtikRiDb0ehK6UEnXb2Hgx/NZAuSbgQKGM2mTv9o061hO4SpeRG7jZxo1X?= =?us-ascii?Q?YmlVgjL+/7dUcRTgXGIEa48BhZPicfNZdy4yeJ9PEmnAjdb3co4nce8OsHEX?= =?us-ascii?Q?NA=3D=3D?= X-MS-Exchange-CrossTenant-Network-Message-Id: f43de36d-6b60-4b81-6d92-08dc7fdb6351 X-MS-Exchange-CrossTenant-AuthSource: DS0PR11MB7309.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 May 2024 12:32:22.2009 (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: lVG1XmxAzmdELEFNDnOCW0CIyyOTy0L1/Yrz7YJqNQflIG9oZPU58vzy40vaXg5hrX4qbrym8utTDLStemJ6ZLiljRcLH5HnjPl+sn2P27Y= X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR11MB7764 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 On Tue, May 28, 2024 at 07:28:31AM +0000, Soumyadeep Hore wrote: > The aim of the changes is to remove NVME dependency on Hi, The intro words "The aim of the changes" is unnecessary. Just shorten the commit log by stating what the patch is for without any intro: "Remove NVME dependency on memory allocations..." If rewording, it would be worth including detail about when the define is expected to be used - will it be defined/configured in a later patch, or is it expected that it's a build-time setting set by user? If latter case, we need a doc update here. One further minor nit below too. /Bruce > memory allocations, and to use a prepared buffer instead. > > The changes do not affect other components. > > Signed-off-by: Soumyadeep Hore > --- > drivers/common/idpf/base/idpf_controlq.c | 27 +++++++++++++++++--- > drivers/common/idpf/base/idpf_controlq_api.h | 9 +++++-- > 2 files changed, 31 insertions(+), 5 deletions(-) > > diff --git a/drivers/common/idpf/base/idpf_controlq.c b/drivers/common/idpf/base/idpf_controlq.c > index a82ca628de..0ba7281a45 100644 > --- a/drivers/common/idpf/base/idpf_controlq.c > +++ b/drivers/common/idpf/base/idpf_controlq.c > @@ -1,5 +1,5 @@ > /* SPDX-License-Identifier: BSD-3-Clause > - * Copyright(c) 2001-2023 Intel Corporation > + * Copyright(c) 2001-2024 Intel Corporation > */ > > #include "idpf_controlq.h" > @@ -145,8 +145,12 @@ int idpf_ctlq_add(struct idpf_hw *hw, > qinfo->buf_size > IDPF_CTLQ_MAX_BUF_LEN) > return -EINVAL; > > +#ifndef NVME_CPF > cq = (struct idpf_ctlq_info *) > idpf_calloc(hw, 1, sizeof(struct idpf_ctlq_info)); > +#else > + cq = *cq_out; > +#endif > if (!cq) > return -ENOMEM; > > @@ -172,10 +176,15 @@ int idpf_ctlq_add(struct idpf_hw *hw, > } > > if (status) > +#ifdef NVME_CPF > + return status; > +#else > goto init_free_q; > +#endif > > if (is_rxq) { > idpf_ctlq_init_rxq_bufs(cq); > +#ifndef NVME_CPF > } else { > /* Allocate the array of msg pointers for TX queues */ > cq->bi.tx_msg = (struct idpf_ctlq_msg **) > @@ -185,6 +194,7 @@ int idpf_ctlq_add(struct idpf_hw *hw, > status = -ENOMEM; > goto init_dealloc_q_mem; > } > +#endif > } > > idpf_ctlq_setup_regs(cq, qinfo); > @@ -195,6 +205,7 @@ int idpf_ctlq_add(struct idpf_hw *hw, > > LIST_INSERT_HEAD(&hw->cq_list_head, cq, cq_list); > > +#ifndef NVME_CPF > *cq_out = cq; > return status; > > @@ -205,6 +216,7 @@ int idpf_ctlq_add(struct idpf_hw *hw, > idpf_free(hw, cq); > cq = NULL; > > +#endif > return status; > } > > @@ -232,8 +244,13 @@ void idpf_ctlq_remove(struct idpf_hw *hw, > * destroyed. This must be called prior to using the individual add/remove > * APIs. > */ > +#ifdef NVME_CPF > +int idpf_ctlq_init(struct idpf_hw *hw, u8 num_q, > + struct idpf_ctlq_create_info *q_info, struct idpf_ctlq_info **ctlq) > +#else > int idpf_ctlq_init(struct idpf_hw *hw, u8 num_q, > struct idpf_ctlq_create_info *q_info) > +#endif > { > struct idpf_ctlq_info *cq = NULL, *tmp = NULL; > int ret_code = 0; > @@ -244,6 +261,10 @@ int idpf_ctlq_init(struct idpf_hw *hw, u8 num_q, > for (i = 0; i < num_q; i++) { > struct idpf_ctlq_create_info *qinfo = q_info + i; > > +#ifdef NVME_CPF > + cq = *(ctlq + i); > + > +#endif > ret_code = idpf_ctlq_add(hw, qinfo, &cq); > if (ret_code) > goto init_destroy_qs; > @@ -398,7 +419,7 @@ int idpf_ctlq_send(struct idpf_hw *hw, struct idpf_ctlq_info *cq, > * ctlq_msgs and free or reuse the DMA buffers. > */ > static int __idpf_ctlq_clean_sq(struct idpf_ctlq_info *cq, u16 *clean_count, > - struct idpf_ctlq_msg *msg_status[], bool force) > + struct idpf_ctlq_msg *msg_status[], bool force) > { > struct idpf_ctlq_desc *desc; > u16 i = 0, num_to_clean; > @@ -469,7 +490,7 @@ static int __idpf_ctlq_clean_sq(struct idpf_ctlq_info *cq, u16 *clean_count, > * ctlq_msgs and free or reuse the DMA buffers. > */ > int idpf_ctlq_clean_sq_force(struct idpf_ctlq_info *cq, u16 *clean_count, > - struct idpf_ctlq_msg *msg_status[]) > + struct idpf_ctlq_msg *msg_status[]) > { > return __idpf_ctlq_clean_sq(cq, clean_count, msg_status, true); > } > diff --git a/drivers/common/idpf/base/idpf_controlq_api.h b/drivers/common/idpf/base/idpf_controlq_api.h > index 38f5d2df3c..bce5187981 100644 > --- a/drivers/common/idpf/base/idpf_controlq_api.h > +++ b/drivers/common/idpf/base/idpf_controlq_api.h > @@ -1,5 +1,5 @@ > /* SPDX-License-Identifier: BSD-3-Clause > - * Copyright(c) 2001-2023 Intel Corporation > + * Copyright(c) 2001-2024 Intel Corporation > */ > > #ifndef _IDPF_CONTROLQ_API_H_ > @@ -158,8 +158,13 @@ enum idpf_mbx_opc { > /* Will init all required q including default mb. "q_info" is an array of > * create_info structs equal to the number of control queues to be created. > */ > +#ifdef NVME_CPF > +int idpf_ctlq_init(struct idpf_hw *hw, u8 num_q, > + struct idpf_ctlq_create_info *q_info, struct idpf_ctlq_info **ctlq); > +#else > int idpf_ctlq_init(struct idpf_hw *hw, u8 num_q, > struct idpf_ctlq_create_info *q_info); > +#endif > > /* Allocate and initialize a single control queue, which will be added to the > * control queue list; returns a handle to the created control queue > @@ -186,7 +191,7 @@ int idpf_ctlq_recv(struct idpf_ctlq_info *cq, u16 *num_q_msg, > > /* Reclaims all descriptors on HW write back */ > int idpf_ctlq_clean_sq_force(struct idpf_ctlq_info *cq, u16 *clean_count, > - struct idpf_ctlq_msg *msg_status[]); > + struct idpf_ctlq_msg *msg_status[]); This is a whitespace change that has slipped in unrelated to the rest of the patch. Not a big deal, just keep an eye out for such things. If there are a few like this, you can consider just putting them in a misc patch at the end. > > /* Reclaims send descriptors on HW write back */ > int idpf_ctlq_clean_sq(struct idpf_ctlq_info *cq, u16 *clean_count, > -- > 2.43.0 >