linux-kernel - Re: [PATCH v5 00/17] Add static

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20200709090210.GL597537@hirez.programming.kicks-ass.net>
Date:   Thu, 9 Jul 2020 11:02:10 +0200
From:   Peter Zijlstra <peterz@...radead.org>
To:     x86@...nel.org
Cc:     linux-kernel@...r.kernel.org, rostedt@...dmis.org,
        mhiramat@...nel.org, bristot@...hat.com, jbaron@...mai.com,
        torvalds@...ux-foundation.org, tglx@...utronix.de,
        mingo@...nel.org, namit@...are.com, hpa@...or.com, luto@...nel.org,
        ard.biesheuvel@...aro.org, jpoimboe@...hat.com,
        pbonzini@...hat.com, mathieu.desnoyers@...icios.com,
        linux@...musvillemoes.dk
Subject: Re: [PATCH v5 00/17] Add static_call()

On Wed, Jun 24, 2020 at 05:30:24PM +0200, Peter Zijlstra wrote:
> Hi all,
> 
> Hopefully for the last time...
> 
> static_call(), is the idea of static_branch() applied to indirect function
> calls. Remove a data load (indirection) by modifying the text.
> 
> The inline implementation still relies on objtool to generate the
> .static_call_sites section, mostly because this is a natural place for x86_64
> and works for both GCC and LLVM.  Other architectures can pick other means
> if/when they implement the inline patching. The out-of-line (aka. trampoline)
> variant doesn't require this.
> 
> Patches go on top of tip/objtool/core + tip/perf/core
> 
> Patches can also be found here:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/peterz/queue.git x86/static_call
> 
> Changes since the last time:
> 
>  - Better documentation
>  - More consistent macro names
>  - Changed key prefix
>  - Folded trampoline definitions
>  - Renamed DEFINE_STATIC_COND_CALL / DEFINE_STATIC_CALL_NULL

v5.1 (patch 16) has a crash fix found by 0day, synthetic tracepoints
don't have static call sites.

Given the overwhelming feedback on this series, I'm thinking everybody
is in violent agreement with the code this time around (or just really
busy, in which case, consider this a gentle prod to send out your ACK
:-).