git: 8209a085c701 - stable/13 - ixgbe: Clarify index name in ixgbe_mc_filter_apply

Kevin Bowling kbowling at FreeBSD.org
Sun Apr 25 07:44:19 UTC 2021


The branch stable/13 has been updated by kbowling (ports committer):

URL: https://cgit.FreeBSD.org/src/commit/?id=8209a085c7016e62e97a9c756d656a41b4d9827e

commit 8209a085c7016e62e97a9c756d656a41b4d9827e
Author:     Kevin Bowling <kbowling at FreeBSD.org>
AuthorDate: 2021-04-17 01:17:43 +0000
Commit:     Kevin Bowling <kbowling at FreeBSD.org>
CommitDate: 2021-04-25 07:44:09 +0000

    ixgbe: Clarify index name in ixgbe_mc_filter_apply
    
    "It looks like it would be less confusing to rename 'count' to
    something like 'idx', since that's what it's used for in this
    function."
    
    Reviewed by:    erj
    MFC after:      1 week
    Differential Revision:  https://reviews.freebsd.org/D29798
    
    (cherry picked from commit 21afed4b1d18578aa8c9fa31e9e677971f8b4300)
---
 sys/dev/ixgbe/if_ix.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/sys/dev/ixgbe/if_ix.c b/sys/dev/ixgbe/if_ix.c
index 6e65f6bae55a..77ef118493a2 100644
--- a/sys/dev/ixgbe/if_ix.c
+++ b/sys/dev/ixgbe/if_ix.c
@@ -3262,15 +3262,15 @@ ixgbe_config_delay_values(struct adapter *adapter)
  *   Called whenever multicast address list is updated.
  ************************************************************************/
 static u_int
-ixgbe_mc_filter_apply(void *arg, struct sockaddr_dl *sdl, u_int count)
+ixgbe_mc_filter_apply(void *arg, struct sockaddr_dl *sdl, u_int idx)
 {
 	struct adapter *adapter = arg;
 	struct ixgbe_mc_addr *mta = adapter->mta;
 
-	if (count == MAX_NUM_MULTICAST_ADDRESSES)
+	if (idx == MAX_NUM_MULTICAST_ADDRESSES)
 		return (0);
-	bcopy(LLADDR(sdl), mta[count].addr, IXGBE_ETH_LENGTH_OF_ADDRESS);
-	mta[count].vmdq = adapter->pool;
+	bcopy(LLADDR(sdl), mta[idx].addr, IXGBE_ETH_LENGTH_OF_ADDRESS);
+	mta[idx].vmdq = adapter->pool;
 
 	return (1);
 } /* ixgbe_mc_filter_apply */


More information about the dev-commits-src-all mailing list