Re: [PATCH v12 4/5] jump_label: adjust inline asm to be consistent

From: "Russell King (Oracle)" <linux@armlinux.org.uk>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: "Alice Ryhl" <aliceryhl@google.com>,
	"Masami Hiramatsu" <mhiramat@kernel.org>,
	"Mathieu Desnoyers" <mathieu.desnoyers@efficios.com>,
	"Peter Zijlstra" <peterz@infradead.org>,
	"Josh Poimboeuf" <jpoimboe@kernel.org>,
	"Jason Baron" <jbaron@akamai.com>,
	"Ard Biesheuvel" <ardb@kernel.org>,
	"Miguel Ojeda" <ojeda@kernel.org>,
	"Alex Gaynor" <alex.gaynor@gmail.com>,
	"Wedson Almeida Filho" <wedsonaf@gmail.com>,
	"Boqun Feng" <boqun.feng@gmail.com>,
	"Gary Guo" <gary@garyguo.net>,
	"Björn Roy Baron" <bjorn3_gh@protonmail.com>,
	"Benno Lossin" <benno.lossin@proton.me>,
	"Andreas Hindborg" <a.hindborg@kernel.org>,
	linux-trace-kernel@vger.kernel.org,
	rust-for-linux@vger.kernel.org, linux-kernel@vger.kernel.org,
	"Arnd Bergmann" <arnd@arndb.de>,
	linux-arch@vger.kernel.org,
	"Thomas Gleixner" <tglx@linutronix.de>,
	"Ingo Molnar" <mingo@redhat.com>,
	"Borislav Petkov" <bp@alien8.de>,
	"Dave Hansen" <dave.hansen@linux.intel.com>,
	x86@kernel.org, "H. Peter Anvin" <hpa@zytor.com>,
	"Sean Christopherson" <seanjc@google.com>,
	"Uros Bizjak" <ubizjak@gmail.com>,
	"Catalin Marinas" <catalin.marinas@arm.com>,
	"Will Deacon" <will@kernel.org>, "Marc Zyngier" <maz@kernel.org>,
	"Oliver Upton" <oliver.upton@linux.dev>,
	"Mark Rutland" <mark.rutland@arm.com>,
	"Ryan Roberts" <ryan.roberts@arm.com>,
	"Fuad Tabba" <tabba@google.com>,
	linux-arm-kernel@lists.infradead.org,
	"Paul Walmsley" <paul.walmsley@sifive.com>,
	"Palmer Dabbelt" <palmer@dabbelt.com>,
	"Albert Ou" <aou@eecs.berkeley.edu>,
	"Anup Patel" <apatel@ventanamicro.com>,
	"Andrew Jones" <ajones@ventanamicro.com>,
	"Alexandre Ghiti" <alexghiti@rivosinc.com>,
	"Conor Dooley" <conor.dooley@microchip.com>,
	"Samuel Holland" <samuel.holland@sifive.com>,
	linux-riscv@lists.infradead.org,
	"Huacai Chen" <chenhuacai@kernel.org>,
	"WANG Xuerui" <kernel@xen0n.name>,
	"Bibo Mao" <maobibo@loongson.cn>,
	"Tiezhu Yang" <yangtiezhu@loongson.cn>,
	"Andrew Morton" <akpm@linux-foundation.org>,
	"Tianrui Zhao" <zhaotianrui@loongson.cn>,
	loongarch@lists.linux.dev, "Palmer Dabbelt" <palmer@rivosinc.com>
Subject: Re: [PATCH v12 4/5] jump_label: adjust inline asm to be consistent
Date: Thu, 31 Oct 2024 00:10:50 +0000	[thread overview]
Message-ID: <ZyLLCmq9-9BX7GBd@shell.armlinux.org.uk> (raw)
In-Reply-To: <20241030191316.2a27ff1b@rorschach.local.home>

On Wed, Oct 30, 2024 at 07:13:16PM -0400, Steven Rostedt wrote:
> On Wed, 30 Oct 2024 16:04:27 +0000
> Alice Ryhl <aliceryhl@google.com> wrote:
> 
> > To avoid duplication of inline asm between C and Rust, we need to
> > import the inline asm from the relevant `jump_label.h` header into Rust.
> > To make that easier, this patch updates the header files to expose the
> > inline asm via a new ARCH_STATIC_BRANCH_ASM macro.
> > 
> > The header files are all updated to define a ARCH_STATIC_BRANCH_ASM that
> > takes the same arguments in a consistent order so that Rust can use the
> > same logic for every architecture.
> > 
> > Suggested-by: Peter Zijlstra (Intel) <peterz@infradead.org>
> > Co-developed-by: Miguel Ojeda <ojeda@kernel.org>
> > Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
> > Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>
> > Acked-by: Catalin Marinas <catalin.marinas@arm.com>
> > Acked-by: Palmer Dabbelt <palmer@rivosinc.com> # RISC-V
> 
> Still missing an ack from the ARM maintainer (32 bit, just added) and loongarch.
> 
> I'll wait till Monday, if I don't hear anything against these patches
> I'll pull them in.

As of what has recently happened, I am not participating in technical
activities until such time that I'm told what is now acceptable. So
far, that has not been forthcoming, and thus I believe it is unwise to
engage in technical discussion.

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!