Commit b81000b6 authored by Richard Weinberger's avatar Richard Weinberger Committed by Artem Bityutskiy
Browse files

UBI: vtbl: Use ubi_eba_atomic_leb_change() 



This is more a cosmetic change than a fix.
By using ubi_eba_atomic_leb_change()
we can guarantee that the first VTBL record is always
correct and we don't really need the second one anymore.
But we have to keep the second one to not break anything.

Artem: add a comment

Signed-off-by: default avatarRichard Weinberger <richard@nod.at>
Signed-off-by: default avatarArtem Bityutskiy <artem.bityutskiy@linux.intel.com>
parent 32608703

drivers/mtd/ubi/vtbl.c

+9 −14
Original line number Diff line number Diff line
@@ -30,9 +30,12 @@ 
 * eraseblock stores one volume table copy, i.e. LEB 0 and LEB 1 duplicate each

 * other. This redundancy guarantees robustness to unclean reboots. The volume

 * table is basically an array of volume table records. Each record contains

 * full information about the volume and protected by a CRC checksum.

 * full information about the volume and protected by a CRC checksum. Note,

 * nowadays we use the atomic LEB change operation when updating the volume

 * table, so we do not really need 2 LEBs anymore, but we preserve the older

 * design for the backward compatibility reasons.

 *

 * The volume table is changed, it is first changed in RAM. Then LEB 0 is

 * When the volume table is changed, it is first changed in RAM. Then LEB 0 is

 * erased, and the updated volume table is written back to LEB 0. Then same for

 * LEB 1. This scheme guarantees recoverability from unclean reboots.

 *
@@ -96,11 +99,7 @@ int ubi_change_vtbl_record(struct ubi_device *ubi, int idx, 


	memcpy(&ubi->vtbl[idx], vtbl_rec, sizeof(struct ubi_vtbl_record));

	for (i = 0; i < UBI_LAYOUT_VOLUME_EBS; i++) {

		err = ubi_eba_unmap_leb(ubi, layout_vol, i);

		if (err)

			return err;



		err = ubi_eba_write_leb(ubi, layout_vol, i, ubi->vtbl, 0,

		err = ubi_eba_atomic_leb_change(ubi, layout_vol, i, ubi->vtbl,

						ubi->vtbl_size);

		if (err)

			return err;
@@ -148,11 +147,7 @@ int ubi_vtbl_rename_volumes(struct ubi_device *ubi, 


	layout_vol = ubi->volumes[vol_id2idx(ubi, UBI_LAYOUT_VOLUME_ID)];

	for (i = 0; i < UBI_LAYOUT_VOLUME_EBS; i++) {

		err = ubi_eba_unmap_leb(ubi, layout_vol, i);

		if (err)

			return err;



		err = ubi_eba_write_leb(ubi, layout_vol, i, ubi->vtbl, 0,

		err = ubi_eba_atomic_leb_change(ubi, layout_vol, i, ubi->vtbl,

						ubi->vtbl_size);

		if (err)

			return err;

CRA Git | Maintained and supported by SUSTech CRA and CCSE