From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by dpdk.org (Postfix) with ESMTP id C403A6CD5 for ; Wed, 19 Oct 2016 11:31:51 +0200 (CEST) Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by fmsmga105.fm.intel.com with ESMTP; 19 Oct 2016 02:31:50 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,513,1473145200"; d="scan'208";a="21154202" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.210.138]) by fmsmga006.fm.intel.com with SMTP; 19 Oct 2016 02:31:48 -0700 Received: by (sSMTP sendmail emulation); Wed, 19 Oct 2016 10:31:48 +0100 Date: Wed, 19 Oct 2016 10:31:48 +0100 From: Bruce Richardson To: Adrien Mazarguil Cc: Ferruh Yigit , Olivier Gournet , dev@dpdk.org, Nelio Laranjeiro Message-ID: <20161019093147.GD27816@bricha3-MOBL3.ger.corp.intel.com> References: <1475072658-29534-1-git-send-email-ogournet@corp.free.fr> <1476708993-62820-1-git-send-email-ogournet@corp.free.fr> <20161017141859.GB5733@6wind.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161017141859.GB5733@6wind.com> Organization: Intel Research and =?iso-8859-1?Q?De=ACvel?= =?iso-8859-1?Q?opment?= Ireland Ltd. User-Agent: Mutt/1.7.1 (2016-10-04) Subject: Re: [dpdk-dev] [PATCH v2] net/mlx5: fix init on secondary process 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: Wed, 19 Oct 2016 09:31:52 -0000 On Mon, Oct 17, 2016 at 04:18:59PM +0200, Adrien Mazarguil wrote: > On Mon, Oct 17, 2016 at 02:52:39PM +0100, Ferruh Yigit wrote: > > Hi Adrien, > > > > On 10/17/2016 1:56 PM, Olivier Gournet wrote: > > > Fixes: 1d88ba171942 ("net/mlx5: refactor Tx data path") > > > Fixes: 21c8bb4928c9 ("net/mlx5: split Tx queue structure") > > > > > > Signed-off-by: Olivier Gournet > > > > According your comment on previous version of it, I think you have your > > Ack on this patch, but can you please confirm? > > Yes it's fine, thanks. > > Acked-by: Adrien Mazarguil > > -- While this patch is acked, I'd still like a bit of detail in the commit message describing what the problem is and how the patch fixes it. The Chuck Norris approach of trying to stare down the code until it tells me just isn't working for me today! :-) Adrien or Olivier, if you can supply a brief description of what this patch is doing and why I'll add it to the commit log on apply. Thanks, /Bruce