From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Wed, 10 Aug 2011 08:11:38 -0700 From: Stephen Hemminger Message-ID: <20110810081138.3a1b2632@nehalam.ftrdhcpuser.net> In-Reply-To: <6680AE1BD102B9419251E785D19F5D039FA037F0@ExchN1.riedel.net> References: <6680AE1BD102B9419251E785D19F5D039FA037F0@ExchN1.riedel.net> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Bridge] possible bridge regression in "bridge: implement [add/del]_slave ops"? List-Id: Linux Ethernet Bridging List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Weirich, Bernhard" Cc: "bridge@lists.linux-foundation.org" On Wed, 10 Aug 2011 09:54:58 +0200 "Weirich, Bernhard" wrote: > Hello, > > I had trouble with the new detection code for interface ports. > In turned out that the carrier was down, but IFF_RUNNING was set. > I changed the code to check for IFF_UP & IFF_LOWER_UP for interface ports and now it works the way it should. > Would it be better to check IFF_LOWER_UP or is my interface device driver incorrect? > > Best regards, > Bernhard The standard for carrier up is IFF_RUNNING. IFF_LOWER_UP is meant for use by modems and other special devies.