Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1634400

[PATCH 2/9] net: thunderx: Optimize RBDR descriptor handling

From sunil.kovvuri@gmail.com
Newsgroups linux.kernel
Subject [PATCH 2/9] net: thunderx: Optimize RBDR descriptor handling
Date 2017-05-02 15:10 +0200
Message-ID <tCFJ0-77a-39@gated-at.bofh.it> (permalink)
References <tCFIZ-77a-9@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


From: Sunil Goutham <sgoutham@cavium.com>

Receive buffer's physical address or iova will anyway not
go beyond 49bits, since it is the max supported HW address.
As per perf, updating bitfields i.e buf_addr:42 in RBDR
descriptor entry consumes lots of cpu cycles, hence changed
it to a 64bit field with alignment requirements taken care of.

Signed-off-by: Sunil Goutham <sgoutham@cavium.com>
---
 drivers/net/ethernet/cavium/thunder/nicvf_queues.c |  8 ++++----
 drivers/net/ethernet/cavium/thunder/q_struct.h     | 10 +---------
 2 files changed, 5 insertions(+), 13 deletions(-)

diff --git a/drivers/net/ethernet/cavium/thunder/nicvf_queues.c b/drivers/net/ethernet/cavium/thunder/nicvf_queues.c
index 12f9709..dfc85a1 100644
--- a/drivers/net/ethernet/cavium/thunder/nicvf_queues.c
+++ b/drivers/net/ethernet/cavium/thunder/nicvf_queues.c
@@ -257,7 +257,7 @@ static int  nicvf_init_rbdr(struct nicvf *nic, struct rbdr *rbdr,
 		}
 
 		desc = GET_RBDR_DESC(rbdr, idx);
-		desc->buf_addr = (u64)rbuf >> NICVF_RCV_BUF_ALIGN;
+		desc->buf_addr = (u64)rbuf & ~(NICVF_RCV_BUF_ALIGN_BYTES - 1);
 	}
 
 	nicvf_get_page(nic);
@@ -286,7 +286,7 @@ static void nicvf_free_rbdr(struct nicvf *nic, struct rbdr *rbdr)
 	/* Release page references */
 	while (head != tail) {
 		desc = GET_RBDR_DESC(rbdr, head);
-		buf_addr = ((u64)desc->buf_addr) << NICVF_RCV_BUF_ALIGN;
+		buf_addr = desc->buf_addr;
 		phys_addr = nicvf_iova_to_phys(nic, buf_addr);
 		dma_unmap_page_attrs(&nic->pdev->dev, buf_addr, RCV_FRAG_LEN,
 				     DMA_FROM_DEVICE, DMA_ATTR_SKIP_CPU_SYNC);
@@ -297,7 +297,7 @@ static void nicvf_free_rbdr(struct nicvf *nic, struct rbdr *rbdr)
 	}
 	/* Release buffer of tail desc */
 	desc = GET_RBDR_DESC(rbdr, tail);
-	buf_addr = ((u64)desc->buf_addr) << NICVF_RCV_BUF_ALIGN;
+	buf_addr = desc->buf_addr;
 	phys_addr = nicvf_iova_to_phys(nic, buf_addr);
 	dma_unmap_page_attrs(&nic->pdev->dev, buf_addr, RCV_FRAG_LEN,
 			     DMA_FROM_DEVICE, DMA_ATTR_SKIP_CPU_SYNC);
@@ -364,7 +364,7 @@ static void nicvf_refill_rbdr(struct nicvf *nic, gfp_t gfp)
 			break;
 
 		desc = GET_RBDR_DESC(rbdr, tail);
-		desc->buf_addr = (u64)rbuf >> NICVF_RCV_BUF_ALIGN;
+		desc->buf_addr = (u64)rbuf & ~(NICVF_RCV_BUF_ALIGN_BYTES - 1);
 		refill_rb_cnt--;
 		new_rb++;
 	}
diff --git a/drivers/net/ethernet/cavium/thunder/q_struct.h b/drivers/net/ethernet/cavium/thunder/q_struct.h
index f363472..e47205a 100644
--- a/drivers/net/ethernet/cavium/thunder/q_struct.h
+++ b/drivers/net/ethernet/cavium/thunder/q_struct.h
@@ -359,15 +359,7 @@ union cq_desc_t {
 };
 
 struct rbdr_entry_t {
-#if defined(__BIG_ENDIAN_BITFIELD)
-	u64   rsvd0:15;
-	u64   buf_addr:42;
-	u64   cache_align:7;
-#elif defined(__LITTLE_ENDIAN_BITFIELD)
-	u64   cache_align:7;
-	u64   buf_addr:42;
-	u64   rsvd0:15;
-#endif
+	u64   buf_addr;
 };
 
 /* TCP reassembly context */
-- 
2.7.4

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 0/9] net: thunderx: Adds XDP support sunil.kovvuri@gmail.com - 2017-05-02 15:10 +0200
  [PATCH 8/9] net: thunderx: Support for XDP header adjustment sunil.kovvuri@gmail.com - 2017-05-02 15:10 +0200
  [PATCH 9/9] net: thunderx: Optimize page recycling for XDP sunil.kovvuri@gmail.com - 2017-05-02 15:10 +0200
  [PATCH 1/9] net: thunderx: Support for page recycling sunil.kovvuri@gmail.com - 2017-05-02 15:10 +0200
  [PATCH 2/9] net: thunderx: Optimize RBDR descriptor handling sunil.kovvuri@gmail.com - 2017-05-02 15:10 +0200
  [PATCH 4/9] net: thunderx: Cleanup receive buffer allocation sunil.kovvuri@gmail.com - 2017-05-02 15:10 +0200
  Re: [PATCH 0/9] net: thunderx: Adds XDP support David Miller <davem@davemloft.net> - 2017-05-02 21:50 +0200
    Re: [PATCH 0/9] net: thunderx: Adds XDP support Sunil Kovvuri <sunil.kovvuri@gmail.com> - 2017-05-03 09:30 +0200
      Re: [PATCH 0/9] net: thunderx: Adds XDP support Rami Rosen <roszenrami@gmail.com> - 2017-05-03 22:50 +0200
        Re: [PATCH 0/9] net: thunderx: Adds XDP support Sunil Kovvuri <sunil.kovvuri@gmail.com> - 2017-05-05 12:20 +0200

csiph-web