Messages in this thread |  | | Date | Fri, 1 Nov 2013 09:01:36 +0100 | From | Ingo Molnar <> | Subject | Re: [PATCH 0/4] per anon_vma lock and turn anon_vma rwsem lock to rwlock_t |
| |
* Yuanhan Liu <yuanhan.liu@linux.intel.com> wrote:
> Patch 1 turns locking the anon_vma's root to locking itself to let it be > a per anon_vma lock, which would reduce contentions. > > In the same time, lock range becomes quite small then, which is bascially > a call of anon_vma_interval_tree_insert(). Patch 2 turn rwsem to rwlock_t. > It's a patch made from Ingo, I just made some change to let it apply based on > patch 1. > > Patch 3 is from Peter. It was a diff, I edited it to be a patch ;) > > Here is the detailed changed stats with this patch applied. The test base is v3.12-rc7, > and 1c00bef768d4341afa7d is patch 1, e3e37183ee805f33e88f is patch 2. > > NOTE: both commits are compared to base v3.12-rc7. > > 1c00bef768d4341afa7d e3e37183ee805f33e88f > ------------------------ ------------------------ > +35.0% +89.9% brickland1/micro/aim7/fork_test > +28.4% +49.3% lkp-ib03/micro/aim7/fork_test > +2.0% +2.7% lkp-ib03/micro/aim7/shared > -0.4% +0.0% lkp-sb03/micro/aim7/dbase > +16.4% +59.0% lkp-sb03/micro/aim7/fork_test > +0.1% +0.3% lkp-sb03/micro/aim7/shared > +2.2% +5.0% TOTAL aim7.2000.jobs-per-min
Impressive!
> 1c00bef768d4341afa7d e3e37183ee805f33e88f > ------------------------ ------------------------ > -25.9% 1008.55 -47.3% 717.39 brickland1/micro/aim7/fork_test > -1.4% 641.19 -3.4% 628.45 brickland1/micro/hackbench/1600%-process-pipe > -1.0% 122.84 +1.1% 125.36 brickland1/micro/netperf/120s-200%-UDP_RR > +0.0% 121.29 +0.2% 121.57 lkp-a04/micro/netperf/120s-200%-TCP_SENDFILE > -22.1% 351.41 -26.3% 332.54 lkp-ib03/micro/aim7/fork_test > -1.9% 31.33 -2.6% 31.11 lkp-ib03/micro/aim7/shared > -0.4% 630.36 +0.4% 635.05 lkp-ib03/micro/hackbench/1600%-process-socket > -0.0% 612.62 +1.8% 623.80 lkp-ib03/micro/hackbench/1600%-threads-socket > -14.1% 340.30 -37.1% 249.26 lkp-sb03/micro/aim7/fork_test > -0.1% 41.31 -0.3% 41.22 lkp-sb03/micro/aim7/shared > -0.0% 614.26 +0.6% 617.81 lkp-sb03/micro/hackbench/1600%-process-socket > -10.4% 4515.47 -18.2% 4123.55 TOTAL time.elapsed_time
Here you scared me for a second with those negative percentages! :-)
> 1c00bef768d4341afa7d e3e37183ee805f33e88f > ------------------------ ------------------------ > +26.7% 323386.33 -75.7% 61980.00 brickland1/micro/aim7/fork_test > -22.9% 67734.00 -64.1% 31531.33 brickland1/micro/aim7/shared > +0.4% 3303.67 -0.8% 3264.33 brickland1/micro/dbench/100% > +0.7% 1871483.67 -0.4% 1850846.00 brickland1/micro/netperf/120s-200%-TCP_MAERTS > -1.0% 109553.00 +0.4% 111038.67 brickland1/micro/pigz/100% > -0.7% 13600.67 +0.1% 13718.67 lkp-a04/micro/netperf/120s-200%-TCP_CRR > -4.6% 995898.00 -85.2% 154621.40 lkp-ib03/micro/aim7/fork_test > -31.8% 32178.00 -50.3% 23442.67 lkp-ib03/micro/aim7/shared > +1.1% 7466432.67 -0.7% 7334831.67 lkp-ib03/micro/hackbench/1600%-threads-pipe > +2.5% 1044936.33 -1.3% 1006084.00 lkp-ib03/micro/hackbench/1600%-threads-socket > -1.3% 5635979.00 +0.2% 5721011.67 lkp-ib03/micro/netperf/120s-200%-TCP_RR > -24.3% 42853.33 -56.8% 24484.33 lkp-nex04/micro/aim7/shared > -23.3% 754297.67 -83.2% 165479.00 lkp-sb03/micro/aim7/fork_test > -7.4% 21586.00 -24.1% 17698.33 lkp-sb03/micro/aim7/shared > +1.1% 3838724.00 +0.3% 3808206.67 lkp-sb03/micro/hackbench/1600%-process-pipe > +0.8% 5143255.00 -1.1% 5046716.67 lkp-sb03/micro/hackbench/1600%-threads-pipe > +2.8% 537048.67 -0.8% 518351.67 lkp-sb03/micro/hackbench/1600%-threads-socket > +4.0% 50446.67 -5.3% 45960.00 lkp-sb03/micro/netperf/120s-200%-TCP_MAERTS > -42.0% 52693.00 -26.4% 66849.67 lkp-sb03/micro/netperf/120s-200%-TCP_STREAM > -0.6% 28005389.67 -7.7% 26006116.73 TOTAL vmstat.system.cs
looks like a win all across, with a few below 1% regressions what might be statistical outliners - it's hard to tell without a stddev column ...
> 1c00bef768d4341afa7d e3e37183ee805f33e88f > ------------------------ ------------------------ > -4.7% 82.00 -98.8% 1.00 brickland1/micro/aim7/fork_test > -20.5% 48.33 -94.1% 3.60 lkp-ib03/micro/aim7/fork_test > -21.2% 36.00 -89.8% 4.67 lkp-sb03/micro/aim7/fork_test > -13.6% 166.33 -95.2% 9.27 TOTAL vmstat.cpu.id > > 1c00bef768d4341afa7d e3e37183ee805f33e88f > ------------------------ ------------------------ > +33.3% 16.00 +705.6% 96.67 brickland1/micro/aim7/fork_test > +19.6% 59.00 +78.4% 88.00 lkp-sb03/micro/aim7/fork_test > +22.3% 75.00 +201.1% 184.67 TOTAL vmstat.cpu.sy
So idle time goes down, system time goes up - we exchange context-switch related idle time with spinning on an rwlock.
Btw., another _really_ interesting comparison would be against the latest rwsem patches. Mind doing such a comparison?
Thanks,
Ingo
|  |