block: merge __bio_map_kern into bio_map_kern
This saves a little code, and allow to simplify the error handling. Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Ming Lei <tom.leiming@gmail.com> Signed-off-by: Jens Axboe <axboe@fb.com>
This commit is contained in:
parent
26e49cfc7e
commit
75c72b8366
1 changed files with 17 additions and 33 deletions
50
block/bio.c
50
block/bio.c
|
@ -1429,8 +1429,18 @@ static void bio_map_kern_endio(struct bio *bio, int err)
|
||||||
bio_put(bio);
|
bio_put(bio);
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct bio *__bio_map_kern(struct request_queue *q, void *data,
|
/**
|
||||||
unsigned int len, gfp_t gfp_mask)
|
* bio_map_kern - map kernel address into bio
|
||||||
|
* @q: the struct request_queue for the bio
|
||||||
|
* @data: pointer to buffer to map
|
||||||
|
* @len: length in bytes
|
||||||
|
* @gfp_mask: allocation flags for bio allocation
|
||||||
|
*
|
||||||
|
* Map the kernel address into a bio suitable for io to a block
|
||||||
|
* device. Returns an error pointer in case of error.
|
||||||
|
*/
|
||||||
|
struct bio *bio_map_kern(struct request_queue *q, void *data, unsigned int len,
|
||||||
|
gfp_t gfp_mask)
|
||||||
{
|
{
|
||||||
unsigned long kaddr = (unsigned long)data;
|
unsigned long kaddr = (unsigned long)data;
|
||||||
unsigned long end = (kaddr + len + PAGE_SIZE - 1) >> PAGE_SHIFT;
|
unsigned long end = (kaddr + len + PAGE_SIZE - 1) >> PAGE_SHIFT;
|
||||||
|
@ -1454,8 +1464,11 @@ static struct bio *__bio_map_kern(struct request_queue *q, void *data,
|
||||||
bytes = len;
|
bytes = len;
|
||||||
|
|
||||||
if (bio_add_pc_page(q, bio, virt_to_page(data), bytes,
|
if (bio_add_pc_page(q, bio, virt_to_page(data), bytes,
|
||||||
offset) < bytes)
|
offset) < bytes) {
|
||||||
break;
|
/* we don't support partial mappings */
|
||||||
|
bio_put(bio);
|
||||||
|
return ERR_PTR(-EINVAL);
|
||||||
|
}
|
||||||
|
|
||||||
data += bytes;
|
data += bytes;
|
||||||
len -= bytes;
|
len -= bytes;
|
||||||
|
@ -1465,35 +1478,6 @@ static struct bio *__bio_map_kern(struct request_queue *q, void *data,
|
||||||
bio->bi_end_io = bio_map_kern_endio;
|
bio->bi_end_io = bio_map_kern_endio;
|
||||||
return bio;
|
return bio;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* bio_map_kern - map kernel address into bio
|
|
||||||
* @q: the struct request_queue for the bio
|
|
||||||
* @data: pointer to buffer to map
|
|
||||||
* @len: length in bytes
|
|
||||||
* @gfp_mask: allocation flags for bio allocation
|
|
||||||
*
|
|
||||||
* Map the kernel address into a bio suitable for io to a block
|
|
||||||
* device. Returns an error pointer in case of error.
|
|
||||||
*/
|
|
||||||
struct bio *bio_map_kern(struct request_queue *q, void *data, unsigned int len,
|
|
||||||
gfp_t gfp_mask)
|
|
||||||
{
|
|
||||||
struct bio *bio;
|
|
||||||
|
|
||||||
bio = __bio_map_kern(q, data, len, gfp_mask);
|
|
||||||
if (IS_ERR(bio))
|
|
||||||
return bio;
|
|
||||||
|
|
||||||
if (bio->bi_iter.bi_size == len)
|
|
||||||
return bio;
|
|
||||||
|
|
||||||
/*
|
|
||||||
* Don't support partial mappings.
|
|
||||||
*/
|
|
||||||
bio_put(bio);
|
|
||||||
return ERR_PTR(-EINVAL);
|
|
||||||
}
|
|
||||||
EXPORT_SYMBOL(bio_map_kern);
|
EXPORT_SYMBOL(bio_map_kern);
|
||||||
|
|
||||||
static void bio_copy_kern_endio(struct bio *bio, int err)
|
static void bio_copy_kern_endio(struct bio *bio, int err)
|
||||||
|
|
Loading…
Add table
Reference in a new issue