# \[PATCH net-next RESEND v2 5/9\] ibmvnic: init\_tx\_pools move loop-invariant code

- From: Sukadev Bhattiprolu \<hidden\>
- Date: 2021-09-15 03:53:36 
- List: netdev
- Page: https://ratatoskr.run/netdev/2021/09/11211235
- Thread: https://ratatoskr.run/netdev/2021/09/11211230/t.md

```text
In init_tx_pools() move some loop-invariant code out of the loop.

Reviewed-by: Rick Lindsley <redacted>
Reviewed-by: Dany Madden <redacted>
Signed-off-by: Sukadev Bhattiprolu <redacted>
---
 drivers/net/ethernet/ibm/ibmvnic.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/net/ethernet/ibm/ibmvnic.c b/drivers/net/ethernet/ibm/ibmvnic.c
index 97041b319beb..bb9b8aec9c9b 100644
--- a/drivers/net/ethernet/ibm/ibmvnic.c
+++ b/drivers/net/ethernet/ibm/ibmvnic.c
@@ -843,11 +843,10 @@ static int init_tx_pools(struct net_device *netdev)
 	 * allocation, release_tx_pools() will know how many to look for.
 	 */
 	adapter->num_active_tx_pools = num_pools;
+	buff_size = adapter->req_mtu + VLAN_HLEN;
+	buff_size = ALIGN(buff_size, L1_CACHE_BYTES);
 
 	for (i = 0; i < num_pools; i++) {
-		buff_size = adapter->req_mtu + VLAN_HLEN;
-		buff_size = ALIGN(buff_size, L1_CACHE_BYTES);
-
 		dev_dbg(dev, "Init tx pool %d [%llu, %llu]\n",
 			i, adapter->req_tx_entries_per_subcrq, buff_size);
 
-- 
2.26.2

```
