From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Monjalon <thomas@monjalon.net>
Subject: Re: [PATCH v3 2/2] net/i40e: add hot plug monitor in i40e
Date: Fri, 07 Jul 2017 12:17:40 +0200
Message-ID: <7030258.umMgtiJvDN@xps>
References: <1498648044-57541-2-git-send-email-jia.guo@intel.com>
 <1498712510-44217-3-git-send-email-jia.guo@intel.com>
 <1888554.fe4JxUsZG9@xps>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7Bit
Cc: dev@dpdk.org, helin.zhang@intel.com, jingjing.wu@intel.com,
 bruce.richardson@intel.com, konstantin.ananyev@intel.com,
 gaetan.rivet@6wind.com, thomas.monjalon@6wind.com, ferruh.yigit@intel.com
To: Jeff Guo <jia.guo@intel.com>
Return-path: <dev-bounces@dpdk.org>
Received: from out2-smtp.messagingengine.com (out2-smtp.messagingengine.com
 [66.111.4.26]) by dpdk.org (Postfix) with ESMTP id 20BC53772
 for <dev@dpdk.org>; Fri,  7 Jul 2017 12:17:42 +0200 (CEST)
In-Reply-To: <1888554.fe4JxUsZG9@xps>
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

07/07/2017 09:56, Thomas Monjalon:
> 29/06/2017 07:01, Jeff Guo:
> > --- a/drivers/net/i40e/i40e_ethdev.c
> > +++ b/drivers/net/i40e/i40e_ethdev.c
> > @@ -1283,6 +1283,7 @@ static inline void i40e_GLQF_reg_init(struct i40e_hw *hw)
> >  
> >  	/* enable uio intr after callback register */
> >  	rte_intr_enable(intr_handle);
> > +
> >  	/*
> >  	 * Add an ethertype filter to drop all flow control frames transmitted
> >  	 * from VSIs. By doing so, we stop VF from sending out PAUSE or PFC
> > @@ -5832,11 +5833,29 @@ struct i40e_vsi *
> >  {
> >  	struct rte_eth_dev *dev = (struct rte_eth_dev *)param;
> >  	struct i40e_hw *hw = I40E_DEV_PRIVATE_TO_HW(dev->data->dev_private);
> > +	struct rte_uevent event;
> >  	uint32_t icr0;
> > +	struct rte_pci_device *pci_dev;
> > +	struct rte_intr_handle *intr_handle;
> > +
> > +	pci_dev = RTE_ETH_DEV_TO_PCI(dev);
> > +	intr_handle = &pci_dev->intr_handle;
> >  
> >  	/* Disable interrupt */
> >  	i40e_pf_disable_irq0(hw);
> >  
> > +	/* check device uevent */
> > +	if (rte_uevent_get(intr_handle->uevent_fd, &event) == 0) {
> > +		if (event.subsystem == RTE_UEVENT_SUBSYSTEM_UIO) {
> > +			if (event.action == RTE_UEVENT_REMOVE) {
> > +				_rte_eth_dev_callback_process(dev,
> > +					RTE_ETH_EVENT_INTR_RMV, NULL);
> > +				return;
> > +			}
> > +		}
> > +		goto done;
> > +	}
> 
> There is nothing specific to i40e in this patch.
> It seems wrong to add such generic code in every drivers.

It should be managed at bus layer and not be specific to ethdev only.