-----Original Message-----
From: Thomas Monjalon [mailto:thomas.monja...@6wind.com] 
Sent: Wednesday, June 18, 2014 5:24 PM
To: Zhang, Helin
Cc: Chen, Jing D; dev at dpdk.org
Subject: Re: [dpdk-dev] [PATCH v2 00/27] Add i40e PMD support

2014-06-18 08:51, Zhang, Helin:
> Thomas Monjalon:
> > Some things could be cleaned up later, especially i40e specific 
> > flags in generic API must be removed. Please work on a patch for next 
> > release.
> 
> Btw, what do you mean the i40e specific flags in generic API? Did you 
> mean the new flags defined in rte_mbuf.h? Yes, we are working on some 
> bug fixes and enhancements which might need to be added in the next release.

I mean that RSS flags are really Intel-specific. In case other NIC vendors 
would use this RSS API, it can be a problem to address. But we can wait for 
this one.
About mbuf flags, they are all zeros and useless. So a patch is needed to get 
some space and define new values (as you already explained).
And last but not the least, RTE_LIBRTE_I40E_16BYTE_RX_DESC handling is very 
strange as it can be seen in this commit:
        http://dpdk.org/browse/dpdk/commit/?id=ac2ece3fb1f5511

-- 
Thomas

--------------------------------------------------------------------------

Hi Thomas

I am now working on several small bug fixes and enhancements which is for the 
next rc, including the rework of the last item you indicated in testpmd. Thank 
you very much!

Regards,
Helin

Reply via email to