Re: [PATCH 019 of 35] Convert bio_for_each_segment to fill in a fresh bio_vec

From: Tejun Heo
Date: Wed Aug 01 2007 - 12:22:02 EST


Hi,

On Tue, Jul 31, 2007 at 12:17:27PM +1000, NeilBrown wrote:
> i.e. instread of providing a pointer to each bio_vec, it provides
> a copy of each bio_vec.
>
> This allows a future patch to cause bio_for_each_segment to
> provide bio_vecs that are not in the bi_io_vec list, thus allowing
> for offsets and length restrictions.
>
> We consequently remove the only call for bio_kmap_atomic,
> and so remove that function as well.
> Also remove bio_kmap_irq. No-one uses it, and bvec_kmap_irq
> is a much more usable interface.

I think this patch can be split into two but it's no big deal.

> +struct bio_iterator {
> + int i;
> +};
> #define bio_for_each_segment(bvl, bio, i) \
> - for (bvl = bio_iovec_idx((bio), 0), i = 0; \
> - i < (bio)->bi_vcnt; \
> - bvl++, i++)
> + for (i.i = 0, bvl = *bio_iovec_idx((bio), i.i); \
> + i.i < (bio)->bi_vcnt; \
> + i.i++, bvl = *bio_iovec_idx((bio), i.i))

How about something like...

struct bio_iterator {
int i;
struct bio_vec tmp_bvec;
/* might put bio here too? */
};

#define bio_for_each_segment(bvl, bio, i) \
for (bvl = bio_iter_init(&i, bio); i.i < (bio)->bi_vcnt; \
bvl = bio_iter_next(&i, bio))

[static inline] struct bio_vec *bio_iter_init(struct bio_iterator *i,
struct bio *bio)
{
i->i = 0;

if (no further restrictions)
return bio_iovec_idx(bio, 0);
else {
i->tmp_bvec = *bio_iovec_idx(bio, 0);
apply restriction;
return &i->tmp_bvec;
}
}

[static inline] struct bio_vec *bio_iter_next(struct bio_iterator *i,
struct bio *bio)
{
i->i++;

if (no further restriction)
return bio_iovec_idx(bio, i->i);
else {
i->tmp_bvec = *bio_iovec_idx(bio, i->i);
apply restriction;
return &i->tmp_bvec;
}
}

--
tejun
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/