UBI: harmonize the update of ubi->beb_rsvd_pebs
Currently, there are several locations where an attempt to reserve more PEBs for bad PEB handling is made, with the same code being duplicated. Harmonize it by introducing 'ubi_update_reserved()'. Also, improve the debug message issued, making it more descriptive. Artem: amended the patch a little. Signed-off-by: Shmulik Ladkani <shmulik.ladkani@gmail.com> Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@linux.intel.com>
This commit is contained in:

committed by
Artem Bityutskiy

parent
5c669a5bd8
commit
87e773c95e
@@ -443,15 +443,7 @@ int ubi_remove_volume(struct ubi_volume_desc *desc, int no_vtbl)
|
||||
spin_lock(&ubi->volumes_lock);
|
||||
ubi->rsvd_pebs -= reserved_pebs;
|
||||
ubi->avail_pebs += reserved_pebs;
|
||||
i = ubi->beb_rsvd_level - ubi->beb_rsvd_pebs;
|
||||
if (i > 0) {
|
||||
i = ubi->avail_pebs >= i ? i : ubi->avail_pebs;
|
||||
ubi->avail_pebs -= i;
|
||||
ubi->rsvd_pebs += i;
|
||||
ubi->beb_rsvd_pebs += i;
|
||||
if (i > 0)
|
||||
ubi_msg("reserve more %d PEBs", i);
|
||||
}
|
||||
ubi_update_reserved(ubi);
|
||||
ubi->vol_count -= 1;
|
||||
spin_unlock(&ubi->volumes_lock);
|
||||
|
||||
@@ -558,15 +550,7 @@ int ubi_resize_volume(struct ubi_volume_desc *desc, int reserved_pebs)
|
||||
spin_lock(&ubi->volumes_lock);
|
||||
ubi->rsvd_pebs += pebs;
|
||||
ubi->avail_pebs -= pebs;
|
||||
pebs = ubi->beb_rsvd_level - ubi->beb_rsvd_pebs;
|
||||
if (pebs > 0) {
|
||||
pebs = ubi->avail_pebs >= pebs ? pebs : ubi->avail_pebs;
|
||||
ubi->avail_pebs -= pebs;
|
||||
ubi->rsvd_pebs += pebs;
|
||||
ubi->beb_rsvd_pebs += pebs;
|
||||
if (pebs > 0)
|
||||
ubi_msg("reserve more %d PEBs", pebs);
|
||||
}
|
||||
ubi_update_reserved(ubi);
|
||||
for (i = 0; i < reserved_pebs; i++)
|
||||
new_mapping[i] = vol->eba_tbl[i];
|
||||
kfree(vol->eba_tbl);
|
||||
|
Reference in New Issue
Block a user