> +extern struct bio_vec *bvec_alloc(gfp_t, int, unsigned long *, mempool_t *); > +extern void bvec_free(mempool_t *, struct bio_vec *, unsigned int); > +extern unsigned int bvec_nr_vecs(unsigned short idx); Please drop the externs while you're at it. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>