Skip to content

Conversation

bee-san
Copy link
Owner

@bee-san bee-san commented Nov 26, 2024

No description provided.

Copy link
Collaborator

@PsypherPunk PsypherPunk left a comment

Choose a reason for hiding this comment

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

LGTM. This is the output (with a lot of Open… lines removed):

Running benches/benchmark_portscan.rs (target/release/deps/benchmark_portscan-b123ee4f74837671)
Benchmarking portscan tcp: Warming up for 3.0000 s
Open 127.0.0.1:631
…
Open 127.0.0.1:631
Benchmarking portscan tcp: Collecting 100 samples in estimated 6.3717 s (200 iterations)
Open 127.0.0.1:631
…
Open 127.0.0.1:631
portscan tcp            time:   [32.337 ms 32.707 ms 33.164 ms]
                   change: [-3.0799% -0.4265% +2.3654%] (p = 0.75 > 0.05)
                   No change in performance detected.
Found 11 outliers among 100 measurements (11.00%)
2 (2.00%) low mild
4 (4.00%) high mild
5 (5.00%) high severe

portscan udp/portscan udp
                   time:   [144.59 ms 146.87 ms 149.07 ms]
                   change: [-4.5106% -2.3084% -0.0888%] (p = 0.04 < 0.05)
                   Change within noise threshold.

@bee-san
Copy link
Owner Author

bee-san commented Nov 26, 2024

portscan tcp            time:   [37.942 ms 38.609 ms 39.280 ms]
                        change: [+0.7704% +3.1799% +5.6135%] (p = 0.01 < 0.05)
                        Change within noise threshold.

portscan udp/portscan udp
                        time:   [62.450 ms 63.154 ms 63.873 ms]
                        change: [-0.7117% +1.1712% +3.0481%] (p = 0.22 > 0.05)
                        No change in performance detected.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild

parse address           time:   [17.086 ns 17.465 ns 17.843 ns]
                        change: [-3.6254% +0.6389% +4.8144%] (p = 0.77 > 0.05)
                        No change in performance detected.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild

port strategy           time:   [614.41 ps 621.38 ps 628.64 ps]
                        change: [-2.1726% -0.8710% +0.5917%] (p = 0.21 > 0.05)
                        No change in performance detected.
Found 9 outliers among 100 measurements (9.00%)
  7 (7.00%) high mild
  2 (2.00%) high severe

@bee-san bee-san merged commit d9dec00 into master Nov 26, 2024
9 checks passed
@PsypherPunk PsypherPunk deleted the bee-add-benches branch November 26, 2024 13: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