Re: [dpdk-web] [PATCH] roadmap: update to 16.11 roadmap items

[Thomas Monjalon <thomas.monjalon@6wind.com>]

2016-07-13 11:03, Mcnamara, John:
> Hi Thomas,
> 
> Comments below.

I agree with everything, thanks


> > -----Original Message-----
> > From: web [mailto:web-bounces@dpdk.org] On Behalf Of Thomas Monjalon
> > Sent: Monday, July 11, 2016 9:49 PM
> > To: Poole, James G <james.g.poole@intel.com>
> > Cc: web@dpdk.org
> > Subject: Re: [dpdk-web] [PATCH] roadmap: update to 16.11 roadmap items
> > 
> > Hi,
> > 
> > Thanks for helping to update the website.
> > Below are some comments:
> > 
> > 2016-07-08 08:27, James Poole:
> > > -		<li>Rx Filtering Rework
> > > -		<li>QEMU Vhost backend Reconnect
> > 
> > Yes it is in 16.07 already.
> 
> Yes. So it is okay to take it off the older roadmap. Right?
> 
> 
> > 
> > > -		<li>Delay Packet Copy in Vhost-user
> > > -		<li>Fix library and symbol Namespace
> > > +		<li>Cryptodev Support for Hardware Acceleration (via Intel(r)
> > QuickAssist Technology) for Additional Algorithms
> > > +		<li>Cryptodev Support for Software Acceleration for Additional
> > > +Algorithms
> > 
> > It's a bit too long.
> > Why not just "More Crypto Algorithms" instead of the above 2 lines?
> 
> Ok. Having them as two item is probably a result of us thinking about them separately. We will update to one item with:
> 
> "Additional Hardware and Software Crypto Algorithms"
> 
> 
> 
> > 
> > > +		<li>Cryptodev Performance Optimisation
> > > +		<li>IPsec Sample App Enhancements
> > 
> > Which IPsec enhancements?
> 
> The details are on Tim's longer announcement to the list: 
> "Add support for AES-GCM, AES-CTR, config file support to remove hard-coding of SAs/SPs etc., use forward cipher function to generate IV on CBC mode.""
> 
> I don't think it helps to get into that level of detail here.
> 
> > 
> > > +		<li>Generic Flow Director/Filtering/Classification API
> > 
> > The old API was already meant to be generic :) I think the "Rx Filtering"
> > wording is a simpler name than "Flow Director/Filtering/Classification".
> 
> The name actually came from Adrien's RFC. :-) No problem changing it. "Rx Filtering API" seems fine to me.
> 
> 
> > 
> > > +		<li>Cuckoo Hash Enhancements
> > 
> > Which enhancements?
> 
> Optimize the Cuckoo Hash lookup stages by using intelligent prefetching for keys and using IA AVX instructions for vector processing of keys. The details are on the longer list.
> 
> 
> > 
> > > +		<li>Add vHost PMD xStats
> > 
> > The previous lines had no verb "Add".
> > 
> > > +		<li>Delay Packet Copy in vHost-User Dequeue
> > 
> 
> No problem. We can just call it " vHost PMD xStats"
> 
> 
> James will make the changes and submit a v2. If you have any other concerns you can just make the changes during merge.
> 
> John
> 
> 
>