diff options
author | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2010-07-25 07:29:23 -0400 |
---|---|---|
committer | Jens Axboe <jaxboe@fusionio.com> | 2010-08-07 12:53:56 -0400 |
commit | b5048a6cb5455a16bdff26a4c5ae9534f070d94c (patch) | |
tree | fe66278b6ff27a6ef9dbef0cd3db1d8640e10bbd /mm | |
parent | 6467716a37673e8d47b4984eb19839bdad0a8353 (diff) |
writeback: remove unnecessary init_timer call
The 'setup_timer()' function also calls 'init_timer()', so the extra
'init_timer()' call is not needed. Indeed, 'setup_timer()' is basically
'init_timer()' plus callback function and data pointers initialization.
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <jaxboe@fusionio.com>
Diffstat (limited to 'mm')
-rw-r--r-- | mm/backing-dev.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/mm/backing-dev.c b/mm/backing-dev.c index cfff7225138c..99890831d557 100644 --- a/mm/backing-dev.c +++ b/mm/backing-dev.c | |||
@@ -236,7 +236,6 @@ static int __init default_bdi_init(void) | |||
236 | sync_supers_tsk = kthread_run(bdi_sync_supers, NULL, "sync_supers"); | 236 | sync_supers_tsk = kthread_run(bdi_sync_supers, NULL, "sync_supers"); |
237 | BUG_ON(IS_ERR(sync_supers_tsk)); | 237 | BUG_ON(IS_ERR(sync_supers_tsk)); |
238 | 238 | ||
239 | init_timer(&sync_supers_timer); | ||
240 | setup_timer(&sync_supers_timer, sync_supers_timer_fn, 0); | 239 | setup_timer(&sync_supers_timer, sync_supers_timer_fn, 0); |
241 | bdi_arm_supers_timer(); | 240 | bdi_arm_supers_timer(); |
242 | 241 | ||