On Mon 02-03-26 15:24:46, Jeff Layton wrote: > Now that i_ino is u64 and the PRIino format macro has been removed, > replace all uses in jbd2 with the concrete format strings. > > Signed-off-by: Jeff Layton <[email protected]>
Looks good. Feel free to add: Reviewed-by: Jan Kara <[email protected]> Honza > --- > fs/jbd2/journal.c | 4 ++-- > fs/jbd2/transaction.c | 2 +- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/fs/jbd2/journal.c b/fs/jbd2/journal.c > index > 9df937f0e15c71028038e1c0c12159421a2444b4..b60918ed8a9965efe42419e6de3f449be50f7ca0 > 100644 > --- a/fs/jbd2/journal.c > +++ b/fs/jbd2/journal.c > @@ -1677,7 +1677,7 @@ journal_t *jbd2_journal_init_inode(struct inode *inode) > return err ? ERR_PTR(err) : ERR_PTR(-EINVAL); > } > > - jbd2_debug(1, "JBD2: inode %s/%" PRIino "u, size %lld, bits %d, blksize > %ld\n", > + jbd2_debug(1, "JBD2: inode %s/%llu, size %lld, bits %d, blksize %ld\n", > inode->i_sb->s_id, inode->i_ino, (long long) inode->i_size, > inode->i_sb->s_blocksize_bits, inode->i_sb->s_blocksize); > > @@ -1689,7 +1689,7 @@ journal_t *jbd2_journal_init_inode(struct inode *inode) > > journal->j_inode = inode; > snprintf(journal->j_devname, sizeof(journal->j_devname), > - "%pg-%" PRIino "u", journal->j_dev, journal->j_inode->i_ino); > + "%pg-%llu", journal->j_dev, journal->j_inode->i_ino); > strreplace(journal->j_devname, '/', '!'); > jbd2_stats_proc_init(journal); > > diff --git a/fs/jbd2/transaction.c b/fs/jbd2/transaction.c > index > 2a03d4eafdee95e5caa8dbd0afe4e32ef4104378..a90f9092706cceea56c1100f7d40ccba0d50adba > 100644 > --- a/fs/jbd2/transaction.c > +++ b/fs/jbd2/transaction.c > @@ -2651,7 +2651,7 @@ static int jbd2_journal_file_inode(handle_t *handle, > struct jbd2_inode *jinode, > return -EROFS; > journal = transaction->t_journal; > > - jbd2_debug(4, "Adding inode %" PRIino "u, tid:%d\n", > jinode->i_vfs_inode->i_ino, > + jbd2_debug(4, "Adding inode %llu, tid:%d\n", jinode->i_vfs_inode->i_ino, > transaction->t_tid); > > spin_lock(&journal->j_list_lock); > > -- > 2.53.0 > -- Jan Kara <[email protected]> SUSE Labs, CR
