On Fri, 2023-11-03 at 16:41 +0000, Benjamin Beichler wrote:
> Although the information for the next requested time was already sent in
> a prior message, this change introduces the otherwise unused time
> variable in the TT message. This addition enables an extra consistency
> check and can otherwise be ignored.

Seems fine to me.

> --- a/include/uapi/linux/um_timetravel.h
> +++ b/include/uapi/linux/um_timetravel.h
> @@ -77,9 +77,9 @@ enum um_timetravel_ops {
>       /**
>        * @UM_TIMETRAVEL_WAIT: Indicate waiting for the previously requested
>        *      runtime, new requests may be made while waiting (e.g. due to
> -      *      interrupts); the time field is ignored. The calendar must 
> process
> -      *      this message and later  send a %UM_TIMETRAVEL_RUN message when
> -      *      the host can run again.
> +      *      interrupts); the time field contains the next requested runtime
> +      *  for consistency checks. The calendar must process this message and
> +      *  later send a %UM_TIMETRAVEL_RUN message when the host can run again.

Indentation got messed up here though.

johannes

_______________________________________________
linux-um mailing list
linux-um@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-um

Reply via email to