From: Brian Foster <bfoster@redhat.com>
To: Christoph Hellwig <hch@lst.de>
Cc: linux-xfs@vger.kernel.org
Subject: Re: [PATCH 10/20] xfs: remove the xfs_log_item_t typedef
Date: Fri, 17 May 2019 13:50:43 -0400 [thread overview]
Message-ID: <20190517175043.GJ7888@bfoster> (raw)
In-Reply-To: <20190517073119.30178-11-hch@lst.de>
On Fri, May 17, 2019 at 09:31:09AM +0200, Christoph Hellwig wrote:
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> ---
Reviewed-by: Brian Foster <bfoster@redhat.com>
> fs/xfs/xfs_buf_item.c | 6 +++---
> fs/xfs/xfs_buf_item.h | 6 +++---
> fs/xfs/xfs_dquot_item.h | 4 ++--
> fs/xfs/xfs_extfree_item.h | 4 ++--
> fs/xfs/xfs_inode.c | 12 +++++------
> fs/xfs/xfs_inode_item.h | 2 +-
> fs/xfs/xfs_log_recover.c | 2 +-
> fs/xfs/xfs_trans.h | 16 +++++++-------
> fs/xfs/xfs_trans_ail.c | 44 +++++++++++++++++++--------------------
> 9 files changed, 48 insertions(+), 48 deletions(-)
>
> diff --git a/fs/xfs/xfs_buf_item.c b/fs/xfs/xfs_buf_item.c
> index 7193ee9ca5b8..05eefc677cd8 100644
> --- a/fs/xfs/xfs_buf_item.c
> +++ b/fs/xfs/xfs_buf_item.c
> @@ -981,9 +981,9 @@ xfs_buf_item_relse(
> */
> void
> xfs_buf_attach_iodone(
> - xfs_buf_t *bp,
> - void (*cb)(xfs_buf_t *, xfs_log_item_t *),
> - xfs_log_item_t *lip)
> + struct xfs_buf *bp,
> + void (*cb)(struct xfs_buf *, struct xfs_log_item *),
> + struct xfs_log_item *lip)
> {
> ASSERT(xfs_buf_islocked(bp));
>
> diff --git a/fs/xfs/xfs_buf_item.h b/fs/xfs/xfs_buf_item.h
> index 90f65f891fab..4a054b11011a 100644
> --- a/fs/xfs/xfs_buf_item.h
> +++ b/fs/xfs/xfs_buf_item.h
> @@ -39,7 +39,7 @@ struct xfs_buf_log_item;
> * locked, and which 128 byte chunks of the buffer are dirty.
> */
> struct xfs_buf_log_item {
> - xfs_log_item_t bli_item; /* common item structure */
> + struct xfs_log_item bli_item; /* common item structure */
> struct xfs_buf *bli_buf; /* real buffer pointer */
> unsigned int bli_flags; /* misc flags */
> unsigned int bli_recur; /* lock recursion count */
> @@ -55,8 +55,8 @@ bool xfs_buf_item_put(struct xfs_buf_log_item *);
> void xfs_buf_item_log(struct xfs_buf_log_item *, uint, uint);
> bool xfs_buf_item_dirty_format(struct xfs_buf_log_item *);
> void xfs_buf_attach_iodone(struct xfs_buf *,
> - void(*)(struct xfs_buf *, xfs_log_item_t *),
> - xfs_log_item_t *);
> + void(*)(struct xfs_buf *, struct xfs_log_item *),
> + struct xfs_log_item *);
> void xfs_buf_iodone_callbacks(struct xfs_buf *);
> void xfs_buf_iodone(struct xfs_buf *, struct xfs_log_item *);
> bool xfs_buf_resubmit_failed_buffers(struct xfs_buf *,
> diff --git a/fs/xfs/xfs_dquot_item.h b/fs/xfs/xfs_dquot_item.h
> index db9df710a308..1aed34ccdabc 100644
> --- a/fs/xfs/xfs_dquot_item.h
> +++ b/fs/xfs/xfs_dquot_item.h
> @@ -12,13 +12,13 @@ struct xfs_mount;
> struct xfs_qoff_logitem;
>
> typedef struct xfs_dq_logitem {
> - xfs_log_item_t qli_item; /* common portion */
> + struct xfs_log_item qli_item; /* common portion */
> struct xfs_dquot *qli_dquot; /* dquot ptr */
> xfs_lsn_t qli_flush_lsn; /* lsn at last flush */
> } xfs_dq_logitem_t;
>
> typedef struct xfs_qoff_logitem {
> - xfs_log_item_t qql_item; /* common portion */
> + struct xfs_log_item qql_item; /* common portion */
> struct xfs_qoff_logitem *qql_start_lip; /* qoff-start logitem, if any */
> unsigned int qql_flags;
> } xfs_qoff_logitem_t;
> diff --git a/fs/xfs/xfs_extfree_item.h b/fs/xfs/xfs_extfree_item.h
> index 2a6a895ca73e..b0dc4ebe8892 100644
> --- a/fs/xfs/xfs_extfree_item.h
> +++ b/fs/xfs/xfs_extfree_item.h
> @@ -51,7 +51,7 @@ struct kmem_zone;
> * AIL, so at this point both the EFI and EFD are freed.
> */
> typedef struct xfs_efi_log_item {
> - xfs_log_item_t efi_item;
> + struct xfs_log_item efi_item;
> atomic_t efi_refcount;
> atomic_t efi_next_extent;
> unsigned long efi_flags; /* misc flags */
> @@ -64,7 +64,7 @@ typedef struct xfs_efi_log_item {
> * have been freed.
> */
> typedef struct xfs_efd_log_item {
> - xfs_log_item_t efd_item;
> + struct xfs_log_item efd_item;
> xfs_efi_log_item_t *efd_efip;
> uint efd_next_extent;
> xfs_efd_log_format_t efd_format;
> diff --git a/fs/xfs/xfs_inode.c b/fs/xfs/xfs_inode.c
> index 419eae485ff3..6076bae6eb21 100644
> --- a/fs/xfs/xfs_inode.c
> +++ b/fs/xfs/xfs_inode.c
> @@ -441,12 +441,12 @@ xfs_lock_inumorder(int lock_mode, int subclass)
> */
> static void
> xfs_lock_inodes(
> - xfs_inode_t **ips,
> - int inodes,
> - uint lock_mode)
> + struct xfs_inode **ips,
> + int inodes,
> + uint lock_mode)
> {
> - int attempts = 0, i, j, try_lock;
> - xfs_log_item_t *lp;
> + int attempts = 0, i, j, try_lock;
> + struct xfs_log_item *lp;
>
> /*
> * Currently supports between 2 and 5 inodes with exclusive locking. We
> @@ -551,7 +551,7 @@ xfs_lock_two_inodes(
> struct xfs_inode *temp;
> uint mode_temp;
> int attempts = 0;
> - xfs_log_item_t *lp;
> + struct xfs_log_item *lp;
>
> ASSERT(hweight32(ip0_mode) == 1);
> ASSERT(hweight32(ip1_mode) == 1);
> diff --git a/fs/xfs/xfs_inode_item.h b/fs/xfs/xfs_inode_item.h
> index 27081eba220c..07a60e74c39c 100644
> --- a/fs/xfs/xfs_inode_item.h
> +++ b/fs/xfs/xfs_inode_item.h
> @@ -14,7 +14,7 @@ struct xfs_inode;
> struct xfs_mount;
>
> typedef struct xfs_inode_log_item {
> - xfs_log_item_t ili_item; /* common portion */
> + struct xfs_log_item ili_item; /* common portion */
> struct xfs_inode *ili_inode; /* inode ptr */
> xfs_lsn_t ili_flush_lsn; /* lsn at last flush */
> xfs_lsn_t ili_last_lsn; /* lsn at last transaction */
> diff --git a/fs/xfs/xfs_log_recover.c b/fs/xfs/xfs_log_recover.c
> index 9329f5adbfbe..76023ea49356 100644
> --- a/fs/xfs/xfs_log_recover.c
> +++ b/fs/xfs/xfs_log_recover.c
> @@ -3463,7 +3463,7 @@ xlog_recover_efd_pass2(
> {
> xfs_efd_log_format_t *efd_formatp;
> xfs_efi_log_item_t *efip = NULL;
> - xfs_log_item_t *lip;
> + struct xfs_log_item *lip;
> uint64_t efi_id;
> struct xfs_ail_cursor cur;
> struct xfs_ail *ailp = log->l_ailp;
> diff --git a/fs/xfs/xfs_trans.h b/fs/xfs/xfs_trans.h
> index a38af44344bf..7a6ee0c2ce20 100644
> --- a/fs/xfs/xfs_trans.h
> +++ b/fs/xfs/xfs_trans.h
> @@ -27,7 +27,7 @@ struct xfs_cud_log_item;
> struct xfs_bui_log_item;
> struct xfs_bud_log_item;
>
> -typedef struct xfs_log_item {
> +struct xfs_log_item {
> struct list_head li_ail; /* AIL pointers */
> struct list_head li_trans; /* transaction list */
> xfs_lsn_t li_lsn; /* last on-disk lsn */
> @@ -48,7 +48,7 @@ typedef struct xfs_log_item {
> struct xfs_log_vec *li_lv; /* active log vector */
> struct xfs_log_vec *li_lv_shadow; /* standby vector */
> xfs_lsn_t li_seq; /* CIL commit seq */
> -} xfs_log_item_t;
> +};
>
> /*
> * li_flags use the (set/test/clear)_bit atomic interfaces because updates can
> @@ -68,15 +68,15 @@ typedef struct xfs_log_item {
>
> struct xfs_item_ops {
> unsigned flags;
> - void (*iop_size)(xfs_log_item_t *, int *, int *);
> - void (*iop_format)(xfs_log_item_t *, struct xfs_log_vec *);
> - void (*iop_pin)(xfs_log_item_t *);
> - void (*iop_unpin)(xfs_log_item_t *, int remove);
> + void (*iop_size)(struct xfs_log_item *, int *, int *);
> + void (*iop_format)(struct xfs_log_item *, struct xfs_log_vec *);
> + void (*iop_pin)(struct xfs_log_item *);
> + void (*iop_unpin)(struct xfs_log_item *, int remove);
> uint (*iop_push)(struct xfs_log_item *, struct list_head *);
> void (*iop_committing)(struct xfs_log_item *, xfs_lsn_t commit_lsn);
> void (*iop_release)(struct xfs_log_item *);
> - xfs_lsn_t (*iop_committed)(xfs_log_item_t *, xfs_lsn_t);
> - void (*iop_error)(xfs_log_item_t *, xfs_buf_t *);
> + xfs_lsn_t (*iop_committed)(struct xfs_log_item *, xfs_lsn_t);
> + void (*iop_error)(struct xfs_log_item *, xfs_buf_t *);
> };
>
> /*
> diff --git a/fs/xfs/xfs_trans_ail.c b/fs/xfs/xfs_trans_ail.c
> index 8509c4b59760..f239344fda1a 100644
> --- a/fs/xfs/xfs_trans_ail.c
> +++ b/fs/xfs/xfs_trans_ail.c
> @@ -74,29 +74,29 @@ xfs_ail_check(
> * Return a pointer to the last item in the AIL. If the AIL is empty, then
> * return NULL.
> */
> -static xfs_log_item_t *
> +static struct xfs_log_item *
> xfs_ail_max(
> struct xfs_ail *ailp)
> {
> if (list_empty(&ailp->ail_head))
> return NULL;
>
> - return list_entry(ailp->ail_head.prev, xfs_log_item_t, li_ail);
> + return list_entry(ailp->ail_head.prev, struct xfs_log_item, li_ail);
> }
>
> /*
> * Return a pointer to the item which follows the given item in the AIL. If
> * the given item is the last item in the list, then return NULL.
> */
> -static xfs_log_item_t *
> +static struct xfs_log_item *
> xfs_ail_next(
> - struct xfs_ail *ailp,
> - xfs_log_item_t *lip)
> + struct xfs_ail *ailp,
> + struct xfs_log_item *lip)
> {
> if (lip->li_ail.next == &ailp->ail_head)
> return NULL;
>
> - return list_first_entry(&lip->li_ail, xfs_log_item_t, li_ail);
> + return list_first_entry(&lip->li_ail, struct xfs_log_item, li_ail);
> }
>
> /*
> @@ -109,10 +109,10 @@ xfs_ail_next(
> */
> xfs_lsn_t
> xfs_ail_min_lsn(
> - struct xfs_ail *ailp)
> + struct xfs_ail *ailp)
> {
> - xfs_lsn_t lsn = 0;
> - xfs_log_item_t *lip;
> + xfs_lsn_t lsn = 0;
> + struct xfs_log_item *lip;
>
> spin_lock(&ailp->ail_lock);
> lip = xfs_ail_min(ailp);
> @@ -128,10 +128,10 @@ xfs_ail_min_lsn(
> */
> static xfs_lsn_t
> xfs_ail_max_lsn(
> - struct xfs_ail *ailp)
> + struct xfs_ail *ailp)
> {
> - xfs_lsn_t lsn = 0;
> - xfs_log_item_t *lip;
> + xfs_lsn_t lsn = 0;
> + struct xfs_log_item *lip;
>
> spin_lock(&ailp->ail_lock);
> lip = xfs_ail_max(ailp);
> @@ -216,13 +216,13 @@ xfs_trans_ail_cursor_clear(
> * ascending traversal. Pass a @lsn of zero to initialise the cursor to the
> * first item in the AIL. Returns NULL if the list is empty.
> */
> -xfs_log_item_t *
> +struct xfs_log_item *
> xfs_trans_ail_cursor_first(
> struct xfs_ail *ailp,
> struct xfs_ail_cursor *cur,
> xfs_lsn_t lsn)
> {
> - xfs_log_item_t *lip;
> + struct xfs_log_item *lip;
>
> xfs_trans_ail_cursor_init(ailp, cur);
>
> @@ -248,7 +248,7 @@ __xfs_trans_ail_cursor_last(
> struct xfs_ail *ailp,
> xfs_lsn_t lsn)
> {
> - xfs_log_item_t *lip;
> + struct xfs_log_item *lip;
>
> list_for_each_entry_reverse(lip, &ailp->ail_head, li_ail) {
> if (XFS_LSN_CMP(lip->li_lsn, lsn) <= 0)
> @@ -327,8 +327,8 @@ xfs_ail_splice(
> */
> static void
> xfs_ail_delete(
> - struct xfs_ail *ailp,
> - xfs_log_item_t *lip)
> + struct xfs_ail *ailp,
> + struct xfs_log_item *lip)
> {
> xfs_ail_check(ailp, lip);
> list_del(&lip->li_ail);
> @@ -358,7 +358,7 @@ xfsaild_push(
> {
> xfs_mount_t *mp = ailp->ail_mount;
> struct xfs_ail_cursor cur;
> - xfs_log_item_t *lip;
> + struct xfs_log_item *lip;
> xfs_lsn_t lsn;
> xfs_lsn_t target;
> long tout;
> @@ -613,10 +613,10 @@ xfsaild(
> */
> void
> xfs_ail_push(
> - struct xfs_ail *ailp,
> - xfs_lsn_t threshold_lsn)
> + struct xfs_ail *ailp,
> + xfs_lsn_t threshold_lsn)
> {
> - xfs_log_item_t *lip;
> + struct xfs_log_item *lip;
>
> lip = xfs_ail_min(ailp);
> if (!lip || XFS_FORCED_SHUTDOWN(ailp->ail_mount) ||
> @@ -701,7 +701,7 @@ xfs_trans_ail_update_bulk(
> int nr_items,
> xfs_lsn_t lsn) __releases(ailp->ail_lock)
> {
> - xfs_log_item_t *mlip;
> + struct xfs_log_item *mlip;
> int mlip_changed = 0;
> int i;
> LIST_HEAD(tmp);
> --
> 2.20.1
>
next prev parent reply other threads:[~2019-05-17 17:50 UTC|newest]
Thread overview: 61+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-17 7:30 misc log item related cleanups Christoph Hellwig
2019-05-17 7:31 ` [PATCH 01/20] xfs: fix a trivial comment typo in the xfs_trans_committed_bulk Christoph Hellwig
2019-05-17 14:04 ` Brian Foster
2019-05-20 22:08 ` Darrick J. Wong
2019-05-17 7:31 ` [PATCH 02/20] xfs: stop using XFS_LI_ABORTED as a parameter flag Christoph Hellwig
2019-05-17 14:04 ` Brian Foster
2019-05-20 6:03 ` Christoph Hellwig
2019-05-17 14:10 ` Eric Sandeen
2019-05-20 6:05 ` Christoph Hellwig
2019-05-20 22:08 ` Darrick J. Wong
2019-06-11 8:46 ` Christoph Hellwig
2019-05-17 7:31 ` [PATCH 03/20] xfs: don't require log items to implement optional methods Christoph Hellwig
2019-05-17 14:06 ` Brian Foster
2019-05-20 6:06 ` Christoph Hellwig
2019-05-17 7:31 ` [PATCH 04/20] xfs: remove the dummy iop_push implementation for inode creation items Christoph Hellwig
2019-05-17 14:07 ` Brian Foster
2019-05-17 7:31 ` [PATCH 05/20] xfs: remove the iop_push implementation for quota off items Christoph Hellwig
2019-05-17 14:08 ` Brian Foster
2019-05-20 6:09 ` Christoph Hellwig
2019-05-17 7:31 ` [PATCH 06/20] xfs: don't use xfs_trans_free_items in the commit path Christoph Hellwig
2019-05-17 14:08 ` Brian Foster
2019-05-17 7:31 ` [PATCH 07/20] xfs: split iop_unlock Christoph Hellwig
2019-05-17 17:49 ` Brian Foster
2019-05-20 6:10 ` Christoph Hellwig
2019-05-20 11:38 ` Brian Foster
2019-05-17 7:31 ` [PATCH 08/20] xfs: add a flag to release log items on commit Christoph Hellwig
2019-05-17 17:50 ` Brian Foster
2019-05-20 6:11 ` Christoph Hellwig
2019-05-17 7:31 ` [PATCH 09/20] xfs: don't cast inode_log_items to get the log_item Christoph Hellwig
2019-05-17 17:50 ` Brian Foster
2019-05-17 7:31 ` [PATCH 10/20] xfs: remove the xfs_log_item_t typedef Christoph Hellwig
2019-05-17 17:50 ` Brian Foster [this message]
2019-05-17 7:31 ` [PATCH 11/20] xfs: use a list_head for iclog callbacks Christoph Hellwig
2019-05-20 13:12 ` Brian Foster
2019-05-20 13:19 ` Christoph Hellwig
2019-05-20 13:25 ` Brian Foster
2019-05-20 13:27 ` Bryan Gurney
2019-05-20 13:31 ` Christoph Hellwig
2019-05-17 7:31 ` [PATCH 12/20] xfs: remove a pointless comment duplicated above all xfs_item_ops instances Christoph Hellwig
2019-05-20 13:12 ` Brian Foster
2019-05-17 7:31 ` [PATCH 13/20] xfs: merge xfs_efd_init into xfs_trans_get_efd Christoph Hellwig
2019-05-17 8:16 ` Nikolay Borisov
2019-05-17 8:27 ` Christoph Hellwig
2019-05-17 18:26 ` Eric Sandeen
2019-05-20 6:03 ` Christoph Hellwig
2019-05-20 13:12 ` Brian Foster
2019-05-17 7:31 ` [PATCH 14/20] xfs: merge xfs_cud_init into xfs_trans_get_cud Christoph Hellwig
2019-05-20 13:13 ` Brian Foster
2019-05-17 7:31 ` [PATCH 15/20] xfs: merge xfs_rud_init into xfs_trans_get_rud Christoph Hellwig
2019-05-20 13:13 ` Brian Foster
2019-05-17 7:31 ` [PATCH 16/20] xfs: merge xfs_bud_init into xfs_trans_get_bud Christoph Hellwig
2019-05-20 13:13 ` Brian Foster
2019-05-17 7:31 ` [PATCH 17/20] xfs: merge xfs_trans_extfree.c into xfs_extfree_item.c Christoph Hellwig
2019-05-20 13:13 ` Brian Foster
2019-05-17 7:31 ` [PATCH 18/20] xfs: merge xfs_trans_refcount.c into xfs_refcount_item.c Christoph Hellwig
2019-05-20 13:14 ` Brian Foster
2019-05-17 7:31 ` [PATCH 19/20] xfs: merge xfs_trans_rmap.c into xfs_rmap_item.c Christoph Hellwig
2019-05-20 13:14 ` Brian Foster
2019-05-17 7:31 ` [PATCH 20/20] xfs: merge xfs_trans_bmap.c into xfs_bmap_item.c Christoph Hellwig
2019-05-20 13:14 ` Brian Foster
-- strict thread matches above, loose matches on Subject: below --
2019-06-13 18:02 misc log item related cleanups v2 Christoph Hellwig
2019-06-13 18:02 ` [PATCH 10/20] xfs: remove the xfs_log_item_t typedef 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=20190517175043.GJ7888@bfoster \
--to=bfoster@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).