Changeset 42741 in vbox for trunk/src/VBox/Devices

Timestamp:

Aug 10, 2012 7:11:09 AM (12 years ago)

Author:

vboxsync

Message:

EFI/VBoxFsDxe: moved to UDK2010.sr1 API.

Location:

trunk/src/VBox/Devices/EFI/Firmware2/VBoxPkg/VBoxFsDxe

Files:

• VBoxHfs.inf (modified) (4 diffs)
• VBoxIso9660.inf (modified) (3 diffs)
• fsw_core.h (modified) (1 diff)
• fsw_efi.c (modified) (9 diffs)
• fsw_iso9660.c (modified) (3 diffs)

trunk/src/VBox/Devices/EFI/Firmware2/VBoxPkg/VBoxFsDxe/VBoxHfs.inf

@@ -36 +36 @@
 [Packages]
         MdePkg/MdePkg.dec
+        MdeModulePkg/MdeModulePkg.dec
         IntelFrameworkModulePkg/IntelFrameworkModulePkg.dec
         VBoxPkg/VBoxPkg.dec
@@ -49 +50 @@
    DebugLib
    PcdLib
+   DevicePathLib
 
 [Guids]
@@ -61 +63 @@
   gEfiUnicodeCollationProtocolGuid
   gEfiUnicodeCollation2ProtocolGuid
+  gEfiDevicePathToTextProtocolGuid                ## CONSUMED
 
 [Pcd]
@@ -67 +70 @@
 
 [BuildOptions.common]
-  GCC:*_*_*_CC_FLAGS = -DFSTYPE=hfs
+  GCC:*_*_*_CC_FLAGS = -DFSTYPE=hfs -DEFI_LOG_ENABLED=1
 # -DFSW_DEBUG_LEVEL=3
   INTEL:*_*_*_CC_FLAGS = -DFSTYPE=hfs

trunk/src/VBox/Devices/EFI/Firmware2/VBoxPkg/VBoxFsDxe/VBoxIso9660.inf

@@ -34 +34 @@
 [Packages]
         MdePkg/MdePkg.dec
+        MdeModulePkg/MdeModulePkg.dec
         IntelFrameworkModulePkg/IntelFrameworkModulePkg.dec
         VBoxPkg/VBoxPkg.dec
@@ -47 +48 @@
    DebugLib
    PcdLib
+   DevicePathLib
 
 [Guids]
@@ -60 +62 @@
   gEfiUnicodeCollationProtocolGuid
   gEfiUnicodeCollation2ProtocolGuid
+  gEfiDevicePathToTextProtocolGuid                ## CONSUMED
 
 [Pcd]

trunk/src/VBox/Devices/EFI/Firmware2/VBoxPkg/VBoxFsDxe/fsw_core.h

@@ -72 +72 @@
 
 /** Indicates that the block cache entry is empty. */
-#define FSW_INVALID_BNO (~0UL)
+#define FSW_INVALID_BNO (~0U)
 
 

trunk/src/VBox/Devices/EFI/Firmware2/VBoxPkg/VBoxFsDxe/fsw_efi.c

@@ -67 +67 @@
 #define FSW_EFI_DRIVER_NAME(t) L"Fsw " FSW_EFI_STRINGIFY(t) L" File System Driver"
 
+
 // function prototypes
 
@@ -222 +223 @@
 
     // first, open DiskIO
+    LogFlowFuncEnter();
+    LogFlowFuncMarkDP(RemainingDevicePath);
+
     Status = BS->OpenProtocol(ControllerHandle,
                               &PROTO_NAME(DiskIoProtocol),
@@ -227 +231 @@
                               This->DriverBindingHandle,
                               ControllerHandle,
-                              EFI_OPEN_PROTOCOL_BY_DRIVER);
+                              EFI_OPEN_PROTOCOL_GET_PROTOCOL);
     if (EFI_ERROR(Status))
-        return Status;
+    {
+        LogFlowFuncLeaveRC(Status);
+        return Status;
+    }
 
     // we were just checking, close it again
@@ -244 +251 @@
                               ControllerHandle,
                               EFI_OPEN_PROTOCOL_TEST_PROTOCOL);
+    LogFlowFuncLeaveRC(Status);
+    return Status;
+}
+
+static EFI_STATUS fsw_efi_ReMount(IN FSW_VOLUME_DATA *pVolume,
+                                       IN EFI_HANDLE      ControllerHandle,
+                                       EFI_DISK_IO        *pDiskIo,
+                                       EFI_BLOCK_IO       *pBlockIo)
+{
+    EFI_STATUS Status;
+    LogFlowFuncEnter();
+    pVolume->Signature       = FSW_VOLUME_DATA_SIGNATURE;
+    pVolume->Handle          = ControllerHandle;
+    pVolume->DiskIo          = pDiskIo;
+    pVolume->MediaId         = pBlockIo->Media->MediaId;
+    pVolume->LastIOStatus    = EFI_SUCCESS;
+
+    // mount the filesystem
+    Status = fsw_efi_map_status(fsw_mount(pVolume, &fsw_efi_host_table,
+                                          &FSW_FSTYPE_TABLE_NAME(FSTYPE), &pVolume->vol),
+                                pVolume);
+
+    LogFlowFuncMarkVar(Status, "%r");
+    if (!EFI_ERROR(Status)) {
+        // register the SimpleFileSystem protocol
+        pVolume->FileSystem.Revision     = EFI_FILE_IO_INTERFACE_REVISION;
+        pVolume->FileSystem.OpenVolume   = fsw_efi_FileSystem_OpenVolume;
+        Status = BS->InstallMultipleProtocolInterfaces(&ControllerHandle,
+                                                       &PROTO_NAME(SimpleFileSystemProtocol), &pVolume->FileSystem,
+                                                       NULL);
+        if (EFI_ERROR(Status))
+            Print(L"Fsw ERROR: InstallMultipleProtocolInterfaces returned %x\n", Status);
+    }
+    LogFlowFuncLeaveRC(Status);
     return Status;
 }
