Re: [PATCH v2 004/110] net: change sock.sk_ino and sock_i_ino() to u64
Marc Kleine-Budde <[email protected]> Tue, 3 Mar 2026 11:03:42 +0100
| Newsgroups | org.kernel.vger.autofs,dev.linux.lists.fsverity,dev.linux.lists.netfs,dev.linux.lists.ntfs3,dev.linux.lists.nvdimm,dev.linux.lists.ocfs2-devel,dev.linux.lists.v9fs,org.freedesktop.lists.amd-gfx,org.freedesktop.lists.dri-devel,org.infradead.lists.linux-mtd,org.kernel.vger.audit,org.kernel.vger.bpf,org.kernel.vger.ceph-devel,org.kernel.vger.ecryptfs,org.kernel.vger.linux-bluetooth,org.kernel.vger.linux-can,org.kernel.vger.linux-cifs,org.kernel.vger.linux-ext4,org.kernel.vger.linux-fscrypt,org.kernel.vger.linux-fsdevel,org.kernel.vger.linux-hams,org.kernel.vger.linux-integrity,org.kernel.vger.linux-kernel,org.kernel.vger.linux-media,org.kernel.vger.linux-nfs,org.kernel.vger.linux-nilfs,org.kernel.vger.linux-perf-users,org.kernel.vger.linux-sctp,org.kernel.vger.linux-security-module,org.kernel.vger.linux-trace-kernel,org.kernel.vger.linux-unionfs,org.kernel.vger.linux-x25,org.kernel.vger.linux-xfs,org.kernel.vger.netdev,org.kernel.vger.selinux,org.kvack.linux-mm |
|---|---|
| Message-ID | <[email protected]> |
--w7xntzjnabubqe7a Content-Type: text/plain; protected-headers=v1; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Subject: Re: [PATCH v2 004/110] net: change sock.sk_ino and sock_i_ino() to u64 MIME-Version: 1.0 On 02.03.2026 15:23:48, Jeff Layton wrote: > inode->i_ino is being converted to a u64. sock.sk_ino (which caches the > inode number) must also be widened to avoid truncation on 32-bit > architectures where unsigned long is only 32 bits. > > Change sk_ino from unsigned long to u64, and update the return type > of sock_i_ino() to match. Fix all format strings that print the > result of sock_i_ino() (%lu -> %llu), and widen the intermediate > variables and function parameters in the diag modules that were > using int to hold the inode number. > > Note that the UAPI socket diag structures (inet_diag_msg.idiag_inode, > unix_diag_msg.udiag_ino, etc.) are all __u32 and cannot be changed > without breaking the ABI. The assignments to those fields will > silently truncate, which is the existing behavior. > > Signed-off-by: Jeff Layton <[email protected]> > --- > net/can/bcm.c | 2 +- Acked-by: Marc Kleine-Budde <[email protected]> # for net/can regards, Marc --=20 Pengutronix e.K. | Marc Kleine-Budde | Embedded Linux | https://www.pengutronix.de | Vertretung N=C3=BCrnberg | Phone: +49-5121-206917-129 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 | --w7xntzjnabubqe7a Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYKAB0WIQSl+MghEFFAdY3pYJLMOmT6rpmt0gUCaaax+QAKCRDMOmT6rpmt 0hl7AQCzPgPTWe6ol8KwtKBDfHnqkx4Ku1cIiwlQY4Hnx5jz1wD5ASVd5abGOb50 lF4hQNIazRvyjJwKRF+va5JwX/SN2go= =y8vr -----END PGP SIGNATURE----- --w7xntzjnabubqe7a--