lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <ZtICWbRiTuPcN6mT@yury-ThinkPad>
Date: Fri, 30 Aug 2024 10:33:13 -0700
From: Yury Norov <yury.norov@...il.com>
To: Mathieu Desnoyers <mathieu.desnoyers@...icios.com>
Cc: Rasmus Villemoes <linux@...musvillemoes.dk>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 5/6] lib: benchmark bitmap sets binary operation find

On Fri, Aug 30, 2024 at 12:07:53PM -0400, Mathieu Desnoyers wrote:
> On 2024-08-30 17:48, Yury Norov wrote:
> > On Thu, Aug 29, 2024 at 09:59:25AM -0400, Mathieu Desnoyers wrote:
> > > Benchmark the following bitmap find functions applying binary operations
> > > on sets of two bitmaps:
> > > 
> > > - find_first_andnot_bit,
> > > - find_first_nor_bit,
> > > - find_next_andnot_bit,
> > > - find_next_nor_bit,
> > > - find_next_or_bit.
> > > 
> > > Note that find_first_or_bit is not part of the current API, so it is not
> > > covered.
> > 
> > Can you please show how the test output looks on your system now? I'll
> > add that in commit message.
> 
> Start testing find_bit() with random-filled bitmap
> find_next_bit:                  576314 ns, 163810 iterations
> find_next_zero_bit:             626847 ns, 163871 iterations
> find_last_bit:                  465050 ns, 163810 iterations
> find_nth_bit:                  2720718 ns,  16329 iterations
> find_first_bit:                1409431 ns,  16330 iterations
> find_first_and_bit:           15216406 ns,  40975 iterations
> find_next_and_bit:              324624 ns,  81708 iterations
> find_first_andnot_bit:           23856039 ns,  40955 iterations
> find_next_andnot_bit:              327734 ns,  82103 iterations
> find_first_nor_bit:           21911075 ns,  40956 iterations
> find_next_nor_bit:              345315 ns,  81919 iterations
> find_next_or_bit:              886338 ns, 245762 iterations
> 
> Start testing find_bit() with sparse bitmap
> find_next_bit:                    8870 ns,    656 iterations
> find_next_zero_bit:            1188951 ns, 327025 iterations
> find_last_bit:                    8380 ns,    656 iterations
> find_nth_bit:                  1110068 ns,    655 iterations
> find_first_bit:                 455799 ns,    656 iterations
> find_first_and_bit:               6521 ns,      2 iterations
> find_next_and_bit:                3540 ns,      2 iterations
> find_first_andnot_bit:             785844 ns,    655 iterations
> find_next_andnot_bit:                8950 ns,    655 iterations
> find_first_nor_bit:          338646832 ns, 326373 iterations

0.3 sec is too much. Consider a slower CPU where it may be 10
times slower... Can you keep that in a range of few milliseconds?
It's more than enough to grab stable perf results.

> find_next_nor_bit:             1264144 ns, 326372 iterations
> find_next_or_bit:               14020 ns,   1309 iterations

Before the columns were all aligned. Can you please send a v3
with this fixed, and address the other comments?

Thanks,
Yury

