This looks like what I've done in D57353 already, except with the assignment to buf moved into the conditional?

OK I update the diff via the web form, which I noticed messes up the diff-to-diff comparison, since the base commit is gone, but hopefully that does not matter when landing this.

Fix the return statement to match the style guide

I wonder if we should start provide the definitive list of MI aromics in atomic(9).

The #ifdef INVARIANTS conditional for _Static_assert() is mostly a no-op, because GENERIC enables them via "include "std.debug"".

This change has been exercised with bird3. And the tcpdump dissector I forgot I wrote almost ~22.25 years to the day...

In D57381#1330788, @zishun.yi.dev_gmail.com wrote:

In D57381#1330138, @markj wrote:

@zishun.yi.dev_gmail.com could you please rebase this patch so that it applies to the latest main? I don't really like the diff to riscv/include/atomic.h, and would like to explore different approaches. (Really I'd like to get rid of all of these interceptors entirely, but the best way to do that is probably to reimplement atomic.h using C11 atomics...)

Yeah, I agree the current code is ugly, and your approach sounds great.

For now I've moved most of the ugly code into a separate revision(D58037) so it doesn't block the riscv work. That part could well be refactored entirely in the future , so keeping it separate from the KASAN changes seems cleaner.

In D58457#1342400, @ziaee wrote:

This line was forgotten to be updated when information about st_birthtime was first added into the manual page.

Nice, so we want to find that commit and put it at the bottom of the commit message like Fixes: <shorthash> ("Commit title").