Re: [PATCH] revl: Add empty Send + Sync implementations for Event

[Philippe Gerum <rpm@xenomai.org>]

Jussi Viiri <ilmai@iki.fi> writes:

> Event wasn't safe to send across threads. Fixed.
>
> ---
>
> From 0782e7d6c5f9189fe85f8f340721e1bcec3d16b2 Mon Sep 17 00:00:00 2001
> From: Jussi Viiri <ilmai@iki.fi>
> Date: Tue, 23 Aug 2022 22:49:01 +0300
> Subject: [PATCH 3/3] Add empty Send + Sync implementations for Event
>
> ---
>  src/event.rs | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/src/event.rs b/src/event.rs
> index 7fdfd23..f469c43 100644
> --- a/src/event.rs
> +++ b/src/event.rs
> @@ -76,6 +76,9 @@ impl WaitTimeoutResult {
>
>  pub struct Event(UnsafeCell<evl_event>);
>
> +unsafe impl Send for Event {}
> +unsafe impl Sync for Event {}
> +
>  impl Event {
>      pub fn new(builder: Builder) -> Result<Self, Error> {
>          let this = Self(UnsafeCell::new(unsafe {

LGTM. We enforce the developer's certificate of origin as defined by the
linux kernel project, please send all patches with a Signed-off-by line
(resending this one as well).

Thanks,

-- 
Philippe.