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
> >>>+ 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... ;)