[PATCH 2/7] xfs: fix min bufsize bugs in two places

Christoph Hellwig hch at infradead.org
Sat Mar 20 11:42:49 CDT 2010


> +/* Number of basic blocks in a log sector */
> +#define xlog_sectbb(log) (1 << (log)->l_sectbb_log)

Looking at all uses of (log)->l_sectbb_log I wonder if we should
bother storing this in the log structure in this form, or rather
as the multipler of the basic block size, ala l_sectsize.  All the

	if (log->l_sectbb_log) {

checks would just become

	if (log->l_sectsize > 1) {

and the xlog_find_verify_cycle/xlog_write_log_records checks
would also be a natural

	if (bufblks < log->l_sectsize)

The comments added are defintively useful, btw.




More information about the xfs mailing list