diff options
| author | Wang Sheng-Hui <shhuiw@gmail.com> | 2012-11-23 07:03:14 +0400 |
|---|---|---|
| committer | Chris Mason <chris.mason@fusionio.com> | 2012-12-17 05:46:06 +0400 |
| commit | 071401258a580dec2a3e0c2700b7e76f3ed43320 (patch) | |
| tree | 7c648128bc1863f6034909a561b6055cc8772820 /fs/btrfs/free-space-cache.c | |
| parent | 3f6bcfbd4149875662773eb40a62294cddf215d4 (diff) | |
| download | linux-071401258a580dec2a3e0c2700b7e76f3ed43320.tar.xz | |
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) static void io_ctl_map_page(struct io_ctl *io_ctl, int clear) { - WARN_ON(io_ctl->cur); BUG_ON(io_ctl->index >= io_ctl->num_pages); io_ctl->page = io_ctl->pages[io_ctl->index++]; io_ctl->cur = kmap(io_ctl->page); |