> 
> Relevant lscpu output:
> 
> Architecture:             x86_64
>   CPU op-mode(s):         32-bit, 64-bit
>   Address sizes:          52 bits physical, 57 bits virtual
>   Byte Order:             Little Endian
> CPU(s):                   384
>   On-line CPU(s) list:    0-383
> Vendor ID:                AuthenticAMD
>   Model name:             AMD EPYC 9654 96-Core Processor
>     CPU family:           25
>     Model:                17
>     Thread(s) per core:   2
>     Core(s) per socket:   96
>     Socket(s):            2
>     Stepping:             1
>     Frequency boost:      enabled
>     CPU(s) scaling MHz:   100%
>     CPU max MHz:          3709.0000
>     CPU min MHz:          400.0000
>     BogoMIPS:             4799.80
>     Flags:                fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext
>                           fxsr_opt pdpe1gb rdtscp lm constant_tsc rep_good amd_lbr_v2 nopl xtopology nonstop_tsc cpuid extd_apicid aperfmperf rap
>                           l pni pclmulqdq monitor ssse3 fma cx16 pcid sse4_1 sse4_2 x2apic movbe popcnt aes xsave avx f16c rdrand lahf_lm cmp_leg
>                           acy svm extapic cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw ibs skinit wdt tce topoext perfctr_core perfctr_nb
>                           bpext perfctr_llc mwaitx cpb cat_l3 cdp_l3 hw_pstate ssbd mba perfmon_v2 ibrs ibpb stibp ibrs_enhanced vmmcall fsgsbase
>                            bmi1 avx2 smep bmi2 erms invpcid cqm rdt_a avx512f avx512dq rdseed adx smap avx512ifma clflushopt clwb avx512cd sha_ni
>                            avx512bw avx512vl xsaveopt xsavec xgetbv1 xsaves cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local user_shstk avx512_b
>                           f16 clzero irperf xsaveerptr rdpru wbnoinvd amd_ppin cppc arat npt lbrv svm_lock nrip_save tsc_scale vmcb_clean flushby
>                           asid decodeassists pausefilter pfthreshold avic v_vmsave_vmload vgif x2avic v_spec_ctrl vnmi avx512vbmi umip pku ospke
>                           avx512_vbmi2 gfni vaes vpclmulqdq avx512_vnni avx512_bitalg avx512_vpopcntdq la57 rdpid overflow_recov succor smca fsrm
>                            flush_l1d debug_swap
> Virtualization features:
>   Virtualization:         AMD-V
> Caches (sum of all):
>   L1d:                    6 MiB (192 instances)
>   L1i:                    6 MiB (192 instances)
>   L2:                     192 MiB (192 instances)
>   L3:                     768 MiB (24 instances)
> NUMA:
>   NUMA node(s):           24
>   NUMA node0 CPU(s):      0-7,192-199
>   NUMA node1 CPU(s):      8-15,200-207
>   NUMA node2 CPU(s):      16-23,208-215
>   NUMA node3 CPU(s):      24-31,216-223
>   NUMA node4 CPU(s):      32-39,224-231
>   NUMA node5 CPU(s):      40-47,232-239
>   NUMA node6 CPU(s):      48-55,240-247
>   NUMA node7 CPU(s):      56-63,248-255
>   NUMA node8 CPU(s):      64-71,256-263
>   NUMA node9 CPU(s):      72-79,264-271
>   NUMA node10 CPU(s):     80-87,272-279
>   NUMA node11 CPU(s):     88-95,280-287
>   NUMA node12 CPU(s):     96-103,288-295
>   NUMA node13 CPU(s):     104-111,296-303
>   NUMA node14 CPU(s):     112-119,304-311
>   NUMA node15 CPU(s):     120-127,312-319
>   NUMA node16 CPU(s):     128-135,320-327
>   NUMA node17 CPU(s):     136-143,328-335
>   NUMA node18 CPU(s):     144-151,336-343
>   NUMA node19 CPU(s):     152-159,344-351
>   NUMA node20 CPU(s):     160-167,352-359
>   NUMA node21 CPU(s):     168-175,360-367
>   NUMA node22 CPU(s):     176-183,368-375
>   NUMA node23 CPU(s):     184-191,376-383
> Vulnerabilities:
>   Gather data sampling:   Not affected
>   Itlb multihit:          Not affected
>   L1tf:                   Not affected
>   Mds:                    Not affected
>   Meltdown:               Not affected
>   Mmio stale data:        Not affected
>   Reg file data sampling: Not affected
>   Retbleed:               Not affected
>   Spec rstack overflow:   Vulnerable
>   Spec store bypass:      Vulnerable
>   Spectre v1:             Vulnerable: __user pointer sanitization and usercopy barriers only; no swapgs barriers
>   Spectre v2:             Vulnerable; IBPB: disabled; STIBP: disabled; PBRSB-eIBRS: Not affected; BHI: Not affected
>   Srbds:                  Not affected
>   Tsx async abort:        Not affected
> 
> 
> 
> > 
> > > 
> > > Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@...icios.com>
> > > Cc: Yury Norov <yury.norov@...il.com>
> > > Cc: Rasmus Villemoes <linux@...musvillemoes.dk>
> > > ---
> > >   lib/find_bit_benchmark.c | 93 ++++++++++++++++++++++++++++++++++++++++
> > >   1 file changed, 93 insertions(+)
> > > 
> > > diff --git a/lib/find_bit_benchmark.c b/lib/find_bit_benchmark.c
> > > index aee2ebb6b3cd..3b16254dec23 100644
> > > --- a/lib/find_bit_benchmark.c
> > > +++ b/lib/find_bit_benchmark.c
> > > @@ -70,6 +70,44 @@ static int __init test_find_first_and_bit(void *bitmap, const void *bitmap2, uns
> > >   	return 0;
> > >   }
> > > +static int __init test_find_first_andnot_bit(void *bitmap, const void *bitmap2, unsigned long len)
> > > +{
> > > +	static DECLARE_BITMAP(cp, BITMAP_LEN) __initdata;
> > > +	unsigned long i, cnt;
> > > +	ktime_t time;
> > > +
> > > +	bitmap_copy(cp, bitmap, BITMAP_LEN);
> > > +
> > > +	time = ktime_get();
> > > +	for (cnt = i = 0; i < len; cnt++) {
> > > +		i = find_first_andnot_bit(cp, bitmap2, len);
> > > +		__clear_bit(i, cp);
> > > +	}
> > > +	time = ktime_get() - time;
> > > +	pr_err("find_first_andnot_bit: %18llu ns, %6ld iterations\n", time, cnt);
> > > +
> > > +	return 0;
> > > +}
> > > +
> > > +static int __init test_find_first_nor_bit(void *bitmap, const void *bitmap2, unsigned long len)
> > > +{
> > > +	static DECLARE_BITMAP(cp, BITMAP_LEN) __initdata;
> > > +	unsigned long i, cnt;
> > > +	ktime_t time;
> > > +
> > > +	bitmap_copy(cp, bitmap, BITMAP_LEN);
> > > +
> > > +	time = ktime_get();
> > > +	for (cnt = i = 0; i < len; cnt++) {
> > > +		i = find_first_nor_bit(cp, bitmap2, len);
> > > +		__set_bit(i, cp);
> > > +	}
> > > +	time = ktime_get() - time;
> > > +	pr_err("find_first_nor_bit: %18llu ns, %6ld iterations\n", time, cnt);
> > > +
> > > +	return 0;
> > > +}
> > > +
> > >   static int __init test_find_next_bit(const void *bitmap, unsigned long len)
> > >   {
> > >   	unsigned long i, cnt;
> > > @@ -148,6 +186,51 @@ static int __init test_find_next_and_bit(const void *bitmap,
> > >   	return 0;
> > >   }
> > > +static int __init test_find_next_andnot_bit(const void *bitmap,
> > > +		const void *bitmap2, unsigned long len)
> > > +{
> > > +	unsigned long i, cnt;
> > > +	ktime_t time;
> > > +
> > > +	time = ktime_get();
> > > +	for (cnt = i = 0; i < BITMAP_LEN; cnt++)
> > > +		i = find_next_andnot_bit(bitmap, bitmap2, BITMAP_LEN, i + 1);
> > > +	time = ktime_get() - time;
> > > +	pr_err("find_next_andnot_bit:  %18llu ns, %6ld iterations\n", time, cnt);
> > > +
> > > +	return 0;
> > > +}
> > > +
> > > +static int __init test_find_next_nor_bit(const void *bitmap,
> > > +		const void *bitmap2, unsigned long len)
> > > +{
> > > +	unsigned long i, cnt;
> > > +	ktime_t time;
> > > +
> > > +	time = ktime_get();
> > > +	for (cnt = i = 0; i < BITMAP_LEN; cnt++)
> > > +		i = find_next_nor_bit(bitmap, bitmap2, BITMAP_LEN, i + 1);
> > > +	time = ktime_get() - time;
> > > +	pr_err("find_next_nor_bit:  %18llu ns, %6ld iterations\n", time, cnt);
> > > +
> > > +	return 0;
> > > +}
> > > +
> > > +static int __init test_find_next_or_bit(const void *bitmap,
> > > +		const void *bitmap2, unsigned long len)
> > > +{
> > > +	unsigned long i, cnt;
> > > +	ktime_t time;
> > > +
> > > +	time = ktime_get();
> > > +	for (cnt = i = 0; i < BITMAP_LEN; cnt++)
> > > +		i = find_next_or_bit(bitmap, bitmap2, BITMAP_LEN, i + 1);
> > > +	time = ktime_get() - time;
> > > +	pr_err("find_next_or_bit:  %18llu ns, %6ld iterations\n", time, cnt);
> > > +
> > > +	return 0;
> > > +}
> > > +
> > >   static int __init find_bit_test(void)
> > >   {
> > >   	unsigned long nbits = BITMAP_LEN / SPARSE;
> > > @@ -169,6 +252,11 @@ static int __init find_bit_test(void)
> > >   	test_find_first_bit(bitmap, BITMAP_LEN / 10);
> > >   	test_find_first_and_bit(bitmap, bitmap2, BITMAP_LEN / 2);
> > >   	test_find_next_and_bit(bitmap, bitmap2, BITMAP_LEN);
> > > +	test_find_first_andnot_bit(bitmap, bitmap2, BITMAP_LEN / 2);
> > > +	test_find_next_andnot_bit(bitmap, bitmap2, BITMAP_LEN);
> > > +	test_find_first_nor_bit(bitmap, bitmap2, BITMAP_LEN / 2);
> > > +	test_find_next_nor_bit(bitmap, bitmap2, BITMAP_LEN);
> > > +	test_find_next_or_bit(bitmap, bitmap2, BITMAP_LEN);
> > >   	pr_err("\nStart testing find_bit() with sparse bitmap\n");
> > > @@ -187,6 +275,11 @@ static int __init find_bit_test(void)
> > >   	test_find_first_bit(bitmap, BITMAP_LEN);
> > >   	test_find_first_and_bit(bitmap, bitmap2, BITMAP_LEN);
> > >   	test_find_next_and_bit(bitmap, bitmap2, BITMAP_LEN);
> > > +	test_find_first_andnot_bit(bitmap, bitmap2, BITMAP_LEN);
> > > +	test_find_next_andnot_bit(bitmap, bitmap2, BITMAP_LEN);
> > > +	test_find_first_nor_bit(bitmap, bitmap2, BITMAP_LEN);
> > > +	test_find_next_nor_bit(bitmap, bitmap2, BITMAP_LEN);
> > > +	test_find_next_or_bit(bitmap, bitmap2, BITMAP_LEN);
> > >   	/*
> > >   	 * Everything is OK. Return error just to let user run benchmark
> > > -- 
> > > 2.39.2
> 
> -- 
> Mathieu Desnoyers
> EfficiOS Inc.
> https://www.efficios.com

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