From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerin Jacob Subject: Re: [PATCH v2 3/5] test: add distributor_perf autotest Date: Thu, 22 Dec 2016 17:49:44 +0530 Message-ID: <20161222121943.GA8778@localhost.localdomain> References: <1480567821-70846-2-git-send-email-david.hunt@intel.com> <1482381428-148094-1-git-send-email-david.hunt@intel.com> <1482381428-148094-4-git-send-email-david.hunt@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Cc: , To: David Hunt Return-path: Received: from NAM03-CO1-obe.outbound.protection.outlook.com (mail-co1nam03on0060.outbound.protection.outlook.com [104.47.40.60]) by dpdk.org (Postfix) with ESMTP id BFEB42C4B for ; Thu, 22 Dec 2016 13:20:05 +0100 (CET) Content-Disposition: inline In-Reply-To: <1482381428-148094-4-git-send-email-david.hunt@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 Thu, Dec 22, 2016 at 04:37:06AM +0000, David Hunt wrote: > Signed-off-by: David Hunt > --- > + * it does nothing but return packets and count them. > + */ > +static int > +handle_work_burst(void *arg) > +{ > + //struct rte_mbuf *pkt = NULL; Seems like their is lot test code with // in this file. Please remove it. > + struct rte_distributor_burst *d = arg; > + unsigned int count = 0; > + unsigned int num = 0; > + unsigned int id = __sync_fetch_and_add(&worker_idx, 1); Use rte_atomic equivalent