|
Michal Suchanek |
56b8c5 |
From 38106b2c433e886344c33c90522343c37edfacea Mon Sep 17 00:00:00 2001
|
|
Michal Suchanek |
56b8c5 |
From: Sukadev Bhattiprolu <sukadev@linux.ibm.com>
|
|
Michal Suchanek |
56b8c5 |
Date: Tue, 14 Sep 2021 20:52:51 -0700
|
|
Michal Suchanek |
56b8c5 |
Subject: [PATCH] ibmvnic: Consolidate code in replenish_rx_pool()
|
|
Michal Suchanek |
56b8c5 |
|
|
Michal Suchanek |
56b8c5 |
References: bsc#1190758 ltc#191943
|
|
Takashi Iwai |
8dc4b3 |
Patch-mainline: v5.16-rc1
|
|
Michal Suchanek |
56b8c5 |
Git-commit: 38106b2c433e886344c33c90522343c37edfacea
|
|
Michal Suchanek |
56b8c5 |
|
|
Michal Suchanek |
56b8c5 |
For better readability, consolidate related code in replenish_rx_pool()
|
|
Michal Suchanek |
56b8c5 |
and add some comments.
|
|
Michal Suchanek |
56b8c5 |
|
|
Michal Suchanek |
56b8c5 |
Reviewed-by: Rick Lindsley <ricklind@linux.vnet.ibm.com>
|
|
Michal Suchanek |
56b8c5 |
Reviewed-by: Dany Madden <drt@linux.ibm.com>
|
|
Michal Suchanek |
56b8c5 |
Signed-off-by: Sukadev Bhattiprolu <sukadev@linux.ibm.com>
|
|
Michal Suchanek |
56b8c5 |
Signed-off-by: David S. Miller <davem@davemloft.net>
|
|
Michal Suchanek |
56b8c5 |
Acked-by: Michal Suchanek <msuchanek@suse.de>
|
|
Michal Suchanek |
56b8c5 |
---
|
|
Michal Suchanek |
56b8c5 |
drivers/net/ethernet/ibm/ibmvnic.c | 10 +++++++---
|
|
Michal Suchanek |
56b8c5 |
1 file changed, 7 insertions(+), 3 deletions(-)
|
|
Michal Suchanek |
56b8c5 |
|
|
Michal Suchanek |
56b8c5 |
diff --git a/drivers/net/ethernet/ibm/ibmvnic.c b/drivers/net/ethernet/ibm/ibmvnic.c
|
|
Michal Suchanek |
56b8c5 |
index a775c69e4fd7..e8b1231be485 100644
|
|
Michal Suchanek |
56b8c5 |
--- a/drivers/net/ethernet/ibm/ibmvnic.c
|
|
Michal Suchanek |
56b8c5 |
+++ b/drivers/net/ethernet/ibm/ibmvnic.c
|
|
Michal Suchanek |
56b8c5 |
@@ -371,6 +371,8 @@ static void replenish_rx_pool(struct ibmvnic_adapter *adapter,
|
|
Michal Suchanek |
56b8c5 |
}
|
|
Michal Suchanek |
56b8c5 |
|
|
Michal Suchanek |
56b8c5 |
index = pool->free_map[pool->next_free];
|
|
Michal Suchanek |
56b8c5 |
+ pool->free_map[pool->next_free] = IBMVNIC_INVALID_MAP;
|
|
Michal Suchanek |
56b8c5 |
+ pool->next_free = (pool->next_free + 1) % pool->size;
|
|
Michal Suchanek |
56b8c5 |
|
|
Michal Suchanek |
56b8c5 |
if (pool->rx_buff[index].skb)
|
|
Michal Suchanek |
56b8c5 |
dev_err(dev, "Inconsistent free_map!\n");
|
|
Michal Suchanek |
56b8c5 |
@@ -380,14 +382,15 @@ static void replenish_rx_pool(struct ibmvnic_adapter *adapter,
|
|
Michal Suchanek |
56b8c5 |
dst = pool->long_term_buff.buff + offset;
|
|
Michal Suchanek |
56b8c5 |
memset(dst, 0, pool->buff_size);
|
|
Michal Suchanek |
56b8c5 |
dma_addr = pool->long_term_buff.addr + offset;
|
|
Michal Suchanek |
56b8c5 |
- pool->rx_buff[index].data = dst;
|
|
Michal Suchanek |
56b8c5 |
|
|
Michal Suchanek |
56b8c5 |
- pool->free_map[pool->next_free] = IBMVNIC_INVALID_MAP;
|
|
Michal Suchanek |
56b8c5 |
+ /* add the skb to an rx_buff in the pool */
|
|
Michal Suchanek |
56b8c5 |
+ pool->rx_buff[index].data = dst;
|
|
Michal Suchanek |
56b8c5 |
pool->rx_buff[index].dma = dma_addr;
|
|
Michal Suchanek |
56b8c5 |
pool->rx_buff[index].skb = skb;
|
|
Michal Suchanek |
56b8c5 |
pool->rx_buff[index].pool_index = pool->index;
|
|
Michal Suchanek |
56b8c5 |
pool->rx_buff[index].size = pool->buff_size;
|
|
Michal Suchanek |
56b8c5 |
|
|
Michal Suchanek |
56b8c5 |
+ /* queue the rx_buff for the next send_subcrq_indirect */
|
|
Michal Suchanek |
56b8c5 |
sub_crq = &ind_bufp->indir_arr[ind_bufp->index++];
|
|
Michal Suchanek |
56b8c5 |
memset(sub_crq, 0, sizeof(*sub_crq));
|
|
Michal Suchanek |
56b8c5 |
sub_crq->rx_add.first = IBMVNIC_CRQ_CMD;
|
|
Michal Suchanek |
56b8c5 |
@@ -405,7 +408,8 @@ static void replenish_rx_pool(struct ibmvnic_adapter *adapter,
|
|
Michal Suchanek |
56b8c5 |
shift = 8;
|
|
Michal Suchanek |
56b8c5 |
#endif
|
|
Michal Suchanek |
56b8c5 |
sub_crq->rx_add.len = cpu_to_be32(pool->buff_size << shift);
|
|
Michal Suchanek |
56b8c5 |
- pool->next_free = (pool->next_free + 1) % pool->size;
|
|
Michal Suchanek |
56b8c5 |
+
|
|
Michal Suchanek |
56b8c5 |
+ /* if send_subcrq_indirect queue is full, flush to VIOS */
|
|
Michal Suchanek |
56b8c5 |
if (ind_bufp->index == IBMVNIC_MAX_IND_DESCS ||
|
|
Michal Suchanek |
56b8c5 |
i == count - 1) {
|
|
Michal Suchanek |
56b8c5 |
lpar_rc =
|
|
Michal Suchanek |
56b8c5 |
--
|
|
Michal Suchanek |
56b8c5 |
2.31.1
|
|
Michal Suchanek |
56b8c5 |
|