From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752077AbcABRzk (ORCPT ); Sat, 2 Jan 2016 12:55:40 -0500 Received: from mout.web.de ([212.227.15.14]:57353 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751792AbcABRzh (ORCPT ); Sat, 2 Jan 2016 12:55:37 -0500 Subject: [PATCH 2/5] xen-netback: Delete an unnecessary goto statement in connect_rings() To: xen-devel@lists.xenproject.org, netdev@vger.kernel.org, Ian Campbell , Wei Liu References: <566ABCD9.1060404@users.sourceforge.net> <56880DD4.2090806@users.sourceforge.net> Cc: LKML , kernel-janitors@vger.kernel.org, Julia Lawall From: SF Markus Elfring Message-ID: <56880F11.7030303@users.sourceforge.net> Date: Sat, 2 Jan 2016 18:55:29 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 In-Reply-To: <56880DD4.2090806@users.sourceforge.net> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:sOThxM3kN6ruz+XQLLjt1f//uR094S6+DAWmamy5phNc2nzV4Eg RLsa9qps1Z2sf6IyLZRX9AtGHqT4voBghCBgp0gFql/vx31nlvNlIaBExX1bzju9xsQ3tMh pOZRolxIPhiVx+JrjMWacUvLzqegeFu1+ZxiAz8of2aL7e2OtUsUfVRuoz41pSuZpyuHBzs hEmaAchll49SM4ytFXxkw== X-UI-Out-Filterresults: notjunk:1;V01:K0:5iDKCvW9JsY=:EF1Cf/pCGqxMjWnln8B3np mPsTXQcPppEz/VapWWI4th8C3g+kxNgRiGMisz398kzkZgzOP7B8AhYg7Bj3uNMaIWTamwv2Z zb1mcCUJ12LnBic2sEVyYFuvchB/T3FoxFt1DIuVKjYLXbzxvxJWkceesmJAdy+afZVni+FqJ 9R2nHDMcu6qIBd19PNzI2DlXPW4GeRJEtxBYHknZ9L9gFrR0ekL/l2TOshnGbW4XKl5JV12Na 9i3XF3NcB8nRXpXzn2Kj8qB3Nn5+UXuFGXF/3n2jFiv2HOBrrHo93AwoiMaWaHlSbTslzEx2L BsUCh7hHIEkn/ecxFeuKFyfXwNsoRzrRRpqPIQyRJaeFIFeU2777S9Z8ZxSt3FO2mh+YBS2+i SPVxcIMqLhPIkRzz/abS8zT5kXwFXgZiK94V9MQe8YmYK4aBaUL6jtLMaT6UMu3ErruVodbN4 1y3ftzOXz0qXM9+kayktCSSiMi42IGpxQEmd8aMqiPekRTcSNeMK4+9lJ49gy5FK/co/Qfc6O hPzlRCfisbBOQMxJpl+O8V06OKaoqT2sRsVCP+xyDIPTQfRCrIZtauGRA8B+EOrhbK299VlRH KQ/eZmlv28gZpnkC5mBieo+NZB0L7e9uue5wF9+sCZwu3ILTH515GMKA4HEz0HUofl3JgTty3 MQy4kaYEFsek8OapBRqfCpGeXuuxmiC4BjGYrvf0tAnOsrgPqPENo5/Cwts30vRlvfOElLuja UBw+FopZvBVMKMFMWYXESWnnl5sxhuUvl2ptFPPJAfHee5I9efFxHqXxiJBGC6dCVTMwx/1HQ MQYQgCj Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Sat, 2 Jan 2016 17:50:21 +0100 One goto statement referred to a source code position directly behind it. Thus omit such an unnecessary jump. Signed-off-by: Markus Elfring --- drivers/net/xen-netback/xenbus.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/net/xen-netback/xenbus.c b/drivers/net/xen-netback/xenbus.c index 7f2895d..d4947e1 100644 --- a/drivers/net/xen-netback/xenbus.c +++ b/drivers/net/xen-netback/xenbus.c @@ -933,13 +933,11 @@ static int connect_rings(struct backend_info *be, struct xenvif_queue *queue) /* Map the shared frame, irq etc. */ err = xenvif_connect(queue, tx_ring_ref, rx_ring_ref, tx_evtchn, rx_evtchn); - if (err) { + if (err) xenbus_dev_fatal(dev, err, "mapping shared-frames %lu/%lu port tx %u rx %u", tx_ring_ref, rx_ring_ref, tx_evtchn, rx_evtchn); - goto err; - } err: /* Regular return falls through with err == 0 */ kfree(xspath); -- 2.6.3