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

From
Sami Tolvanen <samitolvanen@google.com>
Date
2024-10-11 21:26:28
Also in
linux-arch, linux-riscv, lkml, loongarch, rust-for-linux
Reply to
Miguel Ojeda, 2024-10-11
On Fri, Oct 11, 2024 at 12:48 PM Miguel Ojeda
[off-list ref] wrote:
On Fri, Oct 11, 2024 at 7:52 PM Sami Tolvanen [off-list ref] wrote:
quoted
It's just one line per helper, but sure, I do see your point.
I guess we will have a lot of helpers added over time, so even if it
is one line, it may end up being a lot of lines in total. The rules
should stay constant, which would be better. Having said that, it is
true the extra complexity of the rules isn't great either.
My only concern is that custom C build rules must be kept in sync with
Makefile.build changes, while EXPORT_SYMBOL lines should be basically
maintenance-free. However, perhaps this isn't really an issue since
most of the complexity is already contained in rule_cc_o_c that we can
conveniently reuse. Anyway, I think this is something we can worry
about later if we actually run into problems. I was mostly interested
in the reasoning behind these changes.

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