[v2] x86/xen TSC related cleanups

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

Krister Johansen posted 2 patches 1 year, 2 months ago

Diff against v1
Download series mbox

Failed in applying to current master (apply log)

There is a newer version of this series

arch/x86/include/asm/xen/cpuid.h | 22 ++++++++++++++++++----
arch/x86/xen/time.c              |  7 ++-----
2 files changed, 20 insertions(+), 9 deletions(-)

Expand all Fold all

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

Posted by Krister Johansen 1 year, 2 months ago

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(-)

-- 
2.25.1

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

Posted by Juergen Gross 1 year, 2 months ago

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