Re: [PATCH] superbh, fractured blocks, and grouped io

From: Joel Becker (Joel.Becker@oracle.com)
Date: Mon Oct 14 2002 - 13:13:38 EST


On Mon, Oct 14, 2002 at 03:51:00PM +0200, Jens Axboe wrote:

        Just a couple niggles, really. Looking good.

> /*
> - * First step, 'identity mapping' - RAID or LVM might
> - * further remap this.
> + * detach each bh and resubmit, or completely and if its a grouped bh
> */

        The last line of the comment means "completely fail if its
grouped", right?

> +#define MAX_SUPERBH 65535 /* must fit info ->b_size right now */

        Why not sizeof(b_size) in case we ever care?

> +extern int submit_bh_linked(int, struct buffer_head *);
> +extern int submit_bh_grouped(int, struct buffer_head *);

        Why aren't these EXPORT_SYMBOL(), given that a third party
driver may wish to use them (eg, a filesystem doing its own O_DIRECT
work)?

Joel

-- 

"We'd better get back, `cause it'll be dark soon, and they mostly come at night. Mostly."

Joel Becker Senior Member of Technical Staff Oracle Corporation E-mail: joel.becker@oracle.com Phone: (650) 506-8127 - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Tue Oct 15 2002 - 22:00:51 EST