[dpdk-dev] [PATCH v6 0/7] rte_hash_crc reworked to be platform-independent

2015-02-24 Thread Bruce Richardson
On Tue, Feb 24, 2015 at 04:10:34AM +0100, Thomas Monjalon wrote: > 2015-02-24 09:00, Yerden Zhumabekov: > > > > 23.02.2015 23:36, Thomas Monjalon ?: > > > 2015-02-19 15:21, Bruce Richardson: > > >> Confirmed, this worked for me too. > > >> Looking at the patches, they look good. However, one t

[dpdk-dev] [PATCH v6 0/7] rte_hash_crc reworked to be platform-independent

2015-02-24 Thread Yerden Zhumabekov
23.02.2015 23:36, Thomas Monjalon ?: > 2015-02-19 15:21, Bruce Richardson: >> Confirmed, this worked for me too. >> Looking at the patches, they look good. However, one thing I think we are >> missing >> is a unit test to verify that all our CRC implementations give the same >> result. >> Th

[dpdk-dev] [PATCH v6 0/7] rte_hash_crc reworked to be platform-independent

2015-02-24 Thread Thomas Monjalon
2015-02-24 09:00, Yerden Zhumabekov: > > 23.02.2015 23:36, Thomas Monjalon ?: > > 2015-02-19 15:21, Bruce Richardson: > >> Confirmed, this worked for me too. > >> Looking at the patches, they look good. However, one thing I think we are > >> missing > >> is a unit test to verify that all our

[dpdk-dev] [PATCH v6 0/7] rte_hash_crc reworked to be platform-independent

2015-02-23 Thread Thomas Monjalon
2015-02-19 15:21, Bruce Richardson: > Confirmed, this worked for me too. > Looking at the patches, they look good. However, one thing I think we are > missing > is a unit test to verify that all our CRC implementations give the same > result. > That would be useful as a sanity check of the softwa

[dpdk-dev] [PATCH v6 0/7] rte_hash_crc reworked to be platform-independent

2015-02-19 Thread Bruce Richardson
On Mon, Feb 02, 2015 at 11:39:18AM +0600, Yerden Zhumabekov wrote: > > 02.02.2015 9:31, Neil Horman ?: > > On Mon, Feb 02, 2015 at 09:07:45AM +0600, Yerden Zhumabekov wrote: > > > >> I think so, I've just successfully built it against latest snapshot with > >> RTE_TARGET > >> equal to 'x86_64-

[dpdk-dev] [PATCH v6 0/7] rte_hash_crc reworked to be platform-independent

2015-02-02 Thread Yerden Zhumabekov
02.02.2015 9:31, Neil Horman ?: > On Mon, Feb 02, 2015 at 09:07:45AM +0600, Yerden Zhumabekov wrote: > >> I think so, I've just successfully built it against latest snapshot with >> RTE_TARGET >> equal to 'x86_64-native-linuxapp-gcc'. >> > Please confirm that setting the machine type to defaul

[dpdk-dev] [PATCH v6 0/7] rte_hash_crc reworked to be platform-independent

2015-02-02 Thread Yerden Zhumabekov
01.02.2015 20:13, Neil Horman ?: > On Thu, Jan 29, 2015 at 02:48:11PM +0600, Yerden Zhumabekov wrote: >> This is a rework of my previous patches improving performance of >> rte_hash_crc. >> >> Summary of changes: >> * software implementation of CRC32 introduced; >> * in the runtime, algorithm

[dpdk-dev] [PATCH v6 0/7] rte_hash_crc reworked to be platform-independent

2015-02-01 Thread Neil Horman
On Mon, Feb 02, 2015 at 09:07:45AM +0600, Yerden Zhumabekov wrote: > > 01.02.2015 20:13, Neil Horman ?: > > On Thu, Jan 29, 2015 at 02:48:11PM +0600, Yerden Zhumabekov wrote: > >> This is a rework of my previous patches improving performance of > >> rte_hash_crc. > >> > >> Summary of changes:

[dpdk-dev] [PATCH v6 0/7] rte_hash_crc reworked to be platform-independent

2015-02-01 Thread Neil Horman
On Thu, Jan 29, 2015 at 02:48:11PM +0600, Yerden Zhumabekov wrote: > This is a rework of my previous patches improving performance of rte_hash_crc. > > Summary of changes: > * software implementation of CRC32 introduced; > * in the runtime, algorithm can fall back to software version if CPU doesn'

[dpdk-dev] [PATCH v6 0/7] rte_hash_crc reworked to be platform-independent

2015-01-29 Thread Yerden Zhumabekov
This is a rework of my previous patches improving performance of rte_hash_crc. Summary of changes: * software implementation of CRC32 introduced; * in the runtime, algorithm can fall back to software version if CPU doesn't support SSE4.2; * best available algorithm is automatically detected upon