From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f193.google.com (mail-wr0-f193.google.com [209.85.128.193]) by dpdk.org (Postfix) with ESMTP id 22D5B1B2C6 for ; Tue, 13 Feb 2018 14:32:06 +0100 (CET) Received: by mail-wr0-f193.google.com with SMTP id t94so18607301wrc.5 for ; Tue, 13 Feb 2018 05:32:06 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to :user-agent; bh=WnDwDtO/p3riPefcozKfNTLVuhBgYNe8oDSiCYgTO1Y=; b=K8Dt+x0wgPxY0JdLc/9hItI7HS14roJOcBzXobxjtQuzrnRQH9fH44Fqo9GQ8it7/h jDuJNpKTEvyWxlmaO91J/oA2P2PJVvAoC9fkXsgQHKxCtPMfu+02v8NvHN9ROJPzrw+a 63IwbgnuEJZ+GlQrQwCKOYXfJtJb7Wdrv73Hxi+3UrWQTrPa/dbWZqrn/aJfREB27+Ia 1WYGLb2OUEUB6E0K0ehy87DduXZUQ+0flf27qdyeguuiTg/ZXP/a/9WDw/gA7HTENCr7 QRigbRynGzlP9u5seWMkISs1sxZ93C3iLDJcts0aYP07sxgEq9z2d38/CCqET+Tk8WWd vlGA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to:user-agent; bh=WnDwDtO/p3riPefcozKfNTLVuhBgYNe8oDSiCYgTO1Y=; b=fcnfCkJv3dFr6/qc4yqj7Fpay9JNwUvX3F3N8BEnv0rOC8N7wUu2TFMJQ9E8MzUxL0 OPJln3EZJL8JMGsk8swNTI2WP/mBYjFith5lOotkgv71FglOCYon7QrKzv9/edhyub1N dk2fOhyqNzsPzfdHOgI1bJeMmDvmErXw7stZ0mwa+WPLgfV1594FGhLmVsW6ByRBLv7d tYz3MqGnrjam0bDZL5yepMIF1lfaI4HjBpLpjarnwtcRPNMrpwCaQAZJyyohoDEX9eU6 Km6diUyeXRhpPGJyJXwGPSyXaVVbbM4v2JrksG69KBGCkFnU6sPSK8LZtIGB4dy45c8S Jsng== X-Gm-Message-State: APf1xPDJPWnzbkX9oSFqrUTgOj7CPtlVxIYGwPig3y5Eg8X4o8QASs5c lov7WMhlvto+ZEy2QC1LD8QLiGGw X-Google-Smtp-Source: AH8x225rkTInkZdliEyiBzTk0Ivxi6SRGxRsGrlSizLp6pB7Bcb9VPQ2CQFXTdmkZcYQy8IU4gPMRw== X-Received: by 10.223.163.142 with SMTP id l14mr1225498wrb.277.1518528726418; Tue, 13 Feb 2018 05:32:06 -0800 (PST) Received: from bidouze.vm.6wind.com (host.78.145.23.62.rev.coltfrance.com. [62.23.145.78]) by smtp.gmail.com with ESMTPSA id a137sm6562032wme.15.2018.02.13.05.32.05 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Tue, 13 Feb 2018 05:32:05 -0800 (PST) Date: Tue, 13 Feb 2018 14:31:52 +0100 From: =?iso-8859-1?Q?Ga=EBtan?= Rivet To: Matan Azrad Cc: dev@dpdk.org Message-ID: <20180213133152.gmz35ovsbkrrkkef@bidouze.vm.6wind.com> References: <1518369872-12324-1-git-send-email-matan@mellanox.com> <1518468702-16719-1-git-send-email-matan@mellanox.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1518468702-16719-1-git-send-email-matan@mellanox.com> User-Agent: NeoMutt/20170113 (1.7.2) Subject: Re: [dpdk-dev] [PATCH v7 0/3] failsafe: fix hotplug races X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 13 Feb 2018 13:32:07 -0000 Hi Matan, On Mon, Feb 12, 2018 at 08:51:39PM +0000, Matan Azrad wrote: > This series fixes failsafe race between control commands to the asynchronic plug-out\in processes. > Thanks for tackling this complicated issue. For the series: Acked-by: Gaetan Rivet I hope there will be more work on this matter soon. > V7(matan): > improve commit logs. > return back emty line. > return back description wrongly removed. > > V6(matan): > Full lock based fix. > Change the remove flag scope until SW resources release. > > v5(Matan): > Change defines names to failsafe convention (UNSAFE). > split a fix patch. > > v4(Matan): > Rebase on top of 18.02-rc3. > Extend the fix for other control commands. > Fix hotplug alarm cancel. > > V3(Ophir): > Rebase v2. > Add rationales (copy from an email which accompanied v2). > > > > Matan Azrad (3): > net/failsafe: fix hotplug alarm cancel > net/failsafe: fix removal scope > net/failsafe: fix hotplug races > > drivers/net/failsafe/Makefile | 1 + > drivers/net/failsafe/failsafe.c | 53 +++++++++--- > drivers/net/failsafe/failsafe_ether.c | 6 ++ > drivers/net/failsafe/failsafe_flow.c | 20 ++++- > drivers/net/failsafe/failsafe_ops.c | 149 ++++++++++++++++++++++++++------ > drivers/net/failsafe/failsafe_private.h | 56 ++++++++++++ > 6 files changed, 248 insertions(+), 37 deletions(-) > > -- > 1.9.5 > -- Gaëtan Rivet 6WIND