Re: [PATCH linux-next v2 0/2] x86/xen TSC related cleanups

[Juergen Gross <jgross@suse.com>]

[-- Attachment #1.1.1: Type: text/plain, Size: 1014 bytes --]

On 22.02.23 18:54, Krister Johansen wrote:
> Hi,
> Enclosed please find a pair of patches that perform some additional cleanup
> that was suggested by Boris, Jan and Thomas.
> 
> Specifically: this resync's arch/x86/include/asm/xen/cpuid.h from its
> upstream source in the Xen tree, and then uses one of the new #define-s to
> replace a constant in x86/xen/time.c that was previously only numerically
> defined.  Pedantic code from v1 is dropped in favor of a more appropriate
> return statement.
> 
> Changes from v1:
> 
> - Coding style fixup in arch/x86/xen/time.c (Feedback from Thomas Gleixner)
> 
> 
> Krister Johansen (2):
>    xen: update arch/x86/include/asm/xen/cpuid.h
>    x86/xen/time: cleanup xen_tsc_safe_clocksource
> 
>   arch/x86/include/asm/xen/cpuid.h | 22 ++++++++++++++++++----
>   arch/x86/xen/time.c              |  7 ++-----
>   2 files changed, 20 insertions(+), 9 deletions(-)
> 

For both patches:

Reviewed-by: Juergen Gross <jgross@suse.com>


Juergen

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 3149 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]