From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerin Jacob Subject: Re: [PATCH v3 4/4] eventdev: add device started attribute Date: Mon, 11 Sep 2017 23:22:43 +0530 Message-ID: <20170911175242.GB26002@jerin> References: <1504883894-43451-1-git-send-email-harry.van.haaren@intel.com> <1504885015-44642-1-git-send-email-harry.van.haaren@intel.com> <1504885015-44642-5-git-send-email-harry.van.haaren@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org To: Harry van Haaren Return-path: Received: from NAM01-SN1-obe.outbound.protection.outlook.com (mail-sn1nam01on0084.outbound.protection.outlook.com [104.47.32.84]) by dpdk.org (Postfix) with ESMTP id 883B516E for ; Mon, 11 Sep 2017 19:53:06 +0200 (CEST) Content-Disposition: inline In-Reply-To: <1504885015-44642-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" -----Original Message----- > Date: Fri, 8 Sep 2017 16:36:55 +0100 > From: Harry van Haaren > To: dev@dpdk.org > CC: jerin.jacob@caviumnetworks.com, Harry van Haaren > > Subject: [PATCH v3 4/4] eventdev: add device started attribute > X-Mailer: git-send-email 2.7.4 > > This commit adds an attribute to the eventdev, allowing applications > to retrieve if the eventdev is running or stopped. Note that no API > or ABI changes were required in adding the statistic, and code changes > are minimal. > > Signed-off-by: Harry van Haaren > --- > lib/librte_eventdev/rte_eventdev.c | 3 +++ > lib/librte_eventdev/rte_eventdev.h | 4 ++++ > 2 files changed, 7 insertions(+) > > diff --git a/lib/librte_eventdev/rte_eventdev.c b/lib/librte_eventdev/rte_eventdev.c > index 3756ec7..87a1e19 100644 > --- a/lib/librte_eventdev/rte_eventdev.c > +++ b/lib/librte_eventdev/rte_eventdev.c > @@ -740,6 +740,9 @@ rte_event_dev_attr_get(uint8_t dev_id, uint32_t attr_id, I think, the new rte_event_dev_attr_get function, we can move up so that all device specific functions comes back to back in API documentation. Check the output of "make doc-api-html" Other than that, it looks good. Acked-by: Jerin Jacob