diff options
author | Gabor Juhos <juhosg@openwrt.org> | 2009-07-13 11:40:10 +0000 |
---|---|---|
committer | Gabor Juhos <juhosg@openwrt.org> | 2009-07-13 11:40:10 +0000 |
commit | a579511dbf89890992ed6c35e5bd7fd6411c56ef (patch) | |
tree | 992b60685c8f7922325c6770d6571d2a0e1fc74d /target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c | |
parent | 82a084680a527039a3fda555131919cc29a498d7 (diff) | |
download | master-31e0f0ae-a579511dbf89890992ed6c35e5bd7fd6411c56ef.tar.gz master-31e0f0ae-a579511dbf89890992ed6c35e5bd7fd6411c56ef.tar.bz2 master-31e0f0ae-a579511dbf89890992ed6c35e5bd7fd6411c56ef.zip |
ag71xx driver: align descriptors on cache_line boundary
SVN-Revision: 16825
Diffstat (limited to 'target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c')
-rw-r--r-- | target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c | 33 |
1 files changed, 23 insertions, 10 deletions
diff --git a/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c b/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c index 6302bf648c..1aa24b3958 100644 --- a/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c +++ b/target/linux/ar71xx/files/drivers/net/ag71xx/ag71xx_main.c @@ -11,6 +11,7 @@ * by the Free Software Foundation. */ +#include <linux/cache.h> #include "ag71xx.h" #define AG71XX_DEFAULT_MSG_ENABLE \ @@ -86,7 +87,7 @@ static void ag71xx_ring_free(struct ag71xx_ring *ring) kfree(ring->buf); if (ring->descs_cpu) - dma_free_coherent(NULL, ring->size * sizeof(struct ag71xx_desc), + dma_free_coherent(NULL, ring->size * ring->desc_size, ring->descs_cpu, ring->descs_dma); } @@ -95,10 +96,16 @@ static int ag71xx_ring_alloc(struct ag71xx_ring *ring, unsigned int size) int err; int i; - ring->descs_cpu = dma_alloc_coherent(NULL, - size * sizeof(struct ag71xx_desc), - &ring->descs_dma, - GFP_ATOMIC); + ring->desc_size = sizeof(struct ag71xx_desc); + if (ring->desc_size % cache_line_size()) { + DBG("ag71xx: ring %p, desc size %u rounded to %u\n", + ring, ring->desc_size, + roundup(ring->desc_size, cache_line_size())); + ring->desc_size = roundup(ring->desc_size, cache_line_size()); + } + + ring->descs_cpu = dma_alloc_coherent(NULL, size * ring->desc_size, + &ring->descs_dma, GFP_ATOMIC); if (!ring->descs_cpu) { err = -ENOMEM; goto err; @@ -113,8 +120,9 @@ static int ag71xx_ring_alloc(struct ag71xx_ring *ring, unsigned int size) } for (i = 0; i < size; i++) { - struct ag71xx_desc *descs = (struct ag71xx_desc *) ring->descs_cpu; - ring->buf[i].desc = &descs[i]; + ring->buf[i].desc = (struct ag71xx_desc *)&ring->descs_cpu[i * ring->desc_size]; + DBG("ag71xx: ring %p, desc %d at %p\n", + ring, i, ring->buf[i].desc); } return 0; @@ -156,7 +164,7 @@ static void ag71xx_ring_tx_init(struct ag71xx *ag) for (i = 0; i < AG71XX_TX_RING_SIZE; i++) { ring->buf[i].desc->next = (u32) (ring->descs_dma + - sizeof(struct ag71xx_desc) * ((i + 1) % AG71XX_TX_RING_SIZE)); + ring->desc_size * ((i + 1) % AG71XX_TX_RING_SIZE)); ring->buf[i].desc->ctrl = DESC_EMPTY; ring->buf[i].skb = NULL; @@ -190,9 +198,14 @@ static int ag71xx_ring_rx_init(struct ag71xx *ag) int ret; ret = 0; - for (i = 0; i < AG71XX_RX_RING_SIZE; i++) + for (i = 0; i < AG71XX_RX_RING_SIZE; i++) { ring->buf[i].desc->next = (u32) (ring->descs_dma + - sizeof(struct ag71xx_desc) * ((i + 1) % AG71XX_RX_RING_SIZE)); + ring->desc_size * ((i + 1) % AG71XX_RX_RING_SIZE)); + + DBG("ag71xx: RX desc at %p, next is %08x\n", + ring->buf[i].desc, + ring->buf[i].desc->next); + } for (i = 0; i < AG71XX_RX_RING_SIZE; i++) { struct sk_buff *skb; |