Skip to content

Commit

Permalink
Hoist benchmarking steps into reusable GH action
Browse files Browse the repository at this point in the history
Signed-off-by: Hanno Becker <[email protected]>
Signed-off-by: Ry Jones <[email protected]>
  • Loading branch information
hanno-becker authored and ryjones committed Sep 8, 2024
1 parent 083d093 commit 8c77442
Show file tree
Hide file tree
Showing 3 changed files with 72 additions and 15 deletions.
57 changes: 57 additions & 0 deletions .github/actions/bench/action.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,57 @@
# SPDX-License-Identifier: Apache-2.0

name: Bench MLKEM-C-AArch64
description: Run benchmarking script

inputs:
name:
description: Name for the benchmarking run
required: true
perf:
description: Method of obtaining PMU metrics
required: true
default: 'PERF'
type: choice
options:
- NO
- PERF
- PMU
- M1
cflags:
description: CFLAGS to pass to compilation
required: true
archflags:
description: ARCHFLAGS to pass to compilation
required: true
bench_extra_args:
description: Further arguments to be appended to command line for `bench` script
required: false
default: ''
store_results:
description: Whether to push results to GH pages
required: true
default: 'false'
gh_token:
description: GitHub access token
required: true
runs:
using: composite
steps:
- name: Run benchmark
shell: nix develop .#ci -c bash -e {0}
run: |
tests bench -c ${{ inputs.perf }} --cflags "${{ inputs.cflags }}" --arch-flags "${{ inputs.archflags }}" -v --output output.json ${{ inputs.bench_extra_args }}
- name: Dump benchmark
shell: nix develop .#${{ inputs.devShell }} -c bash -e {0}
if: ${{ inputs.store_results != 'true' }}
run: |
cat output.json
- name: Store benchmark result
if: ${{ inputs.store_results == 'true' }}
uses: benchmark-action/github-action-benchmark@v1
with:
name: ${{ inputs.name }}
tool: 'customSmallerIsBetter'
output-file-path: output.json
github-token: ${{ inputs.gh_token }}
auto-push: true
4 changes: 4 additions & 0 deletions .github/workflows/bench_ec2_any.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,9 @@ on:
always_terminate:
description: Indicates if EC2 instance should always be terminated
default: true
bench_extra_args:
description: Additional command line to be appended to `tests bench` script
default: ''
jobs:
bench-ec2-any:
name: Ad-hoc benchmark on $${{ github.event.inputs.ec2_instance_type }}
Expand All @@ -35,4 +38,5 @@ jobs:
name: ${{ github.event.inputs.name }}
store_results: ${{ github.event.inputs.store_results }}
always_terminate: ${{ github.event.inputs.always_terminate }}
bench_extra_args: ${{ github.event.inputs.bench_extra_args }}
secrets: inherit
26 changes: 11 additions & 15 deletions .github/workflows/bench_ec2_reusable.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,10 @@ on:
type: string
description: Indicates if instance should always be terminated, even on failure
default: 'true'
bench_extra_args:
type: string
description: Additional command line to be appended to `bench` script
default: ''
env:
AWS_ROLE: arn:aws:iam::559050233797:role/mlkem-c-aarch64-gh-action
AWS_REGION: us-east-1
Expand Down Expand Up @@ -95,23 +99,15 @@ jobs:
$(cat /proc/cpuinfo)
EOF
- name: Run benchmark
shell: nix develop .#ci -c bash -e {0}
run: |
tests bench -c PERF --cflags "${{ inputs.cflags }}" --arch-flags "${{ inputs.archflags }}" -v --output output.json
- name: Dump benchmark
if: ${{ inputs.store_results != 'true' }}
run: |
cat output.json
- name: Store benchmark result
if: ${{ inputs.store_results == 'true' }}
uses: benchmark-action/github-action-benchmark@v1
uses: ./.github/actions/bench
with:
name: ${{ inputs.name }}
tool: 'customSmallerIsBetter'
output-file-path: output.json
github-token: ${{ secrets.AWS_GITHUB_TOKEN }}
auto-push: true

cflags: ${{ inputs.cflags }}
archflags: ${{ inputs.archflags }}
perf: PERF
store_results: ${{ inputs.store_results }}
bench_extra_args: ${{ inputs.bench_extra_args }}
gh_token: ${{ secrets.AWS_GITHUB_TOKEN }}
stop-ec2-runner:
name: Stop ${{ github.event.inputs.name }} (${{ github.event.inputs.ec2_instance_type }})
permissions:
Expand Down

0 comments on commit 8c77442

Please sign in to comment.