From patchwork Sat Feb 27 22:07:33 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jiri Pirko X-Patchwork-Id: 46454 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.180.67]) by ozlabs.org (Postfix) with ESMTP id 73B7EB6F08 for ; Sun, 28 Feb 2010 09:07:51 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S936325Ab0B0WHq (ORCPT ); Sat, 27 Feb 2010 17:07:46 -0500 Received: from mx1.redhat.com ([209.132.183.28]:31439 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S936317Ab0B0WHp (ORCPT ); Sat, 27 Feb 2010 17:07:45 -0500 Received: from int-mx03.intmail.prod.int.phx2.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.16]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id o1RM7aOq024594 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Sat, 27 Feb 2010 17:07:36 -0500 Received: from localhost (vpn1-4-6.ams2.redhat.com [10.36.4.6]) by int-mx03.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id o1RM7XxO022318; Sat, 27 Feb 2010 17:07:34 -0500 Date: Sat, 27 Feb 2010 23:07:33 +0100 From: Jiri Pirko To: Jason Gunthorpe Cc: Or Gerlitz , netdev@vger.kernel.org, linux-rdma@vger.kernel.org, davem@davemloft.net, Moni Shoua Subject: Re: [net-next-2.6 PATCH] infiniband: convert to use netdev_for_each_mc_addr Message-ID: <20100227220731.GB6550@psychotron.redhat.com> References: <20100224151108.GD2663@psychotron.lab.eng.brq.redhat.com> <4B862E07.7020002@Voltaire.com> <20100225084915.GA3171@psychotron.redhat.com> <20100227060146.GA13231@obsidianresearch.com> <20100227103835.GA2862@psychotron.redhat.com> <20100227184410.GB13231@obsidianresearch.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20100227184410.GB13231@obsidianresearch.com> User-Agent: Mutt/1.5.20 (2009-08-17) X-Scanned-By: MIMEDefang 2.67 on 10.5.11.16 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Sat, Feb 27, 2010 at 07:44:10PM CET, jgunthorpe@obsidianresearch.com wrote: >On Sat, Feb 27, 2010 at 11:38:37AM +0100, Jiri Pirko wrote: > >> >The problem this statement is trying to solve had to do with bonding >> >creating multicast addresess for ethernet rather than infiniband in >> >some cases. This happens because bonding makes a device that switches >> >from ethernet to infiniband during its lifetime. I'm not quite >> >sure what kind of life cycle those addresses go through, but if they >> >somehow stay on the mc_list then ipoib_mcast_addr_is_valid will still >> >need to have the check. > >> Ok I see your point now. But in this case, the length check should be in all >> drivers because if for example bonding device changes from infiniband back to >> eth, the addresses stored would be "cut" to 6 bytes and would make no sense. > >Ideally, but the other wrinkle is due to other problems in the ipoib >driver one bad multicast address blocks further addresses from being >processed - effectively rendering the interface useless for >multicast. Ethernet drivers will just subscribe to a bogus address, no >real harm done. > >I would prefer to see the check remain in ipoib until we can see >through testing that it is no longer triggering. Fair enough. Subject: [net-next-2.6 PATCH] ipoib: returned back addrle check for mc addresses Apparently bogus mc address can brake IPOIB multicast processing. Therefore returning the check for addrlen back until this is resolved in bonding (I don't see any other point from where mc address with non-dev->addr_len length can came from). Signed-off-by: Jiri Pirko --- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/infiniband/ulp/ipoib/ipoib_multicast.c b/drivers/infiniband/ulp/ipoib/ipoib_multicast.c index 19eba3c..d41ea27 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_multicast.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_multicast.c @@ -767,8 +767,11 @@ void ipoib_mcast_dev_flush(struct net_device *dev) } } -static int ipoib_mcast_addr_is_valid(const u8 *addr, const u8 *broadcast) +static int ipoib_mcast_addr_is_valid(const u8 *addr, unsigned int addrlen, + const u8 *broadcast) { + if (addrlen != INFINIBAND_ALEN) + return 0; /* reserved QPN, prefix, scope */ if (memcmp(addr, broadcast, 6)) return 0; @@ -812,6 +815,7 @@ void ipoib_mcast_restart_task(struct work_struct *work) union ib_gid mgid; if (!ipoib_mcast_addr_is_valid(mclist->dmi_addr, + mclist->dmi_addrlen, dev->broadcast)) continue;