From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olivier Matz Subject: Re: [PATCH v7 5/6] lib: added new library for latency stats Date: Wed, 18 Jan 2017 21:11:28 +0100 Message-ID: <20170118211128.78592a1b@platinum> References: <1484583573-30163-1-git-send-email-remy.horton@intel.com> <1484583573-30163-6-git-send-email-remy.horton@intel.com> <20170117042935.GA32676@localhost.localdomain> <20170117123418.GA2611@localhost.localdomain> <20170117162515.GA28925@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: "Mcnamara, John" , "Horton, Remy" , "dev@dpdk.org" , "Pattan, Reshma" , Thomas Monjalon , "Richardson, Bruce" To: Jerin Jacob Return-path: Received: from mail-wm0-f51.google.com (mail-wm0-f51.google.com [74.125.82.51]) by dpdk.org (Postfix) with ESMTP id AF648FAFA for ; Wed, 18 Jan 2017 21:11:32 +0100 (CET) Received: by mail-wm0-f51.google.com with SMTP id r126so260921866wmr.0 for ; Wed, 18 Jan 2017 12:11:32 -0800 (PST) In-Reply-To: <20170117162515.GA28925@localhost.localdomain> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi guys, On Tue, 17 Jan 2017 21:55:16 +0530, Jerin Jacob > Oliver, > > Could you please suggest how to proceed further? > Sorry for the lack of response. I know people are waiting for me, but these days I have too many things to do at the same time, and it's difficult to find time. In few words (I'll provide more detailed answers to the thread by friday): I expected to post the mbuf rework patchset for this release, which includes the structure changes (Jerin's patch for arm access, timestamp, port, nb_segs, refcnt changes). But the patchset is clearly not ready yet, it needs a rebase, and it lacks test. Jerin, I know that you submitted your patch a long time ago, and I'm the only one to blame, please do not see any vendor preference in it. I'll check friday what's the effective state of the patchset in my workspace. If I can extract a minimal patch that only change the structure, I'll send it for discussion. But from what I remember, the mbuf structure rework depends on changing the way we access the refcnt, so it can be moved to the 2nd cache line. If that's not possible, I'll try propose some alternatives. Regards, Olivier