Re: [PATCH v10 1/5] rust: add static_branch_unlikely for static_key_false

From
Sami Tolvanen <samitolvanen@google.com>
Date
2024-10-11 15:23:58
Also in
linux-arch, linux-riscv, lkml, loongarch, rust-for-linux
Reply to
Gary Guo, 2024-10-11
On Fri, Oct 11, 2024 at 5:13 AM Gary Guo [off-list ref] wrote:
On Fri, 11 Oct 2024 10:13:34 +0000
Alice Ryhl [off-list ref] wrote:
quoted
+#ifndef CONFIG_JUMP_LABEL
+int rust_helper_static_key_count(struct static_key *key)
+{
+     return static_key_count(key);
+}
+EXPORT_SYMBOL_GPL(rust_helper_static_key_count);
^ Explicit export should be removed. This only works because we didn't
remove export.h from all helpers.c yet, but there's a patch to do
that and this will stop working.
What's the benefit of removing explicit exports from the Rust helper C
code? It requires special casing things like modversions for these
files, so I assume there's a reason for this. I asked about it here,
but never got a response:

https://lore.kernel.org/rust-for-linux/CABCJKudqAEvLcdqTqyfE2+iW+jeqBpnTGgYJvrZ0by6hGdfevQ@mail.gmail.com/ (local)

Sami
Thread (20 messages) 20 messages, 6 authors, last reply 2024-10-15
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help