> +STATIC int
> +_xfs_buf_get_maps(
Should this really be STATIC which prevents inlining even if the
compiler thinks that's a good idea? (Same for the free side)
Also any reason for the underscore in the naming?
> + bp->b_bn = map[0].bm_bn;
> + bp->b_length = 0;
> + for (i = 0; i < nmaps; i++) {
> + bp->b_maps[i].bm_bn = map[i].bm_bn;
> + bp->b_maps[i].bm_len = map[i].bm_len;
> + bp->b_length += map[i].bm_len;
> + }
> + bp->b_io_length = bp->b_length;
> +
> + for (i = 0; i < nmaps; i++) {
> + numblks += map[i].bm_len;
> + }
no need for the curly braces here.
> + struct xfs_buf *bp,
> + int map,
> + int *buf_offset,
> + int *count,
> + int rw)
> {
> - int rw, map_i, total_nr_pages, nr_pages;
> + int page_index;
The indentation for the arguments and local variables doesn't seem to
line up here.
_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs
