From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.tuxdriver.com (charlotte.tuxdriver.com [70.61.120.58]) by dpdk.org (Postfix) with ESMTP id D43A0B659 for ; Tue, 17 Feb 2015 16:53:02 +0100 (CET) Received: from [67.210.173.2] (helo=localhost) by smtp.tuxdriver.com with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.63) (envelope-from ) id 1YNkSE-0003k2-E0; Tue, 17 Feb 2015 10:53:01 -0500 Date: Tue, 17 Feb 2015 10:52:48 -0500 From: Neil Horman To: Zhou Danny Message-ID: <20150217155248.GB6309@neilslaptop.think-freely.org> References: <1424180839-24483-1-git-send-email-danny.zhou@intel.com> <1424180839-24483-2-git-send-email-danny.zhou@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1424180839-24483-2-git-send-email-danny.zhou@intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-Spam-Score: -2.9 (--) X-Spam-Status: No Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v3 1/5] ethdev: add rx interrupt enable/disable functions X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 17 Feb 2015 15:53:03 -0000 On Tue, Feb 17, 2015 at 09:47:15PM +0800, Zhou Danny wrote: > v3 changes > - Add return value for interrupt enable/disable functions > > Add two dev_ops functions to enable and disable rx queue interrupts > > Signed-off-by: Danny Zhou > Tested-by: Yong Liu > --- > lib/librte_ether/rte_ethdev.c | 43 ++++++++++++++++++++++++++++++++ > lib/librte_ether/rte_ethdev.h | 57 +++++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 100 insertions(+) > > diff --git a/lib/librte_ether/rte_ethdev.c b/lib/librte_ether/rte_ethdev.c > index ea3a1fb..d27469a 100644 > --- a/lib/librte_ether/rte_ethdev.c > +++ b/lib/librte_ether/rte_ethdev.c > @@ -2825,6 +2825,49 @@ _rte_eth_dev_callback_process(struct rte_eth_dev *dev, > } > rte_spinlock_unlock(&rte_eth_dev_cb_lock); > } > + > +int > +rte_eth_dev_rx_queue_intr_enable(uint8_t port_id, > + uint16_t queue_id) > +{ > + struct rte_eth_dev *dev; > + > + if (port_id >= nb_ports) { > + PMD_DEBUG_TRACE("Invalid port_id=%d\n", port_id); > + return (-ENODEV); > + } > + > + dev = &rte_eth_devices[port_id]; > + if (dev == NULL) { > + PMD_DEBUG_TRACE("Invalid port device\n"); > + return (-ENODEV); > + } > + > + FUNC_PTR_OR_ERR_RET(*dev->dev_ops->rx_queue_intr_enable, -ENOTSUP); > + return (*dev->dev_ops->rx_queue_intr_enable)(dev, queue_id); > +} > + > +int > +rte_eth_dev_rx_queue_intr_disable(uint8_t port_id, > + uint16_t queue_id) > +{ > + struct rte_eth_dev *dev; > + > + if (port_id >= nb_ports) { > + PMD_DEBUG_TRACE("Invalid port_id=%d\n", port_id); > + return (-ENODEV); > + } > + > + dev = &rte_eth_devices[port_id]; > + if (dev == NULL) { > + PMD_DEBUG_TRACE("Invalid port device\n"); > + return (-ENODEV); > + } > + > + FUNC_PTR_OR_ERR_RET(*dev->dev_ops->rx_queue_intr_disable, -ENOTSUP); > + return (*dev->dev_ops->rx_queue_intr_disable)(dev, queue_id); > +} > + > #ifdef RTE_NIC_BYPASS > int rte_eth_dev_bypass_init(uint8_t port_id) > { > diff --git a/lib/librte_ether/rte_ethdev.h b/lib/librte_ether/rte_ethdev.h > index 84160c3..0f320a9 100644 > --- a/lib/librte_ether/rte_ethdev.h > +++ b/lib/librte_ether/rte_ethdev.h > @@ -848,6 +848,8 @@ struct rte_eth_fdir { > struct rte_intr_conf { > /** enable/disable lsc interrupt. 0 (default) - disable, 1 enable */ > uint16_t lsc; > + /** enable/disable rxq interrupt. 0 (default) - disable, 1 enable */ > + uint16_t rxq; > }; > > /** > @@ -1109,6 +1111,14 @@ typedef int (*eth_tx_queue_setup_t)(struct rte_eth_dev *dev, > const struct rte_eth_txconf *tx_conf); > /**< @internal Setup a transmit queue of an Ethernet device. */ > > +typedef int (*eth_rx_enable_intr_t)(struct rte_eth_dev *dev, > + uint16_t rx_queue_id); > +/**< @internal Enable interrupt of a receive queue of an Ethernet device. */ > + > +typedef int (*eth_rx_disable_intr_t)(struct rte_eth_dev *dev, > + uint16_t rx_queue_id); > +/**< @internal Disable interrupt of a receive queue of an Ethernet device. */ > + > typedef void (*eth_queue_release_t)(void *queue); > /**< @internal Release memory resources allocated by given RX/TX queue. */ > > @@ -1445,6 +1455,8 @@ struct eth_dev_ops { > eth_queue_start_t tx_queue_start;/**< Start TX for a queue.*/ > eth_queue_stop_t tx_queue_stop;/**< Stop TX for a queue.*/ > eth_rx_queue_setup_t rx_queue_setup;/**< Set up device RX queue.*/ > + eth_rx_enable_intr_t rx_queue_intr_enable; /**< Enable Rx queue interrupt. */ > + eth_rx_disable_intr_t rx_queue_intr_disable; /**< Disable Rx queue interrupt.*/ > eth_queue_release_t rx_queue_release;/**< Release RX queue.*/ > eth_rx_queue_count_t rx_queue_count; /**< Get Rx queue count. */ > eth_rx_descriptor_done_t rx_descriptor_done; /**< Check rxd DD bit */ > @@ -2811,6 +2823,51 @@ void _rte_eth_dev_callback_process(struct rte_eth_dev *dev, > enum rte_eth_event_type event); > > /** > + * When there is no rx packet coming in Rx Queue for a long time, we can > + * sleep lcore related to RX Queue for power saving, and enable rx interrupt > + * to be triggered when rx packect arrives. > + * > + * The rte_eth_dev_rx_queue_intr_enable() function enables rx queue > + * interrupt on specific rx queue of a port. > + * > + * @param port_id > + * The port identifier of the Ethernet device. > + * @param queue_id > + * The index of the receive queue from which to retrieve input packets. > + * The value must be in the range [0, nb_rx_queue - 1] previously supplied > + * to rte_eth_dev_configure(). > + * @return > + * - (0) if successful. > + * - (-ENOTSUP) if underlying hardware OR driver doesn't support > + * that operation. > + * - (-ENODEV) if *port_id* invalid. > + */ > +int rte_eth_dev_rx_queue_intr_enable(uint8_t port_id, > + uint16_t queue_id); > + > +/** > + * When lcore wakes up from rx interrupt indicating packet coming, disable rx > + * interrupt and returns to polling mode. > + * > + * The rte_eth_dev_rx_queue_intr_disable() function disables rx queue > + * interrupt on specific rx queue of a port. > + * > + * @param port_id > + * The port identifier of the Ethernet device. > + * @param queue_id > + * The index of the receive queue from which to retrieve input packets. > + * The value must be in the range [0, nb_rx_queue - 1] previously supplied > + * to rte_eth_dev_configure(). > + * @return > + * - (0) if successful. > + * - (-ENOTSUP) if underlying hardware OR driver doesn't support > + * that operation. > + * - (-ENODEV) if *port_id* invalid. > + */ > +int rte_eth_dev_rx_queue_intr_disable(uint8_t port_id, > + uint16_t queue_id); > + > +/** > * Turn on the LED on the Ethernet device. > * This function turns on the LED on the Ethernet device. > * > -- > 1.8.1.4 > > Need to export these functions Neil