This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Remove atomic operations from malloc.c

From: Leonhard Holz <leonhard dot holz at web dot de>
To: libc-alpha at sourceware dot org
Date: Wed, 11 Feb 2015 14:29:51 +0100
Subject: Re: [PATCH] Remove atomic operations from malloc.c
Authentication-results: sourceware.org; auth=none
References: <54DB130F dot 9070300 at web dot de> <1423652468 dot 9778 dot 250 dot camel at triegel dot csb> <54DB5011 dot 9020405 at linux dot vnet dot ibm dot com>

I did get into the changes itself, but at least for powerpc (POWER8/16c/128T)
I am not seeing improvements with the patch.  In fact it seems to increase
contention:

            time per iteration
nths       master     patch
1    	   51.422    75.046
8          53.077    78.507
16         57.430    89.385
32         71.206   108.359
64        114.370   172.115
128       251.731   330.924

Thank you for testing! Maybe the costs of a mutex_lock are higher on PowerPC thanon i686? Anyway it looks like I have to take a different approach...

Follow-Ups:
- Re: [PATCH] Remove atomic operations from malloc.c
  - From: Adhemerval Zanella

References:
- [PATCH] Remove atomic operations from malloc.c
  - From: Leonhard Holz
- Re: [PATCH] Remove atomic operations from malloc.c
  - From: Torvald Riegel
- Re: [PATCH] Remove atomic operations from malloc.c
  - From: Adhemerval Zanella

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]