Commit 64d4b4c9 authored by Artem Bityutskiy's avatar Artem Bityutskiy
Browse files

UBI: do not warn unnecessarily 



Currently, when UBI attaches an MTD device and cannot reserve all 1% (by
default) of PEBs for bad eraseblocks handling, it prints a warning. However,
Matthew L. Creech <mlcreech@gmail.com> is not very happy to see this warning,
because he did reserve enough of PEB at the beginning, but with time some
PEBs became bad. The warning is not necessary in this case.

This patch makes UBI print the warning
 o if this is a new image
 o of this is used image and the amount of reserved PEBs is only 10% (or less)
   of the size of the reserved PEB pool.

Signed-off-by: default avatarArtem Bityutskiy <Artem.Bityutskiy@nokia.com>
parent 7cdb996e

drivers/mtd/ubi/build.c

+2 −1
Original line number Diff line number Diff line
@@ -593,6 +593,7 @@ static int attach_by_scanning(struct ubi_device *ubi) 
	ubi->good_peb_count = ubi->peb_count - ubi->bad_peb_count;

	ubi->max_ec = si->max_ec;

	ubi->mean_ec = si->mean_ec;

	ubi_msg("max. sequence number:       %llu", si->max_sqnum);



	err = ubi_read_volume_table(ubi, si);

	if (err)

drivers/mtd/ubi/eba.c

+39 −3
Original line number Diff line number Diff line
@@ -1165,6 +1165,44 @@ out_unlock_leb: 
	return err;

}



/**

 * print_rsvd_warning - warn about not having enough reserved PEBs.

 * @ubi: UBI device description object

 *

 * This is a helper function for 'ubi_eba_init_scan()' which is called when UBI

 * cannot reserve enough PEBs for bad block handling. This function makes a

 * decision whether we have to print a warning or not. The algorithm is as

 * follows:

 *   o if this is a new UBI image, then just print the warning

 *   o if this is an UBI image which has already been used for some time, print

 *     a warning only if we can reserve less than 10% of the expected amount of

 *     the reserved PEB.

 *

 * The idea is that when UBI is used, PEBs become bad, and the reserved pool

 * of PEBs becomes smaller, which is normal and we do not want to scare users

 * with a warning every time they attach the MTD device. This was an issue

 * reported by real users.

 */

static void print_rsvd_warning(struct ubi_device *ubi,

			       struct ubi_scan_info *si)

{

	/*

	 * The 1 << 18 (256KiB) number is picked randomly, just a reasonably

	 * large number to distinguish between newly flashed and used images.

	 */

	if (si->max_sqnum > (1 << 18)) {

		int min = ubi->beb_rsvd_level / 10;



		if (!min)

			min = 1;

		if (ubi->beb_rsvd_pebs > min)

			return;

	}



	ubi_warn("cannot reserve enough PEBs for bad PEB handling, reserved %d,"

		 " need %d", ubi->beb_rsvd_pebs, ubi->beb_rsvd_level);

}



/**

 * ubi_eba_init_scan - initialize the EBA sub-system using scanning information.

 * @ubi: UBI device description object
@@ -1237,9 +1275,7 @@ int ubi_eba_init_scan(struct ubi_device *ubi, struct ubi_scan_info *si) 
		if (ubi->avail_pebs < ubi->beb_rsvd_level) {

			/* No enough free physical eraseblocks */

			ubi->beb_rsvd_pebs = ubi->avail_pebs;

			ubi_warn("cannot reserve enough PEBs for bad PEB "

				 "handling, reserved %d, need %d",

				 ubi->beb_rsvd_pebs, ubi->beb_rsvd_level);

			print_rsvd_warning(ubi, si);

		} else

			ubi->beb_rsvd_pebs = ubi->beb_rsvd_level;

CRA Git | Maintained and supported by SUSTech CRA and CCSE