xref: /linux/fs/xfs/xfs_log_priv.h (revision b2ae3a9ef91152931b99620c431cf3805daa1429)
10b61f8a4SDave Chinner // SPDX-License-Identifier: GPL-2.0
21da177e4SLinus Torvalds /*
37b718769SNathan Scott  * Copyright (c) 2000-2003,2005 Silicon Graphics, Inc.
47b718769SNathan Scott  * All Rights Reserved.
51da177e4SLinus Torvalds  */
61da177e4SLinus Torvalds #ifndef	__XFS_LOG_PRIV_H__
71da177e4SLinus Torvalds #define __XFS_LOG_PRIV_H__
81da177e4SLinus Torvalds 
91da177e4SLinus Torvalds struct xfs_buf;
10ad223e60SMark Tinguely struct xlog;
11a844f451SNathan Scott struct xlog_ticket;
121da177e4SLinus Torvalds struct xfs_mount;
131da177e4SLinus Torvalds 
141da177e4SLinus Torvalds /*
15fc06c6d0SDave Chinner  * Flags for log structure
161da177e4SLinus Torvalds  */
17fc06c6d0SDave Chinner #define XLOG_ACTIVE_RECOVERY	0x2	/* in the middle of recovery */
18fc06c6d0SDave Chinner #define	XLOG_RECOVERY_NEEDED	0x4	/* log was recovered */
19fc06c6d0SDave Chinner #define XLOG_IO_ERROR		0x8	/* log hit an I/O error, and being
20fc06c6d0SDave Chinner 					   shutdown */
21fc06c6d0SDave Chinner #define XLOG_TAIL_WARN		0x10	/* log tail verify warning issued */
221da177e4SLinus Torvalds 
231da177e4SLinus Torvalds /*
241da177e4SLinus Torvalds  * get client id from packed copy.
251da177e4SLinus Torvalds  *
261da177e4SLinus Torvalds  * this hack is here because the xlog_pack code copies four bytes
271da177e4SLinus Torvalds  * of xlog_op_header containing the fields oh_clientid, oh_flags
281da177e4SLinus Torvalds  * and oh_res2 into the packed copy.
291da177e4SLinus Torvalds  *
301da177e4SLinus Torvalds  * later on this four byte chunk is treated as an int and the
311da177e4SLinus Torvalds  * client id is pulled out.
321da177e4SLinus Torvalds  *
331da177e4SLinus Torvalds  * this has endian issues, of course.
341da177e4SLinus Torvalds  */
35b53e675dSChristoph Hellwig static inline uint xlog_get_client_id(__be32 i)
3603bea6feSChristoph Hellwig {
37b53e675dSChristoph Hellwig 	return be32_to_cpu(i) >> 24;
3803bea6feSChristoph Hellwig }
391da177e4SLinus Torvalds 
401da177e4SLinus Torvalds /*
411da177e4SLinus Torvalds  * In core log state
421da177e4SLinus Torvalds  */
431858bb0bSChristoph Hellwig enum xlog_iclog_state {
441858bb0bSChristoph Hellwig 	XLOG_STATE_ACTIVE,	/* Current IC log being written to */
451858bb0bSChristoph Hellwig 	XLOG_STATE_WANT_SYNC,	/* Want to sync this iclog; no more writes */
461858bb0bSChristoph Hellwig 	XLOG_STATE_SYNCING,	/* This IC log is syncing */
471858bb0bSChristoph Hellwig 	XLOG_STATE_DONE_SYNC,	/* Done syncing to disk */
481858bb0bSChristoph Hellwig 	XLOG_STATE_CALLBACK,	/* Callback functions now */
491858bb0bSChristoph Hellwig 	XLOG_STATE_DIRTY,	/* Dirty IC log, not ready for ACTIVE status */
501858bb0bSChristoph Hellwig 	XLOG_STATE_IOERROR,	/* IO error happened in sync'ing log */
511858bb0bSChristoph Hellwig };
521da177e4SLinus Torvalds 
53956f6daaSDave Chinner #define XLOG_STATE_STRINGS \
54956f6daaSDave Chinner 	{ XLOG_STATE_ACTIVE,	"XLOG_STATE_ACTIVE" }, \
55956f6daaSDave Chinner 	{ XLOG_STATE_WANT_SYNC,	"XLOG_STATE_WANT_SYNC" }, \
56956f6daaSDave Chinner 	{ XLOG_STATE_SYNCING,	"XLOG_STATE_SYNCING" }, \
57956f6daaSDave Chinner 	{ XLOG_STATE_DONE_SYNC,	"XLOG_STATE_DONE_SYNC" }, \
58956f6daaSDave Chinner 	{ XLOG_STATE_CALLBACK,	"XLOG_STATE_CALLBACK" }, \
59956f6daaSDave Chinner 	{ XLOG_STATE_DIRTY,	"XLOG_STATE_DIRTY" }, \
60956f6daaSDave Chinner 	{ XLOG_STATE_IOERROR,	"XLOG_STATE_IOERROR" }
61956f6daaSDave Chinner 
62*b2ae3a9eSDave Chinner /*
63*b2ae3a9eSDave Chinner  * In core log flags
64*b2ae3a9eSDave Chinner  */
65*b2ae3a9eSDave Chinner #define XLOG_ICL_NEED_FLUSH	(1 << 0)	/* iclog needs REQ_PREFLUSH */
66*b2ae3a9eSDave Chinner #define XLOG_ICL_NEED_FUA	(1 << 1)	/* iclog needs REQ_FUA */
67*b2ae3a9eSDave Chinner 
68*b2ae3a9eSDave Chinner #define XLOG_ICL_STRINGS \
69*b2ae3a9eSDave Chinner 	{ XLOG_ICL_NEED_FLUSH,	"XLOG_ICL_NEED_FLUSH" }, \
70*b2ae3a9eSDave Chinner 	{ XLOG_ICL_NEED_FUA,	"XLOG_ICL_NEED_FUA" }
71*b2ae3a9eSDave Chinner 
72956f6daaSDave Chinner 
731da177e4SLinus Torvalds /*
7470e42f2dSDave Chinner  * Log ticket flags
751da177e4SLinus Torvalds  */
7670e42f2dSDave Chinner #define XLOG_TIC_PERM_RESERV	0x1	/* permanent reservation */
770b1b213fSChristoph Hellwig 
780b1b213fSChristoph Hellwig #define XLOG_TIC_FLAGS \
7910547941SDave Chinner 	{ XLOG_TIC_PERM_RESERV,	"XLOG_TIC_PERM_RESERV" }
800b1b213fSChristoph Hellwig 
811da177e4SLinus Torvalds /*
821da177e4SLinus Torvalds  * Below are states for covering allocation transactions.
831da177e4SLinus Torvalds  * By covering, we mean changing the h_tail_lsn in the last on-disk
841da177e4SLinus Torvalds  * log write such that no allocation transactions will be re-done during
851da177e4SLinus Torvalds  * recovery after a system crash. Recovery starts at the last on-disk
861da177e4SLinus Torvalds  * log write.
871da177e4SLinus Torvalds  *
881da177e4SLinus Torvalds  * These states are used to insert dummy log entries to cover
891da177e4SLinus Torvalds  * space allocation transactions which can undo non-transactional changes
901da177e4SLinus Torvalds  * after a crash. Writes to a file with space
911da177e4SLinus Torvalds  * already allocated do not result in any transactions. Allocations
921da177e4SLinus Torvalds  * might include space beyond the EOF. So if we just push the EOF a
931da177e4SLinus Torvalds  * little, the last transaction for the file could contain the wrong
941da177e4SLinus Torvalds  * size. If there is no file system activity, after an allocation
951da177e4SLinus Torvalds  * transaction, and the system crashes, the allocation transaction
961da177e4SLinus Torvalds  * will get replayed and the file will be truncated. This could
971da177e4SLinus Torvalds  * be hours/days/... after the allocation occurred.
981da177e4SLinus Torvalds  *
991da177e4SLinus Torvalds  * The fix for this is to do two dummy transactions when the
1001da177e4SLinus Torvalds  * system is idle. We need two dummy transaction because the h_tail_lsn
1011da177e4SLinus Torvalds  * in the log record header needs to point beyond the last possible
1021da177e4SLinus Torvalds  * non-dummy transaction. The first dummy changes the h_tail_lsn to
1031da177e4SLinus Torvalds  * the first transaction before the dummy. The second dummy causes
1041da177e4SLinus Torvalds  * h_tail_lsn to point to the first dummy. Recovery starts at h_tail_lsn.
1051da177e4SLinus Torvalds  *
1061da177e4SLinus Torvalds  * These dummy transactions get committed when everything
1071da177e4SLinus Torvalds  * is idle (after there has been some activity).
1081da177e4SLinus Torvalds  *
1091da177e4SLinus Torvalds  * There are 5 states used to control this.
1101da177e4SLinus Torvalds  *
1111da177e4SLinus Torvalds  *  IDLE -- no logging has been done on the file system or
1121da177e4SLinus Torvalds  *		we are done covering previous transactions.
1131da177e4SLinus Torvalds  *  NEED -- logging has occurred and we need a dummy transaction
1141da177e4SLinus Torvalds  *		when the log becomes idle.
1151da177e4SLinus Torvalds  *  DONE -- we were in the NEED state and have committed a dummy
1161da177e4SLinus Torvalds  *		transaction.
1171da177e4SLinus Torvalds  *  NEED2 -- we detected that a dummy transaction has gone to the
1181da177e4SLinus Torvalds  *		on disk log with no other transactions.
1191da177e4SLinus Torvalds  *  DONE2 -- we committed a dummy transaction when in the NEED2 state.
1201da177e4SLinus Torvalds  *
1211da177e4SLinus Torvalds  * There are two places where we switch states:
1221da177e4SLinus Torvalds  *
1231da177e4SLinus Torvalds  * 1.) In xfs_sync, when we detect an idle log and are in NEED or NEED2.
1241da177e4SLinus Torvalds  *	We commit the dummy transaction and switch to DONE or DONE2,
1251da177e4SLinus Torvalds  *	respectively. In all other states, we don't do anything.
1261da177e4SLinus Torvalds  *
1271da177e4SLinus Torvalds  * 2.) When we finish writing the on-disk log (xlog_state_clean_log).
1281da177e4SLinus Torvalds  *
1291da177e4SLinus Torvalds  *	No matter what state we are in, if this isn't the dummy
1301da177e4SLinus Torvalds  *	transaction going out, the next state is NEED.
1311da177e4SLinus Torvalds  *	So, if we aren't in the DONE or DONE2 states, the next state
1321da177e4SLinus Torvalds  *	is NEED. We can't be finishing a write of the dummy record
1331da177e4SLinus Torvalds  *	unless it was committed and the state switched to DONE or DONE2.
1341da177e4SLinus Torvalds  *
1351da177e4SLinus Torvalds  *	If we are in the DONE state and this was a write of the
1361da177e4SLinus Torvalds  *		dummy transaction, we move to NEED2.
1371da177e4SLinus Torvalds  *
1381da177e4SLinus Torvalds  *	If we are in the DONE2 state and this was a write of the
1391da177e4SLinus Torvalds  *		dummy transaction, we move to IDLE.
1401da177e4SLinus Torvalds  *
1411da177e4SLinus Torvalds  *
1421da177e4SLinus Torvalds  * Writing only one dummy transaction can get appended to
1431da177e4SLinus Torvalds  * one file space allocation. When this happens, the log recovery
1441da177e4SLinus Torvalds  * code replays the space allocation and a file could be truncated.
1451da177e4SLinus Torvalds  * This is why we have the NEED2 and DONE2 states before going idle.
1461da177e4SLinus Torvalds  */
1471da177e4SLinus Torvalds 
1481da177e4SLinus Torvalds #define XLOG_STATE_COVER_IDLE	0
1491da177e4SLinus Torvalds #define XLOG_STATE_COVER_NEED	1
1501da177e4SLinus Torvalds #define XLOG_STATE_COVER_DONE	2
1511da177e4SLinus Torvalds #define XLOG_STATE_COVER_NEED2	3
1521da177e4SLinus Torvalds #define XLOG_STATE_COVER_DONE2	4
1531da177e4SLinus Torvalds 
1541da177e4SLinus Torvalds #define XLOG_COVER_OPS		5
1551da177e4SLinus Torvalds 
1567e9c6396STim Shimmin /* Ticket reservation region accounting */
1577e9c6396STim Shimmin #define XLOG_TIC_LEN_MAX	15
1587e9c6396STim Shimmin 
1597e9c6396STim Shimmin /*
1607e9c6396STim Shimmin  * Reservation region
1617e9c6396STim Shimmin  * As would be stored in xfs_log_iovec but without the i_addr which
1627e9c6396STim Shimmin  * we don't care about.
1637e9c6396STim Shimmin  */
1647e9c6396STim Shimmin typedef struct xlog_res {
1651259845dSTim Shimmin 	uint	r_len;	/* region length		:4 */
1661259845dSTim Shimmin 	uint	r_type;	/* region's transaction type	:4 */
1677e9c6396STim Shimmin } xlog_res_t;
1687e9c6396STim Shimmin 
1691da177e4SLinus Torvalds typedef struct xlog_ticket {
17010547941SDave Chinner 	struct list_head   t_queue;	 /* reserve/write queue */
17114a7235fSChristoph Hellwig 	struct task_struct *t_task;	 /* task that owns this ticket */
1721da177e4SLinus Torvalds 	xlog_tid_t	   t_tid;	 /* transaction identifier	 : 4  */
173cc09c0dcSDave Chinner 	atomic_t	   t_ref;	 /* ticket reference count       : 4  */
1741da177e4SLinus Torvalds 	int		   t_curr_res;	 /* current reservation in bytes : 4  */
1751da177e4SLinus Torvalds 	int		   t_unit_res;	 /* unit reservation in bytes    : 4  */
1767e9c6396STim Shimmin 	char		   t_ocnt;	 /* original count		 : 1  */
1777e9c6396STim Shimmin 	char		   t_cnt;	 /* current count		 : 1  */
1787e9c6396STim Shimmin 	char		   t_clientid;	 /* who does this belong to;	 : 1  */
1797e9c6396STim Shimmin 	char		   t_flags;	 /* properties of reservation	 : 1  */
1807e9c6396STim Shimmin 
1817e9c6396STim Shimmin         /* reservation array fields */
1827e9c6396STim Shimmin 	uint		   t_res_num;                    /* num in array : 4 */
1837e9c6396STim Shimmin 	uint		   t_res_num_ophdrs;		 /* num op hdrs  : 4 */
1847e9c6396STim Shimmin 	uint		   t_res_arr_sum;		 /* array sum    : 4 */
1857e9c6396STim Shimmin 	uint		   t_res_o_flow;		 /* sum overflow : 4 */
1861259845dSTim Shimmin 	xlog_res_t	   t_res_arr[XLOG_TIC_LEN_MAX];  /* array of res : 8 * 15 */
1871da177e4SLinus Torvalds } xlog_ticket_t;
1887e9c6396STim Shimmin 
1891da177e4SLinus Torvalds /*
1901da177e4SLinus Torvalds  * - A log record header is 512 bytes.  There is plenty of room to grow the
1911da177e4SLinus Torvalds  *	xlog_rec_header_t into the reserved space.
1921da177e4SLinus Torvalds  * - ic_data follows, so a write to disk can start at the beginning of
1931da177e4SLinus Torvalds  *	the iclog.
19412017fafSDavid Chinner  * - ic_forcewait is used to implement synchronous forcing of the iclog to disk.
1951da177e4SLinus Torvalds  * - ic_next is the pointer to the next iclog in the ring.
1961da177e4SLinus Torvalds  * - ic_log is a pointer back to the global log structure.
19779b54d9bSChristoph Hellwig  * - ic_size is the full size of the log buffer, minus the cycle headers.
1981da177e4SLinus Torvalds  * - ic_offset is the current number of bytes written to in this iclog.
1991da177e4SLinus Torvalds  * - ic_refcnt is bumped when someone is writing to the log.
2001da177e4SLinus Torvalds  * - ic_state is the state of the iclog.
201114d23aaSDavid Chinner  *
202114d23aaSDavid Chinner  * Because of cacheline contention on large machines, we need to separate
203114d23aaSDavid Chinner  * various resources onto different cachelines. To start with, make the
204114d23aaSDavid Chinner  * structure cacheline aligned. The following fields can be contended on
205114d23aaSDavid Chinner  * by independent processes:
206114d23aaSDavid Chinner  *
20789ae379dSChristoph Hellwig  *	- ic_callbacks
208114d23aaSDavid Chinner  *	- ic_refcnt
209114d23aaSDavid Chinner  *	- fields protected by the global l_icloglock
210114d23aaSDavid Chinner  *
211114d23aaSDavid Chinner  * so we need to ensure that these fields are located in separate cachelines.
212114d23aaSDavid Chinner  * We'll put all the read-only and l_icloglock fields in the first cacheline,
213114d23aaSDavid Chinner  * and move everything else out to subsequent cachelines.
2141da177e4SLinus Torvalds  */
215b28708d6SChristoph Hellwig typedef struct xlog_in_core {
216eb40a875SDave Chinner 	wait_queue_head_t	ic_force_wait;
217eb40a875SDave Chinner 	wait_queue_head_t	ic_write_wait;
2181da177e4SLinus Torvalds 	struct xlog_in_core	*ic_next;
2191da177e4SLinus Torvalds 	struct xlog_in_core	*ic_prev;
220ad223e60SMark Tinguely 	struct xlog		*ic_log;
22179b54d9bSChristoph Hellwig 	u32			ic_size;
22279b54d9bSChristoph Hellwig 	u32			ic_offset;
2231858bb0bSChristoph Hellwig 	enum xlog_iclog_state	ic_state;
224eef983ffSDave Chinner 	unsigned int		ic_flags;
2251da177e4SLinus Torvalds 	char			*ic_datap;	/* pointer to iclog data */
22689ae379dSChristoph Hellwig 	struct list_head	ic_callbacks;
227114d23aaSDavid Chinner 
228114d23aaSDavid Chinner 	/* reference counts need their own cacheline */
229114d23aaSDavid Chinner 	atomic_t		ic_refcnt ____cacheline_aligned_in_smp;
230b28708d6SChristoph Hellwig 	xlog_in_core_2_t	*ic_data;
231b28708d6SChristoph Hellwig #define ic_header	ic_data->hic_header
232366fc4b8SChristoph Hellwig #ifdef DEBUG
233366fc4b8SChristoph Hellwig 	bool			ic_fail_crc : 1;
234366fc4b8SChristoph Hellwig #endif
23579b54d9bSChristoph Hellwig 	struct semaphore	ic_sema;
23679b54d9bSChristoph Hellwig 	struct work_struct	ic_end_io_work;
23779b54d9bSChristoph Hellwig 	struct bio		ic_bio;
23879b54d9bSChristoph Hellwig 	struct bio_vec		ic_bvec[];
2391da177e4SLinus Torvalds } xlog_in_core_t;
2401da177e4SLinus Torvalds 
2411da177e4SLinus Torvalds /*
24271e330b5SDave Chinner  * The CIL context is used to aggregate per-transaction details as well be
24371e330b5SDave Chinner  * passed to the iclog for checkpoint post-commit processing.  After being
24471e330b5SDave Chinner  * passed to the iclog, another context needs to be allocated for tracking the
24571e330b5SDave Chinner  * next set of transactions to be aggregated into a checkpoint.
24671e330b5SDave Chinner  */
24771e330b5SDave Chinner struct xfs_cil;
24871e330b5SDave Chinner 
24971e330b5SDave Chinner struct xfs_cil_ctx {
25071e330b5SDave Chinner 	struct xfs_cil		*cil;
2515f9b4b0dSDave Chinner 	xfs_csn_t		sequence;	/* chkpt sequence # */
25271e330b5SDave Chinner 	xfs_lsn_t		start_lsn;	/* first LSN of chkpt commit */
25371e330b5SDave Chinner 	xfs_lsn_t		commit_lsn;	/* chkpt commit record lsn */
25471e330b5SDave Chinner 	struct xlog_ticket	*ticket;	/* chkpt ticket */
25571e330b5SDave Chinner 	int			nvecs;		/* number of regions */
25671e330b5SDave Chinner 	int			space_used;	/* aggregate size of regions */
25771e330b5SDave Chinner 	struct list_head	busy_extents;	/* busy extents in chkpt */
25871e330b5SDave Chinner 	struct xfs_log_vec	*lv_chain;	/* logvecs being pushed */
25989ae379dSChristoph Hellwig 	struct list_head	iclog_entry;
26071e330b5SDave Chinner 	struct list_head	committing;	/* ctx committing list */
2614560e78fSChristoph Hellwig 	struct work_struct	discard_endio_work;
26271e330b5SDave Chinner };
26371e330b5SDave Chinner 
26471e330b5SDave Chinner /*
26571e330b5SDave Chinner  * Committed Item List structure
26671e330b5SDave Chinner  *
26771e330b5SDave Chinner  * This structure is used to track log items that have been committed but not
26871e330b5SDave Chinner  * yet written into the log. It is used only when the delayed logging mount
26971e330b5SDave Chinner  * option is enabled.
27071e330b5SDave Chinner  *
27171e330b5SDave Chinner  * This structure tracks the list of committing checkpoint contexts so
27271e330b5SDave Chinner  * we can avoid the problem of having to hold out new transactions during a
27371e330b5SDave Chinner  * flush until we have a the commit record LSN of the checkpoint. We can
27471e330b5SDave Chinner  * traverse the list of committing contexts in xlog_cil_push_lsn() to find a
27571e330b5SDave Chinner  * sequence match and extract the commit LSN directly from there. If the
27671e330b5SDave Chinner  * checkpoint is still in the process of committing, we can block waiting for
27771e330b5SDave Chinner  * the commit LSN to be determined as well. This should make synchronous
27871e330b5SDave Chinner  * operations almost as efficient as the old logging methods.
27971e330b5SDave Chinner  */
28071e330b5SDave Chinner struct xfs_cil {
281ad223e60SMark Tinguely 	struct xlog		*xc_log;
28271e330b5SDave Chinner 	struct list_head	xc_cil;
28371e330b5SDave Chinner 	spinlock_t		xc_cil_lock;
2844bb928cdSDave Chinner 
2854bb928cdSDave Chinner 	struct rw_semaphore	xc_ctx_lock ____cacheline_aligned_in_smp;
28671e330b5SDave Chinner 	struct xfs_cil_ctx	*xc_ctx;
2874bb928cdSDave Chinner 
2884bb928cdSDave Chinner 	spinlock_t		xc_push_lock ____cacheline_aligned_in_smp;
2895f9b4b0dSDave Chinner 	xfs_csn_t		xc_push_seq;
29071e330b5SDave Chinner 	struct list_head	xc_committing;
291eb40a875SDave Chinner 	wait_queue_head_t	xc_commit_wait;
2925f9b4b0dSDave Chinner 	xfs_csn_t		xc_current_sequence;
2934c2d542fSDave Chinner 	struct work_struct	xc_push_work;
294c7f87f39SDave Chinner 	wait_queue_head_t	xc_push_wait;	/* background push throttle */
2954bb928cdSDave Chinner } ____cacheline_aligned_in_smp;
29671e330b5SDave Chinner 
29771e330b5SDave Chinner /*
29880168676SDave Chinner  * The amount of log space we allow the CIL to aggregate is difficult to size.
29980168676SDave Chinner  * Whatever we choose, we have to make sure we can get a reservation for the
30080168676SDave Chinner  * log space effectively, that it is large enough to capture sufficient
30180168676SDave Chinner  * relogging to reduce log buffer IO significantly, but it is not too large for
30280168676SDave Chinner  * the log or induces too much latency when writing out through the iclogs. We
30380168676SDave Chinner  * track both space consumed and the number of vectors in the checkpoint
30480168676SDave Chinner  * context, so we need to decide which to use for limiting.
305df806158SDave Chinner  *
306df806158SDave Chinner  * Every log buffer we write out during a push needs a header reserved, which
307df806158SDave Chinner  * is at least one sector and more for v2 logs. Hence we need a reservation of
308df806158SDave Chinner  * at least 512 bytes per 32k of log space just for the LR headers. That means
309df806158SDave Chinner  * 16KB of reservation per megabyte of delayed logging space we will consume,
310df806158SDave Chinner  * plus various headers.  The number of headers will vary based on the num of
311df806158SDave Chinner  * io vectors, so limiting on a specific number of vectors is going to result
312df806158SDave Chinner  * in transactions of varying size. IOWs, it is more consistent to track and
313df806158SDave Chinner  * limit space consumed in the log rather than by the number of objects being
314df806158SDave Chinner  * logged in order to prevent checkpoint ticket overruns.
315df806158SDave Chinner  *
316df806158SDave Chinner  * Further, use of static reservations through the log grant mechanism is
317df806158SDave Chinner  * problematic. It introduces a lot of complexity (e.g. reserve grant vs write
318df806158SDave Chinner  * grant) and a significant deadlock potential because regranting write space
319df806158SDave Chinner  * can block on log pushes. Hence if we have to regrant log space during a log
320df806158SDave Chinner  * push, we can deadlock.
321df806158SDave Chinner  *
322df806158SDave Chinner  * However, we can avoid this by use of a dynamic "reservation stealing"
323df806158SDave Chinner  * technique during transaction commit whereby unused reservation space in the
324df806158SDave Chinner  * transaction ticket is transferred to the CIL ctx commit ticket to cover the
325df806158SDave Chinner  * space needed by the checkpoint transaction. This means that we never need to
326df806158SDave Chinner  * specifically reserve space for the CIL checkpoint transaction, nor do we
327df806158SDave Chinner  * need to regrant space once the checkpoint completes. This also means the
328df806158SDave Chinner  * checkpoint transaction ticket is specific to the checkpoint context, rather
329df806158SDave Chinner  * than the CIL itself.
330df806158SDave Chinner  *
33180168676SDave Chinner  * With dynamic reservations, we can effectively make up arbitrary limits for
33280168676SDave Chinner  * the checkpoint size so long as they don't violate any other size rules.
33380168676SDave Chinner  * Recovery imposes a rule that no transaction exceed half the log, so we are
33480168676SDave Chinner  * limited by that.  Furthermore, the log transaction reservation subsystem
33580168676SDave Chinner  * tries to keep 25% of the log free, so we need to keep below that limit or we
33680168676SDave Chinner  * risk running out of free log space to start any new transactions.
33780168676SDave Chinner  *
338108a4235SDave Chinner  * In order to keep background CIL push efficient, we only need to ensure the
339108a4235SDave Chinner  * CIL is large enough to maintain sufficient in-memory relogging to avoid
340108a4235SDave Chinner  * repeated physical writes of frequently modified metadata. If we allow the CIL
341108a4235SDave Chinner  * to grow to a substantial fraction of the log, then we may be pinning hundreds
342108a4235SDave Chinner  * of megabytes of metadata in memory until the CIL flushes. This can cause
343108a4235SDave Chinner  * issues when we are running low on memory - pinned memory cannot be reclaimed,
344108a4235SDave Chinner  * and the CIL consumes a lot of memory. Hence we need to set an upper physical
345108a4235SDave Chinner  * size limit for the CIL that limits the maximum amount of memory pinned by the
346108a4235SDave Chinner  * CIL but does not limit performance by reducing relogging efficiency
347108a4235SDave Chinner  * significantly.
348108a4235SDave Chinner  *
349108a4235SDave Chinner  * As such, the CIL push threshold ends up being the smaller of two thresholds:
350108a4235SDave Chinner  * - a threshold large enough that it allows CIL to be pushed and progress to be
351108a4235SDave Chinner  *   made without excessive blocking of incoming transaction commits. This is
352108a4235SDave Chinner  *   defined to be 12.5% of the log space - half the 25% push threshold of the
353108a4235SDave Chinner  *   AIL.
354108a4235SDave Chinner  * - small enough that it doesn't pin excessive amounts of memory but maintains
355108a4235SDave Chinner  *   close to peak relogging efficiency. This is defined to be 16x the iclog
356108a4235SDave Chinner  *   buffer window (32MB) as measurements have shown this to be roughly the
357108a4235SDave Chinner  *   point of diminishing performance increases under highly concurrent
358108a4235SDave Chinner  *   modification workloads.
3590e7ab7efSDave Chinner  *
3600e7ab7efSDave Chinner  * To prevent the CIL from overflowing upper commit size bounds, we introduce a
3610e7ab7efSDave Chinner  * new threshold at which we block committing transactions until the background
3620e7ab7efSDave Chinner  * CIL commit commences and switches to a new context. While this is not a hard
3630e7ab7efSDave Chinner  * limit, it forces the process committing a transaction to the CIL to block and
3640e7ab7efSDave Chinner  * yeild the CPU, giving the CIL push work a chance to be scheduled and start
3650e7ab7efSDave Chinner  * work. This prevents a process running lots of transactions from overfilling
3660e7ab7efSDave Chinner  * the CIL because it is not yielding the CPU. We set the blocking limit at
3670e7ab7efSDave Chinner  * twice the background push space threshold so we keep in line with the AIL
3680e7ab7efSDave Chinner  * push thresholds.
3690e7ab7efSDave Chinner  *
3700e7ab7efSDave Chinner  * Note: this is not a -hard- limit as blocking is applied after the transaction
3710e7ab7efSDave Chinner  * is inserted into the CIL and the push has been triggered. It is largely a
3720e7ab7efSDave Chinner  * throttling mechanism that allows the CIL push to be scheduled and run. A hard
3730e7ab7efSDave Chinner  * limit will be difficult to implement without introducing global serialisation
3740e7ab7efSDave Chinner  * in the CIL commit fast path, and it's not at all clear that we actually need
3750e7ab7efSDave Chinner  * such hard limits given the ~7 years we've run without a hard limit before
3760e7ab7efSDave Chinner  * finding the first situation where a checkpoint size overflow actually
3770e7ab7efSDave Chinner  * occurred. Hence the simple throttle, and an ASSERT check to tell us that
3780e7ab7efSDave Chinner  * we've overrun the max size.
379df806158SDave Chinner  */
380108a4235SDave Chinner #define XLOG_CIL_SPACE_LIMIT(log)	\
381108a4235SDave Chinner 	min_t(int, (log)->l_logsize >> 3, BBTOB(XLOG_TOTAL_REC_SHIFT(log)) << 4)
382df806158SDave Chinner 
3830e7ab7efSDave Chinner #define XLOG_CIL_BLOCKING_SPACE_LIMIT(log)	\
3840e7ab7efSDave Chinner 	(XLOG_CIL_SPACE_LIMIT(log) * 2)
3850e7ab7efSDave Chinner 
386df806158SDave Chinner /*
38728496968SChristoph Hellwig  * ticket grant locks, queues and accounting have their own cachlines
38828496968SChristoph Hellwig  * as these are quite hot and can be operated on concurrently.
38928496968SChristoph Hellwig  */
39028496968SChristoph Hellwig struct xlog_grant_head {
39128496968SChristoph Hellwig 	spinlock_t		lock ____cacheline_aligned_in_smp;
39228496968SChristoph Hellwig 	struct list_head	waiters;
39328496968SChristoph Hellwig 	atomic64_t		grant;
39428496968SChristoph Hellwig };
39528496968SChristoph Hellwig 
39628496968SChristoph Hellwig /*
3971da177e4SLinus Torvalds  * The reservation head lsn is not made up of a cycle number and block number.
3981da177e4SLinus Torvalds  * Instead, it uses a cycle number and byte number.  Logs don't expect to
3991da177e4SLinus Torvalds  * overflow 31 bits worth of byte offset, so using a byte number will mean
4001da177e4SLinus Torvalds  * that round off problems won't occur when releasing partial reservations.
4011da177e4SLinus Torvalds  */
4029a8d2fdbSMark Tinguely struct xlog {
4034679b2d3SDavid Chinner 	/* The following fields don't need locking */
4044679b2d3SDavid Chinner 	struct xfs_mount	*l_mp;	        /* mount point */
405a9c21c1bSDavid Chinner 	struct xfs_ail		*l_ailp;	/* AIL log is working with */
40671e330b5SDave Chinner 	struct xfs_cil		*l_cilp;	/* CIL log is working with */
4074679b2d3SDavid Chinner 	struct xfs_buftarg	*l_targ;        /* buftarg of log */
4081058d0f5SChristoph Hellwig 	struct workqueue_struct	*l_ioend_workqueue; /* for I/O completions */
409f661f1e0SDave Chinner 	struct delayed_work	l_work;		/* background flush work */
4104679b2d3SDavid Chinner 	uint			l_flags;
4114679b2d3SDavid Chinner 	uint			l_quotaoffs_flag; /* XFS_DQ_*, for QUOTAOFFs */
412d5689eaaSChristoph Hellwig 	struct list_head	*l_buf_cancel_table;
4134679b2d3SDavid Chinner 	int			l_iclog_hsize;  /* size of iclog header */
4144679b2d3SDavid Chinner 	int			l_iclog_heads;  /* # of iclog header sectors */
41548389ef1SAlex Elder 	uint			l_sectBBsize;   /* sector size in BBs (2^n) */
4164679b2d3SDavid Chinner 	int			l_iclog_size;	/* size of log in bytes */
4174679b2d3SDavid Chinner 	int			l_iclog_bufs;	/* number of iclog buffers */
4184679b2d3SDavid Chinner 	xfs_daddr_t		l_logBBstart;   /* start block of log */
4194679b2d3SDavid Chinner 	int			l_logsize;      /* size of log in bytes */
4204679b2d3SDavid Chinner 	int			l_logBBsize;    /* size of log in BB chunks */
4214679b2d3SDavid Chinner 
4221da177e4SLinus Torvalds 	/* The following block of fields are changed while holding icloglock */
423eb40a875SDave Chinner 	wait_queue_head_t	l_flush_wait ____cacheline_aligned_in_smp;
424d748c623SMatthew Wilcox 						/* waiting for iclog flush */
4251da177e4SLinus Torvalds 	int			l_covered_state;/* state of "covering disk
4261da177e4SLinus Torvalds 						 * log entries" */
4271da177e4SLinus Torvalds 	xlog_in_core_t		*l_iclog;       /* head log queue	*/
428b22cd72cSEric Sandeen 	spinlock_t		l_icloglock;    /* grab to change iclog state */
4291da177e4SLinus Torvalds 	int			l_curr_cycle;   /* Cycle number of log writes */
4301da177e4SLinus Torvalds 	int			l_prev_cycle;   /* Cycle number before last
4311da177e4SLinus Torvalds 						 * block increment */
4321da177e4SLinus Torvalds 	int			l_curr_block;   /* current logical log block */
4331da177e4SLinus Torvalds 	int			l_prev_block;   /* previous logical log block */
4341da177e4SLinus Torvalds 
43584f3c683SDave Chinner 	/*
4361c3cb9ecSDave Chinner 	 * l_last_sync_lsn and l_tail_lsn are atomics so they can be set and
4371c3cb9ecSDave Chinner 	 * read without needing to hold specific locks. To avoid operations
4381c3cb9ecSDave Chinner 	 * contending with other hot objects, place each of them on a separate
4391c3cb9ecSDave Chinner 	 * cacheline.
44084f3c683SDave Chinner 	 */
44184f3c683SDave Chinner 	/* lsn of last LR on disk */
44284f3c683SDave Chinner 	atomic64_t		l_last_sync_lsn ____cacheline_aligned_in_smp;
4431c3cb9ecSDave Chinner 	/* lsn of 1st LR with unflushed * buffers */
4441c3cb9ecSDave Chinner 	atomic64_t		l_tail_lsn ____cacheline_aligned_in_smp;
44584f3c683SDave Chinner 
44628496968SChristoph Hellwig 	struct xlog_grant_head	l_reserve_head;
44728496968SChristoph Hellwig 	struct xlog_grant_head	l_write_head;
4483f16b985SDave Chinner 
449baff4e44SBrian Foster 	struct xfs_kobj		l_kobj;
450baff4e44SBrian Foster 
4514679b2d3SDavid Chinner 	/* The following field are used for debugging; need to hold icloglock */
4524679b2d3SDavid Chinner #ifdef DEBUG
4535809d5e0SChristoph Hellwig 	void			*l_iclog_bak[XLOG_MAX_ICLOGS];
4544679b2d3SDavid Chinner #endif
45512818d24SBrian Foster 	/* log recovery lsn tracking (for buffer submission */
45612818d24SBrian Foster 	xfs_lsn_t		l_recovery_lsn;
457a6a65fefSDave Chinner 
458a6a65fefSDave Chinner 	uint32_t		l_iclog_roundoff;/* padding roundoff */
4599a8d2fdbSMark Tinguely };
4601da177e4SLinus Torvalds 
461d5689eaaSChristoph Hellwig #define XLOG_BUF_CANCEL_BUCKET(log, blkno) \
462c8ce540dSDarrick J. Wong 	((log)->l_buf_cancel_table + ((uint64_t)blkno % XLOG_BC_TABLE_SIZE))
463d5689eaaSChristoph Hellwig 
464b941c719SChristoph Hellwig #define XLOG_FORCED_SHUTDOWN(log) \
465b941c719SChristoph Hellwig 	(unlikely((log)->l_flags & XLOG_IO_ERROR))
466cfcbbbd0SNathan Scott 
4671da177e4SLinus Torvalds /* common routines */
4689a8d2fdbSMark Tinguely extern int
4699a8d2fdbSMark Tinguely xlog_recover(
4709a8d2fdbSMark Tinguely 	struct xlog		*log);
4719a8d2fdbSMark Tinguely extern int
4729a8d2fdbSMark Tinguely xlog_recover_finish(
4739a8d2fdbSMark Tinguely 	struct xlog		*log);
474a7a9250eSHariprasad Kelam extern void
475f0b2efadSBrian Foster xlog_recover_cancel(struct xlog *);
4760e446be4SChristoph Hellwig 
477f9668a09SDave Chinner extern __le32	 xlog_cksum(struct xlog *log, struct xlog_rec_header *rhead,
4780e446be4SChristoph Hellwig 			    char *dp, int size);
4791da177e4SLinus Torvalds 
480eb01c9cdSDavid Chinner extern kmem_zone_t *xfs_log_ticket_zone;
481ad223e60SMark Tinguely struct xlog_ticket *
482ad223e60SMark Tinguely xlog_ticket_alloc(
483ad223e60SMark Tinguely 	struct xlog	*log,
484ad223e60SMark Tinguely 	int		unit_bytes,
485ad223e60SMark Tinguely 	int		count,
486ad223e60SMark Tinguely 	char		client,
487ca4f2589SCarlos Maiolino 	bool		permanent);
488eb01c9cdSDavid Chinner 
489e6b1f273SChristoph Hellwig static inline void
490e6b1f273SChristoph Hellwig xlog_write_adv_cnt(void **ptr, int *len, int *off, size_t bytes)
491e6b1f273SChristoph Hellwig {
492e6b1f273SChristoph Hellwig 	*ptr += bytes;
493e6b1f273SChristoph Hellwig 	*len -= bytes;
494e6b1f273SChristoph Hellwig 	*off += bytes;
495e6b1f273SChristoph Hellwig }
496e6b1f273SChristoph Hellwig 
49771e330b5SDave Chinner void	xlog_print_tic_res(struct xfs_mount *mp, struct xlog_ticket *ticket);
498d4ca1d55SBrian Foster void	xlog_print_trans(struct xfs_trans *);
4997ec94921SDave Chinner int	xlog_write(struct xlog *log, struct xfs_log_vec *log_vector,
5007ec94921SDave Chinner 		struct xlog_ticket *tic, xfs_lsn_t *start_lsn,
5013468bb1cSDave Chinner 		struct xlog_in_core **commit_iclog, uint optype);
502f10e925dSDave Chinner int	xlog_commit_record(struct xlog *log, struct xlog_ticket *ticket,
503dd401770SDave Chinner 		struct xlog_in_core **iclog, xfs_lsn_t *lsn);
5048b41e3f9SChristoph Hellwig void	xfs_log_ticket_ungrant(struct xlog *log, struct xlog_ticket *ticket);
5058b41e3f9SChristoph Hellwig void	xfs_log_ticket_regrant(struct xlog *log, struct xlog_ticket *ticket);
50671e330b5SDave Chinner 
5070dc8f7f1SDave Chinner int xlog_state_release_iclog(struct xlog *log, struct xlog_in_core *iclog,
5080dc8f7f1SDave Chinner 		xfs_lsn_t log_tail_lsn);
509eef983ffSDave Chinner 
51071e330b5SDave Chinner /*
5111c3cb9ecSDave Chinner  * When we crack an atomic LSN, we sample it first so that the value will not
5121c3cb9ecSDave Chinner  * change while we are cracking it into the component values. This means we
5131c3cb9ecSDave Chinner  * will always get consistent component values to work from. This should always
51425985edcSLucas De Marchi  * be used to sample and crack LSNs that are stored and updated in atomic
5151c3cb9ecSDave Chinner  * variables.
5161c3cb9ecSDave Chinner  */
5171c3cb9ecSDave Chinner static inline void
5181c3cb9ecSDave Chinner xlog_crack_atomic_lsn(atomic64_t *lsn, uint *cycle, uint *block)
5191c3cb9ecSDave Chinner {
5201c3cb9ecSDave Chinner 	xfs_lsn_t val = atomic64_read(lsn);
5211c3cb9ecSDave Chinner 
5221c3cb9ecSDave Chinner 	*cycle = CYCLE_LSN(val);
5231c3cb9ecSDave Chinner 	*block = BLOCK_LSN(val);
5241c3cb9ecSDave Chinner }
5251c3cb9ecSDave Chinner 
5261c3cb9ecSDave Chinner /*
5271c3cb9ecSDave Chinner  * Calculate and assign a value to an atomic LSN variable from component pieces.
5281c3cb9ecSDave Chinner  */
5291c3cb9ecSDave Chinner static inline void
5301c3cb9ecSDave Chinner xlog_assign_atomic_lsn(atomic64_t *lsn, uint cycle, uint block)
5311c3cb9ecSDave Chinner {
5321c3cb9ecSDave Chinner 	atomic64_set(lsn, xlog_assign_lsn(cycle, block));
5331c3cb9ecSDave Chinner }
5341c3cb9ecSDave Chinner 
5351c3cb9ecSDave Chinner /*
536d0eb2f38SDave Chinner  * When we crack the grant head, we sample it first so that the value will not
537a69ed03cSDave Chinner  * change while we are cracking it into the component values. This means we
538a69ed03cSDave Chinner  * will always get consistent component values to work from.
539a69ed03cSDave Chinner  */
540a69ed03cSDave Chinner static inline void
541d0eb2f38SDave Chinner xlog_crack_grant_head_val(int64_t val, int *cycle, int *space)
542a69ed03cSDave Chinner {
543a69ed03cSDave Chinner 	*cycle = val >> 32;
544a69ed03cSDave Chinner 	*space = val & 0xffffffff;
545a69ed03cSDave Chinner }
546a69ed03cSDave Chinner 
547a69ed03cSDave Chinner static inline void
548d0eb2f38SDave Chinner xlog_crack_grant_head(atomic64_t *head, int *cycle, int *space)
549d0eb2f38SDave Chinner {
550d0eb2f38SDave Chinner 	xlog_crack_grant_head_val(atomic64_read(head), cycle, space);
551d0eb2f38SDave Chinner }
552d0eb2f38SDave Chinner 
553d0eb2f38SDave Chinner static inline int64_t
554d0eb2f38SDave Chinner xlog_assign_grant_head_val(int cycle, int space)
555d0eb2f38SDave Chinner {
556d0eb2f38SDave Chinner 	return ((int64_t)cycle << 32) | space;
557d0eb2f38SDave Chinner }
558d0eb2f38SDave Chinner 
559d0eb2f38SDave Chinner static inline void
560c8a09ff8SDave Chinner xlog_assign_grant_head(atomic64_t *head, int cycle, int space)
561a69ed03cSDave Chinner {
562d0eb2f38SDave Chinner 	atomic64_set(head, xlog_assign_grant_head_val(cycle, space));
563a69ed03cSDave Chinner }
564a69ed03cSDave Chinner 
565a69ed03cSDave Chinner /*
56671e330b5SDave Chinner  * Committed Item List interfaces
56771e330b5SDave Chinner  */
5682c6e24ceSDave Chinner int	xlog_cil_init(struct xlog *log);
5692c6e24ceSDave Chinner void	xlog_cil_init_post_recovery(struct xlog *log);
5702c6e24ceSDave Chinner void	xlog_cil_destroy(struct xlog *log);
5712c6e24ceSDave Chinner bool	xlog_cil_empty(struct xlog *log);
5725f9b4b0dSDave Chinner void	xlog_cil_commit(struct xlog *log, struct xfs_trans *tp,
5735f9b4b0dSDave Chinner 			xfs_csn_t *commit_seq, bool regrant);
57471e330b5SDave Chinner 
575a44f13edSDave Chinner /*
576a44f13edSDave Chinner  * CIL force routines
577a44f13edSDave Chinner  */
5785f9b4b0dSDave Chinner xfs_lsn_t xlog_cil_force_seq(struct xlog *log, xfs_csn_t sequence);
579a44f13edSDave Chinner 
580a44f13edSDave Chinner static inline void
581ad223e60SMark Tinguely xlog_cil_force(struct xlog *log)
582a44f13edSDave Chinner {
5835f9b4b0dSDave Chinner 	xlog_cil_force_seq(log, log->l_cilp->xc_current_sequence);
584a44f13edSDave Chinner }
58571e330b5SDave Chinner 
586955e47adSTim Shimmin /*
587eb40a875SDave Chinner  * Wrapper function for waiting on a wait queue serialised against wakeups
588eb40a875SDave Chinner  * by a spinlock. This matches the semantics of all the wait queues used in the
589eb40a875SDave Chinner  * log code.
590eb40a875SDave Chinner  */
591f7559793SDarrick J. Wong static inline void
592f7559793SDarrick J. Wong xlog_wait(
593f7559793SDarrick J. Wong 	struct wait_queue_head	*wq,
594f7559793SDarrick J. Wong 	struct spinlock		*lock)
595f7559793SDarrick J. Wong 		__releases(lock)
596eb40a875SDave Chinner {
597eb40a875SDave Chinner 	DECLARE_WAITQUEUE(wait, current);
598eb40a875SDave Chinner 
599eb40a875SDave Chinner 	add_wait_queue_exclusive(wq, &wait);
600eb40a875SDave Chinner 	__set_current_state(TASK_UNINTERRUPTIBLE);
601eb40a875SDave Chinner 	spin_unlock(lock);
602eb40a875SDave Chinner 	schedule();
603eb40a875SDave Chinner 	remove_wait_queue(wq, &wait);
604eb40a875SDave Chinner }
6051da177e4SLinus Torvalds 
606a79b28c2SDave Chinner int xlog_wait_on_iclog(struct xlog_in_core *iclog);
607a79b28c2SDave Chinner 
608a45086e2SBrian Foster /*
609a45086e2SBrian Foster  * The LSN is valid so long as it is behind the current LSN. If it isn't, this
610a45086e2SBrian Foster  * means that the next log record that includes this metadata could have a
611a45086e2SBrian Foster  * smaller LSN. In turn, this means that the modification in the log would not
612a45086e2SBrian Foster  * replay.
613a45086e2SBrian Foster  */
614a45086e2SBrian Foster static inline bool
615a45086e2SBrian Foster xlog_valid_lsn(
616a45086e2SBrian Foster 	struct xlog	*log,
617a45086e2SBrian Foster 	xfs_lsn_t	lsn)
618a45086e2SBrian Foster {
619a45086e2SBrian Foster 	int		cur_cycle;
620a45086e2SBrian Foster 	int		cur_block;
621a45086e2SBrian Foster 	bool		valid = true;
622a45086e2SBrian Foster 
623a45086e2SBrian Foster 	/*
624a45086e2SBrian Foster 	 * First, sample the current lsn without locking to avoid added
625a45086e2SBrian Foster 	 * contention from metadata I/O. The current cycle and block are updated
626a45086e2SBrian Foster 	 * (in xlog_state_switch_iclogs()) and read here in a particular order
627a45086e2SBrian Foster 	 * to avoid false negatives (e.g., thinking the metadata LSN is valid
628a45086e2SBrian Foster 	 * when it is not).
629a45086e2SBrian Foster 	 *
630a45086e2SBrian Foster 	 * The current block is always rewound before the cycle is bumped in
631a45086e2SBrian Foster 	 * xlog_state_switch_iclogs() to ensure the current LSN is never seen in
632a45086e2SBrian Foster 	 * a transiently forward state. Instead, we can see the LSN in a
633a45086e2SBrian Foster 	 * transiently behind state if we happen to race with a cycle wrap.
634a45086e2SBrian Foster 	 */
6356aa7de05SMark Rutland 	cur_cycle = READ_ONCE(log->l_curr_cycle);
636a45086e2SBrian Foster 	smp_rmb();
6376aa7de05SMark Rutland 	cur_block = READ_ONCE(log->l_curr_block);
638a45086e2SBrian Foster 
639a45086e2SBrian Foster 	if ((CYCLE_LSN(lsn) > cur_cycle) ||
640a45086e2SBrian Foster 	    (CYCLE_LSN(lsn) == cur_cycle && BLOCK_LSN(lsn) > cur_block)) {
641a45086e2SBrian Foster 		/*
642a45086e2SBrian Foster 		 * If the metadata LSN appears invalid, it's possible the check
643a45086e2SBrian Foster 		 * above raced with a wrap to the next log cycle. Grab the lock
644a45086e2SBrian Foster 		 * to check for sure.
645a45086e2SBrian Foster 		 */
646a45086e2SBrian Foster 		spin_lock(&log->l_icloglock);
647a45086e2SBrian Foster 		cur_cycle = log->l_curr_cycle;
648a45086e2SBrian Foster 		cur_block = log->l_curr_block;
649a45086e2SBrian Foster 		spin_unlock(&log->l_icloglock);
650a45086e2SBrian Foster 
651a45086e2SBrian Foster 		if ((CYCLE_LSN(lsn) > cur_cycle) ||
652a45086e2SBrian Foster 		    (CYCLE_LSN(lsn) == cur_cycle && BLOCK_LSN(lsn) > cur_block))
653a45086e2SBrian Foster 			valid = false;
654a45086e2SBrian Foster 	}
655a45086e2SBrian Foster 
656a45086e2SBrian Foster 	return valid;
657a45086e2SBrian Foster }
658a45086e2SBrian Foster 
6591da177e4SLinus Torvalds #endif	/* __XFS_LOG_PRIV_H__ */
660