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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Thu, 23 Feb 2023 06:43:42 +0100
From:   Juergen Gross <jgross@...e.com>
To:     Krister Johansen <kjlx@...pleofstupid.com>,
        xen-devel@...ts.xenproject.org, linux-kernel@...r.kernel.org
Cc:     Jan Beulich <jbeulich@...e.com>,
        Boris Ostrovsky <boris.ostrovsky@...cle.com>,
        Thomas Gleixner <tglx@...utronix.de>,
        Ingo Molnar <mingo@...hat.com>, Borislav Petkov <bp@...en8.de>,
        Dave Hansen <dave.hansen@...ux.intel.com>, x86@...nel.org,
        "H. Peter Anvin" <hpa@...or.com>,
        Marcelo Tosatti <mtosatti@...hat.com>,
        Anthony Liguori <aliguori@...zon.com>,
        David Reaver <me@...idreaver.com>,
        Brendan Gregg <brendan@...el.com>
Subject: Re: [PATCH linux-next v2 0/2] x86/xen TSC related cleanups

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@...e.com>


Juergen

Download attachment "OpenPGP_0xB0DE9DD628BF132F.asc" of type "application/pgp-keys" (3099 bytes)

Download attachment "OpenPGP_signature" of type "application/pgp-signature" (496 bytes)