hpx icon indicating copy to clipboard operation
hpx copied to clipboard

Adding missing pieces to sync_wait, adding run_loop

Open hkaiser opened this issue 2 years ago • 1 comments

  • implement execution::run_loop
  • implement execution::forwarding_sender_query
  • adding missing pieces to make_future, start_detached, sync_wait (make it work with run_loop)
  • flyby: refine execution::forwarding_env_query and execution::forwarding_scheduler_query
  • flyby: fixing get_completion_scheduler handling for let_xxx, schedule_from, partial algorithm
  • flyby: properly constraining forwarding env, sender, and scheduler queries

hkaiser avatar Jul 28 '22 18:07 hkaiser

Performance test report

HPX Performance

Comparison

BENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTOR
For Each(=)(=)(=)

Info

PropertyBeforeAfter
HPX Datetime2022-05-31T12:57:29+00:002022-08-09T16:07:10+00:00
HPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a95acfe0f6f6e8a333b1d92c11e58a01ed44df9efc1
Hostnamenid01193nid00041
Datetime2022-05-31T15:13:01.357969+02:002022-08-09T18:30:14.987353+02:00
Clusternamedaintdaint
Compiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0
Envfile

Comparison

BENCHMARKNO-EXECUTOR
Future Overhead - Create Thread Hierarchical - Latch(=)

Info

PropertyBeforeAfter
HPX Datetime2022-05-31T12:57:29+00:002022-08-09T16:07:10+00:00
HPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a95acfe0f6f6e8a333b1d92c11e58a01ed44df9efc1
Hostnamenid01193nid00041
Datetime2022-05-31T15:13:18.026239+02:002022-08-09T18:30:31.694307+02:00
Clusternamedaintdaint
Compiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0
Envfile

Comparison

BENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATOR
Stream Benchmark - Add(=)--
Stream Benchmark - Scale(=)(=)(=)
Stream Benchmark - Triad(=)(=)-
Stream Benchmark - Copy(=)(=)(=)

Info

PropertyBeforeAfter
HPX Datetime2021-11-10T19:14:21+00:002022-08-09T16:07:10+00:00
HPX Commit71d8dbe40b737be02ea734f9d98fcc0b1e21f01aacfe0f6f6e8a333b1d92c11e58a01ed44df9efc1
Hostnamenid00120nid00041
Datetime2021-11-10T20:28:18.266961+01:002022-08-09T18:30:47.699804+02:00
Clusternamedaintdaint
Compiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0
Envfile

Explanation of Symbols

SymbolMEANING
=No performance change (confidence interval within ±1%)
(=)Probably no performance change (confidence interval within ±2%)
(+)/(-)Very small performance improvement/degradation (≤1%)
+/-Small performance improvement/degradation (≤5%)
++/--Large performance improvement/degradation (≤10%)
+++/---Very large performance improvement/degradation (>10%)
?Probably no change, but quite large uncertainty (confidence interval with ±5%)
??Unclear result, very large uncertainty (±10%)
???Something unexpected…

StellarBot avatar Aug 09 '22 16:08 StellarBot

retest lsu

hkaiser avatar Aug 19 '22 13:08 hkaiser

bors merge

hkaiser avatar Aug 21 '22 19:08 hkaiser

Build succeeded:

bors[bot] avatar Aug 21 '22 19:08 bors[bot]