[dpdk-dev] examples/dpdk_qat: Fix reference to old mbuf field "data"

Message ID 1416480607-5198-1-git-send-email-pablo.de.lara.guarch@intel.com (mailing list archive)
State Accepted, archived
Headers

Commit Message

De Lara Guarch, Pablo Nov. 20, 2014, 10:50 a.m. UTC
  DPDK QAT app was using the old "data" field in mbuf,
so it has been subtituted with rte_pktmbuf_mtod function.

Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
---
 examples/dpdk_qat/crypto.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)
  

Comments

Thomas Monjalon Nov. 24, 2014, 3:50 p.m. UTC | #1
> DPDK QAT app was using the old "data" field in mbuf,
> so it has been subtituted with rte_pktmbuf_mtod function.
> 
> Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>

Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>

Commit reference 08b563ffb19 ("mbuf: replace data pointer by an offset") added.
Applied

Thanks
  

Patch

diff --git a/examples/dpdk_qat/crypto.c b/examples/dpdk_qat/crypto.c
index 318d47c..f49408a 100644
--- a/examples/dpdk_qat/crypto.c
+++ b/examples/dpdk_qat/crypto.c
@@ -183,7 +183,7 @@  struct glob_keys g_crypto_hash_keys = {
  *
  */
 #define PACKET_DATA_START_PHYS(p) \
-		((p)->buf_physaddr + ((char *)p->data - (char *)p->buf_addr))
+		((p)->buf_physaddr + (p)->data_off)
 
 /*
  * A fixed offset to where the crypto is to be performed, which is the first
@@ -773,7 +773,7 @@  enum crypto_result
 crypto_encrypt(struct rte_mbuf *rte_buff, enum cipher_alg c, enum hash_alg h)
 {
 	CpaCySymDpOpData *opData =
-			(CpaCySymDpOpData *) ((char *) (rte_buff->data)
+			(CpaCySymDpOpData *) (rte_pktmbuf_mtod(rte_buff, char *)
 					+ CRYPTO_OFFSET_TO_OPDATA);
 	uint32_t lcore_id;
 
@@ -848,7 +848,7 @@  enum crypto_result
 crypto_decrypt(struct rte_mbuf *rte_buff, enum cipher_alg c, enum hash_alg h)
 {
 
-	CpaCySymDpOpData *opData = (void*) (((char *) rte_buff->data)
+	CpaCySymDpOpData *opData = (void*) (rte_pktmbuf_mtod(rte_buff, char *)
 			+ CRYPTO_OFFSET_TO_OPDATA);
 	uint32_t lcore_id;