[v5,0/6] ethdev port freeing
Message ID | 20181018012402.1240-1-thomas@monjalon.net (mailing list archive) |
---|---|
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 206407D34; Thu, 18 Oct 2018 03:24:11 +0200 (CEST) Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by dpdk.org (Postfix) with ESMTP id 8BD045F62 for <dev@dpdk.org>; Thu, 18 Oct 2018 03:24:07 +0200 (CEST) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id 020DA220FD; Wed, 17 Oct 2018 21:24:07 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute1.internal (MEProxy); Wed, 17 Oct 2018 21:24:07 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; s=mesmtp; bh=QBVuzwBQzH pYJPvFvYSm9QZoIObSTVfKkqHI7AWdsVo=; b=j/L/S7A41TqDR0rl5ZoHKkJIc0 f3OFEFrWl56HDA2ypjA7EtaSpbjGMO7D2Al43ZMZr30R1qXCJ5IFqnEDa8Qwt0fI t6CFqazeGvDeoVh1yhDdmaNq3Y02DBhB4WLlHUg1V4cgSAHU0vj3rW5N01D6j8+M ISwxd9kXLuCAHaV+o= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:date:from :in-reply-to:message-id:mime-version:references:subject:to :x-me-proxy:x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s= fm1; bh=QBVuzwBQzHpYJPvFvYSm9QZoIObSTVfKkqHI7AWdsVo=; b=THsZr4xR a/tl+Aez8fnMlcxY77pdM2vvhRqEC0Y8LPavuJEW9WnIfWByuRpcvG2AWKUK4Mwl 0Q30SHJT7xafzM3mIcqSXniXdng9j6A79VezaVXqDf62IqYSQmuGk1bGQsUKVNPg GYmdjBpUroFTGMM4cYmIURg3uXbxp4wsqLOQ7CmLcGqaUcfRfjxXAht5V0VQCEqJ lLJuxyZROb4+jI74+bzbfGjF0JpBp0dMIydGpGMKmh3S5SQUkmfatbbDDL3fr6Nv KlaFze+KRjDz3VoximUVSx9HIZ7D0Ii2ubNSMVmlwoWNU68ZCkaG2k+Q7Y3HY2sM 3q68vlvGj7BupA== X-ME-Sender: <xms:teDHW24saWFPaloy_cv6zVi8O2B6HDmTBGz1XS28Ha0skPj3wNfl6Q> X-ME-Proxy: <xmx:teDHW0qd-LQJTrm4e-fmQL1jkFvMXDiZLnEECC0ryOIJSuiI-wY--Q> <xmx:teDHWyjrRTLHVrTjn52rHh8y0xZrC574WaKKmiRLWMDh8BeNgyTOjA> <xmx:teDHW_IoWhWaoS9bKFKtGCEf26F38BWQXU3VxvZIpcTQ24ZTgEEg1A> <xmx:teDHW_i5WIR5E4wCJtrpmV92BES-cepb9lb3DHowm1nTa4E8czHnBg> <xmx:teDHW6jrZdF-9t73_KP8h167igr-HFOs96bt6dul8IOuXyJohI5Hhg> <xmx:tuDHW-sOEXfVJd3TIe4K59GVXMAPWWm06UcoLVEWySuBa-GQDqPmqg> Received: from xps.monjalon.net (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id DD121102EC; Wed, 17 Oct 2018 21:24:04 -0400 (EDT) From: Thomas Monjalon <thomas@monjalon.net> To: ferruh.yigit@intel.com, arybchenko@solarflare.com Cc: dev@dpdk.org, ophirmu@mellanox.com, bernard.iremonger@intel.com, rahul.lakkireddy@chelsio.com Date: Thu, 18 Oct 2018 03:23:56 +0200 Message-Id: <20181018012402.1240-1-thomas@monjalon.net> X-Mailer: git-send-email 2.19.0 In-Reply-To: <20180907233929.21950-1-thomas@monjalon.net> References: <20180907233929.21950-1-thomas@monjalon.net> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH v5 0/6] ethdev port freeing X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://mails.dpdk.org/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://mails.dpdk.org/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://mails.dpdk.org/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Message
Thomas Monjalon
Oct. 18, 2018, 1:23 a.m. UTC
The function rte_eth_dev_detach() is freeing a port and its underlying rte_device object. The issue is that we may have several ports associated to the same rte_device. The right replacement is to free the port, and free the rte_device if no more ports. At ethdev level, the public function for port freeing is rte_eth_dev_close(). The only miss is rte_eth_dev_release_port() to free all port resources. This patchset does a cleanup by moving as much ethdev data freeing as possible in an unique function rte_eth_dev_release_port(), and call this function in rte_eth_dev_close(). Changes in v5: - fix testpmd for detaching multi-ports device - fix testpmd for detaching only stopped ports - add a release note about closed port freeing and state - fix some doxygen comments in rte_eth_dev_data - improve doxygen of rte_eth_dev_release_port() - fix data freeing in some drivers Changes in v4: - do not free data fields which are not dynamically allocated blocks - remove rte_eth_dev_release_port_secondary() - remove testpmd check to detach without closing Changes in v3: - free queues, MAC addresses and private structure in rte_eth_dev_release_port. Thomas Monjalon (5): app/testpmd: allow detaching a port not closed ethdev: fix doxygen comments of shared data fields ethdev: free all common data when releasing port ethdev: remove release function for secondary process ethdev: complete closing of port Wisam Jaddo (1): app/testpmd: fix ports list after removing several at once app/test-pmd/testpmd.c | 64 +++++++++++++---------- doc/guides/rel_notes/release_18_11.rst | 4 ++ drivers/net/af_packet/rte_eth_af_packet.c | 11 ++-- drivers/net/ark/ark_ethdev.c | 1 - drivers/net/avf/avf_ethdev.c | 3 -- drivers/net/avp/avp_ethdev.c | 5 -- drivers/net/axgbe/axgbe_ethdev.c | 3 -- drivers/net/bnxt/bnxt_ethdev.c | 8 --- drivers/net/bonding/rte_eth_bond_pmd.c | 11 ++-- drivers/net/cxgbe/cxgbe_main.c | 16 +----- drivers/net/cxgbe/cxgbevf_main.c | 9 +--- drivers/net/dpaa/dpaa_ethdev.c | 10 ---- drivers/net/dpaa2/dpaa2_ethdev.c | 10 ---- drivers/net/e1000/em_ethdev.c | 3 -- drivers/net/e1000/igb_ethdev.c | 6 --- drivers/net/enetc/enetc_ethdev.c | 4 +- drivers/net/failsafe/failsafe.c | 6 ++- drivers/net/failsafe/failsafe_ops.c | 2 + drivers/net/fm10k/fm10k_ethdev.c | 8 --- drivers/net/i40e/i40e_ethdev.c | 3 -- drivers/net/i40e/i40e_ethdev_vf.c | 3 -- drivers/net/i40e/i40e_vf_representor.c | 5 +- drivers/net/ixgbe/ixgbe_ethdev.c | 9 ---- drivers/net/ixgbe/ixgbe_vf_representor.c | 5 +- drivers/net/kni/rte_eth_kni.c | 7 +-- drivers/net/liquidio/lio_ethdev.c | 3 -- drivers/net/mlx4/mlx4.c | 9 +++- drivers/net/mlx5/mlx5.c | 13 +++-- drivers/net/mvneta/mvneta_ethdev.c | 16 ++---- drivers/net/mvpp2/mrvl_ethdev.c | 16 ++---- drivers/net/netvsc/hn_ethdev.c | 20 +++---- drivers/net/null/rte_eth_null.c | 7 ++- drivers/net/octeontx/octeontx_ethdev.c | 31 +++++------ drivers/net/pcap/rte_eth_pcap.c | 19 ++++--- drivers/net/qede/qede_ethdev.c | 5 -- drivers/net/ring/rte_eth_ring.c | 6 +-- drivers/net/sfc/sfc_ethdev.c | 3 -- drivers/net/softnic/rte_eth_softnic.c | 10 ++-- drivers/net/szedata2/rte_eth_szedata2.c | 1 - drivers/net/tap/rte_eth_tap.c | 11 +++- drivers/net/vhost/rte_eth_vhost.c | 15 ++---- drivers/net/virtio/virtio_ethdev.c | 3 -- drivers/net/virtio/virtio_user_ethdev.c | 3 -- drivers/net/vmxnet3/vmxnet3_ethdev.c | 3 -- lib/librte_ethdev/rte_ethdev.c | 45 +++++----------- lib/librte_ethdev/rte_ethdev.h | 3 +- lib/librte_ethdev/rte_ethdev_core.h | 38 ++++++++------ lib/librte_ethdev/rte_ethdev_driver.h | 23 +++----- lib/librte_ethdev/rte_ethdev_pci.h | 17 ------ lib/librte_ethdev/rte_ethdev_version.map | 7 --- 50 files changed, 196 insertions(+), 347 deletions(-)