From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5460D31AF3B; Thu, 12 Mar 2026 19:50:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773345047; cv=none; b=sI0ewF3jsZ8ctFMS/S6yVPxz+U1uNbkKciGeBbo7ot8FAq+9H5f8zUs8ichb5p8s58NrT39DKRK3g6rK5MzRFzOH9yUhdHyI0i/4S/r3jly/Ar45EAD3n/P69KH0+/i9GQSpHvRX306B3qO4Ai/0zbA5rDn1QHGVQ4RMmT/wxJ8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773345047; c=relaxed/simple; bh=wOHCpU/qUlJ9QLzJ1oUzFBOjTVtulCOI/8sgb9G5W/A=; h=Mime-Version:Content-Type:Date:Message-Id:To:From:Subject:Cc: References:In-Reply-To; b=TFcwts686oOy3CtiUdaspIYY7VGAlua2Z8VhyG1WMLaOFEr7NO5cdnThfJhC+MFgGrUqCSwqTczApE0gtRn+8aRJzErMr3QJL9r0nyAer8BNWaNEZ0EpmtoYC2Ty3/4Vy186duWlg+3iZ+76lO7MJlRBFHzan1Np7rC0H23O8mA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=ZnfOuik6; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="ZnfOuik6" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8BD85C2BC86; Thu, 12 Mar 2026 19:50:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1773345047; bh=wOHCpU/qUlJ9QLzJ1oUzFBOjTVtulCOI/8sgb9G5W/A=; h=Date:To:From:Subject:Cc:References:In-Reply-To:From; b=ZnfOuik6oAR63sev2BS1ZVFdfaVftV+psQSvuZthQeM1iH71vFl8qpXyV13mTPMIf WZWoHZO00Sx38lkpSJQR1Onft7yFww72aa2UEcO7oWjoxdF1a8GyzEE8o9Hah9boZq Mul2o5Cx8SMESt7oEn7gkF8Gp8cqHWwdHBFESmtm3oPOKr1Ufj4kjns2gda6L7B9sg bTHvnDaZAWYquD700jGYAAgzjerLPNNBAf6iSWj6AgqzYQhBMCBXCDlj+t3BQRdvF6 S5Jue3vIToOjPfQUk2B+8bFtIjDviEBVhdmPlXlPa2Cg6V62TwmhdfYtzitIdMVCay Cp2RCyKYBJ83g== Precedence: bulk X-Mailing-List: rust-for-linux@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Thu, 12 Mar 2026 20:50:34 +0100 Message-Id: To: "Miguel Ojeda" , "Joel Fernandes" From: "Danilo Krummrich" Subject: Re: [PATCH v12 1/1] rust: interop: Add list module for C linked list interface Cc: , "Miguel Ojeda" , "Boqun Feng" , "Gary Guo" , =?utf-8?q?Bj=C3=B6rn_Roy_Baron?= , "Benno Lossin" , "Andreas Hindborg" , "Alice Ryhl" , "Trevor Gross" , "Alex Gaynor" , "Dave Airlie" , "David Airlie" , "Maarten Lankhorst" , "Maxime Ripard" , "Thomas Zimmermann" , "Simona Vetter" , "Daniel Almeida" , "Koen Koning" , "Nikola Djukic" , "Alexandre Courbot" , "Philipp Stanner" , "Elle Rhumsaa" , "Jonathan Corbet" , "Alex Deucher" , =?utf-8?q?Christian_K=C3=B6nig?= , "Jani Nikula" , "Joonas Lahtinen" , "Rodrigo Vivi" , "Tvrtko Ursulin" , "Huang Rui" , "Matthew Auld" , "Matthew Brost" , "Lucas De Marchi" , =?utf-8?q?Thomas_Hellstr=C3=B6m?= , "Helge Deller" , "John Hubbard" , "Alistair Popple" , "Timur Tabi" , "Edwin Peer" , "Andrea Righi" , "Andy Ritger" , "Zhi Wang" , "Balbir Singh" , , "Eliot Courtney" , , , , , , , , References: <20260306203648.1136554-1-joelagnelf@nvidia.com> <20260306203648.1136554-2-joelagnelf@nvidia.com> In-Reply-To: On Thu Mar 12, 2026 at 8:20 PM CET, Miguel Ojeda wrote: > On Fri, Mar 6, 2026 at 9:37=E2=80=AFPM Joel Fernandes wrote: >> >> +//! // Create typed [`CList`] from sentinel head. >> +//! // SAFETY: head is valid and initialized, items are `SampleItemC` w= ith >> +//! // embedded `link` field, and `Item` is `#[repr(transparent)]` over= `SampleItemC`. >> +//! let list =3D clist_create!(unsafe { head, Item, SampleItemC, link }= ); > > Was the patch tested with Clippy? It has several issues. I did not give it a shot yet, but given this and the other nits, this is be= yond what I want to fix up when applying the patch. @Joel, can you please resend? Thanks, Danilo