On Tue, Aug 28, 2012 at 01:44:01PM -0700, Tejun Heo wrote:

On Tue, Aug 28, 2012 at 10:37:34AM -0700, Kent Overstreet wrote:

quoted

+static inline struct bio *bio_clone(struct bio *bio, gfp_t gfp_mask)
+{
+	return bio_clone_bioset(bio, gfp_mask, fs_bio_set);
+}
+

...

quoted

+static inline struct bio *bio_clone_kmalloc(struct bio *bio, gfp_t gfp_mask)
+{
+	return bio_clone_bioset(bio, gfp_mask, NULL);
+
+}

Do we really need these wrappers?  I'd prefer requiring users to
explicit choose @bioset when cloning.

bio_clone() is an existing api, I agree but I'd prefer to convert
existing users in a separate patch and when I do that I want to spend
some time actually looking at the existing code instead of doing the
conversion blindly (at least some of the existing users are incorrect
and I'll have to add bio_sets for them).