>>> On 04.01.18 at 14:05, <wei.l...@citrix.com> wrote:
> It is a variant of TSC clock source.
> 
> Signed-off-by: Wei Liu <wei.l...@citrix.com>
> Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com>
> Signed-off-by: Roger Pau Monné <roger....@citrix.com>

Mostly fine, with the TODO addressed, u64 etc replaced by uint64_t
etc, ...

> +static always_inline
> +u64 __read_cycle(const struct vcpu_time_info *info, u64 tsc)

... the double underscores dropped here, and ...

> +static u64 last_value;

... this moved into the only function it's needed in.

Jan

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to