All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vikram Narayanan <vikram186@gmail.com>
To: Stefan Roese <sr@denx.de>
Cc: Huang Shijie <b32955@freescale.com>, linux-mtd@lists.infradead.org
Subject: Re: mtd_oobtest fails with GPMI-NAND
Date: Mon, 13 May 2013 22:08:16 +0530	[thread overview]
Message-ID: <519116F8.1070902@gmail.com> (raw)
In-Reply-To: <518FB09D.3070101@denx.de>

Hello Stefan,

On 5/12/2013 8:39 PM, Stefan Roese wrote:
>> Looks like it is happening at random. Two different physical locations
>> on two different runs.
>
> These locations (blocks) where the errors happen are not always
> identical. But some blocks reoccur. And this Linux test stops
> when one error is detected. The U-Boot reported multiple blocks,
> where some blocks were most of the time defective.
>
> So its not really random.

Ok.

>> The only error I came across in u-boot is "fixable bit-flip" issue. My
>> colleagues have reported it sometime back. But I couldn't reproduce it
>> and neither could they.
>
> Could you please post a link to this thread?

http://lists.denx.de/pipermail/u-boot/2012-December/142476.html

>> Can you please post the logs for the "uncorrectable errors" in u-boot?
>> That might give some hints to Huang.
>
> Sure. Please note that I tracked the error (-74) back to the U-Boot
> imx NAND driver (mxs_nand.c):
>
> 	/* Loop over status bytes, accumulating ECC status. */
> 	status = nand_info->oob_buf + mxs_nand_aux_status_offset();
> 	for (i = 0; i < mxs_nand_ecc_chunk_cnt(mtd->writesize); i++) {
> 		if (status[i] == 0x00)
> 			continue;
>
> 		if (status[i] == 0xff)
> 			continue;
>
> 		if (status[i] == 0xfe) {
> 			failed++;
> 			continue;
> 		}
>
> 		corrected += status[i];
> 	}
>
> I could also add some code to the Linux driver to see, if the error
> happens at the same place (status == 0xfe). I'm pretty sure that this
> is the case though.
>
>
> Here the logs from U-Boot:
>
> => nand erase.part ubi;
> NAND erase.part: device 0 offset 0x1100000, size 0x1ef00000
> Erasing at 0x1ffe0000 -- 100% complete.
> OK
> => ubi part ubi
> UBI: mtd1 is detached from ubi0
> UBI: attaching mtd1 to ubi0
> UBI: physical eraseblock size:   131072 bytes (128 KiB)
> UBI: logical eraseblock size:    126976 bytes
> UBI: smallest flash I/O unit:    2048
> UBI: VID header offset:          2048 (aligned 2048)
> UBI: data offset:                4096
> UBI error: ubi_io_read: error -74 while reading 64 bytes from PEB 3093:0, read 64 bytes
> UBI error: ubi_io_read: error -74 while reading 64 bytes from PEB 3956:0, read 64 bytes
> UBI error: ubi_read_volume_table: the layout volume was not found
> UBI error: ubi_init: cannot attach mtd1
> UBI error: ubi_init: UBI error: cannot initialize UBI, error -22
> UBI init error 22
>
>
> Another thing to mention is, that using this command (nand erase;
> nand erase;ubi part) never triggers this problem. So an additional
> erase somehow seems to fix this problem. Still not sure why this is
> the case.

I'll test this as well with the latest u-boot.

>
> Vikram, which NAND part/chip are you using again?

I'm using "Toshiba, TC58NYG2S0FBAI4".

Regards,
Vikram

  reply	other threads:[~2013-05-13 16:38 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-18 16:52 mtd_oobtest fails with GPMI-NAND Vikram Narayanan
2013-01-21  2:12 ` Huang Shijie
2013-01-28  2:39   ` Vikram Narayanan
2013-01-28  3:20     ` Huang Shijie
2013-01-28 17:04       ` Vikram Narayanan
2013-01-29  2:06         ` Huang Shijie
2013-01-29  2:26           ` Vikram Narayanan
2013-01-29  2:36             ` Huang Shijie
2013-01-29 16:28               ` Vikram Narayanan
2013-01-30  2:27                 ` Huang Shijie
2013-02-02  6:41                   ` Vikram Narayanan
2013-02-02  7:42                     ` Huang Shijie
2013-02-02  7:46                       ` Huang Shijie
2013-05-08 14:33                     ` Stefan Roese
2013-05-09 12:30                       ` Vikram Narayanan
2013-05-10  6:20                         ` Stefan Roese
2013-05-12 12:10                           ` Vikram Narayanan
2013-05-12 15:09                             ` Stefan Roese
2013-05-13 16:38                               ` Vikram Narayanan [this message]
2013-05-13  2:51                           ` Huang Shijie
2013-05-13  8:01                             ` Stefan Roese
2013-05-13  9:01                               ` Huang Shijie
2013-05-13  9:22                                 ` Stefan Roese
2013-05-13  9:34                                   ` Huang Shijie
2013-05-13 10:02                                     ` Stefan Roese
2013-05-14  2:09                                       ` Huang Shijie
2013-05-14  2:11                                       ` Huang Shijie
2013-05-13 16:51                                 ` Vikram Narayanan
2013-05-14  2:23                                   ` Huang Shijie
2013-05-14  2:33                                     ` Vikram Narayanan
2013-05-14  2:47                                       ` Huang Shijie
2013-05-13 16:43                               ` Vikram Narayanan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=519116F8.1070902@gmail.com \
    --to=vikram186@gmail.com \
    --cc=b32955@freescale.com \
    --cc=linux-mtd@lists.infradead.org \
    --cc=sr@denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.