diff options
author | Wang Sheng-Hui <shhuiw@gmail.com> | 2012-11-22 22:03:14 -0500 |
---|---|---|
committer | Chris Mason <chris.mason@fusionio.com> | 2012-12-16 20:46:06 -0500 |
commit | 071401258a580dec2a3e0c2700b7e76f3ed43320 (patch) | |
tree | 7c648128bc1863f6034909a561b6055cc8772820 /fs/btrfs/free-space-cache.c | |
parent | 3f6bcfbd4149875662773eb40a62294cddf215d4 (diff) |
Btrfs: do not warn_on io_ctl->cur in io_ctl_map_page
io_ctl_map_page is called by many functions in free-space-cache.
In most scenarios, the ->cur is not null, e.g. io_ctl_add_entry.
I think we'd better remove the warn_on here.
Signed-off-by: Wang Sheng-Hui <shhuiw@gmail.com>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
Diffstat (limited to 'fs/btrfs/free-space-cache.c')
-rw-r--r-- | fs/btrfs/free-space-cache.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/btrfs/free-space-cache.c b/fs/btrfs/free-space-cache.c index 557502ca1a2a..efdd1d3f441c 100644 --- a/fs/btrfs/free-space-cache.c +++ b/fs/btrfs/free-space-cache.c | |||
@@ -307,7 +307,6 @@ static void io_ctl_unmap_page(struct io_ctl *io_ctl) | |||
307 | 307 | ||
308 | static void io_ctl_map_page(struct io_ctl *io_ctl, int clear) | 308 | static void io_ctl_map_page(struct io_ctl *io_ctl, int clear) |
309 | { | 309 | { |
310 | WARN_ON(io_ctl->cur); | ||
311 | BUG_ON(io_ctl->index >= io_ctl->num_pages); | 310 | BUG_ON(io_ctl->index >= io_ctl->num_pages); |
312 | io_ctl->page = io_ctl->pages[io_ctl->index++]; | 311 | io_ctl->page = io_ctl->pages[io_ctl->index++]; |
313 | io_ctl->cur = kmap(io_ctl->page); | 312 | io_ctl->cur = kmap(io_ctl->page); |