[dpdk-dev] [PATCH v2 19/20] doc: add ICE description and update release note

Varghese, Vipin vipin.varghese at intel.com
Thu Dec 6 05:34:27 CET 2018


Thanks Wenzhuo

snipped
> >
> > Do we support Traffic Manager and Inline Crypto? If not, can we add
> > this also to limitations?
> The style is only listed supported features here. I don't think we'll list everything
> not supported as limitation.
Thanks for the correction in format, hence the suggested action item is update limitation section to state what is not supported. 

> The same below.
> 
> >
> > > +FW version           = Y
> > > +Module EEPROM dump   = Y
> > > +Multiprocess aware   = Y
> > > +BSD nic_uio          = Y
> > > +Linux UIO            = Y
> > > +Linux VFIO           = Y
> > > +x86-32               = Y
> > > +x86-64               = Y
> >
> > Is cross compile for ARM and PPC disabled since it uses Intel specific ISA?
> > Should this be added to limitations?
Is powerpc and arm cross build with non avx and sse ISA supported? If no, will default '.config' has 'ICE_PMD=n'?

> >
> >
> > Snipped
> >
> > > +
> > > +Config File Options
> > > +~~~~~~~~~~~~~~~~~~~
> > > +
> > > +The following options can be modified in the ``config`` file.
> > > +Please note that enabling debugging options may affect system
> > performance.
> >
> > Do we see real performance variance? If yes, can we highlight this in
> > info section?
> IMH, we could get different numbers because of different scenarios. It's only a
> reminder to not enable debug.
Thanks, but then wording should 'Note: enabling debug option for ICE will have difference in performance. Hence recommendation is not to enable unless for debugging the ICE PMD.'
> 
> >
> > snipped



More information about the dev mailing list