]> git.proxmox.com Git - mirror_edk2.git/commitdiff
MdeModulePkg FaultTolerantWriteDxe: Remove assumptions below in FTW driver.
authorStar Zeng <star.zeng@intel.com>
Fri, 10 Oct 2014 02:27:14 +0000 (02:27 +0000)
committerlzeng14 <lzeng14@6f19259b-4bc3-4df7-8a09-765794883524>
Fri, 10 Oct 2014 02:27:14 +0000 (02:27 +0000)
1. Work space and spare block must be in a FVB with FV header.
Updated to work space and spare block could be in independent FVBs that are without FV header.
2. NV region, work space and spare block must have same BlockSize.
Updated to NV region, work space and spare block could have different BlockSize.
3. Works space size must be <= one block size.
Update to work space size could be <= one block size (not span blocks) or > one block size (block size aligned).

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Star Zeng <star.zeng@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@16201 6f19259b-4bc3-4df7-8a09-765794883524

MdeModulePkg/MdeModulePkg.dec
MdeModulePkg/MdeModulePkg.uni
MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWrite.c
MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWrite.h
MdeModulePkg/Universal/FaultTolerantWriteDxe/FtwMisc.c
MdeModulePkg/Universal/FaultTolerantWriteDxe/UpdateWorkingBlock.c

index c5c5ab1063e9abe4ad807de79ea0449bbcb27c8c..60fb20938591708ed13c0f543f475ef65f19e41f 100644 (file)
   gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareSize|0x0|UINT32|0x30000014\r
 \r
   ## Base address of the FTW working block range in flash device.\r
+  # If PcdFlashNvStorageFtwWorkingSize is larger than one block size, this value should be block size aligned.\r
   # @Prompt Base address of flash FTW working block range.\r
   gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwWorkingBase|0x0|UINT32|0x30000010\r
 \r
   ## Size of the FTW working block range.\r
+  # If the value is less than one block size, the work space range should not span blocks.\r
+  # If the value is larger than one block size, it should be block size aligned.\r
   # @Prompt Size of flash FTW working block range.\r
   gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwWorkingSize|0x0|UINT32|0x30000011\r
 \r
   gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareBase64|0x0|UINT64|0x80000013\r
 \r
   ## 64-bit Base address of the FTW working block range in flash device.\r
+  # If PcdFlashNvStorageFtwWorkingSize is larger than one block size, this value should be block size aligned.\r
   # @Prompt 64-bit Base address of flash FTW working block range.\r
   gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwWorkingBase64|0x0|UINT64|0x80000010\r
 \r
index 590f00611367e18e302aa8d7a9bb02fa5b5c8efb..30bc563e261ed5723923de9306c3ef3fe7b9dbd8 100644 (file)
Binary files a/MdeModulePkg/MdeModulePkg.uni and b/MdeModulePkg/MdeModulePkg.uni differ
index 714b5d8d7c433fd919dd11af2136c5f54acbeb96..f08f280b46adf43fbd25cf5fac4b8afbb2a401eb 100644 (file)
@@ -3,7 +3,7 @@
   These are the common Fault Tolerant Write (FTW) functions that are shared \r
   by DXE FTW driver and SMM FTW driver.\r
 \r
-Copyright (c) 2006 - 2013, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>\r
 This program and the accompanying materials                          \r
 are licensed and made available under the terms and conditions of the BSD License         \r
 which accompanies this distribution.  The full text of the license may be found at        \r
