From: "Darrick J. Wong" <djwong@kernel.org>
To: Christoph Hellwig <hch@lst.de>
Cc: linux-xfs@vger.kernel.org, Carlos Maiolino <cmaiolino@redhat.com>
Subject: Re: [PATCH 1/3] xfs: remove support for disabling quota accounting on a mounted file system
Date: Thu, 22 Jul 2021 12:07:03 -0700 [thread overview]
Message-ID: <20210722190703.GI559212@magnolia> (raw)
In-Reply-To: <20210722190307.GA14569@lst.de>
On Thu, Jul 22, 2021 at 09:03:07PM +0200, Christoph Hellwig wrote:
> On Thu, Jul 22, 2021 at 11:22:54AM -0700, Darrick J. Wong wrote:
> > <snip> I think you could delete more...
>
> > ...because I think xfs_dqrele_all_inodes has lost all of its callers.
> > Can you please remove it and XFS_ICWALK_DQRELE from xfs_icache.[ch]?
>
> ---
> From e8d07e55103636ce3207788e430eeff12b9ef153 Mon Sep 17 00:00:00 2001
> From: Christoph Hellwig <hch@lst.de>
> Date: Thu, 22 Jul 2021 20:58:55 +0200
> Subject: xfs: remove xfs_dqrele_all_inodes
>
> xfs_dqrele_all_inodes is unused now, remove it and all supporting code.
>
> Signed-off-by: Christoph Hellwig <hch@lst.de>
For this patch (which I guess is now patch 4?) and the original patch 1,
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
--D
> ---
> fs/xfs/xfs_icache.c | 107 +-------------------------------------------
> fs/xfs/xfs_icache.h | 6 ---
> 2 files changed, 1 insertion(+), 112 deletions(-)
>
> diff --git a/fs/xfs/xfs_icache.c b/fs/xfs/xfs_icache.c
> index 6007683482c625..086a88b8dfdb39 100644
> --- a/fs/xfs/xfs_icache.c
> +++ b/fs/xfs/xfs_icache.c
> @@ -38,9 +38,6 @@
> * radix tree tags when convenient. Avoid existing XFS_IWALK namespace.
> */
> enum xfs_icwalk_goal {
> - /* Goals that are not related to tags; these must be < 0. */
> - XFS_ICWALK_DQRELE = -1,
> -
> /* Goals directly associated with tagged inodes. */
> XFS_ICWALK_BLOCKGC = XFS_ICI_BLOCKGC_TAG,
> XFS_ICWALK_RECLAIM = XFS_ICI_RECLAIM_TAG,
> @@ -64,9 +61,6 @@ static int xfs_icwalk_ag(struct xfs_perag *pag,
> * Private inode cache walk flags for struct xfs_icwalk. Must not
> * coincide with XFS_ICWALK_FLAGS_VALID.
> */
> -#define XFS_ICWALK_FLAG_DROP_UDQUOT (1U << 31)
> -#define XFS_ICWALK_FLAG_DROP_GDQUOT (1U << 30)
> -#define XFS_ICWALK_FLAG_DROP_PDQUOT (1U << 29)
>
> /* Stop scanning after icw_scan_limit inodes. */
> #define XFS_ICWALK_FLAG_SCAN_LIMIT (1U << 28)
> @@ -74,10 +68,7 @@ static int xfs_icwalk_ag(struct xfs_perag *pag,
> #define XFS_ICWALK_FLAG_RECLAIM_SICK (1U << 27)
> #define XFS_ICWALK_FLAG_UNION (1U << 26) /* union filter algorithm */
>
> -#define XFS_ICWALK_PRIVATE_FLAGS (XFS_ICWALK_FLAG_DROP_UDQUOT | \
> - XFS_ICWALK_FLAG_DROP_GDQUOT | \
> - XFS_ICWALK_FLAG_DROP_PDQUOT | \
> - XFS_ICWALK_FLAG_SCAN_LIMIT | \
> +#define XFS_ICWALK_PRIVATE_FLAGS (XFS_ICWALK_FLAG_SCAN_LIMIT | \
> XFS_ICWALK_FLAG_RECLAIM_SICK | \
> XFS_ICWALK_FLAG_UNION)
>
> @@ -817,97 +808,6 @@ xfs_icache_inode_is_allocated(
> return 0;
> }
>
> -#ifdef CONFIG_XFS_QUOTA
> -/* Decide if we want to grab this inode to drop its dquots. */
> -static bool
> -xfs_dqrele_igrab(
> - struct xfs_inode *ip)
> -{
> - bool ret = false;
> -
> - ASSERT(rcu_read_lock_held());
> -
> - /* Check for stale RCU freed inode */
> - spin_lock(&ip->i_flags_lock);
> - if (!ip->i_ino)
> - goto out_unlock;
> -
> - /*
> - * Skip inodes that are anywhere in the reclaim machinery because we
> - * drop dquots before tagging an inode for reclamation.
> - */
> - if (ip->i_flags & (XFS_IRECLAIM | XFS_IRECLAIMABLE))
> - goto out_unlock;
> -
> - /*
> - * The inode looks alive; try to grab a VFS reference so that it won't
> - * get destroyed. If we got the reference, return true to say that
> - * we grabbed the inode.
> - *
> - * If we can't get the reference, then we know the inode had its VFS
> - * state torn down and hasn't yet entered the reclaim machinery. Since
> - * we also know that dquots are detached from an inode before it enters
> - * reclaim, we can skip the inode.
> - */
> - ret = igrab(VFS_I(ip)) != NULL;
> -
> -out_unlock:
> - spin_unlock(&ip->i_flags_lock);
> - return ret;
> -}
> -
> -/* Drop this inode's dquots. */
> -static void
> -xfs_dqrele_inode(
> - struct xfs_inode *ip,
> - struct xfs_icwalk *icw)
> -{
> - if (xfs_iflags_test(ip, XFS_INEW))
> - xfs_inew_wait(ip);
> -
> - xfs_ilock(ip, XFS_ILOCK_EXCL);
> - if (icw->icw_flags & XFS_ICWALK_FLAG_DROP_UDQUOT) {
> - xfs_qm_dqrele(ip->i_udquot);
> - ip->i_udquot = NULL;
> - }
> - if (icw->icw_flags & XFS_ICWALK_FLAG_DROP_GDQUOT) {
> - xfs_qm_dqrele(ip->i_gdquot);
> - ip->i_gdquot = NULL;
> - }
> - if (icw->icw_flags & XFS_ICWALK_FLAG_DROP_PDQUOT) {
> - xfs_qm_dqrele(ip->i_pdquot);
> - ip->i_pdquot = NULL;
> - }
> - xfs_iunlock(ip, XFS_ILOCK_EXCL);
> - xfs_irele(ip);
> -}
> -
> -/*
> - * Detach all dquots from incore inodes if we can. The caller must already
> - * have dropped the relevant XFS_[UGP]QUOTA_ACTIVE flags so that dquots will
> - * not get reattached.
> - */
> -int
> -xfs_dqrele_all_inodes(
> - struct xfs_mount *mp,
> - unsigned int qflags)
> -{
> - struct xfs_icwalk icw = { .icw_flags = 0 };
> -
> - if (qflags & XFS_UQUOTA_ACCT)
> - icw.icw_flags |= XFS_ICWALK_FLAG_DROP_UDQUOT;
> - if (qflags & XFS_GQUOTA_ACCT)
> - icw.icw_flags |= XFS_ICWALK_FLAG_DROP_GDQUOT;
> - if (qflags & XFS_PQUOTA_ACCT)
> - icw.icw_flags |= XFS_ICWALK_FLAG_DROP_PDQUOT;
> -
> - return xfs_icwalk(mp, XFS_ICWALK_DQRELE, &icw);
> -}
> -#else
> -# define xfs_dqrele_igrab(ip) (false)
> -# define xfs_dqrele_inode(ip, priv) ((void)0)
> -#endif /* CONFIG_XFS_QUOTA */
> -
> /*
> * Grab the inode for reclaim exclusively.
> *
> @@ -1647,8 +1547,6 @@ xfs_icwalk_igrab(
> struct xfs_icwalk *icw)
> {
> switch (goal) {
> - case XFS_ICWALK_DQRELE:
> - return xfs_dqrele_igrab(ip);
> case XFS_ICWALK_BLOCKGC:
> return xfs_blockgc_igrab(ip);
> case XFS_ICWALK_RECLAIM:
> @@ -1672,9 +1570,6 @@ xfs_icwalk_process_inode(
> int error = 0;
>
> switch (goal) {
> - case XFS_ICWALK_DQRELE:
> - xfs_dqrele_inode(ip, icw);
> - break;
> case XFS_ICWALK_BLOCKGC:
> error = xfs_blockgc_scan_inode(ip, icw);
> break;
> diff --git a/fs/xfs/xfs_icache.h b/fs/xfs/xfs_icache.h
> index c751cc32dc4634..d0062ebb3f7ac8 100644
> --- a/fs/xfs/xfs_icache.h
> +++ b/fs/xfs/xfs_icache.h
> @@ -68,12 +68,6 @@ void xfs_inode_clear_cowblocks_tag(struct xfs_inode *ip);
>
> void xfs_blockgc_worker(struct work_struct *work);
>
> -#ifdef CONFIG_XFS_QUOTA
> -int xfs_dqrele_all_inodes(struct xfs_mount *mp, unsigned int qflags);
> -#else
> -# define xfs_dqrele_all_inodes(mp, qflags) (0)
> -#endif
> -
> int xfs_icache_inode_is_allocated(struct xfs_mount *mp, struct xfs_trans *tp,
> xfs_ino_t ino, bool *inuse);
>
> --
> 2.30.2
>
next prev parent reply other threads:[~2021-07-22 19:08 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-22 7:26 don't allow disabling quota accounting on a mounted file system v2 Christoph Hellwig
2021-07-22 7:26 ` [PATCH 1/3] xfs: remove support for disabling quota accounting on a mounted file system Christoph Hellwig
2021-07-22 18:22 ` Darrick J. Wong
2021-07-22 19:03 ` Christoph Hellwig
2021-07-22 19:07 ` Darrick J. Wong [this message]
2021-07-23 5:05 ` Christoph Hellwig
2021-07-22 7:26 ` [PATCH 2/3] xfs: remove the flags argument to xfs_qm_dquot_walk Christoph Hellwig
2021-07-22 7:26 ` [PATCH 3/3] xfs: remove the active vs running quota differentiation Christoph Hellwig
-- strict thread matches above, loose matches on Subject: below --
2021-07-12 11:14 don't allow disabling quota accounting on a mounted file system Christoph Hellwig
2021-07-12 11:14 ` [PATCH 1/3] xfs: remove support for " Christoph Hellwig
2021-07-14 11:10 ` Carlos Maiolino
2021-07-20 6:33 ` Christoph Hellwig
2021-07-14 21:44 ` Darrick J. Wong
2021-07-15 17:15 ` Darrick J. Wong
2021-07-20 6:35 ` Christoph Hellwig
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210722190703.GI559212@magnolia \
--to=djwong@kernel.org \
--cc=cmaiolino@redhat.com \
--cc=hch@lst.de \
--cc=linux-xfs@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).