diff options
author | Richard Weinberger <richard@nod.at> | 2012-12-03 14:57:46 -0500 |
---|---|---|
committer | Artem Bityutskiy <artem.bityutskiy@linux.intel.com> | 2012-12-04 09:04:16 -0500 |
commit | ed4b7021cb51fe5a0f260df03298709347a26967 (patch) | |
tree | 483eee94b6c30566e1b7694169e83e9014567b4b /drivers | |
parent | 9489e9dcae718d5fde988e4a684a0f55b5f94d17 (diff) |
UBI: remove PEB from free tree in get_peb_for_wl()
If UBI is built without fastmap, get_peb_for_wl() has to
remove the PEB manially from the free tree.
Otherwise the requested PEB lives in two trees.
Reported-by: Zach Sadecki <zsadecki@itwatchdogs.com>
Signed-off-by: Richard Weinberger <richard@nod.at>
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/mtd/ubi/wl.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/drivers/mtd/ubi/wl.c b/drivers/mtd/ubi/wl.c index da7b44998b40..1f9f5f7a5d0f 100644 --- a/drivers/mtd/ubi/wl.c +++ b/drivers/mtd/ubi/wl.c | |||
@@ -679,7 +679,13 @@ static struct ubi_wl_entry *get_peb_for_wl(struct ubi_device *ubi) | |||
679 | #else | 679 | #else |
680 | static struct ubi_wl_entry *get_peb_for_wl(struct ubi_device *ubi) | 680 | static struct ubi_wl_entry *get_peb_for_wl(struct ubi_device *ubi) |
681 | { | 681 | { |
682 | return find_wl_entry(ubi, &ubi->free, WL_FREE_MAX_DIFF); | 682 | struct ubi_wl_entry *e; |
683 | |||
684 | e = find_wl_entry(ubi, &ubi->free, WL_FREE_MAX_DIFF); | ||
685 | self_check_in_wl_tree(ubi, e, &ubi->free); | ||
686 | rb_erase(&e->u.rb, &ubi->free); | ||
687 | |||
688 | return e; | ||
683 | } | 689 | } |
684 | 690 | ||
685 | int ubi_wl_get_peb(struct ubi_device *ubi) | 691 | int ubi_wl_get_peb(struct ubi_device *ubi) |