@@ -81,7 +81,6 @@ FtwAllocate (
   )\r
 {\r
   EFI_STATUS                      Status;\r
-  UINTN                           Length;\r
   UINTN                           Offset;\r
   EFI_FTW_DEVICE                  *FtwDevice;\r
   EFI_FAULT_TOLERANT_WRITE_HEADER *FtwHeader;\r
@@ -134,14 +133,14 @@ FtwAllocate (
   FtwHeader->PrivateDataSize  = PrivateDataSize;\r
   FtwHeader->HeaderAllocated  = FTW_VALID_STATE;\r
 \r
-  Length                      = sizeof (EFI_FAULT_TOLERANT_WRITE_HEADER);\r
-  Status = FtwDevice->FtwFvBlock->Write (\r
-                                    FtwDevice->FtwFvBlock,\r
-                                    FtwDevice->FtwWorkSpaceLba,\r
-                                    FtwDevice->FtwWorkSpaceBase + Offset,\r
-                                    &Length,\r
-                                    (UINT8 *) FtwHeader\r
-                                    );\r
+  Status = WriteWorkSpaceData (\r
+             FtwDevice->FtwFvBlock,\r
+             FtwDevice->WorkBlockSize,\r
+             FtwDevice->FtwWorkSpaceLba,\r
+             FtwDevice->FtwWorkSpaceBase + Offset,\r
+             sizeof (EFI_FAULT_TOLERANT_WRITE_HEADER),\r
+             (UINT8 *) FtwHeader\r
+             );\r
   if (EFI_ERROR (Status)) {\r
     return EFI_ABORTED;\r
   }\r
@@ -150,6 +149,7 @@ FtwAllocate (
   //\r
   Status = FtwUpdateFvState (\r
             FtwDevice->FtwFvBlock,\r
+            FtwDevice->WorkBlockSize,\r
             FtwDevice->FtwWorkSpaceLba,\r
             FtwDevice->FtwWorkSpaceBase + Offset,\r
             WRITES_ALLOCATED\r
@@ -170,13 +170,14 @@ FtwAllocate (
 \r
 \r
 /**\r
-  Write a record with fault tolerant mannaer.\r
+  Write a record with fault tolerant manner.\r
   Since the content has already backuped in spare block, the write is\r
   guaranteed to be completed with fault tolerant manner.\r
 \r
   @param This            The pointer to this protocol instance. \r
   @param Fvb             The FVB protocol that provides services for\r
                          reading, writing, and erasing the target block.\r
+  @param BlockSize       The size of the block.\r
 \r
   @retval  EFI_SUCCESS          The function completed successfully\r
   @retval  EFI_ABORTED          The function could not complete successfully\r
@@ -185,7 +186,8 @@ FtwAllocate (
 EFI_STATUS\r
 FtwWriteRecord (\r
   IN EFI_FAULT_TOLERANT_WRITE_PROTOCOL     *This,\r
-  IN EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL    *Fvb\r
+  IN EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL    *Fvb,\r
+  IN UINTN                                 BlockSize\r
   )\r
 {\r
   EFI_STATUS                      Status;\r
@@ -193,12 +195,10 @@ FtwWriteRecord (
   EFI_FAULT_TOLERANT_WRITE_HEADER *Header;\r
   EFI_FAULT_TOLERANT_WRITE_RECORD *Record;\r
   UINTN                           Offset;\r
-  EFI_LBA                         WorkSpaceLbaOffset;\r
+  UINTN                           NumberOfWriteBlocks;\r
 \r
   FtwDevice = FTW_CONTEXT_FROM_THIS (This);\r
 \r
-  WorkSpaceLbaOffset = FtwDevice->FtwWorkSpaceLba - FtwDevice->FtwWorkBlockLba;\r
-\r
   //\r
   // Spare Complete but Destination not complete,\r
   // Recover the target block with the spare block.\r
@@ -218,8 +218,9 @@ FtwWriteRecord (
     Offset = (UINT8 *) Record - FtwDevice->FtwWorkSpace;\r
     Status = FtwUpdateFvState (\r
               FtwDevice->FtwBackupFvb,\r
-              FtwDevice->FtwSpareLba + WorkSpaceLbaOffset,\r
-              FtwDevice->FtwWorkSpaceBase + Offset,\r
+              FtwDevice->SpareBlockSize,\r
+              FtwDevice->FtwSpareLba + FtwDevice->FtwWorkSpaceLbaInSpare,\r
+              FtwDevice->FtwWorkSpaceBaseInSpare + Offset,\r
               SPARE_COMPLETED\r
               );\r
     if (EFI_ERROR (Status)) {\r
@@ -227,7 +228,7 @@ FtwWriteRecord (
     }\r
 \r
     Status = FlushSpareBlockToWorkingBlock (FtwDevice);\r
-  } else if (IsBootBlock (FtwDevice, Fvb, Record->Lba)) {\r
+  } else if (IsBootBlock (FtwDevice, Fvb)) {\r
     //\r
     // Update boot block\r
     //\r
@@ -236,7 +237,8 @@ FtwWriteRecord (
     //\r
     // Update blocks other than working block or boot block\r
     //\r
-    Status = FlushSpareBlockToTargetBlock (FtwDevice, Fvb, Record->Lba);\r
+    NumberOfWriteBlocks = FTW_BLOCKS ((UINTN) (Record->Offset + Record->Length), BlockSize);\r
+    Status = FlushSpareBlockToTargetBlock (FtwDevice, Fvb, Record->Lba, BlockSize, NumberOfWriteBlocks);\r
   }\r
 \r
   if (EFI_ERROR (Status)) {\r
@@ -248,6 +250,7 @@ FtwWriteRecord (
   Offset = (UINT8 *) Record - FtwDevice->FtwWorkSpace;\r
   Status = FtwUpdateFvState (\r
             FtwDevice->FtwFvBlock,\r
+            FtwDevice->WorkBlockSize,\r
             FtwDevice->FtwWorkSpaceLba,\r
             FtwDevice->FtwWorkSpaceBase + Offset,\r
             DEST_COMPLETED\r
@@ -266,6 +269,7 @@ FtwWriteRecord (
     Offset = (UINT8 *) Header - FtwDevice->FtwWorkSpace;\r
     Status = FtwUpdateFvState (\r
               FtwDevice->FtwFvBlock,\r
+              FtwDevice->WorkBlockSize,\r
               FtwDevice->FtwWorkSpaceLba,\r
               FtwDevice->FtwWorkSpaceBase + Offset,\r
               WRITES_COMPLETED\r
@@ -330,6 +334,10 @@ FtwWrite (
   UINTN                               Index;\r
   UINT8                               *Ptr;\r
   EFI_PHYSICAL_ADDRESS                FvbPhysicalAddress;\r
+  UINTN                               BlockSize;\r
+  UINTN                               NumberOfBlocks;\r
+  UINTN                               NumberOfWriteBlocks;\r
+  UINTN                               WriteLength;\r
 \r
   FtwDevice = FTW_CONTEXT_FROM_THIS (This);\r
 \r
@@ -383,12 +391,7 @@ FtwWrite (
   if ((Record->SpareComplete == FTW_VALID_STATE) && (Record->DestinationComplete != FTW_VALID_STATE)) {\r
     return EFI_NOT_READY;\r
   }\r
-  //\r
-  // Check if the input data can fit within the target block\r
-  //\r
-  if ((Offset + Length) > FtwDevice->SpareAreaLength) {\r
-    return EFI_BAD_BUFFER_SIZE;\r
-  }\r
+\r
   //\r
   // Get the FVB protocol by handle\r
   //\r
@@ -399,15 +402,39 @@ FtwWrite (
 \r
   Status = Fvb->GetPhysicalAddress (Fvb, &FvbPhysicalAddress);\r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((EFI_D_ERROR, "FtwLite: Get FVB physical address - %r\n", Status));\r
+    DEBUG ((EFI_D_ERROR, "Ftw: Write(), Get FVB physical address - %r\n", Status));\r
+    return EFI_ABORTED;\r
+  }\r
+\r
+  //\r
+  // Now, one FVB has one type of BlockSize.\r
+  //\r
+  Status = Fvb->GetBlockSize (Fvb, 0, &BlockSize, &NumberOfBlocks);\r
+  if (EFI_ERROR (Status)) {\r
+    DEBUG ((EFI_D_ERROR, "Ftw: Write(), Get block size - %r\n", Status));\r
     return EFI_ABORTED;\r
   }\r
 \r
+  NumberOfWriteBlocks = FTW_BLOCKS (Offset + Length, BlockSize);\r
+  DEBUG ((EFI_D_INFO, "Ftw: Write(), BlockSize - 0x%x, NumberOfWriteBlock - 0x%x\n", BlockSize, NumberOfWriteBlocks));\r
+  WriteLength = NumberOfWriteBlocks * BlockSize;\r
+\r
+  //\r
+  // Check if the input data can fit within the spare block.\r
+  //\r
+  if (WriteLength > FtwDevice->SpareAreaLength) {\r
+    return EFI_BAD_BUFFER_SIZE;\r
+  }\r
+\r
   //\r
   // Set BootBlockUpdate FLAG if it's updating boot block.\r
   //\r
-  if (IsBootBlock (FtwDevice, Fvb, Lba)) {\r
+  if (IsBootBlock (FtwDevice, Fvb)) {\r
     Record->BootBlockUpdate = FTW_VALID_STATE;\r
+    //\r
+    // Boot Block and Spare Block should have same block size and block numbers.\r
+    //\r
+    ASSERT ((BlockSize == FtwDevice->SpareBlockSize) && (NumberOfWriteBlocks == FtwDevice->NumberOfSpareBlock));\r
   }\r
   //\r
   // Write the record to the work space.\r
@@ -415,7 +442,7 @@ FtwWrite (
   Record->Lba     = Lba;\r
   Record->Offset  = Offset;\r
   Record->Length  = Length;\r
-  Record->RelativeOffset = (INT64) (FvbPhysicalAddress + (UINTN) Lba * FtwDevice->BlockSize) - (INT64) FtwDevice->SpareAreaAddress;\r
+  Record->RelativeOffset = (INT64) (FvbPhysicalAddress + (UINTN) Lba * BlockSize) - (INT64) FtwDevice->SpareAreaAddress;\r
   if (PrivateData != NULL) {\r
     CopyMem ((Record + 1), PrivateData, (UINTN) Header->PrivateDataSize);\r
   }\r
@@ -423,13 +450,14 @@ FtwWrite (
   MyOffset  = (UINT8 *) Record - FtwDevice->FtwWorkSpace;\r
   MyLength  = FTW_RECORD_SIZE (Header->PrivateDataSize);\r
 \r
-  Status = FtwDevice->FtwFvBlock->Write (\r
-                                    FtwDevice->FtwFvBlock,\r
-                                    FtwDevice->FtwWorkSpaceLba,\r
-                                    FtwDevice->FtwWorkSpaceBase + MyOffset,\r
-                                    &MyLength,\r
-                                    (UINT8 *) Record\r
-                                    );\r
+  Status = WriteWorkSpaceData (\r
+             FtwDevice->FtwFvBlock,\r
+             FtwDevice->WorkBlockSize,\r
+             FtwDevice->FtwWorkSpaceLba,\r
+             FtwDevice->FtwWorkSpaceBase + MyOffset,\r
+             MyLength,\r
+             (UINT8 *) Record\r
+             );\r
   if (EFI_ERROR (Status)) {\r
     return EFI_ABORTED;\r
   }\r
@@ -439,7 +467,7 @@ FtwWrite (
   //\r
   // Allocate a memory buffer\r
   //\r
-  MyBufferSize  = FtwDevice->SpareAreaLength;\r
+  MyBufferSize  = WriteLength;\r
   MyBuffer      = AllocatePool (MyBufferSize);\r
   if (MyBuffer == NULL) {\r
     return EFI_OUT_OF_RESOURCES;\r
@@ -448,8 +476,8 @@ FtwWrite (
   // Read all original data from target block to memory buffer\r
   //\r
   Ptr = MyBuffer;\r
-  for (Index = 0; Index < FtwDevice->NumberOfSpareBlock; Index += 1) {\r
-    MyLength  = FtwDevice->BlockSize;\r
+  for (Index = 0; Index < NumberOfWriteBlocks; Index += 1) {\r
+    MyLength  = BlockSize;\r
     Status    = Fvb->Read (Fvb, Lba + Index, 0, &MyLength, Ptr);\r
     if (EFI_ERROR (Status)) {\r
       FreePool (MyBuffer);\r
@@ -477,7 +505,7 @@ FtwWrite (
 \r
   Ptr = SpareBuffer;\r
   for (Index = 0; Index < FtwDevice->NumberOfSpareBlock; Index += 1) {\r
-    MyLength = FtwDevice->BlockSize;\r
+    MyLength = FtwDevice->SpareBlockSize;\r
     Status = FtwDevice->FtwBackupFvb->Read (\r
                                         FtwDevice->FtwBackupFvb,\r
                                         FtwDevice->FtwSpareLba + Index,\r
@@ -495,11 +523,16 @@ FtwWrite (
   }\r
   //\r
   // Write the memory buffer to spare block\r
+  // Do not assume Spare Block and Target Block have same block size\r
   //\r
   Status  = FtwEraseSpareBlock (FtwDevice);\r
   Ptr     = MyBuffer;\r
-  for (Index = 0; Index < FtwDevice->NumberOfSpareBlock; Index += 1) {\r
-    MyLength = FtwDevice->BlockSize;\r
+  for (Index = 0; MyBufferSize > 0; Index += 1) {\r
+    if (MyBufferSize > FtwDevice->SpareBlockSize) {\r
+      MyLength = FtwDevice->SpareBlockSize;\r
+    } else {\r
+      MyLength = MyBufferSize;\r
+    }\r
     Status = FtwDevice->FtwBackupFvb->Write (\r
                                         FtwDevice->FtwBackupFvb,\r
                                         FtwDevice->FtwSpareLba + Index,\r
@@ -514,6 +547,7 @@ FtwWrite (
     }\r
 \r
     Ptr += MyLength;\r
+    MyBufferSize -= MyLength;\r
   }\r
   //\r
   // Free MyBuffer\r
@@ -526,6 +560,7 @@ FtwWrite (
   MyOffset = (UINT8 *) Record - FtwDevice->FtwWorkSpace;\r
   Status = FtwUpdateFvState (\r
             FtwDevice->FtwFvBlock,\r
+            FtwDevice->WorkBlockSize,\r
             FtwDevice->FtwWorkSpaceLba,\r
             FtwDevice->FtwWorkSpaceBase + MyOffset,\r
             SPARE_COMPLETED\r
@@ -541,7 +576,7 @@ FtwWrite (
   //  Since the content has already backuped in spare block, the write is\r
   //  guaranteed to be completed with fault tolerant manner.\r
   //\r
-  Status = FtwWriteRecord (This, Fvb);\r
+  Status = FtwWriteRecord (This, Fvb, BlockSize);\r
   if (EFI_ERROR (Status)) {\r
     FreePool (SpareBuffer);\r
     return EFI_ABORTED;\r
@@ -552,7 +587,7 @@ FtwWrite (
   Status  = FtwEraseSpareBlock (FtwDevice);\r
   Ptr     = SpareBuffer;\r
   for (Index = 0; Index < FtwDevice->NumberOfSpareBlock; Index += 1) {\r
-    MyLength = FtwDevice->BlockSize;\r
+    MyLength = FtwDevice->SpareBlockSize;\r
     Status = FtwDevice->FtwBackupFvb->Write (\r
                                         FtwDevice->FtwBackupFvb,\r
                                         FtwDevice->FtwSpareLba + Index,\r
@@ -609,6 +644,8 @@ FtwRestart (
   EFI_FAULT_TOLERANT_WRITE_HEADER     *Header;\r
   EFI_FAULT_TOLERANT_WRITE_RECORD     *Record;\r
   EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL  *Fvb;\r
+  UINTN                               BlockSize;\r
+  UINTN                               NumberOfBlocks;\r
 \r
   FtwDevice = FTW_CONTEXT_FROM_THIS (This);\r
 \r
@@ -629,6 +666,15 @@ FtwRestart (
     return EFI_NOT_FOUND;\r
   }\r
 \r
+  //\r
+  // Now, one FVB has one type of BlockSize\r
+  //\r
+  Status = Fvb->GetBlockSize (Fvb, 0, &BlockSize, &NumberOfBlocks);\r
+  if (EFI_ERROR (Status)) {\r
+    DEBUG ((EFI_D_ERROR, "Ftw: Restart(), Get block size - %r\n", Status));\r
+    return EFI_ABORTED;\r
+  }\r
+\r
   //\r
   // Check the COMPLETE flag of last write header\r
   //\r
@@ -651,7 +697,7 @@ FtwRestart (
   //  Since the content has already backuped in spare block, the write is\r
   //  guaranteed to be completed with fault tolerant manner.\r
   //\r
-  Status = FtwWriteRecord (This, Fvb);\r
+  Status = FtwWriteRecord (This, Fvb, BlockSize);\r
   if (EFI_ERROR (Status)) {\r
     return EFI_ABORTED;\r
   }\r
@@ -706,6 +752,7 @@ FtwAbort (
   Offset = (UINT8 *) FtwDevice->FtwLastWriteHeader - FtwDevice->FtwWorkSpace;\r
   Status = FtwUpdateFvState (\r
             FtwDevice->FtwFvBlock,\r
+            FtwDevice->WorkBlockSize,\r
             FtwDevice->FtwWorkSpaceLba,\r
             FtwDevice->FtwWorkSpaceBase + Offset,\r
             WRITES_COMPLETED\r
index a460694d655972a3103e553f8f09c9614e5b240e..849d1f4233cdf41537596112b36d0812cb9689b3 100644 (file)
@@ -1,9 +1,9 @@
 /** @file\r
 \r
   The internal header file includes the common header files, defines\r
-  internal structure and functions used by FtwLite module.\r
+  internal structure and functions used by Ftw module.\r
 \r
-Copyright (c) 2006 - 2013, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>\r
 This program and the accompanying materials                          \r
 are licensed and made available under the terms and conditions of the BSD License         \r
 which accompanies this distribution.  The full text of the license may be found at        \r
@@ -50,6 +50,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #define SPARE_COMPLETED   0x2\r
 #define DEST_COMPLETED    0x4\r
 \r
+#define FTW_BLOCKS(Length, BlockSize) ((UINTN) ((Length) / (BlockSize) + (((Length) & ((BlockSize) - 1)) ? 1 : 0)))\r
 \r
 #define FTW_DEVICE_SIGNATURE  SIGNATURE_32 ('F', 'T', 'W', 'D')\r
 \r
@@ -63,9 +64,11 @@ typedef struct {
   EFI_PHYSICAL_ADDRESS                    WorkSpaceAddress;   // Base address of working space range in flash.\r
   EFI_PHYSICAL_ADDRESS                    SpareAreaAddress;   // Base address of spare range in flash.\r
   UINTN                                   WorkSpaceLength;    // Size of working space range in flash.\r
+  UINTN                                   NumberOfWorkSpaceBlock; // Number of the blocks in work block for work space.\r
+  UINTN                                   WorkBlockSize;      // Block size in bytes of the work blocks in flash\r
   UINTN                                   SpareAreaLength;    // Size of spare range in flash.\r
   UINTN                                   NumberOfSpareBlock; // Number of the blocks in spare block.\r
-  UINTN                                   BlockSize;          // Block size in bytes of the blocks in flash\r
+  UINTN                                   SpareBlockSize;     // Block size in bytes of the spare blocks in flash\r
   EFI_FAULT_TOLERANT_WORKING_BLOCK_HEADER *FtwWorkSpaceHeader;// Pointer to Working Space Header in memory buffer\r
   EFI_FAULT_TOLERANT_WRITE_HEADER         *FtwLastWriteHeader;// Pointer to last record header in memory buffer\r
   EFI_FAULT_TOLERANT_WRITE_RECORD         *FtwLastWriteRecord;// Pointer to last record in memory buffer\r
@@ -73,9 +76,12 @@ typedef struct {
   EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL      *FtwBackupFvb;      // FVB of spare block\r
   EFI_LBA                                 FtwSpareLba;        // Start LBA of spare block\r
   EFI_LBA                                 FtwWorkBlockLba;    // Start LBA of working block that contains working space in its last block.\r
+  UINTN                                   NumberOfWorkBlock;  // Number of the blocks in work block.\r
   EFI_LBA                                 FtwWorkSpaceLba;    // Start LBA of working space\r
   UINTN                                   FtwWorkSpaceBase;   // Offset into the FtwWorkSpaceLba block.\r
   UINTN                                   FtwWorkSpaceSize;   // Size of working space range that stores write record.\r
+  EFI_LBA                                 FtwWorkSpaceLbaInSpare; // Start LBA of working space in spare block.\r
+  UINTN                                   FtwWorkSpaceBaseInSpare;// Offset into the FtwWorkSpaceLbaInSpare block.\r
   UINT8                                   *FtwWorkSpace;      // Point to Work Space in memory buffer \r
   //\r
   // Following a buffer of FtwWorkSpace[FTW_WORK_SPACE_SIZE],\r
@@ -335,7 +341,6 @@ IsWorkingBlock (
 \r
   @param FtwDevice       The private data of FTW driver\r
   @param FvBlock         Fvb protocol instance\r
-  @param Lba             The block specified\r
 \r
   @return A BOOLEAN value indicating in boot block or not.\r
 \r
@@ -343,19 +348,20 @@ IsWorkingBlock (
 BOOLEAN\r
 IsBootBlock (\r
   EFI_FTW_DEVICE                      *FtwDevice,\r
-  EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL  *FvBlock,\r
-  EFI_LBA                             Lba\r
+  EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL  *FvBlock\r
   );\r
 \r
 /**\r
   Copy the content of spare block to a target block. Size is FTW_BLOCK_SIZE.\r
-  Spare block is accessed by FTW backup FVB protocol interface. LBA is 1.\r
-  Target block is accessed by FvbBlock protocol interface. LBA is Lba.\r
+  Spare block is accessed by FTW backup FVB protocol interface.\r
+  Target block is accessed by FvBlock protocol interface.\r
 \r
 \r
   @param FtwDevice       The private data of FTW driver\r
   @param FvBlock         FVB Protocol interface to access target block\r
   @param Lba             Lba of the target block\r
+  @param BlockSize       The size of the block\r
+  @param NumberOfBlocks  The number of consecutive blocks starting with Lba\r
 \r
   @retval  EFI_SUCCESS               Spare block content is copied to target block\r
   @retval  EFI_INVALID_PARAMETER     Input parameter error\r
@@ -367,7 +373,9 @@ EFI_STATUS
 FlushSpareBlockToTargetBlock (\r
   EFI_FTW_DEVICE                      *FtwDevice,\r
   EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL  *FvBlock,\r
-  EFI_LBA                             Lba\r
+  EFI_LBA                             Lba,\r
+  UINTN                               BlockSize,\r
+  UINTN                               NumberOfBlocks\r
   );\r
 \r
 /**\r
@@ -395,8 +403,8 @@ FlushSpareBlockToWorkingBlock (
 \r
 /**\r
   Copy the content of spare block to a boot block. Size is FTW_BLOCK_SIZE.\r
-  Spare block is accessed by FTW working FVB protocol interface. LBA is 1.\r
-  Target block is accessed by FvbBlock protocol interface. LBA is Lba.\r
+  Spare block is accessed by FTW working FVB protocol interface.\r
+  Target block is accessed by FvBlock protocol interface.\r
 \r
   FTW will do extra work on boot block update.\r
   FTW should depend on a protocol of EFI_ADDRESS_RANGE_SWAP_PROTOCOL,\r
@@ -405,7 +413,7 @@ FlushSpareBlockToWorkingBlock (
   1. GetRangeLocation(), if the Range is inside the boot block, FTW know\r
   that boot block will be update. It shall add a FLAG in the working block.\r
   2. When spare block is ready,\r
-  3. SetSwapState(EFI_SWAPPED)\r
+  3. SetSwapState(SWAPPED)\r
   4. erasing boot block,\r
   5. programming boot block until the boot block is ok.\r
   6. SetSwapState(UNSWAPPED)\r
@@ -431,6 +439,7 @@ FlushSpareBlockToBootBlock (
 \r
 \r
   @param FvBlock         FVB Protocol interface to access SrcBlock and DestBlock\r
+  @param BlockSize       The size of the block\r
   @param Lba             Lba of a block\r
   @param Offset          Offset on the Lba\r
   @param NewBit          New value that will override the old value if it can be change\r
@@ -445,6 +454,7 @@ FlushSpareBlockToBootBlock (
 EFI_STATUS\r
 FtwUpdateFvState (\r
   IN EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL  *FvBlock,\r
+  IN UINTN                               BlockSize,\r
   IN EFI_LBA                             Lba,\r
   IN UINTN                               Offset,\r
   IN UINT8                               NewBit\r
@@ -614,7 +624,7 @@ FtwReclaimWorkSpace (
 \r
 /**\r
 \r
-  Get firmware block by address.\r
+  Get firmware volume block by address.\r
 \r
 \r
   @param Address         Address specified the block\r
@@ -707,5 +717,53 @@ VOID
 InitializeLocalWorkSpaceHeader (\r
   VOID\r
   );\r
\r
+\r
+/**\r
+  Read work space data from work block or spare block.\r
+\r
+  @param FvBlock        FVB Protocol interface to access the block.\r
+  @param BlockSize      The size of the block.\r
+  @param Lba            Lba of the block.\r
+  @param Offset         The offset within the block.\r
+  @param Length         The number of bytes to read from the block.\r
+  @param Buffer         The data is read.\r
+\r
+  @retval EFI_SUCCESS   The function completed successfully.\r
+  @retval EFI_ABORTED   The function could not complete successfully.\r
+\r
+**/\r
+EFI_STATUS\r
+ReadWorkSpaceData (\r
+  IN EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL *FvBlock,\r
+  IN UINTN                              BlockSize,\r
+  IN EFI_LBA                            Lba,\r
+  IN UINTN                              Offset,\r
+  IN UINTN                              Length,\r
+  OUT UINT8                             *Buffer\r
+  );\r
+\r
+/**\r
+  Write data to work block.\r
+\r
+  @param FvBlock        FVB Protocol interface to access the block.\r
+  @param BlockSize      The size of the block.\r
+  @param Lba            Lba of the block.\r
+  @param Offset         The offset within the block to place the data.\r
+  @param Length         The number of bytes to write to the block.\r
+  @param Buffer         The data to write.\r
+\r
+  @retval EFI_SUCCESS   The function completed successfully.\r
+  @retval EFI_ABORTED   The function could not complete successfully.\r
+\r
+**/\r
+EFI_STATUS\r
+WriteWorkSpaceData (\r
+  IN EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL *FvBlock,\r
+  IN UINTN                              BlockSize,\r
+  IN EFI_LBA                            Lba,\r
+  IN UINTN                              Offset,\r
+  IN UINTN                              Length,\r
+  IN UINT8                              *Buffer\r
+  );\r
+\r
 #endif\r
index b24d66dde374397261a25eda13abf3948b3d2dd9..09223217e4d0f8085249d97d5bf69487b94befd8 100644 (file)
@@ -48,12 +48,13 @@ IsErasedFlashBuffer (
 }\r
 \r
 /**\r
-  To erase the block with the spare block size.\r
+  To erase the block with specified blocks.\r
 \r
 \r
   @param FtwDevice       The private data of FTW driver\r
   @param FvBlock         FVB Protocol interface\r
   @param Lba             Lba of the firmware block\r
+  @param NumberOfBlocks  The number of consecutive blocks starting with Lba\r
 \r
   @retval  EFI_SUCCESS    Block LBA is Erased successfully\r
   @retval  Others         Error occurs\r
@@ -63,13 +64,14 @@ EFI_STATUS
 FtwEraseBlock (\r
   IN EFI_FTW_DEVICE                   *FtwDevice,\r
   EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL  *FvBlock,\r
-  EFI_LBA                             Lba\r
+  EFI_LBA                             Lba,\r
+  UINTN                               NumberOfBlocks\r
   )\r
 {\r
   return FvBlock->EraseBlocks (\r
                     FvBlock,\r
                     Lba,\r
-                    FtwDevice->NumberOfSpareBlock,\r
+                    NumberOfBlocks,\r
                     EFI_LBA_LIST_TERMINATOR\r
                     );\r
 }\r
@@ -137,7 +139,7 @@ IsWorkingBlock (
 \r
 /**\r
 \r
-  Get firmware block by address.\r
+  Get firmware volume block by address.\r
 \r
 \r
   @param Address         Address specified the block\r
@@ -159,8 +161,9 @@ GetFvbByAddress (
   UINTN                               Index;\r
   EFI_PHYSICAL_ADDRESS                FvbBaseAddress;\r
   EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL  *Fvb;\r
-  EFI_FIRMWARE_VOLUME_HEADER          *FwVolHeader;\r
   EFI_HANDLE                          FvbHandle;\r
+  UINTN                               BlockSize;\r
+  UINTN                               NumberOfBlocks;\r
 \r
   *FvBlock  = NULL;\r
   FvbHandle = NULL;\r
@@ -188,8 +191,15 @@ GetFvbByAddress (
       continue;\r
     }\r
 \r
-    FwVolHeader = (EFI_FIRMWARE_VOLUME_HEADER *) ((UINTN) FvbBaseAddress);\r
-    if ((Address >= FvbBaseAddress) && (Address <= (FvbBaseAddress + (FwVolHeader->FvLength - 1)))) {\r
+    //\r
+    // Now, one FVB has one type of BlockSize\r
+    //\r
+    Status = Fvb->GetBlockSize (Fvb, 0, &BlockSize, &NumberOfBlocks);\r
+    if (EFI_ERROR (Status)) {\r
+      continue;\r
+    }\r
+\r
+    if ((Address >= FvbBaseAddress) && (Address < (FvbBaseAddress + BlockSize * NumberOfBlocks))) {\r
       *FvBlock  = Fvb;\r
       FvbHandle  = HandleBuffer[Index];\r
       break;\r
@@ -206,7 +216,6 @@ GetFvbByAddress (
 \r
   @param FtwDevice       The private data of FTW driver\r
   @param FvBlock         Fvb protocol instance\r
-  @param Lba             The block specified\r
 \r
   @return A BOOLEAN value indicating in boot block or not.\r
 \r
@@ -214,8 +223,7 @@ GetFvbByAddress (
 BOOLEAN\r
 IsBootBlock (\r
   EFI_FTW_DEVICE                      *FtwDevice,\r
-  EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL  *FvBlock,\r
-  EFI_LBA                             Lba\r
+  EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL  *FvBlock\r
   )\r
 {\r
   EFI_STATUS                          Status;\r
@@ -274,8 +282,8 @@ IsBootBlock (
 \r
 /**\r
   Copy the content of spare block to a boot block. Size is FTW_BLOCK_SIZE.\r
-  Spare block is accessed by FTW working FVB protocol interface. LBA is 1.\r
-  Target block is accessed by FvbBlock protocol interface. LBA is Lba.\r
+  Spare block is accessed by FTW working FVB protocol interface.\r
+  Target block is accessed by FvBlock protocol interface.\r
 \r
   FTW will do extra work on boot block update.\r
   FTW should depend on a protocol of EFI_ADDRESS_RANGE_SWAP_PROTOCOL,\r
@@ -284,7 +292,7 @@ IsBootBlock (
   1. GetRangeLocation(), if the Range is inside the boot block, FTW know\r
   that boot block will be update. It shall add a FLAG in the working block.\r
   2. When spare block is ready,\r
-  3. SetSwapState(EFI_SWAPPED)\r
+  3. SetSwapState(SWAPPED)\r
   4. erasing boot block,\r
   5. programming boot block until the boot block is ok.\r
   6. SetSwapState(UNSWAPPED)\r
@@ -357,7 +365,7 @@ FlushSpareBlockToBootBlock (
     BootLba = 0;\r
     Ptr     = Buffer;\r
     for (Index = 0; Index < FtwDevice->NumberOfSpareBlock; Index += 1) {\r
-      Count = FtwDevice->BlockSize;\r
+      Count = FtwDevice->SpareBlockSize;\r
       Status = BootFvb->Read (\r
                           BootFvb,\r
                           BootLba + Index,\r
@@ -378,7 +386,7 @@ FlushSpareBlockToBootBlock (
     //\r
     Ptr = Buffer;\r
     for (Index = 0; Index < FtwDevice->NumberOfSpareBlock; Index += 1) {\r
-      Count = FtwDevice->BlockSize;\r
+      Count = FtwDevice->SpareBlockSize;\r
       Status = FtwDevice->FtwBackupFvb->Read (\r
                                           FtwDevice->FtwBackupFvb,\r
                                           FtwDevice->FtwSpareLba + Index,\r
@@ -416,7 +424,7 @@ FlushSpareBlockToBootBlock (
   //\r
   Ptr = Buffer;\r
   for (Index = 0; Index < FtwDevice->NumberOfSpareBlock; Index += 1) {\r
-    Count = FtwDevice->BlockSize;\r
+    Count = FtwDevice->SpareBlockSize;\r
     Status = FtwDevice->FtwBackupFvb->Write (\r
                                         FtwDevice->FtwBackupFvb,\r
                                         FtwDevice->FtwSpareLba + Index,\r
@@ -444,14 +452,16 @@ FlushSpareBlockToBootBlock (
 }\r
 \r
 /**\r
-  Copy the content of spare block to a target block. Size is FTW_BLOCK_SIZE.\r
-  Spare block is accessed by FTW backup FVB protocol interface. LBA is 1.\r
-  Target block is accessed by FvbBlock protocol interface. LBA is Lba.\r
+  Copy the content of spare block to a target block.\r
+  Spare block is accessed by FTW backup FVB protocol interface.\r
+  Target block is accessed by FvBlock protocol interface.\r
 \r
 \r
   @param FtwDevice       The private data of FTW driver\r
   @param FvBlock         FVB Protocol interface to access target block\r
   @param Lba             Lba of the target block\r
+  @param BlockSize       The size of the block\r
+  @param NumberOfBlocks  The number of consecutive blocks starting with Lba\r
 \r
   @retval  EFI_SUCCESS               Spare block content is copied to target block\r
   @retval  EFI_INVALID_PARAMETER     Input parameter error\r
@@ -463,7 +473,9 @@ EFI_STATUS
 FlushSpareBlockToTargetBlock (\r
   EFI_FTW_DEVICE                      *FtwDevice,\r
   EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL  *FvBlock,\r
-  EFI_LBA                             Lba\r
+  EFI_LBA                             Lba,\r
+  UINTN                               BlockSize,\r
+  UINTN                               NumberOfBlocks\r
   )\r
 {\r
   EFI_STATUS  Status;\r
@@ -489,7 +501,7 @@ FlushSpareBlockToTargetBlock (
   //\r
   Ptr = Buffer;\r
   for (Index = 0; Index < FtwDevice->NumberOfSpareBlock; Index += 1) {\r
-    Count = FtwDevice->BlockSize;\r
+    Count = FtwDevice->SpareBlockSize;\r
     Status = FtwDevice->FtwBackupFvb->Read (\r
                                         FtwDevice->FtwBackupFvb,\r
                                         FtwDevice->FtwSpareLba + Index,\r
@@ -507,17 +519,17 @@ FlushSpareBlockToTargetBlock (
   //\r
   // Erase the target block\r
   //\r
-  Status = FtwEraseBlock (FtwDevice, FvBlock, Lba);\r
+  Status = FtwEraseBlock (FtwDevice, FvBlock, Lba, NumberOfBlocks);\r
   if (EFI_ERROR (Status)) {\r
     FreePool (Buffer);\r
     return EFI_ABORTED;\r
   }\r
   //\r
-  // Write memory buffer to block, using the FvbBlock protocol interface\r
+  // Write memory buffer to block, using the FvBlock protocol interface\r
   //\r
   Ptr = Buffer;\r
-  for (Index = 0; Index < FtwDevice->NumberOfSpareBlock; Index += 1) {\r
-    Count   = FtwDevice->BlockSize;\r
+  for (Index = 0; Index < NumberOfBlocks; Index += 1) {\r
+    Count   = BlockSize;\r
     Status  = FvBlock->Write (FvBlock, Lba + Index, 0, &Count, Ptr);\r
     if (EFI_ERROR (Status)) {\r
       DEBUG ((EFI_D_ERROR, "Ftw: FVB Write block - %r\n", Status));\r
@@ -563,7 +575,6 @@ FlushSpareBlockToWorkingBlock (
   UINTN                                   Count;\r
   UINT8                                   *Ptr;\r
   UINTN                                   Index;\r
-  EFI_LBA                                 WorkSpaceLbaOffset;\r
 \r
   //\r
   // Allocate a memory buffer\r
@@ -574,8 +585,6 @@ FlushSpareBlockToWorkingBlock (
     return EFI_OUT_OF_RESOURCES;\r
   }\r
 \r
-  WorkSpaceLbaOffset = FtwDevice->FtwWorkSpaceLba - FtwDevice->FtwWorkBlockLba;\r
-\r
   //\r
   // To guarantee that the WorkingBlockValid is set on spare block\r
   //\r
@@ -585,8 +594,9 @@ FlushSpareBlockToWorkingBlock (
   //\r
   FtwUpdateFvState (\r
     FtwDevice->FtwBackupFvb,\r
-    FtwDevice->FtwSpareLba + WorkSpaceLbaOffset,\r
-    FtwDevice->FtwWorkSpaceBase + sizeof (EFI_GUID) + sizeof (UINT32),\r
+    FtwDevice->SpareBlockSize,\r
+    FtwDevice->FtwSpareLba + FtwDevice->FtwWorkSpaceLbaInSpare,\r
+    FtwDevice->FtwWorkSpaceBaseInSpare + sizeof (EFI_GUID) + sizeof (UINT32),\r
     WORKING_BLOCK_VALID\r
     );\r
   //\r
@@ -594,7 +604,7 @@ FlushSpareBlockToWorkingBlock (
   //\r
   Ptr = Buffer;\r
   for (Index = 0; Index < FtwDevice->NumberOfSpareBlock; Index += 1) {\r
-    Count = FtwDevice->BlockSize;\r
+    Count = FtwDevice->SpareBlockSize;\r
     Status = FtwDevice->FtwBackupFvb->Read (\r
                                         FtwDevice->FtwBackupFvb,\r
                                         FtwDevice->FtwSpareLba + Index,\r
@@ -612,7 +622,7 @@ FlushSpareBlockToWorkingBlock (
   //\r
   // Clear the CRC and STATE, copy data from spare to working block.\r
   //\r
-  WorkingBlockHeader = (EFI_FAULT_TOLERANT_WORKING_BLOCK_HEADER *) (Buffer + (UINTN) WorkSpaceLbaOffset * FtwDevice->BlockSize + FtwDevice->FtwWorkSpaceBase);\r
+  WorkingBlockHeader = (EFI_FAULT_TOLERANT_WORKING_BLOCK_HEADER *) (Buffer + (UINTN) FtwDevice->FtwWorkSpaceLbaInSpare * FtwDevice->SpareBlockSize + FtwDevice->FtwWorkSpaceBaseInSpare);\r
   InitWorkSpaceHeader (WorkingBlockHeader);\r
   WorkingBlockHeader->WorkingBlockValid   = FTW_ERASE_POLARITY;\r
   WorkingBlockHeader->WorkingBlockInvalid = FTW_ERASE_POLARITY;\r
@@ -629,6 +639,7 @@ FlushSpareBlockToWorkingBlock (
   //\r
   Status = FtwUpdateFvState (\r
             FtwDevice->FtwFvBlock,\r
+            FtwDevice->WorkBlockSize,\r
             FtwDevice->FtwWorkSpaceLba,\r
             FtwDevice->FtwWorkSpaceBase + sizeof (EFI_GUID) + sizeof (UINT32),\r
             WORKING_BLOCK_INVALID\r
@@ -643,17 +654,17 @@ FlushSpareBlockToWorkingBlock (
   //\r
   // Erase the working block\r
   //\r
-  Status = FtwEraseBlock (FtwDevice, FtwDevice->FtwFvBlock, FtwDevice->FtwWorkBlockLba);\r
+  Status = FtwEraseBlock (FtwDevice, FtwDevice->FtwFvBlock, FtwDevice->FtwWorkBlockLba, FtwDevice->NumberOfWorkBlock);\r
   if (EFI_ERROR (Status)) {\r
     FreePool (Buffer);\r
     return EFI_ABORTED;\r
   }\r
   //\r
-  // Write memory buffer to working block, using the FvbBlock protocol interface\r
+  // Write memory buffer to working block, using the FvBlock protocol interface\r
   //\r
   Ptr = Buffer;\r
-  for (Index = 0; Index < FtwDevice->NumberOfSpareBlock; Index += 1) {\r
-    Count = FtwDevice->BlockSize;\r
+  for (Index = 0; Index < FtwDevice->NumberOfWorkBlock; Index += 1) {\r
+    Count = FtwDevice->WorkBlockSize;\r
     Status = FtwDevice->FtwFvBlock->Write (\r
                                       FtwDevice->FtwFvBlock,\r
                                       FtwDevice->FtwWorkBlockLba + Index,\r
@@ -682,6 +693,7 @@ FlushSpareBlockToWorkingBlock (
   //\r
   Status = FtwUpdateFvState (\r
             FtwDevice->FtwFvBlock,\r
+            FtwDevice->WorkBlockSize,\r
             FtwDevice->FtwWorkSpaceLba,\r
             FtwDevice->FtwWorkSpaceBase + sizeof (EFI_GUID) + sizeof (UINT32),\r
             WORKING_BLOCK_VALID\r
@@ -703,6 +715,7 @@ FlushSpareBlockToWorkingBlock (
 \r
 \r
   @param FvBlock         FVB Protocol interface to access SrcBlock and DestBlock\r
+  @param BlockSize       The size of the block\r
   @param Lba             Lba of a block\r
   @param Offset          Offset on the Lba\r
   @param NewBit          New value that will override the old value if it can be change\r
@@ -717,6 +730,7 @@ FlushSpareBlockToWorkingBlock (
 EFI_STATUS\r
 FtwUpdateFvState (\r
   IN EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL  *FvBlock,\r
+  IN UINTN                               BlockSize,\r
   IN EFI_LBA                             Lba,\r
   IN UINTN                               Offset,\r
   IN UINT8                               NewBit\r
@@ -726,6 +740,14 @@ FtwUpdateFvState (
   UINT8       State;\r
   UINTN       Length;\r
 \r
+  //\r
+  // Calculate the real Offset and Lba to write.\r
+  //\r
+  while (Offset >= BlockSize) {\r
+    Offset -= BlockSize;\r
+    Lba++;\r
+  }\r
+\r
   //\r
   // Read state from device, assume State is only one byte.\r
   //\r
@@ -1015,10 +1037,10 @@ FindFvbForFtw (
   UINTN                               Index;\r
   EFI_PHYSICAL_ADDRESS                FvbBaseAddress;\r
   EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL  *Fvb;\r
-  EFI_FIRMWARE_VOLUME_HEADER          *FwVolHeader;\r
   EFI_FVB_ATTRIBUTES_2                Attributes;\r
-  EFI_FV_BLOCK_MAP_ENTRY              *FvbMapEntry;\r
   UINT32                              LbaIndex;\r
+  UINTN                               BlockSize;\r
+  UINTN                               NumberOfBlocks;\r
 \r
   HandleBuffer = NULL;\r
 \r
@@ -1056,80 +1078,92 @@ FindFvbForFtw (
       continue;\r
     }\r
 \r
-    FwVolHeader = (EFI_FIRMWARE_VOLUME_HEADER *) ((UINTN) FvbBaseAddress);\r
+    //\r
+    // Now, one FVB has one type of BlockSize.\r
+    //\r
+    Status = Fvb->GetBlockSize (Fvb, 0, &BlockSize, &NumberOfBlocks);\r
+    if (EFI_ERROR (Status)) {\r
+      continue;\r
+    }\r
+\r
     if ((FtwDevice->FtwFvBlock == NULL) && (FtwDevice->WorkSpaceAddress >= FvbBaseAddress) &&\r
-      ((FtwDevice->WorkSpaceAddress + FtwDevice->WorkSpaceLength) <= (FvbBaseAddress + FwVolHeader->FvLength))\r
-      ) {\r
+        ((FtwDevice->WorkSpaceAddress + FtwDevice->WorkSpaceLength) <= (FvbBaseAddress + BlockSize * NumberOfBlocks))) {\r
       FtwDevice->FtwFvBlock = Fvb;\r
       //\r
       // To get the LBA of work space\r
       //\r
-      if ((FwVolHeader->FvLength) > (FwVolHeader->HeaderLength)) {\r
-        //\r
-        // Now, one FV has one type of BlockLength\r
-        //\r
-        FvbMapEntry = &FwVolHeader->BlockMap[0];\r
-        for (LbaIndex = 1; LbaIndex <= FvbMapEntry->NumBlocks; LbaIndex += 1) {\r
-          if ((FtwDevice->WorkSpaceAddress >= (FvbBaseAddress + FvbMapEntry->Length * (LbaIndex - 1)))\r
-              && (FtwDevice->WorkSpaceAddress < (FvbBaseAddress + FvbMapEntry->Length * LbaIndex))) {\r
-            FtwDevice->FtwWorkSpaceLba = LbaIndex - 1;\r
+      for (LbaIndex = 1; LbaIndex <= NumberOfBlocks; LbaIndex += 1) {\r
+        if ((FtwDevice->WorkSpaceAddress >= (FvbBaseAddress + BlockSize * (LbaIndex - 1)))\r
+            && (FtwDevice->WorkSpaceAddress < (FvbBaseAddress + BlockSize * LbaIndex))) {\r
+          FtwDevice->FtwWorkSpaceLba = LbaIndex - 1;\r
+          //\r
+          // Get the Work space size and Base(Offset)\r
+          //\r
+          FtwDevice->FtwWorkSpaceSize = FtwDevice->WorkSpaceLength;\r
+          FtwDevice->WorkBlockSize    = BlockSize;\r
+          FtwDevice->FtwWorkSpaceBase = (UINTN) (FtwDevice->WorkSpaceAddress - (FvbBaseAddress + FtwDevice->WorkBlockSize * (LbaIndex - 1)));\r
+          FtwDevice->NumberOfWorkSpaceBlock = FTW_BLOCKS (FtwDevice->FtwWorkSpaceBase + FtwDevice->FtwWorkSpaceSize, FtwDevice->WorkBlockSize);\r
+          if (FtwDevice->FtwWorkSpaceSize >= FtwDevice->WorkBlockSize) {\r
             //\r
-            // Get the Work space size and Base(Offset)\r
+            // Check the alignment of work space address and length, they should be block size aligned when work space size is larger than one block size.\r
             //\r
-            FtwDevice->FtwWorkSpaceSize = FtwDevice->WorkSpaceLength;\r
-            FtwDevice->FtwWorkSpaceBase = (UINTN) (FtwDevice->WorkSpaceAddress - (FvbBaseAddress + FvbMapEntry->Length * (LbaIndex - 1)));\r
-            break;\r
+            if (((FtwDevice->WorkSpaceAddress & (FtwDevice->WorkBlockSize - 1)) != 0) ||\r
+                ((FtwDevice->WorkSpaceLength & (FtwDevice->WorkBlockSize - 1)) != 0)) {\r
+              DEBUG ((EFI_D_ERROR, "Ftw: Work space address or length is not block size aligned when work space size is larger than one block size\n"));\r
+              FreePool (HandleBuffer);\r
+              ASSERT (FALSE);\r
+              return EFI_ABORTED;\r
+            }\r
+          } else if ((FtwDevice->FtwWorkSpaceBase + FtwDevice->FtwWorkSpaceSize) > FtwDevice->WorkBlockSize) {\r
+            DEBUG ((EFI_D_ERROR, "Ftw: The work space range should not span blocks when work space size is less than one block size\n"));\r
+            FreePool (HandleBuffer);\r
+            ASSERT (FALSE);\r
+            return EFI_ABORTED;\r
           }\r
+          break;\r
         }\r
       }\r
     }\r
-    \r
+\r
     if ((FtwDevice->FtwBackupFvb == NULL) && (FtwDevice->SpareAreaAddress >= FvbBaseAddress) &&\r
-      ((FtwDevice->SpareAreaAddress + FtwDevice->SpareAreaLength) <= (FvbBaseAddress + FwVolHeader->FvLength))\r
-      ) {\r
+        ((FtwDevice->SpareAreaAddress + FtwDevice->SpareAreaLength) <= (FvbBaseAddress + BlockSize * NumberOfBlocks))) {\r
       FtwDevice->FtwBackupFvb = Fvb;\r
       //\r
       // To get the LBA of spare\r
       //\r
-      if ((FwVolHeader->FvLength) > (FwVolHeader->HeaderLength)) {\r
-        //\r
-        // Now, one FV has one type of BlockLength\r
-        //\r
-        FvbMapEntry = &FwVolHeader->BlockMap[0];\r
-        for (LbaIndex = 1; LbaIndex <= FvbMapEntry->NumBlocks; LbaIndex += 1) {\r
-          if ((FtwDevice->SpareAreaAddress >= (FvbBaseAddress + FvbMapEntry->Length * (LbaIndex - 1)))\r
-              && (FtwDevice->SpareAreaAddress < (FvbBaseAddress + FvbMapEntry->Length * LbaIndex))) {\r
-            //\r
-            // Get the NumberOfSpareBlock and BlockSize\r
-            //\r
-            FtwDevice->FtwSpareLba   = LbaIndex - 1;\r
-            FtwDevice->BlockSize     = FvbMapEntry->Length;\r
-            FtwDevice->NumberOfSpareBlock = FtwDevice->SpareAreaLength / FtwDevice->BlockSize;\r
-            //\r
-            // Check the range of spare area to make sure that it's in FV range\r
-            //\r
-            if ((FtwDevice->FtwSpareLba + FtwDevice->NumberOfSpareBlock) > FvbMapEntry->NumBlocks) {\r
-              DEBUG ((EFI_D_ERROR, "Ftw: Spare area is out of FV range\n"));\r
-              FreePool (HandleBuffer);\r
-              ASSERT (FALSE);\r
-              return EFI_ABORTED;\r
-            }\r
+      for (LbaIndex = 1; LbaIndex <= NumberOfBlocks; LbaIndex += 1) {\r
+        if ((FtwDevice->SpareAreaAddress >= (FvbBaseAddress + BlockSize * (LbaIndex - 1)))\r
+            && (FtwDevice->SpareAreaAddress < (FvbBaseAddress + BlockSize * LbaIndex))) {\r
+          //\r
+          // Get the NumberOfSpareBlock and BlockSize\r
+          //\r
+          FtwDevice->FtwSpareLba        = LbaIndex - 1;\r
+          FtwDevice->SpareBlockSize     = BlockSize;\r
+          FtwDevice->NumberOfSpareBlock = FtwDevice->SpareAreaLength / FtwDevice->SpareBlockSize;\r
+          //\r
+          // Check the range of spare area to make sure that it's in FV range\r
+          //\r
+          if ((FtwDevice->FtwSpareLba + FtwDevice->NumberOfSpareBlock) > NumberOfBlocks) {\r
+            DEBUG ((EFI_D_ERROR, "Ftw: Spare area is out of FV range\n"));\r
+            FreePool (HandleBuffer);\r
+            ASSERT (FALSE);\r
+            return EFI_ABORTED;\r
+          }\r
+          //\r
+          // Check the alignment of spare area address and length, they should be block size aligned\r
+          //\r
+          if (((FtwDevice->SpareAreaAddress & (FtwDevice->SpareBlockSize - 1)) != 0) ||\r
+              ((FtwDevice->SpareAreaLength & (FtwDevice->SpareBlockSize - 1)) != 0)) {\r
+            DEBUG ((EFI_D_ERROR, "Ftw: Spare area address or length is not block size aligned\n"));\r
+            FreePool (HandleBuffer);\r
             //\r
-            // Check the alignment of spare area address and length, they should be block size aligned\r
+            // Report Status Code EFI_SW_EC_ABORTED.\r
             //\r
-            if (((FtwDevice->SpareAreaAddress & (FtwDevice->BlockSize - 1)) != 0) ||\r
-                ((FtwDevice->SpareAreaLength & (FtwDevice->BlockSize - 1)) != 0)) {\r
-              DEBUG ((EFI_D_ERROR, "Ftw: Spare area address or length is not block size aligned\n"));\r
-              FreePool (HandleBuffer);\r
-              //\r
-              // Report Status Code EFI_SW_EC_ABORTED.\r
-              //\r
-              REPORT_STATUS_CODE (  (EFI_ERROR_CODE | EFI_ERROR_UNRECOVERED), (EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_EC_ABORTED));\r
-              ASSERT (FALSE);\r
-              CpuDeadLoop ();\r
-            }\r
-            break;\r
+            REPORT_STATUS_CODE ((EFI_ERROR_CODE | EFI_ERROR_UNRECOVERED), (EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_EC_ABORTED));\r
+            ASSERT (FALSE);\r
+            CpuDeadLoop ();\r
           }\r
+          break;\r
         }\r
       }\r
     }\r
@@ -1140,6 +1174,8 @@ FindFvbForFtw (
     (FtwDevice->FtwWorkSpaceLba == (EFI_LBA) (-1)) || (FtwDevice->FtwSpareLba == (EFI_LBA) (-1))) {\r
     return EFI_ABORTED;\r
   }\r
+  DEBUG ((EFI_D_INFO, "Ftw: FtwWorkSpaceLba - 0x%lx, WorkBlockSize  - 0x%x, FtwWorkSpaceBase - 0x%x\n", FtwDevice->FtwWorkSpaceLba, FtwDevice->WorkBlockSize, FtwDevice->FtwWorkSpaceBase));\r
+  DEBUG ((EFI_D_INFO, "Ftw: FtwSpareLba     - 0x%lx, SpareBlockSize - 0x%x\n", FtwDevice->FtwSpareLba, FtwDevice->SpareBlockSize));\r
 \r
   return EFI_SUCCESS;\r
 }\r
@@ -1161,7 +1197,6 @@ InitFtwProtocol (
 {\r
   EFI_STATUS                          Status;\r
   EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL  *Fvb;\r
-  UINTN                               Length;\r
   EFI_FAULT_TOLERANT_WRITE_HEADER     *FtwHeader;\r
   UINTN                               Offset;\r
   EFI_HANDLE                          FvbHandle;\r
@@ -1174,15 +1209,38 @@ InitFtwProtocol (
   if (EFI_ERROR (Status)) {\r
     return EFI_NOT_FOUND;\r
   }  \r
-  \r
+\r
+  //\r
+  // Calculate the start LBA of working block.\r
+  //\r
+  if (FtwDevice->FtwWorkSpaceSize >= FtwDevice->WorkBlockSize) {\r
+    //\r
+    // Working block is a standalone area which only contains working space.\r
+    //\r
+    FtwDevice->NumberOfWorkBlock = FtwDevice->NumberOfWorkSpaceBlock;\r
+  } else {\r
+    //\r
+    // Working block is an area which\r
+    // contains working space in its last block and has the same size as spare\r
+    // block, unless there are not enough blocks before the block that contains\r
+    // working space.\r
+    //\r
+    FtwDevice->NumberOfWorkBlock = (UINTN) (FtwDevice->FtwWorkSpaceLba + FtwDevice->NumberOfWorkSpaceBlock);\r
+    while (FtwDevice->NumberOfWorkBlock * FtwDevice->WorkBlockSize > FtwDevice->SpareAreaLength) {\r
+      FtwDevice->NumberOfWorkBlock--;\r
+    }\r
+  }\r
+  FtwDevice->FtwWorkBlockLba = FtwDevice->FtwWorkSpaceLba + FtwDevice->NumberOfWorkSpaceBlock - FtwDevice->NumberOfWorkBlock;\r
+  DEBUG ((EFI_D_INFO, "Ftw: NumberOfWorkBlock - 0x%x, FtwWorkBlockLba - 0x%lx\n", FtwDevice->NumberOfWorkBlock, FtwDevice->FtwWorkBlockLba));\r
+\r
   //\r
-  // Calculate the start LBA of working block. Working block is an area which\r
-  // contains working space in its last block and has the same size as spare\r
-  // block, unless there are not enough blocks before the block that contains\r
-  // working space.\r
+  // Calcualte the LBA and base of work space in spare block.\r
+  // Note: Do not assume Spare Block and Work Block have same block size.\r
   //\r
-  FtwDevice->FtwWorkBlockLba = FtwDevice->FtwWorkSpaceLba - FtwDevice->NumberOfSpareBlock + 1;\r
-  ASSERT ((INT64) (FtwDevice->FtwWorkBlockLba) >= 0); \r
+  WorkSpaceLbaOffset = FtwDevice->FtwWorkSpaceLba - FtwDevice->FtwWorkBlockLba;\r
+  FtwDevice->FtwWorkSpaceLbaInSpare = (EFI_LBA) (((UINTN) WorkSpaceLbaOffset * FtwDevice->WorkBlockSize + FtwDevice->FtwWorkSpaceBase) / FtwDevice->SpareBlockSize);\r
+  FtwDevice->FtwWorkSpaceBaseInSpare = ((UINTN) WorkSpaceLbaOffset * FtwDevice->WorkBlockSize + FtwDevice->FtwWorkSpaceBase) % FtwDevice->SpareBlockSize;\r
+  DEBUG ((EFI_D_INFO, "Ftw: WorkSpaceLbaInSpare - 0x%lx, WorkSpaceBaseInSpare - 0x%x\n", FtwDevice->FtwWorkSpaceLbaInSpare, FtwDevice->FtwWorkSpaceBaseInSpare));\r
 \r
   //\r
   // Initialize other parameters, and set WorkSpace as FTW_ERASED_BYTE.\r
@@ -1207,15 +1265,14 @@ InitFtwProtocol (
     //\r
     // Read from spare block\r
     //\r
-    WorkSpaceLbaOffset = FtwDevice->FtwWorkSpaceLba - FtwDevice->FtwWorkBlockLba;\r
-    Length = FtwDevice->FtwWorkSpaceSize;\r
-    Status = FtwDevice->FtwBackupFvb->Read (\r
-                    FtwDevice->FtwBackupFvb,\r
-                    FtwDevice->FtwSpareLba + WorkSpaceLbaOffset,\r
-                    FtwDevice->FtwWorkSpaceBase,\r
-                    &Length,\r
-                    FtwDevice->FtwWorkSpace\r
-                    );\r
+    Status = ReadWorkSpaceData (\r
+               FtwDevice->FtwBackupFvb,\r
+               FtwDevice->SpareBlockSize,\r
+               FtwDevice->FtwSpareLba + FtwDevice->FtwWorkSpaceLbaInSpare,\r
+               FtwDevice->FtwWorkSpaceBaseInSpare,\r
+               FtwDevice->FtwWorkSpaceSize,\r
+               FtwDevice->FtwWorkSpace\r
+               );\r
     ASSERT_EFI_ERROR (Status);\r
 \r
     //\r
@@ -1304,7 +1361,7 @@ InitFtwProtocol (
       FvbHandle = GetFvbByAddress ((EFI_PHYSICAL_ADDRESS) (UINTN) ((INT64) FtwDevice->SpareAreaAddress + FtwDevice->FtwLastWriteRecord->RelativeOffset), &Fvb);\r
       if (FvbHandle != NULL) {\r
         Status = FtwRestart (&FtwDevice->FtwInstance, FvbHandle);\r
-        DEBUG ((EFI_D_ERROR, "FtwLite: Restart last write - %r\n", Status));\r
+        DEBUG ((EFI_D_ERROR, "Ftw: Restart last write - %r\n", Status));\r
         ASSERT_EFI_ERROR (Status);\r
       }\r
       FtwAbort (&FtwDevice->FtwInstance);\r
index a5fa12b0a804e55682992686f8a34789bc307c0c..29252c417393dfd4bc8af3c032e07727f3aa059e 100644 (file)
@@ -125,6 +125,132 @@ InitWorkSpaceHeader (
   return EFI_SUCCESS;\r
 }\r
 \r
+/**\r
+  Read work space data from work block or spare block.\r
+\r
+  @param FtwDevice      The private data of FTW driver.\r
+  @param FvBlock        FVB Protocol interface to access the block.\r
+  @param BlockSize      The size of the block.\r
+  @param Lba            Lba of the block.\r
+  @param Offset         The offset within the block.\r
+  @param Length         The number of bytes to read from the block.\r
+  @param Buffer         The data is read.\r
+\r
+  @retval EFI_SUCCESS   The function completed successfully.\r
+  @retval EFI_ABORTED   The function could not complete successfully.\r
+\r
+**/\r
+EFI_STATUS\r
+ReadWorkSpaceData (\r
+  IN EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL *FvBlock,\r
+  IN UINTN                              BlockSize,\r
+  IN EFI_LBA                            Lba,\r
+  IN UINTN                              Offset,\r
+  IN UINTN                              Length,\r
+  OUT UINT8                             *Buffer\r
+  )\r
+{\r
+  EFI_STATUS            Status;\r
+  UINT8                 *Ptr;\r
+  UINTN                 MyLength;\r
+\r
+  //\r
+  // Calculate the real Offset and Lba to write.\r
+  //\r
+  while (Offset >= BlockSize) {\r
+    Offset -= BlockSize;\r
+    Lba++;\r
+  }\r
+\r
+  Ptr = Buffer;\r
+  while (Length > 0) {\r
+    if ((Offset + Length) > BlockSize) {\r
+      MyLength = BlockSize - Offset;\r
+    } else {\r
+      MyLength = Length;\r
+    }\r
+\r
+    Status = FvBlock->Read (\r
+                        FvBlock,\r
+                        Lba,\r
+                        Offset,\r
+                        &MyLength,\r
+                        Ptr\r
+                        );\r
+    if (EFI_ERROR (Status)) {\r
+      return EFI_ABORTED;\r
+    }\r
+    Offset = 0;\r
+    Length -= MyLength;\r
+    Ptr += MyLength;\r
+    Lba++;\r
+  }\r
+\r
+  return EFI_SUCCESS;\r
+}\r
+\r
+/**\r
+  Write work space data to work block.\r
+\r
+  @param FvBlock        FVB Protocol interface to access the block.\r
+  @param BlockSize      The size of the block.\r
+  @param Lba            Lba of the block.\r
+  @param Offset         The offset within the block to place the data.\r
+  @param Length         The number of bytes to write to the block.\r
+  @param Buffer         The data to write.\r
+\r
+  @retval EFI_SUCCESS   The function completed successfully.\r
+  @retval EFI_ABORTED   The function could not complete successfully.\r
+\r
+**/\r
+EFI_STATUS\r
+WriteWorkSpaceData (\r
+  IN EFI_FIRMWARE_VOLUME_BLOCK_PROTOCOL *FvBlock,\r
+  IN UINTN                              BlockSize,\r
+  IN EFI_LBA                            Lba,\r
+  IN UINTN                              Offset,\r
+  IN UINTN                              Length,\r
+  IN UINT8                              *Buffer\r
+  )\r
+{\r
+  EFI_STATUS            Status;\r
+  UINT8                 *Ptr;\r
+  UINTN                 MyLength;\r
+\r
+  //\r
+  // Calculate the real Offset and Lba to write.\r
+  //\r
+  while (Offset >= BlockSize) {\r
+    Offset -= BlockSize;\r
+    Lba++;\r
+  }\r
+\r
+  Ptr = Buffer;\r
+  while (Length > 0) {\r
+    if ((Offset + Length) > BlockSize) {\r
+      MyLength = BlockSize - Offset;\r
+    } else {\r
+      MyLength = Length;\r
+    }\r
+\r
+    Status = FvBlock->Write (\r
+                        FvBlock,\r
+                        Lba,\r
+                        Offset,\r
+                        &MyLength,\r
+                        Ptr\r
+                        );\r
+    if (EFI_ERROR (Status)) {\r
+      return EFI_ABORTED;\r
+    }\r
+    Offset = 0;\r
+    Length -= MyLength;\r
+    Ptr += MyLength;\r
+    Lba++;\r
+  }\r
+  return EFI_SUCCESS;\r
+}\r
+\r
 /**\r
   Read from working block to refresh the work space in memory.\r
 \r
@@ -140,7 +266,6 @@ WorkSpaceRefresh (
   )\r
 {\r
   EFI_STATUS                      Status;\r
-  UINTN                           Length;\r
   UINTN                           RemainingSpaceSize;\r
 \r
   //\r
@@ -155,14 +280,14 @@ WorkSpaceRefresh (
   //\r
   // Read from working block\r
   //\r
-  Length = FtwDevice->FtwWorkSpaceSize;\r
-  Status = FtwDevice->FtwFvBlock->Read (\r
-                                    FtwDevice->FtwFvBlock,\r
-                                    FtwDevice->FtwWorkSpaceLba,\r
-                                    FtwDevice->FtwWorkSpaceBase,\r
-                                    &Length,\r
-                                    FtwDevice->FtwWorkSpace\r
-                                    );\r
+  Status = ReadWorkSpaceData (\r
+             FtwDevice->FtwFvBlock,\r
+             FtwDevice->WorkBlockSize,\r
+             FtwDevice->FtwWorkSpaceLba,\r
+             FtwDevice->FtwWorkSpaceBase,\r
+             FtwDevice->FtwWorkSpaceSize,\r
+             FtwDevice->FtwWorkSpace\r
+             );\r
   if (EFI_ERROR (Status)) {\r
     return EFI_ABORTED;\r
   }\r
@@ -194,14 +319,14 @@ WorkSpaceRefresh (
     //\r
     // Read from working block again\r
     //\r
-    Length = FtwDevice->FtwWorkSpaceSize;\r
-    Status = FtwDevice->FtwFvBlock->Read (\r
-                                      FtwDevice->FtwFvBlock,\r
-                                      FtwDevice->FtwWorkSpaceLba,\r
-                                      FtwDevice->FtwWorkSpaceBase,\r
-                                      &Length,\r
-                                      FtwDevice->FtwWorkSpace\r
-                                      );\r
+    Status = ReadWorkSpaceData (\r
+               FtwDevice->FtwFvBlock,\r
+               FtwDevice->WorkBlockSize,\r
+               FtwDevice->FtwWorkSpaceLba,\r
+               FtwDevice->FtwWorkSpaceBase,\r
+               FtwDevice->FtwWorkSpaceSize,\r
+               FtwDevice->FtwWorkSpace\r
+               );\r
     if (EFI_ERROR (Status)) {\r
       return EFI_ABORTED;\r
     }\r
@@ -265,15 +390,15 @@ FtwReclaimWorkSpace (
   //\r
   // Read all original data from working block to a memory buffer\r
   //\r
-  TempBufferSize = FtwDevice->SpareAreaLength;\r
+  TempBufferSize = FtwDevice->NumberOfWorkBlock * FtwDevice->WorkBlockSize;\r
   TempBuffer     = AllocateZeroPool (TempBufferSize);\r
   if (TempBuffer == NULL) {\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
 \r
   Ptr = TempBuffer;\r
-  for (Index = 0; Index < FtwDevice->NumberOfSpareBlock; Index += 1) {\r
-    Length = FtwDevice->BlockSize;\r
+  for (Index = 0; Index < FtwDevice->NumberOfWorkBlock; Index += 1) {\r
+    Length = FtwDevice->WorkBlockSize;\r
     Status = FtwDevice->FtwFvBlock->Read (\r
                                           FtwDevice->FtwFvBlock,\r
                                           FtwDevice->FtwWorkBlockLba + Index,\r
@@ -292,7 +417,7 @@ FtwReclaimWorkSpace (
   // Clean up the workspace, remove all the completed records.\r
   //\r
   Ptr = TempBuffer +\r
-        (UINTN) WorkSpaceLbaOffset * FtwDevice->BlockSize +\r
+        (UINTN) WorkSpaceLbaOffset * FtwDevice->WorkBlockSize +\r
         FtwDevice->FtwWorkSpaceBase;\r
 \r
   //\r
@@ -348,7 +473,7 @@ FtwReclaimWorkSpace (
   // Set the WorkingBlockValid and WorkingBlockInvalid as INVALID\r
   //\r
   WorkingBlockHeader                      = (EFI_FAULT_TOLERANT_WORKING_BLOCK_HEADER *) (TempBuffer +\r
-                                            (UINTN) WorkSpaceLbaOffset * FtwDevice->BlockSize +\r
+                                            (UINTN) WorkSpaceLbaOffset * FtwDevice->WorkBlockSize +\r
                                             FtwDevice->FtwWorkSpaceBase);\r
   WorkingBlockHeader->WorkingBlockValid   = FTW_INVALID_STATE;\r
   WorkingBlockHeader->WorkingBlockInvalid = FTW_INVALID_STATE;\r
@@ -366,7 +491,7 @@ FtwReclaimWorkSpace (
 \r
   Ptr = SpareBuffer;\r
   for (Index = 0; Index < FtwDevice->NumberOfSpareBlock; Index += 1) {\r
-    Length = FtwDevice->BlockSize;\r
+    Length = FtwDevice->SpareBlockSize;\r
     Status = FtwDevice->FtwBackupFvb->Read (\r
                                         FtwDevice->FtwBackupFvb,\r
                                         FtwDevice->FtwSpareLba + Index,\r
@@ -387,8 +512,12 @@ FtwReclaimWorkSpace (
   //\r
   Status  = FtwEraseSpareBlock (FtwDevice);\r
   Ptr     = TempBuffer;\r
-  for (Index = 0; Index < FtwDevice->NumberOfSpareBlock; Index += 1) {\r
-    Length = FtwDevice->BlockSize;\r
+  for (Index = 0; TempBufferSize > 0; Index += 1) {\r
+    if (TempBufferSize > FtwDevice->SpareBlockSize) {\r
+      Length = FtwDevice->SpareBlockSize;\r
+    } else {\r
+      Length = TempBufferSize;\r
+    }\r
     Status = FtwDevice->FtwBackupFvb->Write (\r
                                             FtwDevice->FtwBackupFvb,\r
                                             FtwDevice->FtwSpareLba + Index,\r
@@ -403,6 +532,7 @@ FtwReclaimWorkSpace (
     }\r
 \r
     Ptr += Length;\r
+    TempBufferSize -= Length;\r
   }\r
   //\r
   // Free TempBuffer\r
@@ -414,8 +544,9 @@ FtwReclaimWorkSpace (
   //\r
   Status = FtwUpdateFvState (\r
             FtwDevice->FtwBackupFvb,\r
-            FtwDevice->FtwSpareLba + WorkSpaceLbaOffset,\r
-            FtwDevice->FtwWorkSpaceBase + sizeof (EFI_GUID) + sizeof (UINT32),\r
+            FtwDevice->SpareBlockSize,\r
+            FtwDevice->FtwSpareLba + FtwDevice->FtwWorkSpaceLbaInSpare,\r
+            FtwDevice->FtwWorkSpaceBaseInSpare + sizeof (EFI_GUID) + sizeof (UINT32),\r
             WORKING_BLOCK_VALID\r
             );\r
   if (EFI_ERROR (Status)) {\r
@@ -430,6 +561,7 @@ FtwReclaimWorkSpace (
   //\r
   Status = FtwUpdateFvState (\r
             FtwDevice->FtwFvBlock,\r
+            FtwDevice->WorkBlockSize,\r
             FtwDevice->FtwWorkSpaceLba,\r
             FtwDevice->FtwWorkSpaceBase + sizeof (EFI_GUID) + sizeof (UINT32),\r
             WORKING_BLOCK_INVALID\r
@@ -455,7 +587,7 @@ FtwReclaimWorkSpace (
   Status  = FtwEraseSpareBlock (FtwDevice);\r
   Ptr     = SpareBuffer;\r
   for (Index = 0; Index < FtwDevice->NumberOfSpareBlock; Index += 1) {\r
-    Length = FtwDevice->BlockSize;\r
+    Length = FtwDevice->SpareBlockSize;\r
     Status = FtwDevice->FtwBackupFvb->Write (\r
                                         FtwDevice->FtwBackupFvb,\r
                                         FtwDevice->FtwSpareLba + Index,\r