From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 59E61A04DD for ; Thu, 29 Oct 2020 21:10:57 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id D3A1AC9C4; Thu, 29 Oct 2020 21:10:55 +0100 (CET) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by dpdk.org (Postfix) with ESMTP id AE577C99C; Thu, 29 Oct 2020 21:10:51 +0100 (CET) IronPort-SDR: Mm1nKZQELt2QN6H5WtvyVvVfnDZ/hC4QQJXiV4722RoBbfFVxFU+e3HY2yQtFjk2sS2SCtDRG1 /5ssXx0amcVA== X-IronPort-AV: E=McAfee;i="6000,8403,9789"; a="156278306" X-IronPort-AV: E=Sophos;i="5.77,430,1596524400"; d="scan'208";a="156278306" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Oct 2020 13:10:49 -0700 IronPort-SDR: OmtIKaNKF2jrBrg9Gg+iRKfkOBV7GJ7dxadwoCWOQXdEPFi2hiJhpN6u4oGqiNnOq68CKPAv4W qt20Uk4pEIVQ== X-IronPort-AV: E=Sophos;i="5.77,430,1596524400"; d="scan'208";a="536801317" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.213.251.183]) ([10.213.251.183]) by orsmga005-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Oct 2020 13:10:48 -0700 To: Matan Azrad , Long Li , Matan Azrad Cc: "dev@dpdk.org" , NBU-Contact-longli , "stable@dpdk.org" References: <1603354294-16610-1-git-send-email-longli@linuxonhyperv.com> From: Ferruh Yigit Message-ID: Date: Thu, 29 Oct 2020 20:10:44 +0000 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-stable] [Patch v2] net/vdev_netvsc: prevent alarm loss on failed device probe X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" On 10/26/2020 7:14 AM, Matan Azrad wrote: > Hi Long > > From: Long Li >> If a device probe fails, the alarm is canceled and will no longer work for >> previously probed devices. >> >> Fix this by checking if alarm is necessary at the end of each device probe. >> Reset the alarm if there are vdev_netvsc_ctx created. >> >> Change log: >> v2: removed lock and flags, use counter to decide if alarm should be reset >> >> Cc: stable@dpdk.org >> Signed-off-by: Long Li > > I suggest the next title: > net/vdev_netvsc: fix device probing error flow > > and the next fixes line: > Fixes: e7dc5d7becc5 ("net/vdev_netvsc: implement core functionality") > > Acked-by: Matan Azrad > Applied to dpdk-next-net/main, thanks. (Used suggested title)