[dpdk-dev] [PATCH v4 02/15] net/avf: initialization of avf PMD

[Lu, Wenzhuo]

Hi Stephen,

> -----Original Message-----
> From: Stephen Hemminger [mailto:stephen at networkplumber.org]
> Sent: Saturday, January 6, 2018 4:30 AM
> To: Lu, Wenzhuo <wenzhuo.lu at intel.com>
> Cc: dev at dpdk.org; Wu, Jingjing <jingjing.wu at intel.com>
> Subject: Re: [dpdk-dev] [PATCH v4 02/15] net/avf: initialization of avf PMD
> 
> On Fri,  5 Jan 2018 16:21:32 +0800
> Wenzhuo Lu <wenzhuo.lu at intel.com> wrote:
> 
> > From: Jingjing Wu <jingjing.wu at intel.com>
> >
> > Signed-off-by: Jingjing Wu <jingjing.wu at intel.com>
> > ---
> >  config/common_base                      |   5 +
> >  drivers/net/Makefile                    |   1 +
> >  drivers/net/avf/Makefile                |  31 +++
> >  drivers/net/avf/avf.h                   | 187 ++++++++++++++
> >  drivers/net/avf/avf_ethdev.c            | 435
> ++++++++++++++++++++++++++++++++
> >  drivers/net/avf/avf_vchnl.c             | 304 ++++++++++++++++++++++
> >  drivers/net/avf/rte_pmd_avf_version.map |   4 +
> >  mk/rte.app.mk                           |   1 +
> >  8 files changed, 968 insertions(+)
> >  create mode 100644 drivers/net/avf/Makefile  create mode 100644
> > drivers/net/avf/avf.h  create mode 100644 drivers/net/avf/avf_ethdev.c
> > create mode 100644 drivers/net/avf/avf_vchnl.c  create mode 100644
> > drivers/net/avf/rte_pmd_avf_version.map
> >
> > diff --git a/config/common_base b/config/common_base index
> > e74febe..ce4d9bb 100644
> > --- a/config/common_base
> > +++ b/config/common_base
> > @@ -226,6 +226,11 @@
> CONFIG_RTE_LIBRTE_FM10K_RX_OLFLAGS_ENABLE=y
> >  CONFIG_RTE_LIBRTE_FM10K_INC_VECTOR=y
> >
> >  #
> > +# Compile burst-oriented AVF PMD driver #
> CONFIG_RTE_LIBRTE_AVF_PMD=n
> 
> Why is the default not 'y' ?
> Default is yse for IXGBE, EM and I40E already.
We change it to 'y' in patch 4. But you're right, we'd better setting it to 'y' at the beginning. Will change it with a V5.