[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