diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-03-23 13:24:55 +0300 |
|---|---|---|
| committer | Jens Axboe <axboe@fb.com> | 2017-03-23 08:16:11 -0600 |
| commit | 7a88fa191944589b2ed795bbed32ca6e9e2df31f (patch) | |
| tree | 08ffd2bfc3cdf1f9ff08c0eeafc468dce213315c /include/linux/bio.h | |
| parent | a4d907b6a33ba22adbbfc443e22eb3c1aa2c73f6 (diff) | |
| download | op-kernel-dev-7a88fa191944589b2ed795bbed32ca6e9e2df31f.zip op-kernel-dev-7a88fa191944589b2ed795bbed32ca6e9e2df31f.tar.gz | |
block: make nr_iovecs unsigned in bio_alloc_bioset()
There isn't a bug here, but Smatch is not smart enough to know that
"nr_iovecs" can't be negative so it complains about underflows.
Really, it's slightly cleaner to make this parameter unsigned.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to 'include/linux/bio.h')
| -rw-r--r-- | include/linux/bio.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/bio.h b/include/linux/bio.h index 8e52119..4931756 100644 --- a/include/linux/bio.h +++ b/include/linux/bio.h @@ -383,7 +383,7 @@ extern struct bio_set *bioset_create_nobvec(unsigned int, unsigned int); extern void bioset_free(struct bio_set *); extern mempool_t *biovec_create_pool(int pool_entries); -extern struct bio *bio_alloc_bioset(gfp_t, int, struct bio_set *); +extern struct bio *bio_alloc_bioset(gfp_t, unsigned int, struct bio_set *); extern void bio_put(struct bio *); extern void __bio_clone_fast(struct bio *, struct bio *); |
