[PATCH] UBI: fastmap break out of used PEB search

richard -rw- weinberger richard.weinberger at gmail.com
Thu May 2 15:11:03 EDT 2013


On Thu, May 2, 2013 at 2:10 AM, Brian Pomerantz <bapper at gmail.com> wrote:
> While searching for PEB matches for each volume in the used PEB list,
> the search fails to stop when the PEB is found.  This patch adds
> a break in the inner loop to stop the search when it is matched.
>
> Signed-off-by: Brian Pomerantz <bapper at gmail.com>
> ---
>  drivers/mtd/ubi/fastmap.c |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mtd/ubi/fastmap.c b/drivers/mtd/ubi/fastmap.c
> index 0648c69..1542751 100644
> --- a/drivers/mtd/ubi/fastmap.c
> +++ b/drivers/mtd/ubi/fastmap.c
> @@ -727,8 +727,10 @@ static int ubi_attach_fastmap(struct ubi_device *ubi,
>
>                         aeb = NULL;
>                         list_for_each_entry(tmp_aeb, &used, u.list) {
> -                               if (tmp_aeb->pnum == pnum)
> +                               if (tmp_aeb->pnum == pnum) {
>                                         aeb = tmp_aeb;
> +                                       break;
> +                               }
>                         }
>
>                         /* This can happen if a PEB is already in an EBA known
> --
> 1.7.0.1

Acked-by: Richard Weinberger <richard at nod.at>

--
Thanks,
//richard



More information about the linux-mtd mailing list