[<prev] [next>] [day] [month] [year] [list]
Message-ID: <000401d18aeb$de199060$9a4cb120$@cmss.chinamobile.com>
Date: Thu, 31 Mar 2016 09:23:01 +0800
From: 张胜举 <zhangshengju@...s.chinamobile.com>
To: "'David Miller'" <davem@...emloft.net>
Cc: <j.vosburgh@...il.com>, <vfalico@...il.com>,
<gospo@...ulusnetworks.com>, <netdev@...r.kernel.org>
Subject: Re: [net-next] bond: output message before setting slave to inactive
> From: Zhang Shengju <zhangshengju@...s.chinamobile.com>
> Date: Tue, 29 Mar 2016 06:32:57 +0000
>
> > This patch moves output message before setting slave to inactive, this
> > will print the correct status of slave device.
> >
> > Signed-off-by: Zhang Shengju <zhangshengju@...s.chinamobile.com>
>
> I think the message is in the appropriate spot wrt. state, so I will not
apply
> this, thanks.
The message will always print 'backup', since the code set it to inactive
before this output.
I think this will cause a little confused, because even when I detach an
active slave, the output is always 'backup'.
Thanks.
Powered by blists - more mailing lists