projects / yaffs2.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
yaffs Fix incorrect handling of deletion flag
[yaffs2.git] / yaffs_checkptrw.c
diff --git a/yaffs_checkptrw.c b/yaffs_checkptrw.c
index ea03e3a298a9da28417f0333ec4ed6dad0953a17..1304c19c185a56abe8a97204ddc91055fb504669 100644 (file)
--- a/yaffs_checkptrw.c
+++ b/yaffs_checkptrw.c
@@ -1,7 +1,7 @@
 /*
  * YAFFS: Yet Another Flash File System. A NAND-flash specific file system.
  *
- * Copyright (C) 2002-2007 Aleph One Ltd.
+ * Copyright (C) 2002-2010 Aleph One Ltd.
  *   for Toby Churchill Ltd and Brightstar Engineering
  *
  * Created by Charles Manning <charles@aleph1.co.uk>
@@ -11,16 +11,12 @@
  * published by the Free Software Foundation.
  */
 
-const char *yaffs_checkptrw_c_version =
-       "$Id: yaffs_checkptrw.c,v 1.19 2009-06-19 01:35:46 charles Exp $";
-
-
 #include "yaffs_checkptrw.h"
 #include "yaffs_getblockinfo.h"
 
 static int yaffs_CheckpointSpaceOk(yaffs_Device *dev)
 {
-       int blocksAvailable = dev->nErasedBlocks - dev->nReservedBlocks;
+       int blocksAvailable = dev->nErasedBlocks - dev->param.nReservedBlocks;
 
        T(YAFFS_TRACE_CHECKPOINT,
                (TSTR("checkpt blocks available = %d" TENDSTR),
@@ -34,7 +30,7 @@ static int yaffs_CheckpointErase(yaffs_Device *dev)
 {
        int i;
 
-       if (!dev->eraseBlockInNAND)
+       if (!dev->param.eraseBlockInNAND)
                return 0;
        T(YAFFS_TRACE_CHECKPOINT, (TSTR("checking blocks %d to %d"TENDSTR),
                dev->internalStartBlock, dev->internalEndBlock));
@@ -43,12 +39,15 @@ static int yaffs_CheckpointErase(yaffs_Device *dev)
                yaffs_BlockInfo *bi = yaffs_GetBlockInfo(dev, i);
                if (bi->blockState == YAFFS_BLOCK_STATE_CHECKPOINT) {
                        T(YAFFS_TRACE_CHECKPOINT, (TSTR("erasing checkpt block %d"TENDSTR), i));
-                       if (dev->eraseBlockInNAND(dev, i - dev->blockOffset /* realign */)) {
+
+                       dev->nBlockErasures++;
+
+                       if (dev->param.eraseBlockInNAND(dev, i - dev->blockOffset /* realign */)) {
                                bi->blockState = YAFFS_BLOCK_STATE_EMPTY;
                                dev->nErasedBlocks++;
-                               dev->nFreeChunks += dev->nChunksPerBlock;
+                               dev->nFreeChunks += dev->param.nChunksPerBlock;
                        } else {
-                               dev->markNANDBlockBad(dev, i);
+                               dev->param.markNANDBlockBad(dev, i);
                                bi->blockState = YAFFS_BLOCK_STATE_DEAD;
                        }
                }
@@ -63,10 +62,10 @@ static int yaffs_CheckpointErase(yaffs_Device *dev)
 static void yaffs_CheckpointFindNextErasedBlock(yaffs_Device *dev)
 {
        int  i;
-       int blocksAvailable = dev->nErasedBlocks - dev->nReservedBlocks;
+       int blocksAvailable = dev->nErasedBlocks - dev->param.nReservedBlocks;
        T(YAFFS_TRACE_CHECKPOINT,
                (TSTR("allocating checkpt block: erased %d reserved %d avail %d next %d "TENDSTR),
-               dev->nErasedBlocks, dev->nReservedBlocks, blocksAvailable, dev->checkpointNextBlock));
+               dev->nErasedBlocks, dev->param.nReservedBlocks, blocksAvailable, dev->checkpointNextBlock));
 
        if (dev->checkpointNextBlock >= 0 &&
                        dev->checkpointNextBlock <= dev->internalEndBlock &&
@@ -98,10 +97,10 @@ static void yaffs_CheckpointFindNextCheckpointBlock(yaffs_Device *dev)
 
        if (dev->blocksInCheckpoint < dev->checkpointMaxBlocks)
                for (i = dev->checkpointNextBlock; i <= dev->internalEndBlock; i++) {
-                       int chunk = i * dev->nChunksPerBlock;
+                       int chunk = i * dev->param.nChunksPerBlock;
                        int realignedChunk = chunk - dev->chunkOffset;
 
-                       dev->readChunkWithTagsFromNAND(dev, realignedChunk,
+                       dev->param.readChunkWithTagsFromNAND(dev, realignedChunk,
                                        NULL, &tags);
                        T(YAFFS_TRACE_CHECKPOINT, (TSTR("find next checkpt block: search: block %d oid %d seq %d eccr %d" TENDSTR),
                                i, tags.objectId, tags.sequenceNumber, tags.eccResult));
@@ -127,26 +126,26 @@ static void yaffs_CheckpointFindNextCheckpointBlock(yaffs_Device *dev)
 int yaffs_CheckpointOpen(yaffs_Device *dev, int forWriting)
 {
 
+
+       dev->checkpointOpenForWrite = forWriting;
+
        /* Got the functions we need? */
-       if (!dev->writeChunkWithTagsToNAND ||
-                       !dev->readChunkWithTagsFromNAND ||
-                       !dev->eraseBlockInNAND ||
-                       !dev->markNANDBlockBad)
+       if (!dev->param.writeChunkWithTagsToNAND ||
+               !dev->param.readChunkWithTagsFromNAND ||
+               !dev->param.eraseBlockInNAND ||
+               !dev->param.markNANDBlockBad)
                return 0;
 
        if (forWriting && !yaffs_CheckpointSpaceOk(dev))
                return 0;
 
        if (!dev->checkpointBuffer)
-               dev->checkpointBuffer = YMALLOC_DMA(dev->totalBytesPerChunk);
+               dev->checkpointBuffer = YMALLOC_DMA(dev->param.totalBytesPerChunk);
        if (!dev->checkpointBuffer)
                return 0;
 
 
        dev->checkpointPageSequence = 0;
-
-       dev->checkpointOpenForWrite = forWriting;
-
        dev->checkpointByteCount = 0;
        dev->checkpointSum = 0;
        dev->checkpointXor = 0;
@@ -214,7 +213,7 @@ static int yaffs_CheckpointFlushBuffer(yaffs_Device *dev)
                dev->blocksInCheckpoint++;
        }
 
-       chunk = dev->checkpointCurrentBlock * dev->nChunksPerBlock + dev->checkpointCurrentChunk;
+       chunk = dev->checkpointCurrentBlock * dev->param.nChunksPerBlock + dev->checkpointCurrentChunk;
 
 
        T(YAFFS_TRACE_CHECKPOINT, (TSTR("checkpoint wite buffer nand %d(%d:%d) objid %d chId %d" TENDSTR),
@@ -222,12 +221,14 @@ static int yaffs_CheckpointFlushBuffer(yaffs_Device *dev)
 
        realignedChunk = chunk - dev->chunkOffset;
 
-       dev->writeChunkWithTagsToNAND(dev, realignedChunk,
+       dev->nPageWrites++;
+
+       dev->param.writeChunkWithTagsToNAND(dev, realignedChunk,
                        dev->checkpointBuffer, &tags);
        dev->checkpointByteOffset = 0;
        dev->checkpointPageSequence++;
        dev->checkpointCurrentChunk++;
-       if (dev->checkpointCurrentChunk >= dev->nChunksPerBlock) {
+       if (dev->checkpointCurrentChunk >= dev->param.nChunksPerBlock) {
                dev->checkpointCurrentChunk = 0;
                dev->checkpointCurrentBlock = -1;
        }
@@ -305,14 +306,16 @@ int yaffs_CheckpointRead(yaffs_Device *dev, void *data, int nBytes)
                                ok = 0;
                        else {
                                chunk = dev->checkpointCurrentBlock *
-                                       dev->nChunksPerBlock +
+                                       dev->param.nChunksPerBlock +
                                        dev->checkpointCurrentChunk;
 
                                realignedChunk = chunk - dev->chunkOffset;
+                               
+                               dev->nPageReads++;
 
                                /* read in the next chunk */
                                /* printf("read checkpoint page %d\n",dev->checkpointPage); */
-                               dev->readChunkWithTagsFromNAND(dev,
+                               dev->param.readChunkWithTagsFromNAND(dev,
                                                realignedChunk,
                                                dev->checkpointBuffer,
                                                &tags);
@@ -326,7 +329,7 @@ int yaffs_CheckpointRead(yaffs_Device *dev, void *data, int nBytes)
                                dev->checkpointPageSequence++;
                                dev->checkpointCurrentChunk++;
 
-                               if (dev->checkpointCurrentChunk >= dev->nChunksPerBlock)
+                               if (dev->checkpointCurrentChunk >= dev->param.nChunksPerBlock)
                                        dev->checkpointCurrentBlock = -1;
                        }
                }
@@ -351,11 +354,14 @@ int yaffs_CheckpointClose(yaffs_Device *dev)
        if (dev->checkpointOpenForWrite) {
                if (dev->checkpointByteOffset != 0)
                        yaffs_CheckpointFlushBuffer(dev);
-       } else {
+       } else if(dev->checkpointBlockList){
                int i;
                for (i = 0; i < dev->blocksInCheckpoint && dev->checkpointBlockList[i] >= 0; i++) {
-                       yaffs_BlockInfo *bi = yaffs_GetBlockInfo(dev, dev->checkpointBlockList[i]);
-                       if (bi->blockState == YAFFS_BLOCK_STATE_EMPTY)
+                       int blk = dev->checkpointBlockList[i];
+                       yaffs_BlockInfo *bi = NULL;
+                       if( dev->internalStartBlock <= blk && blk <= dev->internalEndBlock)
+                               bi = yaffs_GetBlockInfo(dev, blk);
+                       if (bi && bi->blockState == YAFFS_BLOCK_STATE_EMPTY)
                                bi->blockState = YAFFS_BLOCK_STATE_CHECKPOINT;
                        else {
                                /* Todo this looks odd... */
@@ -365,7 +371,7 @@ int yaffs_CheckpointClose(yaffs_Device *dev)
                dev->checkpointBlockList = NULL;
        }
 
-       dev->nFreeChunks -= dev->blocksInCheckpoint * dev->nChunksPerBlock;
+       dev->nFreeChunks -= dev->blocksInCheckpoint * dev->param.nChunksPerBlock;
        dev->nErasedBlocks -= dev->blocksInCheckpoint;
 
 
@@ -383,12 +389,10 @@ int yaffs_CheckpointClose(yaffs_Device *dev)
 
 int yaffs_CheckpointInvalidateStream(yaffs_Device *dev)
 {
-       /* Erase the first checksum block */
-
-       T(YAFFS_TRACE_CHECKPOINT, (TSTR("checkpoint invalidate"TENDSTR)));
+       /* Erase the checkpoint data */
 
-       if (!yaffs_CheckpointSpaceOk(dev))
-               return 0;
+       T(YAFFS_TRACE_CHECKPOINT, (TSTR("checkpoint invalidate of %d blocks"TENDSTR),
+               dev->blocksInCheckpoint));
 
        return yaffs_CheckpointErase(dev);
 }
The YAFFS2 repository.