Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756947Ab1ESNZs (ORCPT ); Thu, 19 May 2011 09:25:48 -0400 Received: from mx1.redhat.com ([209.132.183.28]:47736 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756252Ab1ESNZr (ORCPT ); Thu, 19 May 2011 09:25:47 -0400 Date: Thu, 19 May 2011 09:25:33 -0400 From: Neil Horman To: Andy Gospodarek Cc: Amerigo Wang , linux-kernel@vger.kernel.org, akpm@linux-foundation.org, Jay Vosburgh , "David S. Miller" , Ian Campbell , "Paul E. McKenney" , Josh Triplett , netdev@vger.kernel.org Subject: Re: [V2 Patch net-next-2.6] netpoll: disable netpoll when enslave a device Message-ID: <20110519132533.GA6729@shamino.rdu.redhat.com> References: <20110518105558.GA3203@hmsreliant.think-freely.org> <1305794393-20775-1-git-send-email-amwang@redhat.com> <20110519113127.GE21309@gospo.rdu.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20110519113127.GE21309@gospo.rdu.redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1472 Lines: 33 On Thu, May 19, 2011 at 07:31:27AM -0400, Andy Gospodarek wrote: > On Thu, May 19, 2011 at 04:39:53PM +0800, Amerigo Wang wrote: > [...] > > diff --git a/include/linux/notifier.h b/include/linux/notifier.h > > index 621dfa1..3d82867 100644 > > --- a/include/linux/notifier.h > > +++ b/include/linux/notifier.h > > @@ -211,6 +211,7 @@ static inline int notifier_to_errno(int ret) > > #define NETDEV_UNREGISTER_BATCH 0x0011 > > #define NETDEV_BONDING_DESLAVE 0x0012 > > #define NETDEV_NOTIFY_PEERS 0x0013 > > +#define NETDEV_ENSLAVE 0x0014 > > > > #define SYS_DOWN 0x0001 /* Notify of system down */ > > #define SYS_RESTART SYS_DOWN > > Neil just noted the same concern I had -- the asymmetry between > NETDEV_ENSLAVE and NETDEV_BONDING_DESLAVE bothers me a bit. I also > don't really like the followup patch that uses 'ENSLAVE' in the bridging > code when we typically use that language for bonding only. > > What about changing NETDEV_BONDING_DESLAVE to NETDEV_RELEASE and create > NETDEV_JOIN instead of NETDEV_ENSLAVE? I would prefer that or something > else that might use more generic language that could be applied to all > for stacked interfaces. JOIN and RELEASE (or perhaps LEAVE) sounds good to me. Neil -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/