From patchwork Tue Jul 2 00:57:07 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Darrick J. Wong" X-Patchwork-Id: 13718764 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 803DF6FC3 for ; Tue, 2 Jul 2024 00:57:08 +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=1719881828; cv=none; b=fwam//MCPaZn4kxgunpfHEFkdU9lOy4aj8HjI/IuuDiyfKiulYSKiOT1C84gZyQV4JiEYYzu2+mSgdSaJh8V/ZPnufQD9GcZ5XBq+PaorMZ1UaM4UNX1325NObiWIe1ek3Iu++BCerQ0nCK0AW1yrAbd6O7Ny/YSVj2GaU6yrj0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1719881828; c=relaxed/simple; bh=C4xplnEop356k8k4mzzu0yisNV+Pf+o7bA8bi6kLjEU=; h=Date:Subject:From:To:Cc:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=VzLi9Zw4fHUSK97HI2gAGKFgd53SE6hn9Ksro2KowoVlpmgS9R/3UFVifWT9eSI5vFkO14VfCeQczQqi5PHCmQORgCGTeDS+WD2cXYLMeSIQW+WCrnhZP5ZE3zld/HKIspqPKcmYvKE8+VPkicIVrF7jnkzrwEGg1hZcixlbyUo= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=Jc+yKMxU; 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="Jc+yKMxU" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5EF3BC116B1; Tue, 2 Jul 2024 00:57:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1719881828; bh=C4xplnEop356k8k4mzzu0yisNV+Pf+o7bA8bi6kLjEU=; h=Date:Subject:From:To:Cc:In-Reply-To:References:From; b=Jc+yKMxUp4SYcErt7wqwV4BKSFBiBJT7t0ZxWdL/SRa0KZ1E9eCW7Av2Zsdd5nSLE eKVM/7OQRYseTuve7XtThWIgsTnXLUe58zxOuA0ul1yHDPxZ5zya7ehYV+c77MD1Ty HbQhCKjEO04B28WIOilLzVETPkTfcjjcRD7JmTwl6Go9vEKxu6vJcGzbAyFXUQV9lI jZbZF7FZnsDYnPITpuZMLUSsY0RkfFhSifgaLaypCZ82bGyokpRVAtBFiFObZrPmTg MPheEy7WvaJBc1QVzg2CGTctAa3Pb3N5kaBS55Eau9Mf1hMWPUOoIFZmwiOz/gMzwR +f+tWJSriwytw== Date: Mon, 01 Jul 2024 17:57:07 -0700 Subject: [PATCH 2/3] libxfs: port the bumplink function from the kernel From: "Darrick J. Wong" To: djwong@kernel.org, cem@kernel.org Cc: linux-xfs@vger.kernel.org, hch@lst.de Message-ID: <171988117255.2006964.327302486814604596.stgit@frogsfrogsfrogs> In-Reply-To: <171988117219.2006964.1550137506522221205.stgit@frogsfrogsfrogs> References: <171988117219.2006964.1550137506522221205.stgit@frogsfrogsfrogs> User-Agent: StGit/0.19 Precedence: bulk X-Mailing-List: linux-xfs@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 From: Darrick J. Wong Port the xfs_bumplink function from the kernel and use it to replace raw calls to inc_nlink. The next patch will need this common function to prevent integer overflows in the link count. Signed-off-by: Darrick J. Wong Reviewed-by: Christoph Hellwig --- include/xfs_inode.h | 2 ++ libxfs/util.c | 17 +++++++++++++++++ mkfs/proto.c | 4 ++-- repair/phase6.c | 10 +++++----- 4 files changed, 26 insertions(+), 7 deletions(-) diff --git a/include/xfs_inode.h b/include/xfs_inode.h index c6e4f84bdc16..45339b42621a 100644 --- a/include/xfs_inode.h +++ b/include/xfs_inode.h @@ -359,6 +359,8 @@ extern void libxfs_trans_ichgtime(struct xfs_trans *, struct xfs_inode *, int); extern int libxfs_iflush_int (struct xfs_inode *, struct xfs_buf *); +void libxfs_bumplink(struct xfs_trans *tp, struct xfs_inode *ip); + /* Inode Cache Interfaces */ extern int libxfs_iget(struct xfs_mount *, struct xfs_trans *, xfs_ino_t, uint, struct xfs_inode **); diff --git a/libxfs/util.c b/libxfs/util.c index 2656c99a8ea7..dc54e3ee66db 100644 --- a/libxfs/util.c +++ b/libxfs/util.c @@ -240,6 +240,23 @@ xfs_inode_propagate_flags( ip->i_diflags |= di_flags; } +/* + * Increment the link count on an inode & log the change. + */ +void +libxfs_bumplink( + struct xfs_trans *tp, + struct xfs_inode *ip) +{ + struct inode *inode = VFS_I(ip); + + xfs_trans_ichgtime(tp, ip, XFS_ICHGTIME_CHG); + + inc_nlink(inode); + + xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE); +} + /* * Initialise a newly allocated inode and return the in-core inode to the * caller locked exclusively. diff --git a/mkfs/proto.c b/mkfs/proto.c index 5125ee44f493..a9a9b704a3ca 100644 --- a/mkfs/proto.c +++ b/mkfs/proto.c @@ -591,7 +591,7 @@ parseproto( &creds, fsxp, &ip); if (error) fail(_("Inode allocation failed"), error); - inc_nlink(VFS_I(ip)); /* account for . */ + libxfs_bumplink(tp, ip); /* account for . */ if (!pip) { pip = ip; mp->m_sb.sb_rootino = ip->i_ino; @@ -601,7 +601,7 @@ parseproto( libxfs_trans_ijoin(tp, pip, 0); xname.type = XFS_DIR3_FT_DIR; newdirent(mp, tp, pip, &xname, ip->i_ino); - inc_nlink(VFS_I(pip)); + libxfs_bumplink(tp, pip); libxfs_trans_log_inode(tp, pip, XFS_ILOG_CORE); } newdirectory(mp, tp, ip, pip); diff --git a/repair/phase6.c b/repair/phase6.c index 92a58db0d236..47dd9de2741c 100644 --- a/repair/phase6.c +++ b/repair/phase6.c @@ -947,7 +947,7 @@ mk_orphanage(xfs_mount_t *mp) do_error(_("%s inode allocation failed %d\n"), ORPHANAGE, error); } - inc_nlink(VFS_I(ip)); /* account for . */ + libxfs_bumplink(tp, ip); /* account for . */ ino = ip->i_ino; irec = find_inode_rec(mp, @@ -999,7 +999,7 @@ mk_orphanage(xfs_mount_t *mp) * for .. in the new directory, and update the irec copy of the * on-disk nlink so we don't fail the link count check later. */ - inc_nlink(VFS_I(pip)); + libxfs_bumplink(tp, pip); irec = find_inode_rec(mp, XFS_INO_TO_AGNO(mp, mp->m_sb.sb_rootino), XFS_INO_TO_AGINO(mp, mp->m_sb.sb_rootino)); add_inode_ref(irec, 0); @@ -1094,7 +1094,7 @@ mv_orphanage( if (irec) add_inode_ref(irec, ino_offset); else - inc_nlink(VFS_I(orphanage_ip)); + libxfs_bumplink(tp, orphanage_ip); libxfs_trans_log_inode(tp, orphanage_ip, XFS_ILOG_CORE); err = -libxfs_dir_createname(tp, ino_p, &xfs_name_dotdot, @@ -1103,7 +1103,7 @@ mv_orphanage( do_error( _("creation of .. entry failed (%d)\n"), err); - inc_nlink(VFS_I(ino_p)); + libxfs_bumplink(tp, ino_p); libxfs_trans_log_inode(tp, ino_p, XFS_ILOG_CORE); err = -libxfs_trans_commit(tp); if (err) @@ -1128,7 +1128,7 @@ mv_orphanage( if (irec) add_inode_ref(irec, ino_offset); else - inc_nlink(VFS_I(orphanage_ip)); + libxfs_bumplink(tp, orphanage_ip); libxfs_trans_log_inode(tp, orphanage_ip, XFS_ILOG_CORE); /*