DPDK patches and discussions
 help / color / Atom feed
From: "Di, ChenxuX" <chenxux.di@intel.com>
To: "Yang, Qiming" <qiming.yang@intel.com>, "dev@dpdk.org" <dev@dpdk.org>
Cc: "Xing, Beilei" <beilei.xing@intel.com>
Subject: Re: [dpdk-dev] [PATCH v2 1/5] net/i40e: add private APIs
Date: Wed, 1 Jul 2020 01:30:54 +0000
Message-ID: <350673efbd454a77ad9d59ce7456f268@intel.com> (raw)
In-Reply-To: <BN6PR11MB001791440846691C7D113BABE56F0@BN6PR11MB0017.namprd11.prod.outlook.com>



> -----Original Message-----
> From: Yang, Qiming
> Sent: Tuesday, June 30, 2020 6:25 PM
> To: Di, ChenxuX <chenxux.di@intel.com>; dev@dpdk.org
> Cc: Xing, Beilei <beilei.xing@intel.com>
> Subject: RE: [PATCH v2 1/5] net/i40e: add private APIs
> 
> 
> 
> > -----Original Message-----
> > From: Di, ChenxuX <chenxux.di@intel.com>
> > Sent: Monday, June 15, 2020 10:19
> > To: dev@dpdk.org
> > Cc: Xing, Beilei <beilei.xing@intel.com>; Yang, Qiming
> > <qiming.yang@intel.com>; Di, ChenxuX <chenxux.di@intel.com>
> > Subject: [PATCH v2 1/5] net/i40e: add private APIs
> >
> > The legacy filter API will be superseded by rte_flow.
> > There are also several small features which can not be implemented in
> > rte_flow. This patch re-implemented these features as private API.
> > Three APIs are added:
> > rte_pmd_i40e_get_fdir_info.
> > rte_pmd_i40e_get_fdir_stats.
> > rte_pmd_i40e_set_gre_key_len.
> >
> > Signed-off-by: Chenxu Di <chenxux.di@intel.com>
> > ---
> >  doc/guides/rel_notes/release_20_08.rst    | 14 +++++++
> >  drivers/net/i40e/i40e_ethdev.c            |  2 +-
> >  drivers/net/i40e/i40e_ethdev.h            |  5 +++
> >  drivers/net/i40e/i40e_fdir.c              |  4 +-
> >  drivers/net/i40e/rte_pmd_i40e.c           | 51 +++++++++++++++++++++++
> >  drivers/net/i40e/rte_pmd_i40e.h           | 49 ++++++++++++++++++++++
> >  drivers/net/i40e/rte_pmd_i40e_version.map |  3 ++
> >  7 files changed, 125 insertions(+), 3 deletions(-)
> >
> > diff --git a/doc/guides/rel_notes/release_20_08.rst
> > b/doc/guides/rel_notes/release_20_08.rst
> > index dee4ccbb5..550ad33aa 100644
> > --- a/doc/guides/rel_notes/release_20_08.rst
> > +++ b/doc/guides/rel_notes/release_20_08.rst
> > @@ -62,6 +62,20 @@ New Features
> >
> >    * Added new PMD devarg ``reclaim_mem_mode``.
> >
> > +* **Updated the Intel i40e driver.**
> > +
> > +  Updated the Intel i40e driver with new features and improvements,
> > including:
> > +
> > +  * Re-implemented get_fdir_info and get_fdir_stat in private API.
> > +  * Re-implemented set_gre_key_len in private API.
> > +  * Added support for flow query RSS.
> > +
> > +* **Updated the Intel ixgbe driver.**
> > +
> > +  Updated the Intel ixgbe driver with new features and improvements,
> > including:
> > +
> > +  * Re-implemented get_fdir_info and get_fdir_stat in private API.
> > +
> >
> >  Removed Items
> >  -------------
> > diff --git a/drivers/net/i40e/i40e_ethdev.c
> > b/drivers/net/i40e/i40e_ethdev.c index 970a31cb2..674b711d9 100644
> > --- a/drivers/net/i40e/i40e_ethdev.c
> > +++ b/drivers/net/i40e/i40e_ethdev.c
> > @@ -8787,7 +8787,7 @@ i40e_tunnel_filter_param_check(struct i40e_pf
> > *pf,
> >
> >  #define I40E_GL_PRS_FVBM_MSK_ENA 0x80000000
> >  #define I40E_GL_PRS_FVBM(_i)     (0x00269760 + ((_i) * 4))
> > -static int
> > +int
> >  i40e_dev_set_gre_key_len(struct i40e_hw *hw, uint8_t len)  {  struct
> > i40e_pf *pf = &((struct i40e_adapter *)hw->back)->pf; diff -- git
> > a/drivers/net/i40e/i40e_ethdev.h b/drivers/net/i40e/i40e_ethdev.h
> > index e5d0ce53f..192636927 100644
> > --- a/drivers/net/i40e/i40e_ethdev.h
> > +++ b/drivers/net/i40e/i40e_ethdev.h
> > @@ -1270,6 +1270,11 @@ enum i40e_filter_pctype  uint16_t flow_type);
> > uint16_t i40e_pctype_to_flowtype(const struct i40e_adapter *adapter,
> >   enum i40e_filter_pctype pctype);
> > +int i40e_dev_set_gre_key_len(struct i40e_hw *hw, uint8_t len); void
> > +i40e_fdir_info_get(struct rte_eth_dev *dev, struct rte_eth_fdir_info
> > +*fdir); void i40e_fdir_stats_get(struct rte_eth_dev *dev,  struct
> > +rte_eth_fdir_stats *stat);
> >  int i40e_fdir_ctrl_func(struct rte_eth_dev *dev,
> >    enum rte_filter_op filter_op,
> >    void *arg);
> 
> It doesn't make sense to add the definition here as these functions only called by
> private API.
> I think it better to move these function to private also
> 

For the reason the legacy filter API and function is not really removed, it will call the function.
It may not be the right time to move the functions to private.

The same as ixgbe.

> [...]
> 
> 
> > --
> > 2.17.1
> 


  reply index

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-11  6:01 [dpdk-dev] [PATCH 0/5] re-implement legacy filter functions by private API Chenxu Di
2020-06-11  6:01 ` [dpdk-dev] [PATCH 1/5] net/i40e: add private APIs Chenxu Di
2020-06-11  6:01 ` [dpdk-dev] [PATCH 2/5] net/ixgbe: " Chenxu Di
2020-06-11  6:01 ` [dpdk-dev] [PATCH 3/5] app/testpmd: re-implement commands by using private API Chenxu Di
2020-06-16 20:12   ` Kevin Traynor
2020-06-17 10:12     ` Di, ChenxuX
2020-06-30  6:20       ` Jeff Guo
2020-06-11  6:01 ` [dpdk-dev] [PATCH 4/5] net/i40e: enable flow query RSS Chenxu Di
2020-06-11  6:01 ` [dpdk-dev] [PATCH 5/5] app/testpmd: support query RSS config in flow query Chenxu Di
2020-06-15  2:18 ` [dpdk-dev] [PATCH v2 0/5] re-implement legacy filter functions by private API Chenxu Di
2020-06-15  2:18   ` [dpdk-dev] [PATCH v2 1/5] net/i40e: add private APIs Chenxu Di
2020-06-30  4:22     ` Jeff Guo
2020-06-30 10:24     ` Yang, Qiming
2020-07-01  1:30       ` Di, ChenxuX [this message]
2020-06-15  2:18   ` [dpdk-dev] [PATCH v2 2/5] net/ixgbe: " Chenxu Di
2020-06-30  5:25     ` Jeff Guo
2020-06-30 10:25     ` Yang, Qiming
2020-06-15  2:18   ` [dpdk-dev] [PATCH v2 3/5] app/testpmd: re-implement commands by using private API Chenxu Di
2020-06-30 10:34     ` Yang, Qiming
2020-06-15  2:18   ` [dpdk-dev] [PATCH v2 4/5] net/i40e: enable flow query RSS Chenxu Di
2020-06-30  6:48     ` Jeff Guo
2020-06-30 10:27     ` Yang, Qiming
2020-07-01  1:15       ` Di, ChenxuX
2020-06-15  2:18   ` [dpdk-dev] [PATCH v2 5/5] app/testpmd: support query RSS config in flow query Chenxu Di
2020-06-30  6:37     ` Jeff Guo
2020-06-30 10:40     ` Yang, Qiming
2020-07-01  1:25       ` Di, ChenxuX
2020-07-01  8:24 ` [dpdk-dev] [PATCH v3 0/5] re-implement legacy filter functions by private API Chenxu Di
2020-07-01  8:24   ` [dpdk-dev] [PATCH v3 1/5] net/i40e: add private APIs Chenxu Di
2020-07-03  5:44     ` Jeff Guo
2020-07-01  8:24   ` [dpdk-dev] [PATCH v3 2/5] net/ixgbe: " Chenxu Di
2020-07-03  5:44     ` Jeff Guo
2020-07-01  8:24   ` [dpdk-dev] [PATCH v3 3/5] app/testpmd: re-implement commands by using private API Chenxu Di
2020-07-03  5:54     ` Jeff Guo
2020-07-06  9:10     ` Yang, Qiming
2020-07-01  8:24   ` [dpdk-dev] [PATCH v3 4/5] net/i40e: enable flow query RSS Chenxu Di
2020-07-03  5:57     ` Jeff Guo
2020-07-01  8:24   ` [dpdk-dev] [PATCH v3 5/5] app/testpmd: support query RSS config in flow query Chenxu Di
2020-07-08  1:18 ` [dpdk-dev] [PATCH v4 0/5] re-implement legacy filter functions by private API Chenxu Di
2020-07-08  1:18   ` [dpdk-dev] [PATCH v4 1/5] net/i40e: add private APIs Chenxu Di
2020-07-08  1:18   ` [dpdk-dev] [PATCH v4 2/5] net/ixgbe: " Chenxu Di
2020-07-08  1:18   ` [dpdk-dev] [PATCH v4 3/5] app/testpmd: re-implement commands by using private API Chenxu Di
2020-07-08  3:31     ` Yang, Qiming
2020-07-08  1:18   ` [dpdk-dev] [PATCH v4 4/5] net/i40e: enable flow query RSS Chenxu Di
2020-07-08  1:18   ` [dpdk-dev] [PATCH v4 5/5] app/testpmd: support query RSS config in flow query Chenxu Di
2020-07-08  3:32     ` Yang, Qiming
2020-07-08  9:05   ` [dpdk-dev] [PATCH v4 0/5] re-implement legacy filter functions by private API Zhang, Qi Z

Reply instructions:

You may reply publically to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=350673efbd454a77ad9d59ce7456f268@intel.com \
    --to=chenxux.di@intel.com \
    --cc=beilei.xing@intel.com \
    --cc=dev@dpdk.org \
    --cc=qiming.yang@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

DPDK patches and discussions

Archives are clonable:
	git clone --mirror http://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ http://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev


Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/ public-inbox