-----Original Message-----
From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Didier Pallard
Sent: Friday, February 19, 2016 11:00 AM
To: dev@dpdk.org
Subject: [dpdk-dev] [PATCH v3 0/2] Fix CRC32c computation

CRC32c computation is not valid when buffer length is not a multiple of 4
bytes.
Values returned by rte_hash_crc functions does not match the one
computed by a trivial crc32c implementation.

First patch fixes crc hash function autotests, to outline the problem.
Second patch fixes CRC32c computation.

Didier Pallard (2):
  test: fix CRC hash function autotest
  hash: fix CRC32c computation

 app/test/test_hash_functions.c         |  17 +++--
 doc/guides/rel_notes/release_16_04.rst |   5 ++
 lib/librte_hash/rte_crc_arm64.h        |  64 +++++++++++++++++
 lib/librte_hash/rte_hash_crc.h         | 125 ++++++++++++++++++++++++++-----
--
 4 files changed, 178 insertions(+), 33 deletions(-)

--
2.1.4

Series-acked-by: Pablo de Lara [off-list ref]

Not sure if you need to include a "Fixes" line in the commit messages.
In the first commit, probably you should, the commit that you are fixing is
6298d2c55ae8 ("app/test: add new functional tests for hash functions").
In the second commit, it is a bit more difficult, as we don't know that the commit is,
that code was integrated a while ago, before 1.2.3, which is the first public release in dpdk.org.

Also, there is a typo "lengthes", in both commit messages.

You can leave the ack in both patches. Thanks!!