block: remove bio_clone() and all references.
authorNeilBrown <neilb@suse.com>
Sun, 18 Jun 2017 04:38:59 +0000 (14:38 +1000)
committerJens Axboe <axboe@kernel.dk>
Sun, 18 Jun 2017 18:40:59 +0000 (12:40 -0600)
bio_clone() is no longer used.
Only bio_clone_bioset() or bio_clone_fast().
This is for the best, as bio_clone() used fs_bio_set,
and filesystems are unlikely to want to use bio_clone().

So remove bio_clone() and all references.
This includes a fix to some incorrect documentation.

Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: NeilBrown <neilb@suse.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Documentation/block/biodoc.txt
block/bio.c
block/blk-merge.c
drivers/md/md.c
include/linux/bio.h

index 01ddeaf64b0f6cdba051a66c84e70918198e6e1e..9490f2845f06950f044ae679f40d04c603eeed22 100644 (file)
@@ -632,7 +632,7 @@ to i/o submission, if the bio fields are likely to be accessed after the
 i/o is issued (since the bio may otherwise get freed in case i/o completion
 happens in the meantime).
 
-The bio_clone() routine may be used to duplicate a bio, where the clone
+The bio_clone_fast() routine may be used to duplicate a bio, where the clone
 shares the bio_vec_list with the original bio (i.e. both point to the
 same bio_vec_list). This would typically be used for splitting i/o requests
 in lvm or md.
index 2bd064906e068995cdc5163b795424c6fb30fff8..89a51bd49ab75e814c20371d630430e722dea649 100644 (file)
@@ -547,7 +547,7 @@ EXPORT_SYMBOL(zero_fill_bio);
  *
  * Description:
  *   Put a reference to a &struct bio, either one you have gotten with
- *   bio_alloc, bio_get or bio_clone. The last put of a bio will free it.
+ *   bio_alloc, bio_get or bio_clone_*. The last put of a bio will free it.
  **/
 void bio_put(struct bio *bio)
 {
index 51c84540d3bb4cf3d01d2abd579fa828894421e5..e7862e9dcc395a84b64735ffcb2b4449547a497f 100644 (file)
@@ -115,13 +115,13 @@ static struct bio *blk_bio_segment_split(struct request_queue *q,
                 * With arbitrary bio size, the incoming bio may be very
                 * big. We have to split the bio into small bios so that
                 * each holds at most BIO_MAX_PAGES bvecs because
-                * bio_clone() can fail to allocate big bvecs.
+                * bio_clone_bioset() can fail to allocate big bvecs.
                 *
-                * Those drivers which will need to use bio_clone()
+                * Those drivers which will need to use bio_clone_bioset()
                 * should tell us in some way.  For now, impose the
                 * BIO_MAX_PAGES limit on all queues.
                 *
-                * TODO: handle users of bio_clone() differently.
+                * TODO: handle users of bio_clone_bioset() differently.
                 */
                if (bvecs++ >= BIO_MAX_PAGES)
                        goto split;
index 07fe780ccd29459df74e05b5810f223a5ed544f8..31bcbfb09fefaf0be7256d5bb3af8eaa36bce45f 100644 (file)
@@ -185,7 +185,7 @@ static int start_readonly;
 static bool create_on_open = true;
 
 /* bio_clone_mddev
- * like bio_clone, but with a local bio set
+ * like bio_clone_bioset, but with a local bio set
  */
 
 struct bio *bio_alloc_mddev(gfp_t gfp_mask, int nr_iovecs,
index 32c786baa10ab4e6f8572c988e6e8b7921b36f7b..40d0541852771eec1c686702bc1159e91813494e 100644 (file)
@@ -395,11 +395,6 @@ static inline struct bio *bio_alloc(gfp_t gfp_mask, unsigned int nr_iovecs)
        return bio_alloc_bioset(gfp_mask, nr_iovecs, fs_bio_set);
 }
 
-static inline struct bio *bio_clone(struct bio *bio, gfp_t gfp_mask)
-{
-       return bio_clone_bioset(bio, gfp_mask, fs_bio_set);
-}
-
 static inline struct bio *bio_kmalloc(gfp_t gfp_mask, unsigned int nr_iovecs)
 {
        return bio_alloc_bioset(gfp_mask, nr_iovecs, NULL);