From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from smtp4.osuosl.org (smtp4.osuosl.org [140.211.166.137]) (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 E94DAEE4996 for ; Mon, 21 Aug 2023 14:47:48 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp4.osuosl.org (Postfix) with ESMTP id 825D540967; Mon, 21 Aug 2023 14:47:48 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.11.0 smtp4.osuosl.org 825D540967 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=osuosl.org; s=default; t=1692629268; bh=Ki0cTxgcISou7VI2eTwziBGmMWb+pHrc3q0XHU2aT3g=; h=Date:From:To:References:In-Reply-To:Subject:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: Cc:From; b=JgGCIFG9oUFhkO00x1p7IDZxahOFSh8wRkkHqPaSkRLdR4P1J7G9xmgaNqFA3j62p VGdLfSOMWgnAD23Q7jUSFGFZ4nB54uuaP8RrXR4BDBlPmPXtxZ/ECHCgOW4rDlfAGe u7CMZk1L+nGVTM2EBWaAcUrxmH7v8jw2auVXv6gBEApaHJE7R9JmzKViGkfJpYttVY LyxQiRRFTfNas/WgTLnmb98Ok0RgRk1GDjDF9cZGjyxRR36kPUo1g/qW7EYdFf1zYG u0KwbZkWQy+nkoqu/MWtSDVNtzKxajcCMV+hsgWxBKyYfsmkvUEoy8JL4kQOjI8rOn Wep6mHTYCufDA== X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp4.osuosl.org ([127.0.0.1]) by localhost (smtp4.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 8Z8jUvTruq4E; Mon, 21 Aug 2023 14:47:47 +0000 (UTC) Received: from ash.osuosl.org (ash.osuosl.org [140.211.166.34]) by smtp4.osuosl.org (Postfix) with ESMTP id 20EEC405A4; Mon, 21 Aug 2023 14:47:47 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.11.0 smtp4.osuosl.org 20EEC405A4 Received: from smtp2.osuosl.org (smtp2.osuosl.org [140.211.166.133]) by ash.osuosl.org (Postfix) with ESMTP id F198F1BF3E8 for ; Mon, 21 Aug 2023 14:47:44 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp2.osuosl.org (Postfix) with ESMTP id CA5D040A40 for ; Mon, 21 Aug 2023 14:47:44 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.11.0 smtp2.osuosl.org CA5D040A40 X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp2.osuosl.org ([127.0.0.1]) by localhost (smtp2.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id GmA2zrh98QGB for ; Mon, 21 Aug 2023 14:47:44 +0000 (UTC) Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by smtp2.osuosl.org (Postfix) with ESMTPS id 02BEE400DD for ; Mon, 21 Aug 2023 14:47:43 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.11.0 smtp2.osuosl.org 02BEE400DD Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 495EA63ADA; Mon, 21 Aug 2023 14:47:43 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1E8D6C433C7; Mon, 21 Aug 2023 14:47:40 +0000 (UTC) Date: Mon, 21 Aug 2023 16:47:38 +0200 From: Simon Horman To: "Guo, Junfeng" Message-ID: <20230821144738.GD2711035@kernel.org> References: <20230605054641.2865142-1-junfeng.guo@intel.com> <20230821023833.2700902-1-junfeng.guo@intel.com> <20230821023833.2700902-2-junfeng.guo@intel.com> <20230821072037.GB2711035@kernel.org> <20230821073000.GC2711035@kernel.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Mailman-Original-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1692629262; bh=ohAcEJTeRNpfSePyAT88M5+2s0rI40PRRuwjdeOMi14=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=fMiWpotesV+64iRnHBzBQddVetyPg+SDSgnsFMkX+mx1EeBC8uTmzFyta2dwIe9ns YYqb+b/DY0eKEWJx+Nq3PipjGEfBMMS/ONxHTc0rh3unOS62tYgVCCax4DreLhDLeQ 87ZG5i5G20XSr9/hkcKccEG1FqXAg6rnJWdDCKfLiJy/CxsXg+FB4WNXrXNU99lSnm pHW5OOrw5WQaHGdOgEZNUTnBERgJaeEipehR1PB3CjL2nQo43F729pZXXUr3zXq0Xz HBbUpLcgKWamzwU6UyeL1oTD1GVU3cEZDmAzs3NKX7ppok5hOi5qbmn3kwfBtixpqg 4T+W2tiFet8eQ== X-Mailman-Original-Authentication-Results: smtp2.osuosl.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20201202 header.b=fMiWpote Subject: Re: [Intel-wired-lan] [PATCH iwl-next v5 01/15] ice: add parser create and destroy skeleton X-BeenThere: intel-wired-lan@osuosl.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel Wired Ethernet Linux Kernel Driver Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: ivecera , "netdev@vger.kernel.org" , "Brandeburg, Jesse" , "Nguyen, Anthony L" , "Zhang, Qi Z" , "intel-wired-lan@lists.osuosl.org" Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: intel-wired-lan-bounces@osuosl.org Sender: "Intel-wired-lan" On Mon, Aug 21, 2023 at 07:34:38AM +0000, Guo, Junfeng wrote: > > > > -----Original Message----- > > From: Simon Horman > > Sent: Monday, August 21, 2023 15:30 > > To: Guo, Junfeng > > Cc: intel-wired-lan@lists.osuosl.org; netdev@vger.kernel.org; Nguyen, > > Anthony L ; Brandeburg, Jesse > > ; Zhang, Qi Z ; > > ivecera ; Samudrala, Sridhar > > > > Subject: Re: [PATCH iwl-next v5 01/15] ice: add parser create and > > destroy skeleton > > > > On Mon, Aug 21, 2023 at 09:20:37AM +0200, Simon Horman wrote: > > > On Mon, Aug 21, 2023 at 10:38:19AM +0800, Junfeng Guo wrote: > > > > Add new parser module which can parse a packet in binary > > > > and generate information like ptype, protocol/offset pairs > > > > and flags which can be used to feed the FXP profile creation > > > > directly. > > > > > > > > The patch added skeleton of the create and destroy APIs: > > > > ice_parser_create > > > > ice_parser_destroy > > > > > > > > Signed-off-by: Junfeng Guo > > > > > > Hi Junfeng Guo, > > > > > > some minor feedback from my side. > > > > > > > --- > > > > drivers/net/ethernet/intel/ice/ice_common.h | 4 +++ > > > > drivers/net/ethernet/intel/ice/ice_ddp.c | 10 +++--- > > > > drivers/net/ethernet/intel/ice/ice_ddp.h | 13 ++++++++ > > > > drivers/net/ethernet/intel/ice/ice_parser.c | 34 > > +++++++++++++++++++++ > > > > > > Perhaps I am missing something, but it seems that although > > > ice_parser.c is added by this patch-set, it is not added to > > > the build by this patch-set. This seems a little odd to me. > > > > Sorry, somehow I wasn't looking at the entire series. > > I now see that ice_parser.c is compiled as of patch 12/15 of this series. > > Yes, thanks for the carefully review! > > > > > > > > > > drivers/net/ethernet/intel/ice/ice_parser.h | 13 ++++++++ > > > > 5 files changed, 69 insertions(+), 5 deletions(-) > > > > create mode 100644 drivers/net/ethernet/intel/ice/ice_parser.c > > > > create mode 100644 drivers/net/ethernet/intel/ice/ice_parser.h > > > > > > ... > > > > > > > diff --git a/drivers/net/ethernet/intel/ice/ice_parser.c > > b/drivers/net/ethernet/intel/ice/ice_parser.c > > > > new file mode 100644 > > > > index 000000000000..42602cac7e45 > > > > --- /dev/null > > > > +++ b/drivers/net/ethernet/intel/ice/ice_parser.c > > > > @@ -0,0 +1,34 @@ > > > > +// SPDX-License-Identifier: GPL-2.0 > > > > +/* Copyright (C) 2023 Intel Corporation */ > > > > + > > > > +#include "ice_common.h" > > > > + > > > > +/** > > > > + * ice_parser_create - create a parser instance > > > > + * @hw: pointer to the hardware structure > > > > + * @psr: output parameter for a new parser instance be created > > > > + */ > > > > +int ice_parser_create(struct ice_hw *hw, struct ice_parser **psr) > > > > +{ > > > > + struct ice_parser *p; > > > > + > > > > + p = devm_kzalloc(ice_hw_to_dev(hw), sizeof(struct ice_parser), > > > > + GFP_KERNEL); > > > > + if (!p) > > > > + return -ENOMEM; > > > > + > > > > + p->hw = hw; > > > > + p->rt.psr = p; > > > > > > It is, perhaps academic if this file isn't compiled, but the rt field of > > > struct ice_parser doesn't exist at this point of the patch-set: it is > > added > > > by the last patch of the patch-set. > > > > And I see this field is added in patch 10/15, rather than the last patch > > (15/15) as I previously stated. > > Thanks for the comments! > Yes, the setting for rt field should be moved to patch 10/15. > Will update in the new version patch set. Thanks! Likewise, thanks. If you are going to address this you may also want to look at what seems to be similar problem with both ICE_PARSER_FLG_NUM and ICE_ERR_NOT_IMPL appearing in code before they are defined. ... _______________________________________________ Intel-wired-lan mailing list Intel-wired-lan@osuosl.org https://lists.osuosl.org/mailman/listinfo/intel-wired-lan