From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out5-smtp.messagingengine.com (out5-smtp.messagingengine.com [66.111.4.29]) by dpdk.org (Postfix) with ESMTP id 37D972904 for ; Fri, 21 Apr 2017 00:45:15 +0200 (CEST) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id CA1F6207FC; Thu, 20 Apr 2017 18:45:14 -0400 (EDT) Received: from frontend2 ([10.202.2.161]) by compute1.internal (MEProxy); Thu, 20 Apr 2017 18:45:14 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-sender :x-me-sender:x-sasl-enc:x-sasl-enc; s=mesmtp; bh=tedBBp0AtHmU0uf BIJchEktxTiULWKMnuUZ4GaLrAqY=; b=e4Oz4WlyydCB1N4itTNnWPAbO6p5KzP 7bHHSNLuzkVjm6fKC3P9EkOfaXpW/2Pa8mDmf0/u4Pzq4wFz1qjGFeSQ4uoMP1/9 PkzmamFq+cJblz0zvJ+6bT/VYzsxtQp54BMAF58Uv5qL3UqJJDPqXPG5wfFLWsxN WLOs27zmUpdI= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-sender:x-me-sender:x-sasl-enc:x-sasl-enc; s= fm1; bh=tedBBp0AtHmU0ufBIJchEktxTiULWKMnuUZ4GaLrAqY=; b=Z1StSCr8 M2LPV9jM9z6ydbaXuYlzcN2eBuYjz7FmjAtILKDluKKeYEsfcrZ94PPcdIS+kZdz W+WGh7qP616vmjdsNYBqB9r3OX4gu0qsfJmjWEkzacxrC8zHecYjB1+iRi7HQE9O wqKhKAWFlTG7jhLhjyyBxn0/lHKvyoMy7SBlbJMt3AyVTomLxjC1GF03BzR5gw7L O80EB2zUbNwtq/dVyhdE0HjbXyVZVfLiMRfpIYjUgWm0jpJQbPrmM98UhHS3r3QC gVkXk7cBDy1sA4fYKBFO1b80R7X6HhU0ge/9hj6r4VRhcbfTU8PKFKRuaE0SGxGP UcBZUqHt+vw2Dw== X-ME-Sender: X-Sasl-enc: Ez+SY0p4M37z0eEpe68OAJ0xCCiztwFkCAu9S8Gh0GEB 1492728314 Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id 783A1245DF; Thu, 20 Apr 2017 18:45:14 -0400 (EDT) From: Thomas Monjalon To: Gaetan Rivet Cc: dev@dpdk.org, Jingjing Wu , Adrien Mazarguil , Nelio Laranjeiro Date: Fri, 21 Apr 2017 00:45:13 +0200 Message-ID: <1745480.obotDYhibW@xps> In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH v2 0/5] add device removal event 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: Thu, 20 Apr 2017 22:45:15 -0000 18/04/2017 14:17, Gaetan Rivet: > This new event represents the sudden removal of a device from its bus. > The underlying resources exposed by the bus are expected not to be available > anymore. The application should thus be able to react and possibly clean up > related resources that it reserved for the removed device. > > This event is different from the current hotplug API available in the DPDK > for two reasons: > > 1. It is a reactive design: the application reacts to a device that has been > removed instead of removing a device from its pool. > > 2. The event itself is going further than the current detaching of a device > from a DPDK application. If the bus is a hardware one, it is expected of > the underlying resources to not be available anymore. > > This series adds a new event type to ethdev and implements it in mlx4. > Testpmd is also updated to report all asynchronous ethdev events including > this one for testing purposes and as a practical usage example. > > This series depends on the series titled > [PATCH 1/2] net/mlx4: split the definitions to the header file > > v1 --> v2: > * integrated the series with the new PCI rte_bus implementation. > > I planned on working out a more generic implementation of the RMV event > for the v17.05, however while writing it I found that I had to evolve the > support of interrupts in the PCI rte_bus, which did not seem wise at this > point of the release cycle. > > I consider that this event should be generalized along the LSC event, > going from pure PCI events to generic rte_bus events, given that they can > be relevant to other busses (vdev at least). I would reserve this evolution > for future release however, once a real discussion has taken place. > > Gaetan Rivet (5): > ethdev: introduce device removal event > net/mlx4: device removal event support > app/testpmd: generic event handler > app/testpmd: request link status interrupt > app/testpmd: request device removal interrupt Applied, thanks