From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f74.google.com (mail-wm1-f74.google.com [209.85.128.74]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 531DC41A77F for ; Tue, 7 Jul 2026 13:12:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.74 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783429957; cv=none; b=eo74gO0xusZcVRRjSkqE1hyCSsTUrx5EFAKpavtZ8X1qNC5+reIAshZ3Cfn3gZ5nht0qNmXF5x+l+GaVNRYngPRWKzDdbDaOTHks/ZiwMdmEms13Irqre8/WTiCLO/V+NRe82ajmaCDDS8ez8tQ4nQkpC/j06iLFJup7GCsOJnQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783429957; c=relaxed/simple; bh=9LOCtbgGr2wUzylvSSY/QfURYDLYnVmVvVmuKL95+wU=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=R7NpmP7mDBdeFrmA0fOBtoihusqoyNNow3LKv/PYwpsZOtWgLn/t43QfVJWdw6JqCCPsJWUQj378FtGerlFvKjB/asOPJRboi4F5oJ9AETAmKP1VWDkdjR367lLPNjMcEr50FwippAO6TdkTozAJqwhTDrRhMm4JNMs7ya2HwxY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--aliceryhl.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=RfjJzCO/; arc=none smtp.client-ip=209.85.128.74 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--aliceryhl.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="RfjJzCO/" Received: by mail-wm1-f74.google.com with SMTP id 5b1f17b1804b1-493c20d0468so56753255e9.1 for ; Tue, 07 Jul 2026 06:12:36 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20251104; t=1783429955; x=1784034755; darn=vger.kernel.org; h=content-type:cc:to:from:subject:message-id:references:mime-version :in-reply-to:date:from:to:cc:subject:date:message-id:reply-to :content-type; bh=Of9sO2EnIhsNa8bQoFBjU1/utC4MlhlC0hhDaEkOu5U=; b=RfjJzCO/MeCghv0B6q9bqiXzx2ev8sTZ2fC2IdvajAdZHomB3bohH1aqsD6eIAn59M uKS4ZntRK4mUAHV28S3QTRYOgd+kSCMRxYED4gNNAueE7jv0d4q9SAUT7AUCymtCRhnx n3zOdx1prvHL4D6xh+MgP0uGEjeFK+k+Xobk445dw8dgu23TLs9irqBaMGDZB8nBdGLU gmEMBoWex5icJQG7J5ipRpg6foQ2tnUWzwLMMEZp0wNpS0Yf6+bmktfzVQAQq0vIWCjG r/lOg5j4Eb3iJfKYHpqPOBmIJ2SGfar8FYzJloZtZA5BF2uoFwl7ppLzPYnqlodnMK2u 6h8g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783429955; x=1784034755; h=content-type:cc:to:from:subject:message-id:references:mime-version :in-reply-to:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=Of9sO2EnIhsNa8bQoFBjU1/utC4MlhlC0hhDaEkOu5U=; b=dBjvb9HM1nE8LvVSLh4XyPfW3ZDz/HbLywl7FaQXssKyk3UIOCwwkmfpuFgr/QDYqq tExCzCUF4KBug4/uhUXFLlwdWsR4gv/jfCe3wk+/OI2VNUXQ1vV4QPAEy9cP3NpGRMlA 2sDbygFfO8oyGE7Cc7AL3tP9m0GQviNUFpquiZ8Tcnzg35Gyff0Cux76E/8UmaUA/v9e CnBmSzHmfn2LTDwNIxnNoLgXnbsyKvznOaF/jz4aoIuBuwanG0EPsRXlmC2B0VqLzRID WsQ0CDjetQcuxcqRVQS6RH36QRjFuXe8JLe6JFMjHYtbn1aTeaXyscYwGi16b3cRskU9 IVng== X-Forwarded-Encrypted: i=1; AHgh+Rrbky+YDC7GAPcxMLURqzT08qIbWhhatqBrZCdtdyszBSv4ZeC+0JSs4XFl0L07+MTXLXVMqQ6BCeyUPgs=@vger.kernel.org X-Gm-Message-State: AOJu0Yyx/QDPDhg3YLLpqE16Hmt+h7dxvW8R5GHlDQYykiuiXYhizMUo DbVHYM0lg0Oy61uwlIXwAfZMXc7ZaJIQeKoSo2vsDJ8WTDVefG12/yoRX4oPbv4YItnhahWHkYY c5NfrzSOJUbWUQnG4vA== X-Received: from wmj4.prod.google.com ([2002:a05:600c:244:b0:493:bddc:b22]) (user=aliceryhl job=prod-delivery.src-stubby-dispatcher) by 2002:a05:600c:3f0a:b0:492:7084:32af with SMTP id 5b1f17b1804b1-493df080c62mr56205555e9.23.1783429954428; Tue, 07 Jul 2026 06:12:34 -0700 (PDT) Date: Tue, 7 Jul 2026 13:12:33 +0000 In-Reply-To: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20260623-pr-ratelimited-v1-0-cc922f544dc0@google.com> <20260623-pr-ratelimited-v1-2-cc922f544dc0@google.com> Message-ID: Subject: Re: [PATCH 2/5] rust: sync: add const constructor for raw_spinlock_t From: Alice Ryhl To: Carlos Llamas Cc: Greg Kroah-Hartman , Boqun Feng , Gary Guo , "Onur =?utf-8?B?w5Z6a2Fu?=" , Andreas Hindborg , Benno Lossin , "=?utf-8?B?QmrDtnJu?= Roy Baron" , Daniel Almeida , Danilo Krummrich , Ingo Molnar , Lyude Paul , Miguel Ojeda , Peter Zijlstra , Trevor Gross , Waiman Long , Will Deacon , linux-kernel@vger.kernel.org, rust-for-linux@vger.kernel.org Content-Type: text/plain; charset="utf-8" On Fri, Jun 26, 2026 at 08:52:54PM +0000, Carlos Llamas wrote: > On Tue, Jun 23, 2026 at 03:38:05PM +0000, Alice Ryhl wrote: > > +// Explicitly list architectures where this logic is checked correct. > > +#[cfg(any( > > + CONFIG_ARM, > > + CONFIG_ARM64, > > + CONFIG_LOONGARCH, > > + CONFIG_PPC, > > + CONFIG_RISCV, > > + CONFIG_S390, > > + CONFIG_X86, > > +))] > > I guess this is done explicitly because we can't test if > __ARCH_SPIN_LOCK_UNLOCKED is defined by the arch? There are some arches where zeros is not the right value here, so when someone adds Rust on a new arch, I want to make sure that person has to look at this function and determine whether all-zeros is valid for that particular arch or not. > Looks good to me, > > Reviewed-by: Carlos Llamas Thanks! Alice