From patchwork Thu Aug 14 13:19:47 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: chas williams - CONTRACTOR X-Patchwork-Id: 379930 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 37CF91400B5 for ; Thu, 14 Aug 2014 23:20:14 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754334AbaHNNUH (ORCPT ); Thu, 14 Aug 2014 09:20:07 -0400 Received: from hedwig.cmf.nrl.navy.mil ([134.207.12.162]:35771 "EHLO hedwig.cmf.nrl.navy.mil" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753902AbaHNNUG convert rfc822-to-8bit (ORCPT ); Thu, 14 Aug 2014 09:20:06 -0400 Received: from thirdoffive.cmf.nrl.navy.mil ([IPv6:2001:480:23:c:2e0:81ff:fe78:9314]) by hedwig.cmf.nrl.navy.mil (8.14.2/8.14.2) with ESMTP id s7EDJmgL026974 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT); Thu, 14 Aug 2014 09:19:49 -0400 Received: from thirdoffive.cmf.nrl.navy.mil (localhost [127.0.0.1]) by thirdoffive.cmf.nrl.navy.mil (8.14.7/8.14.4) with ESMTP id s7EDJl9l031767; Thu, 14 Aug 2014 09:19:47 -0400 Message-Id: <201408141319.s7EDJl9l031767@thirdoffive.cmf.nrl.navy.mil> From: Chas Williams - CONTRACTOR To: netdev@vger.kernel.org, linux-atm-general@lists.sourceforge.net Subject: [PATCH net-next] lec: Use rtnl lock/unlock when updating MTU MIME-Version: 1.0 Content-ID: <31765.1408022387.1@thirdoffive.cmf.nrl.navy.mil> Date: Thu, 14 Aug 2014 09:19:47 -0400 X-NRLCMF-Spam-Score: () hits=-0.001 X-NRLCMF-Virus-Scanned: No virus found X-Scanned-By: MIMEDefang 2.68 on IPv6:2001:480:23:c::13 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org The LECS response contains the MTU that should be used. Correctly synchronize with other layers when updating. Signed-off-by: Chas Williams - CONTRACTOR --- net/atm/lec.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/net/atm/lec.c b/net/atm/lec.c index 4c5b8ba..6e13369 100644 --- a/net/atm/lec.c +++ b/net/atm/lec.c @@ -410,9 +410,11 @@ static int lec_atm_send(struct atm_vcc *vcc, struct sk_buff *skb) priv->lane2_ops = NULL; if (priv->lane_version > 1) priv->lane2_ops = &lane2_ops; + rtnl_lock(); if (dev_set_mtu(dev, mesg->content.config.mtu)) pr_info("%s: change_mtu to %d failed\n", dev->name, mesg->content.config.mtu); + rtnl_unlock(); priv->is_proxy = mesg->content.config.is_proxy; break; case l_flush_tran_id: