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

sys/event: add event_is_queued() #19966

Merged
merged 1 commit into from
Oct 11, 2023
Merged

Conversation

fabian18
Copy link
Contributor

@fabian18 fabian18 commented Oct 6, 2023

Contribution description

It can be useful to know if an event is already queued. I was looking for such an API.
I tried event_timeout_is_set() but this only checks if the timer is scheduled which would put the event in the event list, which is fine IMO.
This checks if the event is in the event queue.

Testing procedure

Given the other event_* functions, implementation is straight forward, I would say.

Issues/PRs references

#19963

@fabian18 fabian18 requested a review from kaspar030 as a code owner October 6, 2023 16:31
@github-actions github-actions bot added the Area: sys Area: System label Oct 6, 2023
@benpicco benpicco requested a review from maribu October 7, 2023 14:18
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Oct 7, 2023
@riot-ci
Copy link

riot-ci commented Oct 7, 2023

Murdock results

✔️ PASSED

3047bef sys/event: add event_is_queued()

Success Failures Total Runtime
7937 0 7937 16m:40s

Artifacts

Copy link
Member

@maribu maribu left a comment

Choose a reason for hiding this comment

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

Looks good to me. The code is trivial, but unit test are always appreciated. The ACK remains valid either way.

Note that we are in soft feature freeze. I'd say it is unscary, but let's get @MrKevinWeiss into the loop.

@benpicco
Copy link
Contributor

bors merge

bors bot added a commit that referenced this pull request Oct 10, 2023
19954: sys/psa_crypto: Ed25519 (EdDSA) support r=benpicco a=mguetschow

### Contribution description

- implement [`psa_sign_message()`](https://armmbed.github.io/mbed-crypto/html/api/ops/sign.html#c.psa_sign_message) and [`psa_verify_message()`](https://armmbed.github.io/mbed-crypto/html/api/ops/sign.html#c.psa_verify_message) for the two already supported [`PSA_ALG_ECDSA`](https://armmbed.github.io/mbed-crypto/html/api/ops/sign.html#c.PSA_ALG_ECDSA) algorithms, together with the CryptoCell and `micro-ecc` backends (*not* for the SE backend)
- add support for [`PSA_ALG_PURE_EDDSA`](https://armmbed.github.io/mbed-crypto/html/api/ops/sign.html#c.PSA_ALG_PURE_EDDSA), together with the CryptoCell hardware and `c25519` software backend (*not* for the SE backend)
- wipe private key data from stack for both ECDSA and EdDSA algorithms using `explicit_bzero()` (opinions from experienced Riot maintainers about usage of `goto` to avoid duplicating that function call before every `return`?)


### Testing procedure

- `examples/psa_crypto` has been updated to include EdDSA
- successfully tested configurations:
  - `nrf52840dk` with cryptocell (hardware) and `c25519` (software) backend
  - `native` with software backend


### Issues/PRs references

Thanks `@Einhornhool` for the PSA Crypto framework implementation #18547  which is great to work with!

19966: sys/event: add event_is_queued() r=benpicco a=fabian18



Co-authored-by: Mikolai Gütschow <mikolai.guetschow@tu-dresden.de>
Co-authored-by: Fabian Hüßler <fabian.huessler@ml-pa.com>
@benpicco
Copy link
Contributor

bors merge

1 similar comment
@benpicco
Copy link
Contributor

bors merge

@bors
Copy link
Contributor

bors bot commented Oct 11, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 7bc783e into RIOT-OS:master Oct 11, 2023
25 checks passed
@MrKevinWeiss MrKevinWeiss added this to the Release 2023.10 milestone Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: sys Area: System CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants