From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from wes1-so1.wedos.net (wes1-so1.wedos.net [46.28.106.15]) by dpdk.org (Postfix) with ESMTP id CB8B6559C for ; Mon, 8 Feb 2016 11:24:32 +0100 (CET) Received: from jvn (dynamic-109-81-211-84.ipv4.broadband.iol.cz [109.81.211.84]) by wes1-so1.wedos.net (Postfix) with ESMTPSA id 3pzNkm1DN2z8g1; Mon, 8 Feb 2016 11:24:32 +0100 (CET) Date: Mon, 8 Feb 2016 11:24:30 +0100 From: Jan Viktorin To: Jerin Jacob Message-ID: <20160208112430.4cbc543a@jvn> In-Reply-To: <20160208093157.GF10260@localhost.localdomain> References: <1450067576-18803-1-git-send-email-jerin.jacob@caviumnetworks.com> <1454053555-28850-1-git-send-email-jerin.jacob@caviumnetworks.com> <20160208093157.GF10260@localhost.localdomain> Organization: RehiveTech X-Mailer: Claws Mail 3.13.0 (GTK+ 2.24.28; x86_64-unknown-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v4 0/4] fix performance/cache resource issues with 128-byte cache line targets 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: Mon, 08 Feb 2016 10:24:33 -0000 Hello Jerin, I failed to apply the v4 series. I've tried multiple bases. The v3 applied successfully to v2.2.0-rc4. Could you give the proper working base? Applying patch #10232 using 'git am' Description: [dpdk-dev,v4,2/4] mbuf: fix performance/cache resource issue with 128-byte cache line targets Applying: mbuf: fix performance/cache resource issue with 128-byte cache line targets error: patch failed: lib/librte_eal/linuxapp/eal/include/exec-env/rte_kni_common.h:68 error: lib/librte_eal/linuxapp/eal/include/exec-env/rte_kni_common.h: patch does not apply Patch failed at 0001 mbuf: fix performance/cache resource issue with 128-byte cache line targets The copy of the patch that failed is found in: /home/jviki/upstream/dpdk/.git/worktrees/dpdk-pw/rebase-apply/patch When you have resolved this problem, run "git am --continue". If you prefer to skip this patch, run "git am --skip" instead. To restore the original branch and stop patching, run "git am --abort". 'git am' failed with exit status 128 Try to fix git-am manually or exit... Regards Jan On Mon, 8 Feb 2016 15:01:58 +0530 Jerin Jacob wrote: > On Fri, Jan 29, 2016 at 01:15:51PM +0530, Jerin Jacob wrote: > > This patchset fixes performance/cache resource issues with 128-byte cache line targets > > found in mbuf and bitmap DPDK libraries > > > > Currently, we have two DPDK targets(ThunderX and ppc_64) which are based on > > 128-bytes cache line size target. > > > > This patchset doesn't introduce any performance degradation > > for 64-bytes cache line size targets. > > > > v1..v2 > > - Introduced new cache macro definitions as Suggested by Konstantin > > - Reduced the cache alignment requirement for 128-byte cache targets in > > slow-path data structures to save the memory > > - Verified x86(a 64byte cacheline target) does not have any impact on these changes by > > verifying the md5sum of app/test,app/testpmd, app/testacl binaries with > > or without this patch set > > > > v2..v3 > > > > revert the cache alignment of rte_ring_debug_stats, > > rte_mempool_debug_stats structures > > > > v3..v4 > > replaced RTE_CACHE_MIN_LINE_SIZE with RTE_CACHE_LINE_MIN_SIZE as suggested by > > olivier.matz@6wind.com > > > > For clean git am, "config: cleanup existing RTE_CACHE_LINE_SIZE selection scheme" > > patch needs to apply first > > > > Jerin Jacob (4): > > eal: Introduce new cache line macro definitions > > mbuf: fix performance/cache resource issue with 128-byte cache line > > targets > > bitmap: optimize for 128-bytes cache line targets > > cache/slow-path: reduce cache align requirement for 128-byte cache > > targets > > > > ping for review/merge > > > app/test/test_mbuf.c | 2 +- > > lib/librte_eal/common/include/rte_memory.h | 16 ++++++++++++++++ > > .../linuxapp/eal/include/exec-env/rte_kni_common.h | 4 +++- > > lib/librte_ether/rte_ethdev.h | 4 ++-- > > lib/librte_mbuf/rte_mbuf.h | 2 +- > > lib/librte_sched/rte_bitmap.h | 10 +++++----- > > 6 files changed, 28 insertions(+), 10 deletions(-) > > > > -- > > 2.1.0 > > -- Jan Viktorin E-mail: Viktorin@RehiveTech.com System Architect Web: www.RehiveTech.com RehiveTech Brno, Czech Republic