* [PATCH] xfs: remove the debug-only q_transp field from struct xfs_dquot
@ 2019-05-15 8:10 Christoph Hellwig
2019-05-15 12:49 ` Brian Foster
0 siblings, 1 reply; 2+ messages in thread
From: Christoph Hellwig @ 2019-05-15 8:10 UTC (permalink / raw)
To: linux-xfs
The field is only used for a few assertations. Shrink the dqout
structure instead, similarly to what commit f3ca87389dbf
("xfs: remove i_transp") did for the xfs_inode.
Signed-off-by: Christoph Hellwig <hch@lst.de>
---
fs/xfs/xfs_dquot.h | 1 -
fs/xfs/xfs_dquot_item.c | 5 -----
fs/xfs/xfs_trans_dquot.c | 10 ----------
3 files changed, 16 deletions(-)
diff --git a/fs/xfs/xfs_dquot.h b/fs/xfs/xfs_dquot.h
index 64bd8640f6e8..4fe85709d55d 100644
--- a/fs/xfs/xfs_dquot.h
+++ b/fs/xfs/xfs_dquot.h
@@ -34,7 +34,6 @@ typedef struct xfs_dquot {
uint dq_flags; /* various flags (XFS_DQ_*) */
struct list_head q_lru; /* global free list of dquots */
struct xfs_mount*q_mount; /* filesystem this relates to */
- struct xfs_trans*q_transp; /* trans this belongs to currently */
uint q_nrefs; /* # active refs from inodes */
xfs_daddr_t q_blkno; /* blkno of dquot buffer */
int q_bufoffset; /* off of dq in buffer (# dquots) */
diff --git a/fs/xfs/xfs_dquot_item.c b/fs/xfs/xfs_dquot_item.c
index 7dedd17c4813..87b23ae44397 100644
--- a/fs/xfs/xfs_dquot_item.c
+++ b/fs/xfs/xfs_dquot_item.c
@@ -223,11 +223,6 @@ xfs_qm_dquot_logitem_unlock(
ASSERT(XFS_DQ_IS_LOCKED(dqp));
- /*
- * Clear the transaction pointer in the dquot
- */
- dqp->q_transp = NULL;
-
/*
* dquots are never 'held' from getting unlocked at the end of
* a transaction. Their locking and unlocking is hidden inside the
diff --git a/fs/xfs/xfs_trans_dquot.c b/fs/xfs/xfs_trans_dquot.c
index cd664a03613f..ba3de1f03b98 100644
--- a/fs/xfs/xfs_trans_dquot.c
+++ b/fs/xfs/xfs_trans_dquot.c
@@ -29,7 +29,6 @@ xfs_trans_dqjoin(
xfs_trans_t *tp,
xfs_dquot_t *dqp)
{
- ASSERT(dqp->q_transp != tp);
ASSERT(XFS_DQ_IS_LOCKED(dqp));
ASSERT(dqp->q_logitem.qli_dquot == dqp);
@@ -37,15 +36,8 @@ xfs_trans_dqjoin(
* Get a log_item_desc to point at the new item.
*/
xfs_trans_add_item(tp, &dqp->q_logitem.qli_item);
-
- /*
- * Initialize d_transp so we can later determine if this dquot is
- * associated with this transaction.
- */
- dqp->q_transp = tp;
}
-
/*
* This is called to mark the dquot as needing
* to be logged when the transaction is committed. The dquot must
@@ -61,7 +53,6 @@ xfs_trans_log_dquot(
xfs_trans_t *tp,
xfs_dquot_t *dqp)
{
- ASSERT(dqp->q_transp == tp);
ASSERT(XFS_DQ_IS_LOCKED(dqp));
tp->t_flags |= XFS_TRANS_DIRTY;
@@ -347,7 +338,6 @@ xfs_trans_apply_dquot_deltas(
break;
ASSERT(XFS_DQ_IS_LOCKED(dqp));
- ASSERT(dqp->q_transp == tp);
/*
* adjust the actual number of blocks used
--
2.20.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] xfs: remove the debug-only q_transp field from struct xfs_dquot
2019-05-15 8:10 [PATCH] xfs: remove the debug-only q_transp field from struct xfs_dquot Christoph Hellwig
@ 2019-05-15 12:49 ` Brian Foster
0 siblings, 0 replies; 2+ messages in thread
From: Brian Foster @ 2019-05-15 12:49 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: linux-xfs
On Wed, May 15, 2019 at 10:10:45AM +0200, Christoph Hellwig wrote:
> The field is only used for a few assertations. Shrink the dqout
> structure instead, similarly to what commit f3ca87389dbf
> ("xfs: remove i_transp") did for the xfs_inode.
>
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> ---
Reviewed-by: Brian Foster <bfoster@redhat.com>
> fs/xfs/xfs_dquot.h | 1 -
> fs/xfs/xfs_dquot_item.c | 5 -----
> fs/xfs/xfs_trans_dquot.c | 10 ----------
> 3 files changed, 16 deletions(-)
>
> diff --git a/fs/xfs/xfs_dquot.h b/fs/xfs/xfs_dquot.h
> index 64bd8640f6e8..4fe85709d55d 100644
> --- a/fs/xfs/xfs_dquot.h
> +++ b/fs/xfs/xfs_dquot.h
> @@ -34,7 +34,6 @@ typedef struct xfs_dquot {
> uint dq_flags; /* various flags (XFS_DQ_*) */
> struct list_head q_lru; /* global free list of dquots */
> struct xfs_mount*q_mount; /* filesystem this relates to */
> - struct xfs_trans*q_transp; /* trans this belongs to currently */
> uint q_nrefs; /* # active refs from inodes */
> xfs_daddr_t q_blkno; /* blkno of dquot buffer */
> int q_bufoffset; /* off of dq in buffer (# dquots) */
> diff --git a/fs/xfs/xfs_dquot_item.c b/fs/xfs/xfs_dquot_item.c
> index 7dedd17c4813..87b23ae44397 100644
> --- a/fs/xfs/xfs_dquot_item.c
> +++ b/fs/xfs/xfs_dquot_item.c
> @@ -223,11 +223,6 @@ xfs_qm_dquot_logitem_unlock(
>
> ASSERT(XFS_DQ_IS_LOCKED(dqp));
>
> - /*
> - * Clear the transaction pointer in the dquot
> - */
> - dqp->q_transp = NULL;
> -
> /*
> * dquots are never 'held' from getting unlocked at the end of
> * a transaction. Their locking and unlocking is hidden inside the
> diff --git a/fs/xfs/xfs_trans_dquot.c b/fs/xfs/xfs_trans_dquot.c
> index cd664a03613f..ba3de1f03b98 100644
> --- a/fs/xfs/xfs_trans_dquot.c
> +++ b/fs/xfs/xfs_trans_dquot.c
> @@ -29,7 +29,6 @@ xfs_trans_dqjoin(
> xfs_trans_t *tp,
> xfs_dquot_t *dqp)
> {
> - ASSERT(dqp->q_transp != tp);
> ASSERT(XFS_DQ_IS_LOCKED(dqp));
> ASSERT(dqp->q_logitem.qli_dquot == dqp);
>
> @@ -37,15 +36,8 @@ xfs_trans_dqjoin(
> * Get a log_item_desc to point at the new item.
> */
> xfs_trans_add_item(tp, &dqp->q_logitem.qli_item);
> -
> - /*
> - * Initialize d_transp so we can later determine if this dquot is
> - * associated with this transaction.
> - */
> - dqp->q_transp = tp;
> }
>
> -
> /*
> * This is called to mark the dquot as needing
> * to be logged when the transaction is committed. The dquot must
> @@ -61,7 +53,6 @@ xfs_trans_log_dquot(
> xfs_trans_t *tp,
> xfs_dquot_t *dqp)
> {
> - ASSERT(dqp->q_transp == tp);
> ASSERT(XFS_DQ_IS_LOCKED(dqp));
>
> tp->t_flags |= XFS_TRANS_DIRTY;
> @@ -347,7 +338,6 @@ xfs_trans_apply_dquot_deltas(
> break;
>
> ASSERT(XFS_DQ_IS_LOCKED(dqp));
> - ASSERT(dqp->q_transp == tp);
>
> /*
> * adjust the actual number of blocks used
> --
> 2.20.1
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-05-15 12:49 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-15 8:10 [PATCH] xfs: remove the debug-only q_transp field from struct xfs_dquot Christoph Hellwig
2019-05-15 12:49 ` Brian Foster
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.