From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gaetan Rivet Subject: [PATCH v2 0/6] Remove RTE_ETH_DEV_DETACHABLE Date: Tue, 24 Oct 2017 12:35:36 +0200 Message-ID: References: Cc: Gaetan Rivet , Ajit Khaparde , Alejandro Lucero , Beilei Xing , Bruce Richardson , Cristian Dumitrescu , Declan Doherty , Ferruh Yigit , Jing Chen , John Miller , "John W. Linville" , Konstantin Ananyev , Matt Peters , Nelio Laranjeiro , Pascal Mazon , Shrikrishna Khare , Srisivasubramanian Srinivasan , Tetsuya Mukawa , Wenzhuo Lu , Yongseok Koh To: dev@dpdk.org Return-path: Received: from mail-wm0-f68.google.com (mail-wm0-f68.google.com [74.125.82.68]) by dpdk.org (Postfix) with ESMTP id 2B4EF1B79A for ; Tue, 24 Oct 2017 12:36:00 +0200 (CEST) Received: by mail-wm0-f68.google.com with SMTP id 78so12430367wmb.1 for ; Tue, 24 Oct 2017 03:36:00 -0700 (PDT) In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" This flag was deprecated in the previous release, and this is its removal. We are going from an explicit support of device detach, to an implicit one, with specific checks in place for PMDs not supporting the feature. Attempting to Cc all relevant maintainers: Cc: Ajit Khaparde Cc: Alejandro Lucero Cc: Beilei Xing Cc: Bruce Richardson Cc: Cristian Dumitrescu Cc: Declan Doherty Cc: Ferruh Yigit Cc: Jing Chen Cc: John Miller Cc: John W. Linville Cc: Konstantin Ananyev Cc: Matt Peters Cc: Nelio Laranjeiro Cc: Pascal Mazon Cc: Shrikrishna Khare Cc: Srisivasubramanian Srinivasan Cc: Tetsuya Mukawa Cc: Wenzhuo Lu Cc: Yongseok Koh Sorry for any undue ML traffic going your way if you are not interested in this series. I'd like to highlight Alejandro Lucero for the NFP PMD, as the patch is somewhat more involved for this PMD and I don't have the hardware available to validate the change. v2: + Split ether patch: First remove the use of the flag, then remove the flag + Restore RTE_ETH_DEV_BONDED_SLAVE check, previously hastily removed along the DETACHABLE one. + Added release note, removed deprecation notice Gaetan Rivet (6): ethdev: do not rely on detachable flag in detach ethdev: remove detachable device flag pci: propagate PMD removal error value for unplug doc: follow detachable flag disappearance doc: update detachable flag deprecation notice doc: note ether API change for detachable flag doc/guides/prog_guide/port_hotplug_framework.rst | 8 ++------ doc/guides/rel_notes/deprecation.rst | 8 -------- doc/guides/rel_notes/release_17_11.rst | 6 ++++++ drivers/net/af_packet/rte_eth_af_packet.c | 1 - drivers/net/ark/ark_ethdev.c | 1 - drivers/net/avp/avp_ethdev.c | 2 -- drivers/net/bnxt/bnxt_ethdev.c | 1 - drivers/net/bonding/rte_eth_bond_pmd.c | 3 +-- drivers/net/e1000/em_ethdev.c | 1 - drivers/net/e1000/igb_ethdev.c | 2 -- drivers/net/fm10k/fm10k_ethdev.c | 1 - drivers/net/i40e/i40e_ethdev.c | 1 - drivers/net/i40e/i40e_ethdev_vf.c | 1 - drivers/net/ixgbe/ixgbe_ethdev.c | 2 -- drivers/net/kni/rte_eth_kni.c | 2 -- drivers/net/liquidio/lio_ethdev.c | 1 - drivers/net/mlx4/mlx4.c | 1 - drivers/net/mlx5/mlx5.c | 1 - drivers/net/nfp/nfp_net.c | 19 ++++++++++++++--- drivers/net/null/rte_eth_null.c | 2 -- drivers/net/pcap/rte_eth_pcap.c | 1 - drivers/net/ring/rte_eth_ring.c | 1 - drivers/net/sfc/sfc_ethdev.c | 2 -- drivers/net/softnic/rte_eth_softnic.c | 1 - drivers/net/tap/rte_eth_tap.c | 2 +- drivers/net/vhost/rte_eth_vhost.c | 3 +-- drivers/net/virtio/virtio_ethdev.c | 1 - drivers/net/virtio/virtio_user_ethdev.c | 1 - drivers/net/vmxnet3/vmxnet3_ethdev.c | 1 - lib/librte_eal/common/eal_common_pci.c | 14 +++++++++---- lib/librte_ether/rte_ethdev.c | 26 ++++++++---------------- lib/librte_ether/rte_ethdev.h | 2 -- 32 files changed, 46 insertions(+), 73 deletions(-) -- 2.1.4