Skip to content

feat: Add scripts for remote running zswap kprobe benchmarks#98

Open
dariusgrassi wants to merge 14 commits intoldos-project:mainfrom
dariusgrassi:zswap-probe-bench
Open

feat: Add scripts for remote running zswap kprobe benchmarks#98
dariusgrassi wants to merge 14 commits intoldos-project:mainfrom
dariusgrassi:zswap-probe-bench

Conversation

@dariusgrassi
Copy link
Copy Markdown
Contributor

  • Add remote_zswap_bench.py for running zswap benchmarks remotely
  • Add process_dataframes.py for parsing and plotting benchmark dataframes
  • Add zswap.yaml configuration file for Redis benchmark parameters

In a separate PR, we can consider removing the existing files in benchmark/zswap.

@dariusgrassi dariusgrassi requested a review from a team as a code owner March 18, 2025 16:46
@tewaro
Copy link
Copy Markdown
Collaborator

tewaro commented Mar 18, 2025

Can you explain in some simple documentation what the goal of the zswap benchmark is?

Comment thread config/zswap.yaml
---
benchmark_config:
generic:
benchmark: redis
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

How is this using the remote_zswap_bench?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Other way around. remote_zswap_bench is the remote runner script that uses this configuration file (zswap_redis.yaml) for the benchmark. the only change from the other redis configs is that it adds a probe for zswap_runtime.

@dariusgrassi dariusgrassi force-pushed the zswap-probe-bench branch 2 times, most recently from 1e0eded to 328d28f Compare April 16, 2025 05:02
@dariusgrassi dariusgrassi added the wontfix This will not be worked on label Apr 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wontfix This will not be worked on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants