blob: 0d4267be2ea5ff75ed2aa8f5c9fd2a21ab73dde7 [file] [log] [blame]
Damjan Marion19414de2016-06-07 12:46:55 +02001From d91c4e2de969086ebc8c3a1dfa30913ea3de37b4 Mon Sep 17 00:00:00 2001
John Loacc22d72016-04-27 16:43:44 -04002From: John Daley <johndale@cisco.com>
3Date: Mon, 25 Apr 2016 16:24:53 -0700
Damjan Marion19414de2016-06-07 12:46:55 +02004Subject: [PATCH 11/17] enic: fix misalignment of Rx mbuf data
John Loacc22d72016-04-27 16:43:44 -04005
6Data DMA used m->data_off of uninitialized mbufs instead of
7RTE_PKTMBUF_HEADROOM, potentially causing Rx data to be
8placed at the wrong alignment in the mbuf.
9
10Fixes: 947d860c821f ("enic: improve Rx performance")
11Signed-off-by: John Daley <johndale@cisco.com>
12---
13 drivers/net/enic/enic_main.c | 5 +++--
14 drivers/net/enic/enic_rx.c | 6 ++++--
15 2 files changed, 7 insertions(+), 4 deletions(-)
16
17diff --git a/drivers/net/enic/enic_main.c b/drivers/net/enic/enic_main.c
Damjan Marion19414de2016-06-07 12:46:55 +020018index 06cacd4..b164307 100644
John Loacc22d72016-04-27 16:43:44 -040019--- a/drivers/net/enic/enic_main.c
20+++ b/drivers/net/enic/enic_main.c
Damjan Marion19414de2016-06-07 12:46:55 +020021@@ -355,10 +355,11 @@ enic_alloc_rx_queue_mbufs(struct enic *enic, struct vnic_rq *rq)
John Loacc22d72016-04-27 16:43:44 -040022 return -ENOMEM;
23 }
24
25- dma_addr = (dma_addr_t)(mb->buf_physaddr + mb->data_off);
26+ dma_addr = (dma_addr_t)(mb->buf_physaddr
27+ + RTE_PKTMBUF_HEADROOM);
28
29 rq_enet_desc_enc(rqd, dma_addr, RQ_ENET_TYPE_ONLY_SOP,
30- mb->buf_len);
31+ mb->buf_len - RTE_PKTMBUF_HEADROOM);
32 rq->mbuf_ring[i] = mb;
33 }
34
35diff --git a/drivers/net/enic/enic_rx.c b/drivers/net/enic/enic_rx.c
36index 232987a..39bb55c 100644
37--- a/drivers/net/enic/enic_rx.c
38+++ b/drivers/net/enic/enic_rx.c
39@@ -314,9 +314,11 @@ enic_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts,
40 + rx_id);
41
42 /* Push descriptor for newly allocated mbuf */
43- dma_addr = (dma_addr_t)(nmb->buf_physaddr + nmb->data_off);
44+ dma_addr = (dma_addr_t)(nmb->buf_physaddr
45+ + RTE_PKTMBUF_HEADROOM);
46 rqd_ptr->address = rte_cpu_to_le_64(dma_addr);
47- rqd_ptr->length_type = cpu_to_le16(nmb->buf_len);
48+ rqd_ptr->length_type = cpu_to_le16(nmb->buf_len
49+ - RTE_PKTMBUF_HEADROOM);
50
51 /* Fill in the rest of the mbuf */
52 rxmb->data_off = RTE_PKTMBUF_HEADROOM;
53--
Damjan Marion19414de2016-06-07 12:46:55 +0200542.7.4
John Loacc22d72016-04-27 16:43:44 -040055