badblocks: rename badblocks_free to badblocks_exit
For symmetry with badblocks_init() make it clear that this path only destroys incremental allocations of a badblocks instance, and does not free the badblocks instance itself. Signed-off-by: Dan Williams <dan.j.williams@intel.com>
This commit is contained in:
parent
ad9a8bde2c
commit
d3b407fb3f
|
@ -550,12 +550,12 @@ int badblocks_init(struct badblocks *bb, int enable)
|
|||
EXPORT_SYMBOL_GPL(badblocks_init);
|
||||
|
||||
/**
|
||||
* badblocks_free() - free the badblocks structure
|
||||
* badblocks_exit() - free the badblocks structure
|
||||
* @bb: the badblocks structure that holds all badblock information
|
||||
*/
|
||||
void badblocks_free(struct badblocks *bb)
|
||||
void badblocks_exit(struct badblocks *bb)
|
||||
{
|
||||
kfree(bb->page);
|
||||
bb->page = NULL;
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(badblocks_free);
|
||||
EXPORT_SYMBOL_GPL(badblocks_exit);
|
||||
|
|
|
@ -671,7 +671,7 @@ void del_gendisk(struct gendisk *disk)
|
|||
blk_unregister_region(disk_devt(disk), disk->minors);
|
||||
|
||||
if (disk->bb) {
|
||||
badblocks_free(disk->bb);
|
||||
badblocks_exit(disk->bb);
|
||||
kfree(disk->bb);
|
||||
}
|
||||
|
||||
|
|
|
@ -710,7 +710,7 @@ void md_rdev_clear(struct md_rdev *rdev)
|
|||
put_page(rdev->bb_page);
|
||||
rdev->bb_page = NULL;
|
||||
}
|
||||
badblocks_free(&rdev->badblocks);
|
||||
badblocks_exit(&rdev->badblocks);
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(md_rdev_clear);
|
||||
|
||||
|
|
|
@ -48,6 +48,6 @@ ssize_t badblocks_show(struct badblocks *bb, char *page, int unack);
|
|||
ssize_t badblocks_store(struct badblocks *bb, const char *page, size_t len,
|
||||
int unack);
|
||||
int badblocks_init(struct badblocks *bb, int enable);
|
||||
void badblocks_free(struct badblocks *bb);
|
||||
void badblocks_exit(struct badblocks *bb);
|
||||
|
||||
#endif
|
||||
|
|
Loading…
Reference in New Issue