Re: [PATCH v2 076/110] ceph: replace PRIino with %llu/%llx format strings
Viacheslav Dubeyko <[email protected]> Mon, 2 Mar 2026 22:39:24 +0000
| Newsgroups | org.kernel.vger.linux-x25,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,net.sourceforge.lists.linux-f2fs-devel,org.freedesktop.lists.amd-gfx,org.freedesktop.lists.dri-devel,org.infradead.lists.linux-mtd,org.kernel.vger.audit,org.kernel.vger.autofs,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-xfs,org.kernel.vger.netdev,org.kernel.vger.selinux,org.kvack.linux-mm |
|---|---|
| Message-ID | <[email protected]> |
On Mon, 2026-03-02 at 15:25 -0500, Jeff Layton wrote: > Now that i_ino is u64 and the PRIino format macro has been removed, > replace all uses in ceph with the concrete format strings. > > Signed-off-by: Jeff Layton <[email protected]> > --- > fs/ceph/crypto.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/fs/ceph/crypto.c b/fs/ceph/crypto.c > index 718c194ba5d8ce22c6a5d1dd687ec3761263e7e1..64d2407592772ed61fb95b62401db194112f48a8 100644 > --- a/fs/ceph/crypto.c > +++ b/fs/ceph/crypto.c > @@ -272,7 +272,7 @@ int ceph_encode_encrypted_dname(struct inode *parent, char *buf, int elen) > /* To understand the 240 limit, see CEPH_NOHASH_NAME_MAX comments */ > WARN_ON(elen > 240); > if (dir != parent) // leading _ is already there; append _<inum> > - elen += 1 + sprintf(p + elen, "_%" PRIino "u", dir->i_ino); > + elen += 1 + sprintf(p + elen, "_%llu", dir->i_ino); > > out: > kfree(cryptbuf); > @@ -377,7 +377,7 @@ int ceph_fname_to_usr(const struct ceph_fname *fname, struct fscrypt_str *tname, > if (!ret && (dir != fname->dir)) { > char tmp_buf[BASE64_CHARS(NAME_MAX)]; > > - name_len = snprintf(tmp_buf, sizeof(tmp_buf), "_%.*s_%" PRIino "u", > + name_len = snprintf(tmp_buf, sizeof(tmp_buf), "_%.*s_%llu", > oname->len, oname->name, dir->i_ino); > memcpy(oname->name, tmp_buf, name_len); > oname->len = name_len; Reviewed-by: Viacheslav Dubeyko <[email protected]> Thanks, Slava.