From patchwork Thu Oct 22 13:54:05 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jiri Pirko X-Patchwork-Id: 36661 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by ozlabs.org (Postfix) with ESMTP id 9848AB7BB4 for ; Fri, 23 Oct 2009 00:54:27 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755989AbZJVNyR (ORCPT ); Thu, 22 Oct 2009 09:54:17 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755964AbZJVNyR (ORCPT ); Thu, 22 Oct 2009 09:54:17 -0400 Received: from mx1.redhat.com ([209.132.183.28]:42933 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755413AbZJVNyQ (ORCPT ); Thu, 22 Oct 2009 09:54:16 -0400 Received: from int-mx01.intmail.prod.int.phx2.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id n9MDs1CI016786; Thu, 22 Oct 2009 09:54:01 -0400 Received: from localhost (psychotron.englab.brq.redhat.com [10.34.32.135]) by int-mx01.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id n9MDs081012327; Thu, 22 Oct 2009 09:54:01 -0400 Date: Thu, 22 Oct 2009 15:54:05 +0200 From: Jiri Pirko To: netdev@vger.kernel.org Cc: davem@davemloft.net, eric.dumazet@gmail.com, jeffrey.t.kirsher@intel.com, jesse.brandeburg@intel.com, bruce.w.allan@intel.com, peter.p.waskiewicz.jr@intel.com, john.ronciak@intel.com, e1000-devel@lists.sourceforge.net, mchehab@infradead.org, linux-media@vger.kernel.org Subject: [PATCH net-next-2.6 3/4] e1000e: use mc helpers to access multicast list Message-ID: <20091022135404.GF2868@psychotron.lab.eng.brq.redhat.com> References: <20091022135120.GC2868@psychotron.lab.eng.brq.redhat.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20091022135120.GC2868@psychotron.lab.eng.brq.redhat.com> User-Agent: Mutt/1.5.19 (2009-01-05) X-Scanned-By: MIMEDefang 2.67 on 10.5.11.11 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Signed-off-by: Jiri Pirko --- drivers/net/e1000e/netdev.c | 34 +++++++++++++++++++--------------- 1 files changed, 19 insertions(+), 15 deletions(-) diff --git a/drivers/net/e1000e/netdev.c b/drivers/net/e1000e/netdev.c index 3769248..97cd106 100644 --- a/drivers/net/e1000e/netdev.c +++ b/drivers/net/e1000e/netdev.c @@ -2529,6 +2529,17 @@ static void e1000_update_mc_addr_list(struct e1000_hw *hw, u8 *mc_addr_list, } /** + * e1000_mc_walker - helper function + **/ +static void e1000_mc_walker(void *data, unsigned char *addr) +{ + u8 **mta_list_i = data; + + memcpy(*mta_list_i, addr, ETH_ALEN); + *mta_list_i += ETH_ALEN; +} + +/** * e1000_set_multi - Multicast and Promiscuous mode set * @netdev: network interface device structure * @@ -2542,10 +2553,9 @@ static void e1000_set_multi(struct net_device *netdev) struct e1000_adapter *adapter = netdev_priv(netdev); struct e1000_hw *hw = &adapter->hw; struct e1000_mac_info *mac = &hw->mac; - struct dev_mc_list *mc_ptr; - u8 *mta_list; + u8 *mta_list, *mta_list_i; u32 rctl; - int i; + int mc_count; /* Check for Promiscuous and All Multicast modes */ @@ -2567,23 +2577,17 @@ static void e1000_set_multi(struct net_device *netdev) ew32(RCTL, rctl); - if (netdev->mc_count) { - mta_list = kmalloc(netdev->mc_count * 6, GFP_ATOMIC); + mc_count = netdev_mc_count(netdev); + if (mc_count) { + mta_list = kmalloc(mc_count * ETH_ALEN, GFP_ATOMIC); if (!mta_list) return; /* prepare a packed array of only addresses. */ - mc_ptr = netdev->mc_list; - - for (i = 0; i < netdev->mc_count; i++) { - if (!mc_ptr) - break; - memcpy(mta_list + (i*ETH_ALEN), mc_ptr->dmi_addr, - ETH_ALEN); - mc_ptr = mc_ptr->next; - } + mta_list_i = mta_list; + netdev_mc_walk(netdev, e1000_mc_walker, &mta_list_i); - e1000_update_mc_addr_list(hw, mta_list, i, 1, + e1000_update_mc_addr_list(hw, mta_list, mc_count, 1, mac->rar_entry_count); kfree(mta_list); } else {