@@ -269 +310 @@
     FSW_VOLUME_DATA     *Volume;
 
-#if DEBUG_LEVEL
-    Print(L"fsw_efi_DriverBinding_Start\n");
-#endif
-
+    LogFlowFuncEnter();
     // open consumed protocols
     Status = BS->OpenProtocol(ControllerHandle,
@@ -281 +319 @@
                               EFI_OPEN_PROTOCOL_GET_PROTOCOL);   // NOTE: we only want to look at the MediaId
     if (EFI_ERROR(Status)) {
-        Print(L"Fsw ERROR: OpenProtocol(BlockIo) returned %x\n", Status);
+        LogFlowFuncLeaveRC(Status);
         return Status;
     }
@@ -292 +330 @@
                               EFI_OPEN_PROTOCOL_BY_DRIVER);
     if (EFI_ERROR(Status)) {
-        Print(L"Fsw ERROR: OpenProtocol(DiskIo) returned %x\n", Status);
+        LogFlowFuncLeaveRC(Status);
         return Status;
     }
@@ -298 +336 @@
     // allocate volume structure
     Volume = AllocateZeroPool(sizeof(FSW_VOLUME_DATA));
-    Volume->Signature       = FSW_VOLUME_DATA_SIGNATURE;
-    Volume->Handle          = ControllerHandle;
-    Volume->DiskIo          = DiskIo;
-    Volume->MediaId         = BlockIo->Media->MediaId;
-    Volume->LastIOStatus    = EFI_SUCCESS;
-
-    // mount the filesystem
-    Status = fsw_efi_map_status(fsw_mount(Volume, &fsw_efi_host_table,
-                                          &FSW_FSTYPE_TABLE_NAME(FSTYPE), &Volume->vol),
-                                Volume);
-
-    if (!EFI_ERROR(Status)) {
-        // register the SimpleFileSystem protocol
-        Volume->FileSystem.Revision     = EFI_FILE_IO_INTERFACE_REVISION;
-        Volume->FileSystem.OpenVolume   = fsw_efi_FileSystem_OpenVolume;
-        Status = BS->InstallMultipleProtocolInterfaces(&ControllerHandle,
-                                                       &PROTO_NAME(SimpleFileSystemProtocol), &Volume->FileSystem,
-                                                       NULL);
-        if (EFI_ERROR(Status))
-            Print(L"Fsw ERROR: InstallMultipleProtocolInterfaces returned %x\n", Status);
-    }
+    Status = fsw_efi_ReMount(Volume, ControllerHandle, DiskIo, BlockIo);
 
     // on errors, close the opened protocols
@@ -326 +344 @@
         FreePool(Volume);
 
-        BS->CloseProtocol(ControllerHandle,
-                          &PROTO_NAME(DiskIoProtocol),
-                          This->DriverBindingHandle,
-                          ControllerHandle);
-    }
-
+#if 0
+        if (Status == EFI_MEDIA_CHANGED)
+            Status = fsw_efi_ReMount(Volume, ControllerHandle, DiskIo, BlockIo);
+        else
+#endif
+            BS->CloseProtocol(ControllerHandle,
+                              &PROTO_NAME(DiskIoProtocol),
+                              This->DriverBindingHandle,
+                              ControllerHandle);
+    }
+
+    LogFlowFuncLeaveRC(Status);
     return Status;
 }

trunk/src/VBox/Devices/EFI/Firmware2/VBoxPkg/VBoxFsDxe/fsw_iso9660.c

@@ -88 +88 @@
 static fsw_status_t rr_find_nm(struct fsw_iso9660_volume *vol, struct iso9660_dirrec *dirrec, int off, struct fsw_string *str);
 static fsw_status_t rr_read_ce(struct fsw_iso9660_volume *vol, union fsw_rock_ridge_susp_ce *ce, fsw_u8 *begin);
-static void dump_dirrec(struct iso9660_dirrec *dirrec);
 //
 // Dispatch Table
@@ -226 +225 @@
 {
     int rc;
-    int i;
-    fsw_u8 *r = begin + ISOINT(ce->X.offset);
-    int len = ISOINT(ce->X.len);
     rc = vol->g.host_table->read_block(&vol->g, ISOINT(ce->X.block_loc), begin);
     if (rc != FSW_SUCCESS)
         return rc;
-    for (i = 0; i < len; ++i)
-    {
-        DEBUG((DEBUG_INFO, "%d: (%d:%x)%c ", i, r[i], r[i], r[i]));
-    }
-    return FSW_SUCCESS;
-}
-static void dump_dirrec(struct iso9660_dirrec *dirrec)
-{
-    int i;
-    fsw_u8 *r = (fsw_u8 *)dirrec + dirrec->file_identifier_length;
-    int len = dirrec->dirrec_length;
-    for (i = dirrec->file_identifier_length; i < len; ++i)
-    {
-        DEBUG((DEBUG_INFO, "%d: (%d:%x)%c ", i, r[i], r[i], r[i]));
-    }
+    return FSW_SUCCESS;
 }
 /**
@@ -624 +606 @@
         return FSW_VOLUME_CORRUPTED;
 
-     dump_dirrec(dirrec);
      if (vol->fRockRidge)
      {