From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerin Jacob Subject: Re: [PATCH v2 04/15] event/sw: add device capabilities function Date: Mon, 6 Feb 2017 14:04:29 +0530 Message-ID: <20170206083428.GE25242@localhost.localdomain> References: <1484580885-148524-1-git-send-email-harry.van.haaren@intel.com> <1485879273-86228-1-git-send-email-harry.van.haaren@intel.com> <1485879273-86228-5-git-send-email-harry.van.haaren@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Cc: , Bruce Richardson To: Harry van Haaren Return-path: Received: from NAM01-BN3-obe.outbound.protection.outlook.com (mail-bn3nam01on0083.outbound.protection.outlook.com [104.47.33.83]) by dpdk.org (Postfix) with ESMTP id 10D69293B for ; Mon, 6 Feb 2017 09:34:50 +0100 (CET) Content-Disposition: inline In-Reply-To: <1485879273-86228-5-git-send-email-harry.van.haaren@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, Jan 31, 2017 at 04:14:22PM +0000, Harry van Haaren wrote: > From: Bruce Richardson > > Add in the info_get function to return details on the queues, flow, > prioritization capabilities, etc. that this device has. > > Signed-off-by: Bruce Richardson > Signed-off-by: Harry van Haaren > --- > drivers/event/sw/sw_evdev.c | 23 +++++++++++++++++++++++ > drivers/event/sw/sw_evdev.h | 10 ++++++++++ > 2 files changed, 33 insertions(+) > > diff --git a/drivers/event/sw/sw_evdev.c b/drivers/event/sw/sw_evdev.c > index d60f00f..4dca4cf 100644 > --- a/drivers/event/sw/sw_evdev.c > +++ b/drivers/event/sw/sw_evdev.c > @@ -44,6 +44,28 @@ > #define SCHED_QUANTA_ARG "sched_quanta" > #define CREDIT_QUANTA_ARG "credit_quanta" > > +static void > +sw_info_get(struct rte_eventdev *dev, struct rte_event_dev_info *info) > +{ > + RTE_SET_USED(dev); > + > + static const struct rte_event_dev_info evdev_sw_info = { > + .driver_name = PMD_NAME, > + .max_event_queues = RTE_EVENT_MAX_QUEUES_PER_DEV, > + .max_event_queue_flows = SW_QID_NUM_FIDS, > + .max_event_queue_priority_levels = SW_Q_PRIORITY_MAX, > + .max_event_priority_levels = SW_IQS_MAX, > + .max_event_ports = SW_PORTS_MAX, > + .max_event_port_dequeue_depth = MAX_SW_CONS_Q_DEPTH, > + .max_event_port_enqueue_depth = MAX_SW_PROD_Q_DEPTH, > + .max_num_events = SW_INFLIGHT_EVENTS_TOTAL, > + .event_dev_cap = (RTE_EVENT_DEV_CAP_QUEUE_QOS | > + RTE_EVENT_DEV_CAP_EVENT_QOS), > + }; > + > + *info = evdev_sw_info; > +} > + > static int > assign_numa_node(const char *key __rte_unused, const char *value, void *opaque) > { > @@ -78,6 +100,7 @@ static int > sw_probe(const char *name, const char *params) > { > static const struct rte_eventdev_ops evdev_sw_ops = { > + .dev_infos_get = sw_info_get, > }; > > static const char *const args[] = { > diff --git a/drivers/event/sw/sw_evdev.h b/drivers/event/sw/sw_evdev.h > index 9494659..6e3cb36 100644 > --- a/drivers/event/sw/sw_evdev.h > +++ b/drivers/event/sw/sw_evdev.h > @@ -36,8 +36,18 @@ > #include > #include > > +#define PMD_NAME "event_sw" Better to add SW_ name space. > + > #define SW_DEFAULT_CREDIT_QUANTA 32 > #define SW_DEFAULT_SCHED_QUANTA 128 > +#define SW_QID_NUM_FIDS 16384 > +#define SW_IQS_MAX 4 > +#define SW_Q_PRIORITY_MAX 255 > +#define SW_PORTS_MAX 64 > +#define MAX_SW_CONS_Q_DEPTH 128 > +#define SW_INFLIGHT_EVENTS_TOTAL 4096 > +/* allow for lots of over-provisioning */ > +#define MAX_SW_PROD_Q_DEPTH 4096 > > struct sw_evdev { > struct rte_eventdev_data *data; > -- > 2.7.4 >