From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 6 Mar 2012 09:09:05 -0800 From: Stephen Hemminger Message-ID: <20120306090905.0bc1a527@nehalam.linuxnetplumber.net> In-Reply-To: <4F56208E.2090801@gmail.com> References: <20120306142219.31783.30201.stgit@localhost.localdomain> <4F56208E.2090801@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Bridge] [PATCH] bridge: fix "entering disabled state" logging List-Id: Linux Ethernet Bridging List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paulius Zaleckas Cc: netdev@vger.kernel.org, bridge@lists.linux-foundation.org On Tue, 06 Mar 2012 16:34:54 +0200 Paulius Zaleckas wrote: > On 03/06/2012 04:22 PM, Paulius Zaleckas wrote: > > Now we have: > > eth0: link down > > br0: port 1(eth0) entering forwarding state > > > > State should be logged *after* it was changed, not before. > > The funny thing is that it was introduced: > 2010-05-16 stephen hemminger > bridge: change console message interface > 28a16c97963d3bc36a2c192859f6d8025ef2967a > > and no one noticed since then :D The message could be corrected to have correct verb tense. s/entering/entered/ Acked-by: Stephen Hemminger