Skip to content

Conversation

@fogti
Copy link
Contributor

@fogti fogti commented Nov 22, 2025

No description provided.

@fogti fogti changed the title Use less locking for romfs Use less locking for romfs, and some idiomacy fixes Nov 22, 2025
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Benchmark Results

Benchmark Current: ee5c9c8 Previous: dfff928 Performance Ratio
startup_benchmark Build Time 119.83 s 115.50 s 1.04
startup_benchmark File Size 0.90 MB 0.91 MB 0.99
Startup Time - 1 core 1.03 s (±0.02 s) 0.98 s (±0.03 s) 1.05
Startup Time - 2 cores 1.03 s (±0.02 s) 1.01 s (±0.03 s) 1.02
Startup Time - 4 cores 1.05 s (±0.03 s) 0.97 s (±0.03 s) 1.07
multithreaded_benchmark Build Time 118.33 s 114.18 s 1.04
multithreaded_benchmark File Size 1.00 MB 1.01 MB 0.99
Multithreaded Pi Efficiency - 2 Threads 90.64 % (±5.97 %) 86.76 % (±9.05 %) 1.04
Multithreaded Pi Efficiency - 4 Threads 44.34 % (±2.53 %) 42.81 % (±3.94 %) 1.04
Multithreaded Pi Efficiency - 8 Threads 25.62 % (±1.38 %) 24.76 % (±2.13 %) 1.03
micro_benchmarks Build Time 298.87 s 302.18 s 0.99
micro_benchmarks File Size 1.01 MB 1.01 MB 0.99
Scheduling time - 1 thread 181.19 ticks (±29.16 ticks) 180.66 ticks (±26.40 ticks) 1.00
Scheduling time - 2 threads 106.75 ticks (±16.83 ticks) 106.54 ticks (±20.76 ticks) 1.00
Micro - Time for syscall (getpid) 10.79 ticks (±5.00 ticks) 10.26 ticks (±4.54 ticks) 1.05
Memcpy speed - (built_in) block size 4096 61315.73 MByte/s (±43546.70 MByte/s) 59079.71 MByte/s (±42599.27 MByte/s) 1.04
Memcpy speed - (built_in) block size 1048576 13322.18 MByte/s (±10853.65 MByte/s) 13978.78 MByte/s (±11494.79 MByte/s) 0.95
Memcpy speed - (built_in) block size 16777216 9657.05 MByte/s (±7789.56 MByte/s) 9947.95 MByte/s (±8060.21 MByte/s) 0.97
Memset speed - (built_in) block size 4096 61382.24 MByte/s (±43598.93 MByte/s) 59027.78 MByte/s (±42561.15 MByte/s) 1.04
Memset speed - (built_in) block size 1048576 13706.76 MByte/s (±11103.31 MByte/s) 14226.68 MByte/s (±11623.76 MByte/s) 0.96
Memset speed - (built_in) block size 16777216 9821.41 MByte/s (±7867.63 MByte/s) 10170.81 MByte/s (±8183.13 MByte/s) 0.97
Memcpy speed - (rust) block size 4096 57234.27 MByte/s (±41628.44 MByte/s) 52661.56 MByte/s (±38922.39 MByte/s) 1.09
Memcpy speed - (rust) block size 1048576 13367.62 MByte/s (±11013.51 MByte/s) 13891.94 MByte/s (±11460.61 MByte/s) 0.96
Memcpy speed - (rust) block size 16777216 9672.29 MByte/s (±7822.84 MByte/s) 9844.05 MByte/s (±7952.61 MByte/s) 0.98
Memset speed - (rust) block size 4096 57679.56 MByte/s (±41864.54 MByte/s) 51584.65 MByte/s (±38575.24 MByte/s) 1.12
Memset speed - (rust) block size 1048576 13833.25 MByte/s (±11314.81 MByte/s) 14258.89 MByte/s (±11685.68 MByte/s) 0.97
Memset speed - (rust) block size 16777216 9870.23 MByte/s (±7929.13 MByte/s) 10068.10 MByte/s (±8079.19 MByte/s) 0.98
alloc_benchmarks Build Time 294.02 s 299.14 s 0.98
alloc_benchmarks File Size 0.97 MB 0.98 MB 0.99
Allocations - Allocation success 100.00 % 100.00 % 1
Allocations - Deallocation success 100.00 % 100.00 % 1
Allocations - Pre-fail Allocations 100.00 % 100.00 % 1
Allocations - Average Allocation time 31241.78 Ticks (±1555.25 Ticks) 34147.57 Ticks (±2031.29 Ticks) 0.91
Allocations - Average Allocation time (no fail) 31241.78 Ticks (±1555.25 Ticks) 34147.57 Ticks (±2031.29 Ticks) 0.91
Allocations - Average Deallocation time 3128.79 Ticks (±1557.01 Ticks) 3484.32 Ticks (±2131.76 Ticks) 0.90
mutex_benchmark Build Time 297.32 s 302.30 s 0.98
mutex_benchmark File Size 1.01 MB 1.01 MB 0.99
Mutex Stress Test Average Time per Iteration - 1 Threads 37.72 ns (±3.18 ns) 38.16 ns (±4.61 ns) 0.99
Mutex Stress Test Average Time per Iteration - 2 Threads 30.54 ns (±3.12 ns) 30.82 ns (±3.34 ns) 0.99

This comment was automatically generated by workflow using github-action-benchmark.

@fogti fogti force-pushed the romfs-less-locking branch from 2e98e02 to ee5c9c8 Compare November 29, 2025 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants