xor: use static_call for xor_gen
Avoid the indirect call for xor_generation by using a static_call. Link: https://lkml.kernel.org/r/20260327061704.3707577-28-hch@lst.de Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Eric Biggers <ebiggers@kernel.org> Tested-by: Eric Biggers <ebiggers@kernel.org> Cc: Albert Ou <aou@eecs.berkeley.edu> Cc: Alexander Gordeev <agordeev@linux.ibm.com> Cc: Alexandre Ghiti <alex@ghiti.fr> Cc: Andreas Larsson <andreas@gaisler.com> Cc: Anton Ivanov <anton.ivanov@cambridgegreys.com> Cc: Ard Biesheuvel <ardb@kernel.org> Cc: Arnd Bergmann <arnd@arndb.de> Cc: "Borislav Petkov (AMD)" <bp@alien8.de> Cc: Catalin Marinas <catalin.marinas@arm.com> Cc: Chris Mason <clm@fb.com> Cc: Christian Borntraeger <borntraeger@linux.ibm.com> Cc: Dan Williams <dan.j.williams@intel.com> Cc: David S. Miller <davem@davemloft.net> Cc: David Sterba <dsterba@suse.com> Cc: Heiko Carstens <hca@linux.ibm.com> Cc: Herbert Xu <herbert@gondor.apana.org.au> Cc: "H. Peter Anvin" <hpa@zytor.com> Cc: Huacai Chen <chenhuacai@kernel.org> Cc: Ingo Molnar <mingo@redhat.com> Cc: Jason A. Donenfeld <jason@zx2c4.com> Cc: Johannes Berg <johannes@sipsolutions.net> Cc: Li Nan <linan122@huawei.com> Cc: Madhavan Srinivasan <maddy@linux.ibm.com> Cc: Magnus Lindholm <linmag7@gmail.com> Cc: Matt Turner <mattst88@gmail.com> Cc: Michael Ellerman <mpe@ellerman.id.au> Cc: Nicholas Piggin <npiggin@gmail.com> Cc: Palmer Dabbelt <palmer@dabbelt.com> Cc: Richard Henderson <richard.henderson@linaro.org> Cc: Richard Weinberger <richard@nod.at> Cc: Russell King <linux@armlinux.org.uk> Cc: Song Liu <song@kernel.org> Cc: Sven Schnelle <svens@linux.ibm.com> Cc: Ted Ts'o <tytso@mit.edu> Cc: Vasily Gorbik <gor@linux.ibm.com> Cc: WANG Xuerui <kernel@xen0n.name> Cc: Will Deacon <will@kernel.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>master
parent
80dcf0a783
commit
a21921dd02
|
|
@ -11,10 +11,10 @@
|
|||
#include <linux/raid/xor.h>
|
||||
#include <linux/jiffies.h>
|
||||
#include <linux/preempt.h>
|
||||
#include <linux/static_call.h>
|
||||
#include "xor_impl.h"
|
||||
|
||||
/* The xor routine to use. */
|
||||
static struct xor_block_template *active_template;
|
||||
DEFINE_STATIC_CALL_NULL(xor_gen_impl, *xor_block_8regs.xor_gen);
|
||||
|
||||
/**
|
||||
* xor_gen - generate RAID-style XOR information
|
||||
|
|
@ -37,13 +37,13 @@ void xor_gen(void *dest, void **srcs, unsigned int src_cnt, unsigned int bytes)
|
|||
WARN_ON_ONCE(bytes == 0);
|
||||
WARN_ON_ONCE(bytes & 511);
|
||||
|
||||
active_template->xor_gen(dest, srcs, src_cnt, bytes);
|
||||
static_call(xor_gen_impl)(dest, srcs, src_cnt, bytes);
|
||||
}
|
||||
EXPORT_SYMBOL(xor_gen);
|
||||
|
||||
/* Set of all registered templates. */
|
||||
static struct xor_block_template *__initdata template_list;
|
||||
static bool __initdata xor_forced = false;
|
||||
static struct xor_block_template *forced_template;
|
||||
|
||||
/**
|
||||
* xor_register - register a XOR template
|
||||
|
|
@ -69,7 +69,7 @@ void __init xor_register(struct xor_block_template *tmpl)
|
|||
*/
|
||||
void __init xor_force(struct xor_block_template *tmpl)
|
||||
{
|
||||
active_template = tmpl;
|
||||
forced_template = tmpl;
|
||||
}
|
||||
|
||||
#define BENCH_SIZE 4096
|
||||
|
|
@ -111,7 +111,7 @@ static int __init calibrate_xor_blocks(void)
|
|||
void *b1, *b2;
|
||||
struct xor_block_template *f, *fastest;
|
||||
|
||||
if (xor_forced)
|
||||
if (forced_template)
|
||||
return 0;
|
||||
|
||||
b1 = (void *) __get_free_pages(GFP_KERNEL, 2);
|
||||
|
|
@ -128,7 +128,7 @@ static int __init calibrate_xor_blocks(void)
|
|||
if (f->speed > fastest->speed)
|
||||
fastest = f;
|
||||
}
|
||||
active_template = fastest;
|
||||
static_call_update(xor_gen_impl, fastest->xor_gen);
|
||||
pr_info("xor: using function: %s (%d MB/sec)\n",
|
||||
fastest->name, fastest->speed);
|
||||
|
||||
|
|
@ -156,10 +156,10 @@ static int __init xor_init(void)
|
|||
* If this arch/cpu has a short-circuited selection, don't loop through
|
||||
* all the possible functions, just use the best one.
|
||||
*/
|
||||
if (active_template) {
|
||||
if (forced_template) {
|
||||
pr_info("xor: automatically using best checksumming function %-10s\n",
|
||||
active_template->name);
|
||||
xor_forced = true;
|
||||
forced_template->name);
|
||||
static_call_update(xor_gen_impl, forced_template->xor_gen);
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
@ -170,7 +170,7 @@ static int __init xor_init(void)
|
|||
* Pick the first template as the temporary default until calibration
|
||||
* happens.
|
||||
*/
|
||||
active_template = template_list;
|
||||
static_call_update(xor_gen_impl, template_list->xor_gen);
|
||||
return 0;
|
||||
#endif
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue