Re: [PATCH v2 01/12] xfs: add a helper to figure out the space log reservation per item

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Jan 19, 2013 at 12:18:21PM -0600, Mark Tinguely wrote:
> On 01/18/13 21:28, Jeff Liu wrote:
> >On 01/19/2013 07:03 AM, Mark Tinguely wrote:
> >>On 01/10/13 07:47, Jeff Liu wrote:
> >>>+/*
> >>>+ * A buffer has a format structure overhead in the log in addition
> >>>+ * to the data, so we need to take this into account when reserving
> >>>+ * space in a transaction for a buffer.  Round the space required up
> >>>+ * to a multiple of 128 bytes so that we don't change the historical
> >>>+ * reservation that has been used for this overhead.
> >>>+ */
> >>>+STATIC uint
> >>>+xfs_buf_log_overhead(void)
> >>>+{
> >>>+	return round_up(sizeof(struct xlog_op_header) +
> >>>+			sizeof(struct xfs_buf_log_format), 128);
> >>>+}
> >>
> >>Not used directly except below. inline or fold below?
> >I should be inline.  Maybe it's proper to make xfs_calc_buf_res() inline
> >as well?
> 
> Looks like these routines just get run at boot time - the existing

They run at mount time, not boot time... ;)

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs


[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux