From mboxrd@z Thu Jan 1 00:00:00 1970 From: "John W. Linville" Subject: [patch netdev-2.6] vlan_dev: return 0 on vlan_dev_change_mtu success Date: Thu, 18 Nov 2004 14:04:36 -0500 Message-ID: <20041118140436.A16007@tuxdriver.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: greearb@candelatech.com, vlan@scry.wanfear.com Return-path: To: linux-kernel@vger.kernel.org, linux-net@vger.kernel.org, netdev@oss.sgi.com Content-Disposition: inline Sender: netdev-bounce@oss.sgi.com Errors-to: netdev-bounce@oss.sgi.com List-Id: netdev.vger.kernel.org The VLAN net driver needs to return 0 from vlan_dev_change_mtu() on success. Signed-off-by: John W. Linville --- The proper sucessful return code for the change_mtu() method is zero. For some reason, vlan_dev_change_mtu() is returning the new mtu value instead. net/8021q/vlan_dev.c | 2 +- 1 files changed, 1 insertion(+), 1 deletion(-) --- 1.23/net/8021q/vlan_dev.c 2004-10-28 19:23:09 -04:00 +++ 1.24/net/8021q/vlan_dev.c 2004-11-18 11:12:36 -05:00 @@ -527,7 +527,7 @@ dev->mtu = new_mtu; - return new_mtu; + return 0; } int vlan_dev_set_ingress_priority(char *dev_name, __u32 skb_prio, short vlan_prio)