diff options
author | Jaak Ristioja <Ristioja@gmail.com> | 2010-01-15 05:05:07 -0500 |
---|---|---|
committer | Jiri Kosina <jkosina@suse.cz> | 2010-02-05 06:22:36 -0500 |
commit | db18efac0bd644e4b7af757394ca0bef8d91dcee (patch) | |
tree | bffe9f3b834f50cbd1cb0cf778eb58ef34dbe56d /fs/bio.c | |
parent | 6382dc334064bb0b41a95df0e3c438de35f2ffb7 (diff) |
bio: Fix outdated comment about bio_alloc_bioset()
In commit 451a9ebf653d28337ba53ed5b4b70b0b9543cca1 bio_alloc_bioset()
was refactored not to take NULL as a valid argument for bs. This patch
changes the comment for that function accordingly. Currently, passing
NULL as argument to parameter bs would result in a NULL pointer
dereference.
Signed-off-by: Jaak Ristioja <ristioja@gmail.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'fs/bio.c')
-rw-r--r-- | fs/bio.c | 7 |
1 files changed, 3 insertions, 4 deletions
@@ -264,13 +264,12 @@ EXPORT_SYMBOL(bio_init); | |||
264 | * bio_alloc_bioset - allocate a bio for I/O | 264 | * bio_alloc_bioset - allocate a bio for I/O |
265 | * @gfp_mask: the GFP_ mask given to the slab allocator | 265 | * @gfp_mask: the GFP_ mask given to the slab allocator |
266 | * @nr_iovecs: number of iovecs to pre-allocate | 266 | * @nr_iovecs: number of iovecs to pre-allocate |
267 | * @bs: the bio_set to allocate from. If %NULL, just use kmalloc | 267 | * @bs: the bio_set to allocate from. |
268 | * | 268 | * |
269 | * Description: | 269 | * Description: |
270 | * bio_alloc_bioset will first try its own mempool to satisfy the allocation. | 270 | * bio_alloc_bioset will try its own mempool to satisfy the allocation. |
271 | * If %__GFP_WAIT is set then we will block on the internal pool waiting | 271 | * If %__GFP_WAIT is set then we will block on the internal pool waiting |
272 | * for a &struct bio to become free. If a %NULL @bs is passed in, we will | 272 | * for a &struct bio to become free. |
273 | * fall back to just using @kmalloc to allocate the required memory. | ||
274 | * | 273 | * |
275 | * Note that the caller must set ->bi_destructor on successful return | 274 | * Note that the caller must set ->bi_destructor on successful return |
276 | * of a bio, to do the appropriate freeing of the bio once the reference | 275 | * of a bio, to do the appropriate freeing of the bio once the reference |