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

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

> +/* 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.