From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jerin Jacob <jerin.jacob@caviumnetworks.com>
Subject: Re: [PATCH] eventdev: fix incorrect MP/MC tail updates
	in	rte_event_ring
Date: Mon, 2 Apr 2018 08:25:22 +0530
Message-ID: <20180402025521.GA23108@jerin>
References: <1522228611-4838-1-git-send-email-hofors@lysator.liu.se>
 <E923DB57A917B54B9182A2E928D00FA65E018DD7@IRSMSX101.ger.corp.intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Transfer-Encoding: 8bit
Cc: Mattias =?iso-8859-1?Q?R=F6nnblom?= <hofors@lysator.liu.se>,
 "dev@dpdk.org" <dev@dpdk.org>
To: "Van Haaren, Harry" <harry.van.haaren@intel.com>
Return-path: <dev-bounces@dpdk.org>
Received: from NAM02-SN1-obe.outbound.protection.outlook.com
 (mail-sn1nam02on0069.outbound.protection.outlook.com [104.47.36.69])
 by dpdk.org (Postfix) with ESMTP id C1B1F5B40
 for <dev@dpdk.org>; Mon,  2 Apr 2018 04:55:41 +0200 (CEST)
Content-Disposition: inline
In-Reply-To: <E923DB57A917B54B9182A2E928D00FA65E018DD7@IRSMSX101.ger.corp.intel.com>
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://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: <https://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

-----Original Message-----
> Date: Thu, 29 Mar 2018 12:38:01 +0000
> From: "Van Haaren, Harry" <harry.van.haaren@intel.com>
> To: Mattias Rönnblom <hofors@lysator.liu.se>, "dev@dpdk.org" <dev@dpdk.org>
> Subject: Re: [dpdk-dev] [PATCH] eventdev: fix incorrect MP/MC tail updates
>  in	rte_event_ring
> 
> > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Mattias Rönnblom
> > Sent: Wednesday, March 28, 2018 10:17 AM
> > To: dev@dpdk.org
> > Cc: Mattias Rönnblom <hofors@lysator.liu.se>
> > Subject: [dpdk-dev] [PATCH] eventdev: fix incorrect MP/MC tail updates in
> > rte_event_ring
> > 
> > rte_event_ring enqueue and dequeue tail updates were hardcoded for a
> > SC/SP configuration.
> > 
> > Signed-off-by: Mattias Rönnblom <hofors@lysator.liu.se>
> > ---
> 
> Hi Mattias,
> 
> 
> Thanks for your patch, a few notes;
> 
> 1)
> You can CC the maintainer of a section - the event rings falls under
> the eventdev library, and Jerin Jacob is the maintainer.
> 
> 
> 2)
> In DPDK we note patches that are "Fixes",
> so we can track what commit they fix. Fixes patches are
> also often candidates for backporting.
> 
> Details on getting the fixes line here:
> https://dpdk.org/doc/guides/contributing/patches.html#commit-messages-body
> 
> For this patch, the following is the fixline:
> 
> Fixes: dc39e2f359b5 ("eventdev: add ring structure for events")
> Cc: bruce.richardson@intel.com
> 

Hi Mattias,

Could you please the v2 version addressing Harry's comments.