Hi Reshma,

On Fri, Jun 09, 2017 at 02:24:58PM +0000, Pattan, Reshma wrote:
> Hi,
> 
> > -----Original Message-----
> > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Gaetan Rivet
> > Sent: Thursday, June 8, 2017 12:59 AM
> > To: dev@dpdk.org
> > Cc: Gaetan Rivet <gaetan.ri...@6wind.com>
> > Subject: [dpdk-dev] [PATCH v2 07/12] pdump: disabled by default
> > 
> > Signed-off-by: Gaetan Rivet <gaetan.ri...@6wind.com>
> > ---
> >  config/common_base | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/config/common_base b/config/common_base index
> > cade611..8ec5e4e 100644
> > --- a/config/common_base
> > +++ b/config/common_base
> > @@ -700,7 +700,7 @@ CONFIG_RTE_KNI_PREEMPT_DEFAULT=y  #  # Compile
> > the pdump library  # -CONFIG_RTE_LIBRTE_PDUMP=y
> > +CONFIG_RTE_LIBRTE_PDUMP=n
> > 
> >  #
> >  # Compile vhost user library
> > --
> > 2.1.4
> 
> Since, you already mentioned in other mail to Ferruh that config flag 
> disabling patches are only for testers compilation purpose and you have plans 
> to make proper fix by end of June.  I will wait on for actual patch. 
> 

I said I planned to do so, but found out that I would not have enough
time before the end of June. Sorry about the ambiguous phrasing.

Do you think you will be able to fix this library in time?

> Please see if rte_pci.h can be removed from the includes of rte_pdump.c , 
> might be unnecessary include.
> 
> Thanks,
> Reshma

-- 
Gaëtan Rivet
6WIND

Reply via email to