summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArtem Bityutskiy <Artem.Bityutskiy@nokia.com>2010-07-25 13:29:23 +0200
committerJens Axboe <jaxboe@fusionio.com>2010-08-07 18:53:56 +0200
commitb5048a6cb5455a16bdff26a4c5ae9534f070d94c (patch)
treefe66278b6ff27a6ef9dbef0cd3db1d8640e10bbd
parentwriteback: optimize periodic bdi thread wakeups (diff)
downloadlinux-b5048a6cb5455a16bdff26a4c5ae9534f070d94c.tar.xz
linux-b5048a6cb5455a16bdff26a4c5ae9534f070d94c.zip
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>
-rw-r--r--mm/backing-dev.c1
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)
sync_supers_tsk = kthread_run(bdi_sync_supers, NULL, "sync_supers");
BUG_ON(IS_ERR(sync_supers_tsk));
- init_timer(&sync_supers_timer);
setup_timer(&sync_supers_timer, sync_supers_timer_fn, 0);
bdi_arm_supers_timer();