From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerin Jacob Subject: Re: [PATCH v4 0/4] fix performance/cache resource issues with 128-byte cache line targets Date: Mon, 8 Feb 2016 16:22:17 +0530 Message-ID: <20160208105216.GA14217@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> <20160208112430.4cbc543a@jvn> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Cc: dev@dpdk.org To: Jan Viktorin Return-path: Received: from na01-by2-obe.outbound.protection.outlook.com (mail-by2on0093.outbound.protection.outlook.com [207.46.100.93]) by dpdk.org (Postfix) with ESMTP id CAF93559C for ; Mon, 8 Feb 2016 11:52:43 +0100 (CET) Content-Disposition: inline In-Reply-To: <20160208112430.4cbc543a@jvn> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Mon, Feb 08, 2016 at 11:24:30AM +0100, Jan Viktorin wrote: > 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? Hello Jan, Please apply the depended "config: cleanup existing RTE_CACHE_LINE_SIZE selection scheme" patch first as mentioned in the cover letter. http://dpdk.org/dev/patchwork/patch/9388/ > > 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