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

crypto/riscvcap: fix function declaration for hwprobe_to_cap #24373

Conversation

ZenithalHourlyRate
Copy link
Contributor

@ZenithalHourlyRate ZenithalHourlyRate commented May 12, 2024

When compiling with gcc from ubuntu 24.04 with --strict-warnings, it would complain

crypto/riscvcap.c:88:13: error: function declaration isn't a prototype [-Werror=strict-prototypes]
   88 | static void hwprobe_to_cap()
      |             ^~~~~~~~~~~~~~

This was found during an attempt to migrate riscv64 cross-compile CI for openssl to ubuntu 24.04. In 24.04, the header <asm/hwprobe.h> exists so the #ifdef for this function was enabled, whereas it is not the case for 22.04.

Checklist

error: function declaration isn't a prototype [-Werror=strict-prototypes]

Fixes: 66ad636 ("riscv: use hwprobe syscall for capability detection")
@github-actions github-actions bot added the severity: fips change The pull request changes FIPS provider sources label May 12, 2024
@paulidale paulidale added branch: master Merge to master branch approval: review pending This pull request needs review by a committer labels May 12, 2024
@t8m t8m added triaged: bug The issue/pr is/fixes a bug tests: exempted The PR is exempt from requirements for testing labels May 13, 2024
@t8m t8m added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels May 13, 2024
@openssl-machine openssl-machine removed the approval: done This pull request has the required number of approvals label May 14, 2024
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

@openssl-machine openssl-machine added the approval: ready to merge The 24 hour grace period has passed, ready to merge label May 14, 2024
@t8m
Copy link
Member

t8m commented May 14, 2024

Merged to the master branch. Thank you for your contribution.

@t8m t8m closed this May 14, 2024
openssl-machine pushed a commit that referenced this pull request May 14, 2024
error: function declaration isn't a prototype [-Werror=strict-prototypes]

Fixes: 66ad636 ("riscv: use hwprobe syscall for capability detection")

Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #24373)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Merge to master branch severity: fips change The pull request changes FIPS provider sources tests: exempted The PR is exempt from requirements for testing triaged: bug The issue/pr is/fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants