Hi, Yaffs friends
I have one question about yaffs_handleChunkError function calling code logic.
In yaffs_ReadChunkWithTagsFromNAND:
if (tags &&
tags->eccResult > YAFFS_ECC_RESULT_NO_ERROR) {
yaffs_BlockInfo *bi;
bi = yaffs_GetBlockInfo(dev, chunkInNAND/dev->param.nChunksPerBlock);
yaffs_HandleChunkError(dev, bi);
}
Why use "tags->eccResult > YAFFS_ECC_RESULT_NO_ERROR " instead of "tags->eccResult > YAFFS_ECC_RESULT_FIXED" ?
If the flash has some one bit errors and these errors are corrected, Tags->eccResult will be set to YAFFS_ECC_RESULT_FIXED. in this condition, there should be no ecc problem.
Why need to call yaffs_HandleChunkError ?
The problem is : if call yaffs_HandleChunkError, bi->chunkErrorStrikes++ will be executed. After chunkErrorStrikes > 3, the block will be retired. It will be marked as bad. But the block is not really bad. We can still use it since it only has 4 times of fixed ECC error happened.
thanks
Xiaoguang