On Wed, Oct 7, 2020 at 11:44 AM Axel Rasmussen <axelrasmus...@google.com> wrote:
> It's common [1] to define tracepoint fields as "bool" when they contain
> a true / false value. Currently, defining a synthetic event with a
> "bool" field yields EINVAL. It's possible to work around this by using
> e.g. u8 (assuming sizeof(bool) is 1, and bool is unsigned; if either of
> these properties don't match, you get EINVAL [2]).
>
> Supporting "bool" explicitly makes hooking this up easier and more
> portable for userspace.

Acked-by: Michel Lespinasse <wal...@google.com>

Looks fine to me, but you really want to get Steven's opinion over mine here :)

Reply via email to