Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add lockhammer benchmark to PKB #5247

Open
jellyfishcake opened this issue Oct 4, 2024 · 0 comments
Open

Add lockhammer benchmark to PKB #5247

jellyfishcake opened this issue Oct 4, 2024 · 0 comments
Labels
good first issue This is a good issue for someone new to PKB

Comments

@jellyfishcake
Copy link
Contributor

ARM has a lockhammer benchmark for open-source cloud evaluations: https://github.com/ARM-software/synchronization-benchmarks/tree/master/benchmarks/lockhammer

Add the benchmarking framework
create lockhammer_benchmark.py file in linux_benchmarks
populate BENCHMARK_NAME and BENCHMARK_CONFIG constants so that the benchmark can be found by PKB
create GetConfig, Prepare, Run and Cleanup functions handlers with pass/return [] as function content. At this point you can run your new benchmark in PKB (though it will not do anything yet).
Install lockhammer
try installing on your machine using the guide on github.
Create a new package called lockhammer.py in linux_packages.
create a new function, Install, that takes a vm as an input and installs lockhammer on that vm. You should be able to wrap shell commands as vm.RemoteCommand add unit tests to make sure the remote commands are issued using mock to mock the vm.
Installing on Ubuntu2404 is top priority, followed by other Linux distributions e.g. Debian, Rhel, Centos etc.
Run lockhammer
Download and run lockhammer locally, you don't need to use PKB to run lockhammer. This part is about parsing the output into some sensible format.
Store the lockhammer output as a file in the data directory under data/lockhammer. This file will be your raw data for parsing and parsing unit tests.
Add a function in lockhammer.py that you added to linux_packages with a sensible name, e.g. ParseResults
Parse results should take a str as input and produce a list of PKB Samples as output. You goal is to parse the output into useful samples, where each sample as a metric name, metric value, metric unit, metric metadata. Each row of lockhammer's output should be a separate metric. Test the parser function

@jellyfishcake jellyfishcake added the good first issue This is a good issue for someone new to PKB label Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue This is a good issue for someone new to PKB
Projects
None yet
Development

No branches or pull requests

1 participant