xfs
[Top] [All Lists]

[PATCH 3/4] xfs: simplify xfs_ail_max

To: "xfs@xxxxxxxxxxx" <xfs@xxxxxxxxxxx>
Subject: [PATCH 3/4] xfs: simplify xfs_ail_max
From: Jeff Liu <jeff.liu@xxxxxxxxxx>
Date: Tue, 24 Dec 2013 20:48:26 +0800
Delivered-to: xfs@xxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.1.0
From: Jie Liu <jeff.liu@xxxxxxxxxx>

We have already simplified xfs_ail_min() with a new list helper, i.e,
list_first_entry_or_null(), but xfs_ail_max() still remains as same
as there is no corresponding list helper we can use for now.  It's
fairly easy to simulate the list behavior of getting the last item
in the AIL, therefore we can simplify xfs_ail_max() and move it to
the header file to make this pair of AIL routines looks symmetrical.

Signed-off-by: Jie Liu <jeff.liu@xxxxxxxxxx>
---
 fs/xfs/xfs_trans_ail.c  | 14 --------------
 fs/xfs/xfs_trans_priv.h | 13 +++++++++++++
 2 files changed, 13 insertions(+), 14 deletions(-)

diff --git a/fs/xfs/xfs_trans_ail.c b/fs/xfs/xfs_trans_ail.c
index a728735..af605d0 100644
--- a/fs/xfs/xfs_trans_ail.c
+++ b/fs/xfs/xfs_trans_ail.c
@@ -62,20 +62,6 @@ xfs_ail_check(
 #endif /* DEBUG */
 
 /*
- * Return a pointer to the last item in the AIL.  If the AIL is empty, then
- * return NULL.
- */
-static xfs_log_item_t *
-xfs_ail_max(
-       struct xfs_ail  *ailp)
-{
-       if (list_empty(&ailp->xa_ail))
-               return NULL;
-
-       return list_entry(ailp->xa_ail.prev, xfs_log_item_t, 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.
  */
diff --git a/fs/xfs/xfs_trans_priv.h b/fs/xfs/xfs_trans_priv.h
index 12e86af..c564e94 100644
--- a/fs/xfs/xfs_trans_priv.h
+++ b/fs/xfs/xfs_trans_priv.h
@@ -97,6 +97,19 @@ xfs_ail_min(
                                        li_ail);
 }
 
+/*
+ * Return a pointer to the last item in the AIL.  If the AIL is empty, then
+ * return NULL.
+ */
+static inline struct xfs_log_item *
+xfs_ail_max(
+       struct xfs_ail  *ailp)
+{
+       return list_empty(&ailp->xa_ail) ? NULL :
+               list_last_entry(&ailp->xa_ail, struct xfs_log_item, li_ail);
+}
+
+
 static inline void
 xfs_trans_ail_update(
        struct xfs_ail          *ailp,
-- 
1.8.3.2

<Prev in Thread] Current Thread [Next in Thread>