From: qhuang8 Date: Thu, 20 Sep 2007 10:06:50 +0000 (+0000) Subject: 1. Set the Target array to zero before fill the target id. X-Git-Tag: edk2-stable201903~22116 X-Git-Url: https://git.proxmox.com/?p=mirror_edk2.git;a=commitdiff_plain;h=f36d6e669c9709284c04d76b87debbedfe480cb4 1. Set the Target array to zero before fill the target id. 2. Get the command/control register base address for each channel through IDE common registers. 3. Correctify some return status to sync with newest Uefi Spec 2.1 git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@3901 6f19259b-4bc3-4df7-8a09-765794883524 --- diff --git a/MdeModulePkg/Bus/Pci/AtapiPassThruDxe/AtapiPassThru.c b/MdeModulePkg/Bus/Pci/AtapiPassThruDxe/AtapiPassThru.c index f14d9e3482..b6dceeb83a 100644 --- a/MdeModulePkg/Bus/Pci/AtapiPassThruDxe/AtapiPassThru.c +++ b/MdeModulePkg/Bus/Pci/AtapiPassThruDxe/AtapiPassThru.c @@ -12,13 +12,6 @@ #include "AtapiPassThru.h" -/// -/// IDE registers' fixed address -/// -static IDE_BASE_REGISTERS gAtapiIoPortRegisters[2] = { - { 0x1f0, { 0x1f1 }, 0x1f2, 0x1f3, 0x1f4, 0x1f5, 0x1f6, { 0x1f7 }, { 0x3f6 }, 0x3f7, 0 }, - { 0x170, { 0x171 }, 0x172, 0x173, 0x174, 0x175, 0x176, { 0x177 }, { 0x376 }, 0x377, 0 } -}; static SCSI_COMMAND_SET gEndTable = { 0xff, (DATA_DIRECTION) 0xff }; @@ -97,6 +90,7 @@ AtapiScsiPassThruDriverBindingSupported ( EFI_PCI_IO_PROTOCOL *PciIo; PCI_TYPE00 Pci; + // // Open the IO Abstraction(s) needed to perform the supported test // @@ -125,11 +119,11 @@ AtapiScsiPassThruDriverBindingSupported ( ); if (EFI_ERROR (Status)) { gBS->CloseProtocol ( - Controller, - &gEfiPciIoProtocolGuid, - This->DriverBindingHandle, - Controller - ); + Controller, + &gEfiPciIoProtocolGuid, + This->DriverBindingHandle, + Controller + ); return EFI_UNSUPPORTED; } @@ -139,11 +133,11 @@ AtapiScsiPassThruDriverBindingSupported ( } gBS->CloseProtocol ( - Controller, - &gEfiPciIoProtocolGuid, - This->DriverBindingHandle, - Controller - ); + Controller, + &gEfiPciIoProtocolGuid, + This->DriverBindingHandle, + Controller + ); return Status; } @@ -169,7 +163,9 @@ AtapiScsiPassThruDriverBindingStart ( ) { EFI_STATUS Status; + EFI_STATUS DisableStatus; EFI_PCI_IO_PROTOCOL *PciIo; + UINT64 Supports; PciIo = NULL; Status = gBS->OpenProtocol ( @@ -186,10 +182,21 @@ AtapiScsiPassThruDriverBindingStart ( Status = PciIo->Attributes ( PciIo, - EfiPciIoAttributeOperationEnable, - EFI_PCI_IO_ATTRIBUTE_IDE_PRIMARY_IO | EFI_PCI_IO_ATTRIBUTE_IDE_SECONDARY_IO | EFI_PCI_DEVICE_ENABLE, - NULL + EfiPciIoAttributeOperationSupported, + 0, + &Supports ); + if (!EFI_ERROR (Status)) { + Supports &= (EFI_PCI_DEVICE_ENABLE | + EFI_PCI_IO_ATTRIBUTE_IDE_PRIMARY_IO | + EFI_PCI_IO_ATTRIBUTE_IDE_SECONDARY_IO); + Status = PciIo->Attributes ( + PciIo, + EfiPciIoAttributeOperationEnable, + Supports, + NULL + ); + } if (EFI_ERROR (Status)) { goto Done; } @@ -202,20 +209,31 @@ AtapiScsiPassThruDriverBindingStart ( Done: if (EFI_ERROR (Status)) { if (PciIo) { - PciIo->Attributes ( - PciIo, - EfiPciIoAttributeOperationDisable, - EFI_PCI_IO_ATTRIBUTE_IDE_PRIMARY_IO | EFI_PCI_IO_ATTRIBUTE_IDE_SECONDARY_IO | EFI_PCI_DEVICE_ENABLE, - NULL - ); + DisableStatus = PciIo->Attributes ( + PciIo, + EfiPciIoAttributeOperationSupported, + 0, + &Supports + ); + if (!EFI_ERROR (DisableStatus)) { + Supports &= (EFI_PCI_DEVICE_ENABLE | + EFI_PCI_IO_ATTRIBUTE_IDE_PRIMARY_IO | + EFI_PCI_IO_ATTRIBUTE_IDE_SECONDARY_IO); + DisableStatus = PciIo->Attributes ( + PciIo, + EfiPciIoAttributeOperationDisable, + Supports, + NULL + ); + } } gBS->CloseProtocol ( - Controller, - &gEfiPciIoProtocolGuid, - This->DriverBindingHandle, - Controller - ); + Controller, + &gEfiPciIoProtocolGuid, + This->DriverBindingHandle, + Controller + ); } return Status; @@ -246,6 +264,7 @@ AtapiScsiPassThruDriverBindingStop ( EFI_STATUS Status; EFI_SCSI_PASS_THRU_PROTOCOL *ScsiPassThru; ATAPI_SCSI_PASS_THRU_DEV *AtapiScsiPrivate; + UINT64 Supports; Status = gBS->OpenProtocol ( Controller, @@ -272,19 +291,30 @@ AtapiScsiPassThruDriverBindingStop ( // // Release Pci Io protocol on the controller handle. // - AtapiScsiPrivate->PciIo->Attributes ( - AtapiScsiPrivate->PciIo, - EfiPciIoAttributeOperationDisable, - EFI_PCI_IO_ATTRIBUTE_IDE_PRIMARY_IO | EFI_PCI_IO_ATTRIBUTE_IDE_SECONDARY_IO | EFI_PCI_DEVICE_ENABLE, - NULL - ); + Status = AtapiScsiPrivate->PciIo->Attributes ( + AtapiScsiPrivate->PciIo, + EfiPciIoAttributeOperationSupported, + 0, + &Supports + ); + if (!EFI_ERROR (Status)) { + Supports &= (EFI_PCI_DEVICE_ENABLE | + EFI_PCI_IO_ATTRIBUTE_IDE_PRIMARY_IO | + EFI_PCI_IO_ATTRIBUTE_IDE_SECONDARY_IO); + Status = AtapiScsiPrivate->PciIo->Attributes ( + AtapiScsiPrivate->PciIo, + EfiPciIoAttributeOperationDisable, + Supports, + NULL + ); + } gBS->CloseProtocol ( - Controller, - &gEfiPciIoProtocolGuid, - This->DriverBindingHandle, - Controller - ); + Controller, + &gEfiPciIoProtocolGuid, + This->DriverBindingHandle, + Controller + ); gBS->FreePool (AtapiScsiPrivate); @@ -313,20 +343,36 @@ RegisterAtapiScsiPassThru ( { EFI_STATUS Status; ATAPI_SCSI_PASS_THRU_DEV *AtapiScsiPrivate; - UINT64 Attributes; + UINT64 Supports; + IDE_REGISTERS_BASE_ADDR IdeRegsBaseAddr[ATAPI_MAX_CHANNEL]; AtapiScsiPrivate = AllocateZeroPool (sizeof (ATAPI_SCSI_PASS_THRU_DEV)); if (AtapiScsiPrivate == NULL) { return EFI_OUT_OF_RESOURCES; } - Attributes = EFI_PCI_IO_ATTRIBUTE_IDE_PRIMARY_IO | EFI_PCI_IO_ATTRIBUTE_IDE_SECONDARY_IO | EFI_PCI_DEVICE_ENABLE; CopyMem (AtapiScsiPrivate->ChannelName, gAtapiChannelString, sizeof (gAtapiChannelString)); // // Enable channel // - PciIo->Attributes (PciIo, EfiPciIoAttributeOperationSet, Attributes, NULL); + Status = PciIo->Attributes ( + PciIo, + EfiPciIoAttributeOperationSupported, + 0, + &Supports + ); + if (!EFI_ERROR (Status)) { + Supports &= (EFI_PCI_DEVICE_ENABLE | + EFI_PCI_IO_ATTRIBUTE_IDE_PRIMARY_IO | + EFI_PCI_IO_ATTRIBUTE_IDE_SECONDARY_IO); + Status = PciIo->Attributes ( + PciIo, + EfiPciIoAttributeOperationEnable, + Supports, + NULL + ); + } AtapiScsiPrivate->Signature = ATAPI_SCSI_PASS_THRU_DEV_SIGNATURE; AtapiScsiPrivate->Handle = Controller; @@ -334,10 +380,19 @@ RegisterAtapiScsiPassThru ( // // will reset the IoPort inside each API function. // - AtapiScsiPrivate->IoPort = gAtapiIoPortRegisters; + AtapiScsiPrivate->IoPort = NULL; AtapiScsiPrivate->PciIo = PciIo; // + // Obtain IDE IO port registers' base addresses + // + Status = GetIdeRegistersBaseAddr (PciIo, IdeRegsBaseAddr); + if (EFI_ERROR (Status)) { + return Status; + } + + InitAtapiIoPortRegisters(AtapiScsiPrivate, IdeRegsBaseAddr); + // initialize SCSI Pass Thru Protocol interface // AtapiScsiPrivate->ScsiPassThru.Mode = &AtapiScsiPrivate->ScsiPassThruMode; @@ -364,9 +419,9 @@ RegisterAtapiScsiPassThru ( AtapiScsiPrivate->ScsiPassThruMode.IoAlign = 0; // - // Initialize the LatestTargetId to 0xFFFFFFFF (for the GetNextDevice() call). + // Initialize the LatestTargetId. // - AtapiScsiPrivate->LatestTargetId = 0xFFFFFFFF; + AtapiScsiPrivate->LatestTargetId = 4; AtapiScsiPrivate->LatestLun = 0; Status = gBS->InstallProtocolInterface ( @@ -411,15 +466,15 @@ AtapiScsiPassThruFunction ( IN EFI_EVENT Event OPTIONAL ) { - ATAPI_SCSI_PASS_THRU_DEV *AtapiScsiPrivate; - EFI_STATUS Status; + ATAPI_SCSI_PASS_THRU_DEV *AtapiScsiPrivate; + EFI_STATUS Status; AtapiScsiPrivate = ATAPI_SCSI_PASS_THRU_DEV_FROM_THIS (This); // // Target is not allowed beyond MAX_TARGET_ID // - if (Target > MAX_TARGET_ID) { + if ((Target > MAX_TARGET_ID) || (Lun != 0)) { return EFI_INVALID_PARAMETER; } @@ -442,13 +497,15 @@ AtapiScsiPassThruFunction ( // // According to Target ID, reset the Atapi I/O Register mapping - // (Target Id in [0,1] area, using gAtapiIoPortRegisters[0], - // Target Id in [2,3] area, using gAtapiIoPortRegisters[1] + // (Target Id in [0,1] area, using AtapiIoPortRegisters[0], + // Target Id in [2,3] area, using AtapiIoPortRegisters[1] // if ((Target / 2) == 0) { - AtapiScsiPrivate->IoPort = &gAtapiIoPortRegisters[0]; + Target = Target % 2; + AtapiScsiPrivate->IoPort = &AtapiScsiPrivate->AtapiIoPortRegisters[0]; } else { - AtapiScsiPrivate->IoPort = &gAtapiIoPortRegisters[1]; + Target = Target % 2; + AtapiScsiPrivate->IoPort = &AtapiScsiPrivate->AtapiIoPortRegisters[1]; } // @@ -572,6 +629,7 @@ AtapiScsiPassThruBuildDevicePath ( { EFI_DEV_PATH *Node; + // // Validate parameters passed in. // @@ -691,8 +749,10 @@ AtapiScsiPassThruResetChannel ( UINT8 DeviceControlValue; ATAPI_SCSI_PASS_THRU_DEV *AtapiScsiPrivate; UINT8 Index; + BOOLEAN ResetFlag; AtapiScsiPrivate = ATAPI_SCSI_PASS_THRU_DEV_FROM_THIS (This); + ResetFlag = FALSE; // // Reset both Primary channel and Secondary channel. @@ -702,7 +762,7 @@ AtapiScsiPassThruResetChannel ( // // Reset // - AtapiScsiPrivate->IoPort = &gAtapiIoPortRegisters[Index]; + AtapiScsiPrivate->IoPort = &AtapiScsiPrivate->AtapiIoPortRegisters[Index]; DeviceControlValue = 0; // @@ -735,12 +795,16 @@ AtapiScsiPassThruResetChannel ( // // slave device needs at most 31s to clear BSY // - if (StatusWaitForBSYClear (AtapiScsiPrivate, 31000) == EFI_TIMEOUT) { - return EFI_DEVICE_ERROR; + if (StatusWaitForBSYClear (AtapiScsiPrivate, 31000000) != EFI_TIMEOUT) { + ResetFlag = TRUE; } } - return EFI_SUCCESS; + if (ResetFlag) { + return EFI_SUCCESS; + } + + return EFI_TIMEOUT; } /** @@ -776,7 +840,7 @@ AtapiScsiPassThruResetTarget ( AtapiScsiPrivate = ATAPI_SCSI_PASS_THRU_DEV_FROM_THIS (This); - if (Target > MAX_TARGET_ID) { + if ((Target > MAX_TARGET_ID) || (Lun != 0)) { return EFI_INVALID_PARAMETER; } // @@ -788,13 +852,13 @@ AtapiScsiPassThruResetTarget ( // // According to Target ID, reset the Atapi I/O Register mapping - // (Target Id in [0,1] area, using gAtapiIoPortRegisters[0], - // Target Id in [2,3] area, using gAtapiIoPortRegisters[1] + // (Target Id in [0,1] area, using AtapiIoPortRegisters[0], + // Target Id in [2,3] area, using AtapiIoPortRegisters[1] // if ((Target / 2) == 0) { - AtapiScsiPrivate->IoPort = &gAtapiIoPortRegisters[0]; + AtapiScsiPrivate->IoPort = &AtapiScsiPrivate->AtapiIoPortRegisters[0]; } else { - AtapiScsiPrivate->IoPort = &gAtapiIoPortRegisters[1]; + AtapiScsiPrivate->IoPort = &AtapiScsiPrivate->AtapiIoPortRegisters[1]; } // @@ -813,8 +877,8 @@ AtapiScsiPassThruResetTarget ( // when reset is complete. // slave device needs at most 31s to clear BSY // - if (EFI_ERROR (StatusWaitForBSYClear (AtapiScsiPrivate, 31000))) { - return EFI_DEVICE_ERROR; + if (EFI_ERROR (StatusWaitForBSYClear (AtapiScsiPrivate, 31000000))) { + return EFI_TIMEOUT; } // @@ -825,25 +889,159 @@ AtapiScsiPassThruResetTarget ( return EFI_SUCCESS; } +EFI_STATUS +GetIdeRegistersBaseAddr ( + IN EFI_PCI_IO_PROTOCOL *PciIo, + OUT IDE_REGISTERS_BASE_ADDR *IdeRegsBaseAddr + ) +/*++ + +Routine Description: + Get IDE IO port registers' base addresses by mode. In 'Compatibility' mode, + use fixed addresses. In Native-PCI mode, get base addresses from BARs in + the PCI IDE controller's Configuration Space. + +Arguments: + PciIo - Pointer to the EFI_PCI_IO_PROTOCOL instance + IdeRegsBaseAddr - Pointer to IDE_REGISTERS_BASE_ADDR to + receive IDE IO port registers' base addresses + +Returns: + + EFI_STATUS -/** - Checks the parameters in the SCSI Request Packet to make sure - they are valid for a SCSI Pass Thru request. +--*/ +{ + EFI_STATUS Status; + PCI_TYPE00 PciData; - @todo function comment is missing 'Routine Description:' - @todo function comment is missing 'Arguments:' - @todo function comment is missing 'Returns:' - @todo Packet - add argument and description to function comment - @todo EFI_INVALID_PARAMETER - add return value to function comment - @todo EFI_INVALID_PARAMETER - add return value to function comment - @todo EFI_INVALID_PARAMETER - add return value to function comment - @todo EFI_UNSUPPORTED - add return value to function comment - @todo EFI_SUCCESS - add return value to function comment -**/ + Status = PciIo->Pci.Read ( + PciIo, + EfiPciIoWidthUint8, + 0, + sizeof (PciData), + &PciData + ); + + if (EFI_ERROR (Status)) { + return Status; + } + + if ((PciData.Hdr.ClassCode[0] & IDE_PRIMARY_OPERATING_MODE) == 0) { + IdeRegsBaseAddr[IdePrimary].CommandBlockBaseAddr = 0x1f0; + IdeRegsBaseAddr[IdePrimary].ControlBlockBaseAddr = 0x3f6; + } else { + // + // The BARs should be of IO type + // + if ((PciData.Device.Bar[0] & BIT0) == 0 || + (PciData.Device.Bar[1] & BIT0) == 0) { + return EFI_UNSUPPORTED; + } + + IdeRegsBaseAddr[IdePrimary].CommandBlockBaseAddr = + (UINT16) (PciData.Device.Bar[0] & 0x0000fff8); + IdeRegsBaseAddr[IdePrimary].ControlBlockBaseAddr = + (UINT16) ((PciData.Device.Bar[1] & 0x0000fffc) + 2); + } + + if ((PciData.Hdr.ClassCode[0] & IDE_SECONDARY_OPERATING_MODE) == 0) { + IdeRegsBaseAddr[IdeSecondary].CommandBlockBaseAddr = 0x170; + IdeRegsBaseAddr[IdeSecondary].ControlBlockBaseAddr = 0x376; + } else { + // + // The BARs should be of IO type + // + if ((PciData.Device.Bar[2] & BIT0) == 0 || + (PciData.Device.Bar[3] & BIT0) == 0) { + return EFI_UNSUPPORTED; + } + + IdeRegsBaseAddr[IdeSecondary].CommandBlockBaseAddr = + (UINT16) (PciData.Device.Bar[2] & 0x0000fff8); + IdeRegsBaseAddr[IdeSecondary].ControlBlockBaseAddr = + (UINT16) ((PciData.Device.Bar[3] & 0x0000fffc) + 2); + } + + return EFI_SUCCESS; +} + +VOID +InitAtapiIoPortRegisters ( + IN ATAPI_SCSI_PASS_THRU_DEV *AtapiScsiPrivate, + IN IDE_REGISTERS_BASE_ADDR *IdeRegsBaseAddr + ) +/*++ + +Routine Description: + + Initialize each Channel's Base Address of CommandBlock and ControlBlock. + +Arguments: + + AtapiScsiPrivate - The pointer of ATAPI_SCSI_PASS_THRU_DEV + IdeRegsBaseAddr - The pointer of IDE_REGISTERS_BASE_ADDR + +Returns: + + None + +--*/ +{ + + UINT8 IdeChannel; + UINT16 CommandBlockBaseAddr; + UINT16 ControlBlockBaseAddr; + IDE_BASE_REGISTERS *RegisterPointer; + + + for (IdeChannel = 0; IdeChannel < ATAPI_MAX_CHANNEL; IdeChannel++) { + + RegisterPointer = &AtapiScsiPrivate->AtapiIoPortRegisters[IdeChannel]; + + // + // Initialize IDE IO port addresses, including Command Block registers + // and Control Block registers + // + CommandBlockBaseAddr = IdeRegsBaseAddr[IdeChannel].CommandBlockBaseAddr; + ControlBlockBaseAddr = IdeRegsBaseAddr[IdeChannel].ControlBlockBaseAddr; + + RegisterPointer->Data = CommandBlockBaseAddr; + (*(UINT16 *) &RegisterPointer->Reg1) = (UINT16) (CommandBlockBaseAddr + 0x01); + RegisterPointer->SectorCount = (UINT16) (CommandBlockBaseAddr + 0x02); + RegisterPointer->SectorNumber = (UINT16) (CommandBlockBaseAddr + 0x03); + RegisterPointer->CylinderLsb = (UINT16) (CommandBlockBaseAddr + 0x04); + RegisterPointer->CylinderMsb = (UINT16) (CommandBlockBaseAddr + 0x05); + RegisterPointer->Head = (UINT16) (CommandBlockBaseAddr + 0x06); + (*(UINT16 *) &RegisterPointer->Reg) = (UINT16) (CommandBlockBaseAddr + 0x07); + + (*(UINT16 *) &RegisterPointer->Alt) = ControlBlockBaseAddr; + RegisterPointer->DriveAddress = (UINT16) (ControlBlockBaseAddr + 0x01); + } + +} + + EFI_STATUS CheckSCSIRequestPacket ( EFI_SCSI_PASS_THRU_SCSI_REQUEST_PACKET *Packet ) +/*++ + +Routine Description: + + Checks the parameters in the SCSI Request Packet to make sure + they are valid for a SCSI Pass Thru request. + +Arguments: + + Packet - The pointer of EFI_SCSI_PASS_THRU_SCSI_REQUEST_PACKET + +Returns: + + EFI_STATUS + +--*/ { if (Packet == NULL) { return EFI_INVALID_PARAMETER; @@ -971,11 +1169,23 @@ SubmitBlockingIoCommand ( Packet->SenseDataLength = 0; return PacketCommandStatus; } + + // + // Check if SenseData meets the alignment requirement. + // + if ((AtapiScsiPrivate->ScsiPassThru.Mode->IoAlign != 0) \ + && (AtapiScsiPrivate->ScsiPassThru.Mode->IoAlign != 1)) { + if (((UINTN)Packet->SenseData % AtapiScsiPrivate->ScsiPassThru.Mode->IoAlign) != 0) { + return EFI_INVALID_PARAMETER; + } + } + + // // Return SenseData if PacketCommandStatus matches // the following return codes. // - if ((PacketCommandStatus == EFI_WARN_BUFFER_TOO_SMALL) || + if ((PacketCommandStatus == EFI_BAD_BUFFER_SIZE) || (PacketCommandStatus == EFI_DEVICE_ERROR) || (PacketCommandStatus == EFI_TIMEOUT)) { @@ -1096,18 +1306,25 @@ AtapiPacketCommand ( EFI_STATUS Status; // - // Set all the command parameters by fill related registers. - // Before write to all the following registers, BSY and DRQ must be 0. + // Check if the buffer meets the alignment requirement. // - Status = StatusDRQClear (AtapiScsiPrivate, TimeoutInMicroSeconds); - if (EFI_ERROR (Status)) { - if (Status == EFI_ABORTED) { - Status = EFI_DEVICE_ERROR; + if ((AtapiScsiPrivate->ScsiPassThru.Mode->IoAlign != 0) \ + && (AtapiScsiPrivate->ScsiPassThru.Mode->IoAlign != 1)) { + if (((UINTN)Buffer % AtapiScsiPrivate->ScsiPassThru.Mode->IoAlign) != 0) { + return EFI_INVALID_PARAMETER; } + } - *ByteCount = 0; - return Status; + // + // Set all the command parameters by fill related registers. + // Before write to all the following registers, BSY must be 0. + // + Status = StatusWaitForBSYClear (AtapiScsiPrivate, TimeoutInMicroSeconds); + if (EFI_ERROR (Status)) { + return EFI_DEVICE_ERROR; } + + // // Select device via Device/Head Register. // "Target = 0" indicates device 0; "Target = 1" indicates device 1 @@ -1118,6 +1335,20 @@ AtapiPacketCommand ( (UINT8) ((Target << 4) | DEFAULT_CMD) // DEFAULT_CMD: 0xa0 (1010,0000) ); + // + // Set all the command parameters by fill related registers. + // Before write to all the following registers, BSY DRQ must be 0. + // + Status = StatusDRQClear(AtapiScsiPrivate, TimeoutInMicroSeconds); + + if (EFI_ERROR (Status)) { + if (Status == EFI_ABORTED) { + Status = EFI_DEVICE_ERROR; + } + *ByteCount = 0; + return Status; + } + // // No OVL; No DMA (by setting feature register) // @@ -1282,7 +1513,7 @@ AtapiPassThruPioReadWriteData ( // ActualWordCount > 0 // if (ActualWordCount < RequiredWordCount) { - return EFI_WARN_BUFFER_TOO_SMALL; + return EFI_BAD_BUFFER_SIZE; } } // diff --git a/MdeModulePkg/Bus/Pci/AtapiPassThruDxe/AtapiPassThru.h b/MdeModulePkg/Bus/Pci/AtapiPassThruDxe/AtapiPassThru.h index d61c27bd11..0cb169786a 100644 --- a/MdeModulePkg/Bus/Pci/AtapiPassThruDxe/AtapiPassThru.h +++ b/MdeModulePkg/Bus/Pci/AtapiPassThruDxe/AtapiPassThru.h @@ -56,6 +56,28 @@ typedef union { UINT16 DeviceControl; /* when write */ } IDE_AltStatus_OR_DeviceControl; + +typedef enum { + IdePrimary = 0, + IdeSecondary = 1, + IdeMaxChannel = 2 +} EFI_IDE_CHANNEL; + +/// + + +// +// Bit definitions in Programming Interface byte of the Class Code field +// in PCI IDE controller's Configuration Space +// +#define IDE_PRIMARY_OPERATING_MODE BIT0 +#define IDE_PRIMARY_PROGRAMMABLE_INDICATOR BIT1 +#define IDE_SECONDARY_OPERATING_MODE BIT2 +#define IDE_SECONDARY_PROGRAMMABLE_INDICATOR BIT3 + + +#define ATAPI_MAX_CHANNEL 2 + /// /// IDE registers set /// @@ -68,36 +90,37 @@ typedef struct { UINT16 CylinderMsb; UINT16 Head; IDE_CMD_OR_STATUS Reg; - IDE_AltStatus_OR_DeviceControl Alt; UINT16 DriveAddress; - - UINT16 MasterSlave; } IDE_BASE_REGISTERS; #define ATAPI_SCSI_PASS_THRU_DEV_SIGNATURE EFI_SIGNATURE_32 ('a', 's', 'p', 't') typedef struct { - UINTN Signature; - - EFI_HANDLE Handle; - EFI_SCSI_PASS_THRU_PROTOCOL ScsiPassThru; - EFI_SCSI_PASS_THRU_MODE ScsiPassThruMode; - EFI_PCI_IO_PROTOCOL *PciIo; - + UINTN Signature; + EFI_HANDLE Handle; + EFI_SCSI_PASS_THRU_PROTOCOL ScsiPassThru; + EFI_SCSI_PASS_THRU_MODE ScsiPassThruMode; + EFI_PCI_IO_PROTOCOL *PciIo; // // Local Data goes here // - IDE_BASE_REGISTERS *IoPort; - - CHAR16 ControllerName[100]; - CHAR16 ChannelName[100]; - - UINT32 LatestTargetId; - UINT64 LatestLun; - + IDE_BASE_REGISTERS *IoPort; + IDE_BASE_REGISTERS AtapiIoPortRegisters[2]; + CHAR16 ControllerName[100]; + CHAR16 ChannelName[100]; + UINT32 LatestTargetId; + UINT64 LatestLun; } ATAPI_SCSI_PASS_THRU_DEV; +// +// IDE registers' base addresses +// +typedef struct { + UINT16 CommandBlockBaseAddr; + UINT16 ControlBlockBaseAddr; +} IDE_REGISTERS_BASE_ADDR; + #define ATAPI_SCSI_PASS_THRU_DEV_FROM_THIS(a) \ CR (a, \ ATAPI_SCSI_PASS_THRU_DEV, \ @@ -824,4 +847,51 @@ AtapiPassThruCheckErrorStatus ( ATAPI_SCSI_PASS_THRU_DEV *AtapiScsiPrivate ) ; +EFI_STATUS +GetIdeRegistersBaseAddr ( + IN EFI_PCI_IO_PROTOCOL *PciIo, + OUT IDE_REGISTERS_BASE_ADDR *IdeRegsBaseAddr + ) +/*++ + +Routine Description: + Get IDE IO port registers' base addresses by mode. In 'Compatibility' mode, + use fixed addresses. In Native-PCI mode, get base addresses from BARs in + the PCI IDE controller's Configuration Space. + +Arguments: + PciIo - Pointer to the EFI_PCI_IO_PROTOCOL instance + IdeRegsBaseAddr - Pointer to IDE_REGISTERS_BASE_ADDR to + receive IDE IO port registers' base addresses + +Returns: + + EFI_STATUS + +--*/ +; + +VOID +InitAtapiIoPortRegisters ( + IN ATAPI_SCSI_PASS_THRU_DEV *AtapiScsiPrivate, + IN IDE_REGISTERS_BASE_ADDR *IdeRegsBaseAddr + ) +/*++ + +Routine Description: + + Initialize each Channel's Base Address of CommandBlock and ControlBlock. + +Arguments: + + AtapiScsiPrivate - The pointer of ATAPI_SCSI_PASS_THRU_DEV + IdeRegsBaseAddr - The pointer of IDE_REGISTERS_BASE_ADDR + +Returns: + + None + +--*/ +; + #endif diff --git a/MdeModulePkg/Bus/Scsi/ScsiBusDxe/ScsiBus.c b/MdeModulePkg/Bus/Scsi/ScsiBusDxe/ScsiBus.c index 1110310f92..2ad32d2b08 100644 --- a/MdeModulePkg/Bus/Scsi/ScsiBusDxe/ScsiBus.c +++ b/MdeModulePkg/Bus/Scsi/ScsiBusDxe/ScsiBus.c @@ -120,7 +120,6 @@ InitializeScsiBus( ); ASSERT_EFI_ERROR (Status); - return Status; } @@ -231,124 +230,156 @@ Returns: EFI_DEVICE_ERROR - This driver cannot be started due to device Error --*/ -// TODO: This - add argument and description to function comment -// TODO: Controller - add argument and description to function comment -// TODO: RemainingDevicePath - add argument and description to function comment { - EFI_STATUS Status; - UINT64 Lun; - BOOLEAN ScanOtherPuns; - SCSI_BUS_DEVICE *ScsiBusDev; - BOOLEAN FromFirstTarget; - SCSI_TARGET_ID *ScsiTargetId; - UINT8 *TargetId; - - TargetId = NULL; - ScanOtherPuns = TRUE; + UINT64 Lun; + UINT8 *TargetId; + BOOLEAN ScanOtherPuns; + BOOLEAN FromFirstTarget; + BOOLEAN ExtScsiSupport; + EFI_STATUS Status; + EFI_STATUS DevicePathStatus; + EFI_STATUS PassThruStatus; + SCSI_BUS_DEVICE *ScsiBusDev; + SCSI_TARGET_ID *ScsiTargetId; + EFI_DEVICE_PATH_PROTOCOL *ParentDevicePath; + EFI_SCSI_PASS_THRU_PROTOCOL *ScsiInterface; + EFI_EXT_SCSI_PASS_THRU_PROTOCOL *ExtScsiInterface; + EFI_SCSI_BUS_PROTOCOL *BusIdentify; + + TargetId = NULL; + ScsiTargetId = NULL; + ScanOtherPuns = TRUE; FromFirstTarget = FALSE; - // - // Allocate SCSI_BUS_DEVICE structure - // - ScsiBusDev = NULL; - ScsiBusDev = AllocateZeroPool (sizeof (SCSI_BUS_DEVICE)); - if (ScsiBusDev == NULL) { - return EFI_OUT_OF_RESOURCES; - } - - ScsiTargetId = NULL; - ScsiTargetId = AllocateZeroPool (sizeof (SCSI_TARGET_ID)); + ExtScsiSupport = FALSE; + PassThruStatus = EFI_SUCCESS; + + ScsiTargetId = AllocateZeroPool(sizeof(SCSI_TARGET_ID)); if (ScsiTargetId == NULL) { return EFI_OUT_OF_RESOURCES; } TargetId = &ScsiTargetId->ScsiId.ExtScsi[0]; - - Status = gBS->OpenProtocol ( - Controller, - &gEfiDevicePathProtocolGuid, - (VOID **) &(ScsiBusDev->DevicePath), - This->DriverBindingHandle, - Controller, - EFI_OPEN_PROTOCOL_BY_DRIVER - ); - if (EFI_ERROR (Status) && Status != EFI_ALREADY_STARTED) { - gBS->FreePool (ScsiBusDev); - return Status; + + DevicePathStatus = gBS->OpenProtocol ( + Controller, + &gEfiDevicePathProtocolGuid, + (VOID **) &ParentDevicePath, + This->DriverBindingHandle, + Controller, + EFI_OPEN_PROTOCOL_BY_DRIVER + ); + if (EFI_ERROR (DevicePathStatus) && (DevicePathStatus != EFI_ALREADY_STARTED)) { + return DevicePathStatus; } // - // First consume Extended SCSI Pass Thru protocol, if fail, then consume - // SCSI Pass Thru protocol + // To keep backward compatibility, UEFI ExtPassThru Protocol is supported as well as + // EFI PassThru Protocol. From priority perspective, ExtPassThru Protocol is firstly + // tried to open on host controller handle. If fails, then PassThru Protocol is tried instead. // Status = gBS->OpenProtocol ( Controller, &gEfiExtScsiPassThruProtocolGuid, - (VOID **) &(ScsiBusDev->ExtScsiInterface), + (VOID **) &ExtScsiInterface, This->DriverBindingHandle, Controller, EFI_OPEN_PROTOCOL_BY_DRIVER ); - if (EFI_ERROR (Status) && Status != EFI_ALREADY_STARTED) { + // + // Fail to open UEFI ExtendPassThru Protocol, then try to open EFI PassThru Protocol instead. + // + if (EFI_ERROR(Status) && (Status != EFI_ALREADY_STARTED)) { Status = gBS->OpenProtocol ( Controller, &gEfiScsiPassThruProtocolGuid, - (VOID **) &(ScsiBusDev->ScsiInterface), + (VOID **) &ScsiInterface, This->DriverBindingHandle, Controller, EFI_OPEN_PROTOCOL_BY_DRIVER ); - if (EFI_ERROR (Status) && Status != EFI_ALREADY_STARTED) { - gBS->CloseProtocol ( - Controller, - &gEfiDevicePathProtocolGuid, - This->DriverBindingHandle, - Controller - ); - gBS->FreePool (ScsiBusDev); + // + // Fail to open EFI PassThru Protocol, Close the DevicePathProtocol if it is opened by this time. + // + if (EFI_ERROR (Status) && (Status != EFI_ALREADY_STARTED)) { + if (!EFI_ERROR(DevicePathStatus)) { + gBS->CloseProtocol ( + Controller, + &gEfiDevicePathProtocolGuid, + This->DriverBindingHandle, + Controller + ); + } return Status; - } - DEBUG ((EFI_D_INFO, "Open Scsi Pass Thrugh Protocol\n")); - ScsiBusDev->ExtScsiSupport = FALSE; + } } else { - DEBUG ((EFI_D_INFO, "Open Extended Scsi Pass Thrugh Protocol\n")); - ScsiBusDev->ExtScsiSupport = TRUE; + // + // Succeed to open ExtPassThru Protocol, and meanwhile open PassThru Protocol + // with BY_DRIVER if it is also present on the handle. The intent is to prevent + // another SCSI Bus Driver to work on the same host handle. + // + ExtScsiSupport = TRUE; + PassThruStatus = gBS->OpenProtocol ( + Controller, + &gEfiScsiPassThruProtocolGuid, + (VOID **) &ScsiInterface, + This->DriverBindingHandle, + Controller, + EFI_OPEN_PROTOCOL_BY_DRIVER + ); } - - ScsiBusDev->Signature = SCSI_BUS_DEVICE_SIGNATURE; - // - // Attach EFI_SCSI_BUS_PROTOCOL to controller handle - // - Status = gBS->InstallProtocolInterface ( - &Controller, - &mScsiBusProtocolGuid, - EFI_NATIVE_INTERFACE, - &ScsiBusDev->BusIdentify - ); - - if (EFI_ERROR (Status)) { - gBS->CloseProtocol ( - Controller, - &gEfiDevicePathProtocolGuid, - This->DriverBindingHandle, - Controller - ); + + if (Status != EFI_ALREADY_STARTED) { + // + // Go through here means either ExtPassThru or PassThru Protocol is successfully opened + // on this handle for this time. Then construct Host controller private data. + // + ScsiBusDev = NULL; + ScsiBusDev = AllocateZeroPool(sizeof(SCSI_BUS_DEVICE)); + if (ScsiBusDev == NULL) { + Status = EFI_OUT_OF_RESOURCES; + goto ErrorExit; + } + ScsiBusDev->Signature = SCSI_BUS_DEVICE_SIGNATURE; + ScsiBusDev->ExtScsiSupport = ExtScsiSupport; + ScsiBusDev->DevicePath = ParentDevicePath; if (ScsiBusDev->ExtScsiSupport) { - gBS->CloseProtocol ( - Controller, - &gEfiExtScsiPassThruProtocolGuid, - This->DriverBindingHandle, - Controller - ); + ScsiBusDev->ExtScsiInterface = ExtScsiInterface; } else { - gBS->CloseProtocol ( - Controller, - &gEfiScsiPassThruProtocolGuid, - This->DriverBindingHandle, - Controller - ); + ScsiBusDev->ScsiInterface = ScsiInterface; } - gBS->FreePool (ScsiBusDev); - return Status; + + // + // Install EFI_SCSI_BUS_PROTOCOL to the controller handle, So ScsiBusDev could be + // retrieved on this controller handle. With ScsiBusDev, we can know which PassThru + // Protocol is present on the handle, UEFI ExtPassThru Protocol or EFI PassThru Protocol. + // + Status = gBS->InstallProtocolInterface ( + &Controller, + &mScsiBusProtocolGuid, + EFI_NATIVE_INTERFACE, + &ScsiBusDev->BusIdentify + ); + if (EFI_ERROR (Status)) { + goto ErrorExit; + } + } else { + // + // Go through here means Start() is re-invoked again, nothing special is required to do except + // picking up Host controller private information. + // + Status = gBS->OpenProtocol ( + Controller, + &mScsiBusProtocolGuid, + (VOID **) &BusIdentify, + This->DriverBindingHandle, + Controller, + EFI_OPEN_PROTOCOL_GET_PROTOCOL + ); + + if (EFI_ERROR (Status)) { + return Status; + } + ScsiBusDev = SCSI_BUS_CONTROLLER_DEVICE_FROM_THIS (BusIdentify); } if (RemainingDevicePath == NULL) { @@ -357,7 +388,7 @@ Returns: FromFirstTarget = TRUE; } else { if (ScsiBusDev->ExtScsiSupport) { - ScsiBusDev->ExtScsiInterface->GetTargetLun (ScsiBusDev->ExtScsiInterface, RemainingDevicePath, &TargetId, &Lun); + ScsiBusDev->ExtScsiInterface->GetTargetLun (ScsiBusDev->ExtScsiInterface, RemainingDevicePath, &TargetId, &Lun); } else { ScsiBusDev->ScsiInterface->GetTargetLun (ScsiBusDev->ScsiInterface, RemainingDevicePath, &ScsiTargetId->ScsiId.Scsi, &Lun); } @@ -401,6 +432,38 @@ Returns: // Status = ScsiScanCreateDevice (This, Controller, ScsiTargetId, Lun, ScsiBusDev); } + gBS->FreePool (ScsiTargetId); + return EFI_SUCCESS; + +ErrorExit: + + if (ScsiBusDev != NULL) { + gBS->FreePool (ScsiBusDev); + } + + if (ExtScsiSupport) { + gBS->CloseProtocol ( + Controller, + &gEfiExtScsiPassThruProtocolGuid, + This->DriverBindingHandle, + Controller + ); + if (!EFI_ERROR (PassThruStatus)) { + gBS->CloseProtocol ( + Controller, + &gEfiScsiPassThruProtocolGuid, + This->DriverBindingHandle, + Controller + ); + } + } else { + gBS->CloseProtocol ( + Controller, + &gEfiScsiPassThruProtocolGuid, + This->DriverBindingHandle, + Controller + ); + } return Status; } @@ -414,20 +477,23 @@ SCSIBusDriverBindingStop ( ) /*++ - Routine Description: +Routine Description: - Arguments: + Stop this driver on ControllerHandle. Support stoping any child handles + created by this driver. - Returns: +Arguments: + This - Protocol instance pointer. + Controller - Handle of device to stop driver on + NumberOfChildren - Number of Children in the ChildHandleBuffer + ChildHandleBuffer - List of handles for the children we need to stop. + +Returns: + + EFI_SUCCESS + Others --*/ -// TODO: This - add argument and description to function comment -// TODO: Controller - add argument and description to function comment -// TODO: NumberOfChildren - add argument and description to function comment -// TODO: ChildHandleBuffer - add argument and description to function comment -// TODO: EFI_SUCCESS - add return value to function comment -// TODO: EFI_DEVICE_ERROR - add return value to function comment -// TODO: EFI_SUCCESS - add return value to function comment { EFI_STATUS Status; BOOLEAN AllChildrenStopped; @@ -582,17 +648,21 @@ ScsiGetDeviceType ( ) /*++ - Routine Description: - Retrieves the device type information of the SCSI Controller. +Routine Description: - Arguments: - This - Protocol instance pointer. - DeviceType - A pointer to the device type information - retrieved from the SCSI Controller. + Retrieves the device type information of the SCSI Controller. + +Arguments: - Returns: - EFI_SUCCESS - Retrieves the device type information successfully. - EFI_INVALID_PARAMETER - The DeviceType is NULL. + This - Protocol instance pointer. + DeviceType - A pointer to the device type information + retrieved from the SCSI Controller. + +Returns: + + EFI_SUCCESS - Retrieves the device type information successfully. + EFI_INVALID_PARAMETER - The DeviceType is NULL. + --*/ { SCSI_IO_DEV *ScsiIoDevice; @@ -614,19 +684,24 @@ ScsiGetDeviceLocation ( OUT UINT64 *Lun ) /*++ - Routine Description: - Retrieves the device location in the SCSI channel. - - Arguments: - This - Protocol instance pointer. - Target - A pointer to the Target ID of a SCSI device - on the SCSI channel. - Lun - A pointer to the LUN of the SCSI device on - the SCSI channel. - - Returns: - EFI_SUCCESS - Retrieves the device location successfully. - EFI_INVALID_PARAMETER - The Target or Lun is NULL. + +Routine Description: + + Retrieves the device location in the SCSI channel. + +Arguments: + + This - Protocol instance pointer. + Target - A pointer to the Target Array which represents ID of a SCSI device + on the SCSI channel. + Lun - A pointer to the LUN of the SCSI device on + the SCSI channel. + +Returns: + + EFI_SUCCESS - Retrieves the device location successfully. + EFI_INVALID_PARAMETER - The Target or Lun is NULL. + --*/ { SCSI_IO_DEV *ScsiIoDevice; @@ -635,11 +710,11 @@ ScsiGetDeviceLocation ( return EFI_INVALID_PARAMETER; } - ScsiIoDevice = SCSI_IO_DEV_FROM_THIS (This); + ScsiIoDevice = SCSI_IO_DEV_FROM_THIS (This); CopyMem (*Target,&ScsiIoDevice->Pun, TARGET_MAX_BYTES); - *Lun = ScsiIoDevice->Lun; + *Lun = ScsiIoDevice->Lun; return EFI_SUCCESS; } @@ -651,19 +726,22 @@ ScsiResetBus ( ) /*++ - Routine Description: - Resets the SCSI Bus that the SCSI Controller is attached to. +Routine Description: - Arguments: - This - Protocol instance pointer. + Resets the SCSI Bus that the SCSI Controller is attached to. + +Arguments: - Returns: - EFI_SUCCESS - The SCSI bus is reset successfully. - EFI_DEVICE_ERROR - Errors encountered when resetting the SCSI bus. - EFI_UNSUPPORTED - The bus reset operation is not supported by the - SCSI Host Controller. - EFI_TIMEOUT - A timeout occurred while attempting to reset - the SCSI bus. + This - Protocol instance pointer. + +Returns: + + EFI_SUCCESS - The SCSI bus is reset successfully. + EFI_DEVICE_ERROR - Errors encountered when resetting the SCSI bus. + EFI_UNSUPPORTED - The bus reset operation is not supported by the + SCSI Host Controller. + EFI_TIMEOUT - A timeout occurred while attempting to reset + the SCSI bus. --*/ { SCSI_IO_DEV *ScsiIoDevice; @@ -684,24 +762,26 @@ ScsiResetDevice ( ) /*++ - Routine Description: - Resets the SCSI Controller that the device handle specifies. +Routine Description: - Arguments: - This - Protocol instance pointer. + Resets the SCSI Controller that the device handle specifies. + +Arguments: + This - Protocol instance pointer. + +Returns: - Returns: - EFI_SUCCESS - Reset the SCSI controller successfully. - EFI_DEVICE_ERROR - Errors are encountered when resetting the - SCSI Controller. - EFI_UNSUPPORTED - The SCSI bus does not support a device - reset operation. - EFI_TIMEOUT - A timeout occurred while attempting to - reset the SCSI Controller. + EFI_SUCCESS - Reset the SCSI controller successfully. + EFI_DEVICE_ERROR - Errors are encountered when resetting the + SCSI Controller. + EFI_UNSUPPORTED - The SCSI bus does not support a device + reset operation. + EFI_TIMEOUT - A timeout occurred while attempting to + reset the SCSI Controller. --*/ { - SCSI_IO_DEV *ScsiIoDevice; + SCSI_IO_DEV *ScsiIoDevice; UINT8 Target[TARGET_MAX_BYTES]; ScsiIoDevice = SCSI_IO_DEV_FROM_THIS (This); @@ -732,64 +812,67 @@ ScsiExecuteSCSICommand ( ) /*++ - Routine Description: - Sends a SCSI Request Packet to the SCSI Controller for execution. - - Arguments: - This - Protocol instance pointer. - Packet - The SCSI request packet to send to the SCSI - Controller specified by the device handle. - Event - If the SCSI bus where the SCSI device is attached - does not support non-blocking I/O, then Event is - ignored, and blocking I/O is performed. - If Event is NULL, then blocking I/O is performed. - If Event is not NULL and non-blocking I/O is - supported, then non-blocking I/O is performed, - and Event will be signaled when the SCSI Request - Packet completes. - Returns: - EFI_SUCCESS - The SCSI Request Packet was sent by the host - successfully, and TransferLength bytes were - transferred to/from DataBuffer.See - HostAdapterStatus, TargetStatus, - SenseDataLength, and SenseData in that order - for additional status information. - EFI_WARN_BUFFER_TOO_SMALL - The SCSI Request Packet was executed, - but the entire DataBuffer could not be transferred. - The actual number of bytes transferred is returned - in TransferLength. See HostAdapterStatus, - TargetStatus, SenseDataLength, and SenseData in - that order for additional status information. - EFI_NOT_READY - The SCSI Request Packet could not be sent because - there are too many SCSI Command Packets already - queued.The caller may retry again later. - EFI_DEVICE_ERROR - A device error occurred while attempting to send - the SCSI Request Packet. See HostAdapterStatus, - TargetStatus, SenseDataLength, and SenseData in - that order for additional status information. - EFI_INVALID_PARAMETER - The contents of CommandPacket are invalid. - The SCSI Request Packet was not sent, so no - additional status information is available. - EFI_UNSUPPORTED - The command described by the SCSI Request Packet - is not supported by the SCSI initiator(i.e., SCSI - Host Controller). The SCSI Request Packet was not - sent, so no additional status information is - available. - EFI_TIMEOUT - A timeout occurred while waiting for the SCSI - Request Packet to execute. See HostAdapterStatus, - TargetStatus, SenseDataLength, and SenseData in - that order for additional status information. +Routine Description: + + Sends a SCSI Request Packet to the SCSI Controller for execution. + +Arguments: + + This - Protocol instance pointer. + Packet - The SCSI request packet to send to the SCSI + Controller specified by the device handle. + Event - If the SCSI bus where the SCSI device is attached + does not support non-blocking I/O, then Event is + ignored, and blocking I/O is performed. + If Event is NULL, then blocking I/O is performed. + If Event is not NULL and non-blocking I/O is + supported, then non-blocking I/O is performed, + and Event will be signaled when the SCSI Request + Packet completes. +Returns: + + EFI_SUCCESS - The SCSI Request Packet was sent by the host + successfully, and TransferLength bytes were + transferred to/from DataBuffer.See + HostAdapterStatus, TargetStatus, + SenseDataLength, and SenseData in that order + for additional status information. + EFI_BAD_BUFFER_SIZE - The SCSI Request Packet was executed, + but the entire DataBuffer could not be transferred. + The actual number of bytes transferred is returned + in TransferLength. See HostAdapterStatus, + TargetStatus, SenseDataLength, and SenseData in + that order for additional status information. + EFI_NOT_READY - The SCSI Request Packet could not be sent because + there are too many SCSI Command Packets already + queued.The caller may retry again later. + EFI_DEVICE_ERROR - A device error occurred while attempting to send + the SCSI Request Packet. See HostAdapterStatus, + TargetStatus, SenseDataLength, and SenseData in + that order for additional status information. + EFI_INVALID_PARAMETER - The contents of CommandPacket are invalid. + The SCSI Request Packet was not sent, so no + additional status information is available. + EFI_UNSUPPORTED - The command described by the SCSI Request Packet + is not supported by the SCSI initiator(i.e., SCSI + Host Controller). The SCSI Request Packet was not + sent, so no additional status information is + available. + EFI_TIMEOUT - A timeout occurred while waiting for the SCSI + Request Packet to execute. See HostAdapterStatus, + TargetStatus, SenseDataLength, and SenseData in + that order for additional status information. --*/ { - SCSI_IO_DEV *ScsiIoDevice; - EFI_STATUS Status; + SCSI_IO_DEV *ScsiIoDevice; + EFI_STATUS Status; UINT8 Target[TARGET_MAX_BYTES]; EFI_EVENT PacketEvent; EFI_EXT_SCSI_PASS_THRU_SCSI_REQUEST_PACKET *ExtRequestPacket; - SCSI_EVENT_DATA EventData; + SCSI_EVENT_DATA EventData; PacketEvent = NULL; - + if (Packet == NULL) { return EFI_INVALID_PARAMETER; } diff --git a/MdeModulePkg/Bus/Scsi/ScsiBusDxe/ScsiBus.h b/MdeModulePkg/Bus/Scsi/ScsiBusDxe/ScsiBus.h index be13755707..e6210773ce 100644 --- a/MdeModulePkg/Bus/Scsi/ScsiBusDxe/ScsiBus.h +++ b/MdeModulePkg/Bus/Scsi/ScsiBusDxe/ScsiBus.h @@ -18,9 +18,8 @@ Abstract: Header file for SCSI Bus Driver. Revision History -++*/ +--*/ -// TODO: fix comment to end with --*/ #ifndef _SCSI_BUS_H #define _SCSI_BUS_H @@ -325,77 +324,16 @@ DiscoverScsiDevice ( Routine Description: - TODO: Add function description - -Arguments: - - ScsiIoDevice - TODO: add argument description - -Returns: - - TODO: add return values - ---*/ -; - -EFI_STATUS -GetLunList ( - EFI_SCSI_PASS_THRU_PROTOCOL *ScsiPassThru, - UINT32 Target, - UINT64 **LunArray, - UINTN *NumberOfLuns - ) -/*++ - -Routine Description: - - TODO: Add function description - -Arguments: - - ScsiPassThru - TODO: add argument description - Target - TODO: add argument description - LunArray - TODO: add argument description - NumberOfLuns - TODO: add argument description - -Returns: - - TODO: add return values - ---*/ -; - -EFI_STATUS -ScsiBusSubmitReportLunCommand ( - EFI_SCSI_PASS_THRU_PROTOCOL *ScsiPassThru, - UINT32 Target, - UINTN AllocationLength, - VOID *Buffer, - EFI_SCSI_SENSE_DATA *SenseData, - UINT8 *SenseDataLength, - UINT8 *HostAdapterStatus, - UINT8 *TargetStatus - ) -/*++ - -Routine Description: - - TODO: Add function description + Discovery SCSI Device Arguments: - ScsiPassThru - TODO: add argument description - Target - TODO: add argument description - AllocationLength - TODO: add argument description - Buffer - TODO: add argument description - SenseData - TODO: add argument description - SenseDataLength - TODO: add argument description - HostAdapterStatus - TODO: add argument description - TargetStatus - TODO: add argument description + ScsiIoDevice - The pointer of SCSI_IO_DEV Returns: - TODO: add return values + TRUE - Find SCSI Device and verify it. + FALSE - Unable to find SCSI Device. --*/ ; diff --git a/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ComponentName.c b/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ComponentName.c index 334aae58af..939af33ba1 100644 --- a/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ComponentName.c +++ b/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ComponentName.c @@ -1,6 +1,6 @@ /*++ -Copyright (c) 2006, Intel Corporation +Copyright (c) 2004 - 2007, Intel Corporation All rights reserved. This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at diff --git a/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.c b/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.c index 61d6e4068e..100ebf6d62 100644 --- a/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.c +++ b/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.c @@ -24,6 +24,7 @@ Abstract: #include #include #include +#include #include #include @@ -88,17 +89,25 @@ ScsiDiskDriverBindingSupported ( IN EFI_DEVICE_PATH_PROTOCOL *RemainingDevicePath ) /*++ + +Routine Description: + + Test to see if this driver supports ControllerHandle. Any ControllerHandle + that has ScsiIoProtocol installed will be supported. + +Arguments: + + This - Protocol instance pointer. + Controller - Handle of device to test + RemainingDevicePath - Not used + +Returns: + + EFI_SUCCESS - This driver supports this device. + EFI_UNSUPPORTED - This driver does not support this device. - Routine Description: - - Arguments: - - Returns: --*/ -// TODO: This - add argument and description to function comment -// TODO: Controller - add argument and description to function comment -// TODO: RemainingDevicePath - add argument and description to function comment { EFI_STATUS Status; EFI_SCSI_IO_PROTOCOL *ScsiIo; @@ -126,11 +135,11 @@ ScsiDiskDriverBindingSupported ( } gBS->CloseProtocol ( - Controller, - &gEfiScsiIoProtocolGuid, - This->DriverBindingHandle, - Controller - ); + Controller, + &gEfiScsiIoProtocolGuid, + This->DriverBindingHandle, + Controller + ); return Status; } @@ -142,19 +151,24 @@ ScsiDiskDriverBindingStart ( IN EFI_DEVICE_PATH_PROTOCOL *RemainingDevicePath ) /*++ + +Routine Description: + + Start SCSI Disk Driver, and attach BlockIoProtocol to it. - Routine Description: - - Arguments: +Arguments: + + This - Protocol instance pointer. + Controller - Handle of device to test + RemainingDevicePath - Not used + +Returns: + + EFI_SUCCESS - This driver supports this device. + EFI_UNSUPPORTED - This driver does not support this device. - Returns: --*/ -// TODO: This - add argument and description to function comment -// TODO: Controller - add argument and description to function comment -// TODO: RemainingDevicePath - add argument and description to function comment -// TODO: EFI_DEVICE_ERROR - add return value to function comment -// TODO: EFI_SUCCESS - add return value to function comment { EFI_STATUS Status; EFI_SCSI_IO_PROTOCOL *ScsiIo; @@ -245,11 +259,11 @@ ScsiDiskDriverBindingStart ( if (!NeedRetry) { gBS->FreePool (ScsiDiskDevice->SenseData); gBS->CloseProtocol ( - Controller, - &gEfiScsiIoProtocolGuid, - This->DriverBindingHandle, - Controller - ); + Controller, + &gEfiScsiIoProtocolGuid, + This->DriverBindingHandle, + Controller + ); gBS->FreePool (ScsiDiskDevice); return EFI_DEVICE_ERROR; } @@ -271,11 +285,11 @@ ScsiDiskDriverBindingStart ( if (EFI_ERROR (Status)) { gBS->FreePool (ScsiDiskDevice->SenseData); gBS->CloseProtocol ( - Controller, - &gEfiScsiIoProtocolGuid, - This->DriverBindingHandle, - Controller - ); + Controller, + &gEfiScsiIoProtocolGuid, + This->DriverBindingHandle, + Controller + ); gBS->FreePool (ScsiDiskDevice); return Status; } @@ -285,7 +299,7 @@ ScsiDiskDriverBindingStart ( "eng", gScsiDiskComponentName.SupportedLanguages, &ScsiDiskDevice->ControllerNameTable, - (CHAR16 *) L"SCSI Disk Device" + L"SCSI Disk Device" ); return EFI_SUCCESS; @@ -301,19 +315,26 @@ ScsiDiskDriverBindingStop ( IN EFI_HANDLE *ChildHandleBuffer ) /*++ - - Routine Description: - - Arguments: - - Returns: + +Routine Description: + + Stop this driver on ControllerHandle. Support stoping any child handles + created by this driver. + +Arguments: + + This - Protocol instance pointer. + Controller - Handle of device to stop driver on + NumberOfChildren - Number of Children in the ChildHandleBuffer + ChildHandleBuffer - List of handles for the children we need to stop. + +Returns: + + EFI_SUCCESS + EFI_DEVICE_ERROR + others --*/ -// TODO: This - add argument and description to function comment -// TODO: Controller - add argument and description to function comment -// TODO: NumberOfChildren - add argument and description to function comment -// TODO: ChildHandleBuffer - add argument and description to function comment -// TODO: EFI_SUCCESS - add return value to function comment { EFI_BLOCK_IO_PROTOCOL *BlkIo; SCSI_DISK_DEV *ScsiDiskDevice; @@ -339,11 +360,11 @@ ScsiDiskDriverBindingStop ( ); if (!EFI_ERROR (Status)) { gBS->CloseProtocol ( - Controller, - &gEfiScsiIoProtocolGuid, - This->DriverBindingHandle, - Controller - ); + Controller, + &gEfiScsiIoProtocolGuid, + This->DriverBindingHandle, + Controller + ); ReleaseScsiDiskDeviceResources (ScsiDiskDevice); @@ -355,9 +376,6 @@ ScsiDiskDriverBindingStop ( return Status; } -// -// Block I/O Protocol Interface -// EFI_STATUS EFIAPI @@ -369,22 +387,22 @@ ScsiDiskReset ( Routine Description: - TODO: Add function description + Reset SCSI Disk Arguments: - This - TODO: add argument description - ExtendedVerification - TODO: add argument description + This - The pointer of EFI_BLOCK_IO_PROTOCOL + ExtendedVerification - The flag about if extend verificate Returns: - TODO: add return values + EFI_STATUS --*/ { + EFI_TPL OldTpl; SCSI_DISK_DEV *ScsiDiskDevice; EFI_STATUS Status; - EFI_TPL OldTpl; OldTpl = gBS->RaiseTPL (TPL_CALLBACK); @@ -416,27 +434,24 @@ ScsiDiskReadBlocks ( Routine Description: - TODO: Add function description + The function is to Read Block from SCSI Disk Arguments: - This - TODO: add argument description - MediaId - TODO: add argument description - LBA - TODO: add argument description - BufferSize - TODO: add argument description - Buffer - TODO: add argument description + This - The pointer of EFI_BLOCK_IO_PROTOCOL + MediaId - The Id of Media detected + LBA - The logic block address + BufferSize - The size of Buffer + Buffer - The buffer to fill the read out data Returns: - EFI_INVALID_PARAMETER - TODO: Add description for return value - EFI_SUCCESS - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_NO_MEDIA - TODO: Add description for return value - EFI_MEDIA_CHANGED - TODO: Add description for return value - EFI_BAD_BUFFER_SIZE - TODO: Add description for return value - EFI_INVALID_PARAMETER - TODO: Add description for return value - EFI_INVALID_PARAMETER - TODO: Add description for return value - EFI_INVALID_PARAMETER - TODO: Add description for return value + EFI_INVALID_PARAMETER - Invalid parameter passed in. + EFI_SUCCESS - Successfully to read out block. + EFI_DEVICE_ERROR - Fail to detect media. + EFI_NO_MEDIA - Media is not present. + EFI_MEDIA_CHANGED - Media has changed. + EFI_BAD_BUFFER_SIZE - The buffer size is not multiple of BlockSize. --*/ { @@ -515,9 +530,9 @@ Returns: Status = EFI_INVALID_PARAMETER; goto Done; } - + // - // if all the parameters are valid, then perform read sectors command + // If all the parameters are valid, then perform read sectors command // to transfer data from device to host. // Status = ScsiDiskReadSectors (ScsiDiskDevice, Buffer, LBA, NumberOfBlocks); @@ -540,27 +555,24 @@ ScsiDiskWriteBlocks ( Routine Description: - TODO: Add function description + The function is to Write Block to SCSI Disk Arguments: - This - TODO: add argument description - MediaId - TODO: add argument description - LBA - TODO: add argument description - BufferSize - TODO: add argument description - Buffer - TODO: add argument description + This - The pointer of EFI_BLOCK_IO_PROTOCOL + MediaId - The Id of Media detected + LBA - The logic block address + BufferSize - The size of Buffer + Buffer - The buffer to fill the read out data Returns: - EFI_INVALID_PARAMETER - TODO: Add description for return value - EFI_SUCCESS - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_NO_MEDIA - TODO: Add description for return value - EFI_MEDIA_CHANGED - TODO: Add description for return value - EFI_BAD_BUFFER_SIZE - TODO: Add description for return value - EFI_INVALID_PARAMETER - TODO: Add description for return value - EFI_INVALID_PARAMETER - TODO: Add description for return value - EFI_INVALID_PARAMETER - TODO: Add description for return value + EFI_INVALID_PARAMETER - Invalid parameter passed in. + EFI_SUCCESS - Successfully to read out block. + EFI_DEVICE_ERROR - Fail to detect media. + EFI_NO_MEDIA - Media is not present. + EFI_MEDIA_CHANGED - Media has changed. + EFI_BAD_BUFFER_SIZE - The buffer size is not multiple of BlockSize. --*/ { @@ -647,7 +659,6 @@ Returns: Done: gBS->RestoreTPL (OldTpl); - return Status; } @@ -660,15 +671,15 @@ ScsiDiskFlushBlocks ( Routine Description: - TODO: Add function description + Flush Block to Disk Arguments: - This - TODO: add argument description + This - The pointer of EFI_BLOCK_IO_PROTOCOL Returns: - EFI_SUCCESS - TODO: Add description for return value + EFI_SUCCESS --*/ { @@ -688,20 +699,18 @@ ScsiDiskDetectMedia ( Routine Description: - TODO: Add function description + Dectect Device and read out capacity ,if error occurs, parse the sense key. Arguments: - ScsiDiskDevice - TODO: add argument description - MustReadCapacity - TODO: add argument description - MediaChange - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + MustReadCapacity - The flag about reading device capacity + MediaChange - The pointer of flag indicates if media has changed Returns: - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_SUCCESS - TODO: Add description for return value + EFI_DEVICE_ERROR - Indicates that error occurs + EFI_SUCCESS - Successfully to detect media --*/ { @@ -722,11 +731,9 @@ Returns: NumberOfSenseKeys = 0; NeedReadCapacity = FALSE; CopyMem (&OldMedia, ScsiDiskDevice->BlkIo.Media, sizeof (OldMedia)); - // OldMedia = *(ScsiDiskDevice->BlkIo.Media); - *MediaChange = FALSE; - MaxRetry = 3; + for (Index = 0; Index < MaxRetry; Index++) { Status = ScsiDiskTestUnitReady ( ScsiDiskDevice, @@ -765,13 +772,12 @@ Returns: } else { NeedReadCapacity = TRUE; } - + // // either NeedReadCapacity is TRUE, or MustReadCapacity is TRUE, // retrieve capacity via Read Capacity command // if (NeedReadCapacity || MustReadCapacity) { - // // retrieve media information // @@ -885,25 +891,17 @@ ScsiDiskInquiryDevice ( Routine Description: - TODO: Add function description + Send out Inquiry command to Device Arguments: - ScsiDiskDevice - TODO: add argument description - NeedRetry - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + NeedRetry - Indicates if needs try again when error happens Returns: - EFI_SUCCESS - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value + EFI_DEVICE_ERROR - Indicates that error occurs + EFI_SUCCESS - Successfully to detect media --*/ { @@ -931,37 +929,34 @@ Returns: &InquiryDataLength, FALSE ); - if ((Status == EFI_SUCCESS) || (Status == EFI_WARN_BUFFER_TOO_SMALL)) { // // no need to check HostAdapterStatus and TargetStatus // - ParseInquiryData (ScsiDiskDevice); - return EFI_SUCCESS; - } else if (Status == EFI_NOT_READY) { - // - // no need to check HostAdapterStatus and TargetStatus - // - *NeedRetry = TRUE; - return EFI_DEVICE_ERROR; - } else if ((Status == EFI_INVALID_PARAMETER) || (Status == EFI_UNSUPPORTED)) { - // - // no need to check HostAdapterStatus and TargetStatus - // - *NeedRetry = FALSE; - return EFI_DEVICE_ERROR; - } - // - // go ahead to check HostAdapterStatus and TargetStatus - // (EFI_TIMEOUT, EFI_DEVICE_ERROR) - // - Status = CheckHostAdapterStatus (HostAdapterStatus); - if ((Status == EFI_TIMEOUT) || (Status == EFI_NOT_READY)) { - *NeedRetry = TRUE; - return EFI_DEVICE_ERROR; - } else if (Status == EFI_DEVICE_ERROR) { - // - // reset the scsi channel - // + if ((Status == EFI_SUCCESS) || (Status == EFI_WARN_BUFFER_TOO_SMALL)) { + ParseInquiryData (ScsiDiskDevice); + return EFI_SUCCESS; + + } else if (Status == EFI_NOT_READY) { + *NeedRetry = TRUE; + return EFI_DEVICE_ERROR; + + } else if ((Status == EFI_INVALID_PARAMETER) || (Status == EFI_UNSUPPORTED)) { + *NeedRetry = FALSE; + return EFI_DEVICE_ERROR; + } + // + // go ahead to check HostAdapterStatus and TargetStatus + // (EFI_TIMEOUT, EFI_DEVICE_ERROR) + // + + Status = CheckHostAdapterStatus (HostAdapterStatus); + if ((Status == EFI_TIMEOUT) || (Status == EFI_NOT_READY)) { + *NeedRetry = TRUE; + return EFI_DEVICE_ERROR; + } else if (Status == EFI_DEVICE_ERROR) { + // + // reset the scsi channel + // ScsiDiskDevice->ScsiIo->ResetBus (ScsiDiskDevice->ScsiIo); *NeedRetry = FALSE; return EFI_DEVICE_ERROR; @@ -975,19 +970,19 @@ Returns: ScsiDiskDevice->ScsiIo->ResetDevice (ScsiDiskDevice->ScsiIo); *NeedRetry = TRUE; return EFI_DEVICE_ERROR; + } else if (Status == EFI_DEVICE_ERROR) { *NeedRetry = FALSE; return EFI_DEVICE_ERROR; } // - // if goes here, meant ScsiInquiryCommand() failed. + // if goes here, meant SubmitInquiryCommand() failed. // if ScsiDiskRequestSenseKeys() succeeds at last, - // better retry ScsiInquiryCommand(). (by setting *NeedRetry = TRUE) + // better retry SubmitInquiryCommand(). (by setting *NeedRetry = TRUE) // MaxRetry = 3; for (Index = 0; Index < MaxRetry; Index++) { - Status = ScsiDiskRequestSenseKeys ( ScsiDiskDevice, NeedRetry, @@ -1019,30 +1014,27 @@ ScsiDiskTestUnitReady ( EFI_SCSI_SENSE_DATA **SenseDataArray, UINTN *NumberOfSenseKeys ) -// TODO: function comment should start with '/*++' -/* - When Test Unit Ready command succeeds, - retrieve Sense Keys via Request Sense; + /*++ + +Routine Description: + + When Test Unit Ready command succeeds, retrieve Sense Keys via Request Sense; When Test Unit Ready command encounters any error caused by host adapter or target, return error without retrieving Sense Keys. -*/ -// TODO: function comment should end with '--*/' -// TODO: function comment is missing 'Routine Description:' -// TODO: function comment is missing 'Arguments:' -// TODO: function comment is missing 'Returns:' -// TODO: ScsiDiskDevice - add argument and description to function comment -// TODO: NeedRetry - add argument and description to function comment -// TODO: SenseDataArray - add argument and description to function comment -// TODO: NumberOfSenseKeys - add argument and description to function comment -// TODO: EFI_DEVICE_ERROR - add return value to function comment -// TODO: EFI_DEVICE_ERROR - add return value to function comment -// TODO: EFI_DEVICE_ERROR - add return value to function comment -// TODO: EFI_DEVICE_ERROR - add return value to function comment -// TODO: EFI_DEVICE_ERROR - add return value to function comment -// TODO: EFI_DEVICE_ERROR - add return value to function comment -// TODO: EFI_SUCCESS - add return value to function comment -// TODO: EFI_DEVICE_ERROR - add return value to function comment -// TODO: EFI_DEVICE_ERROR - add return value to function comment + +Arguments: + + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + NeedRetry - The pointer of flag indicates try again + SenseDataArray - The pointer of an array of sense data + NumberOfSenseKeys - The pointer of the number of sense data array + +Returns: + + EFI_DEVICE_ERROR - Indicates that error occurs + EFI_SUCCESS - Successfully to test unit + +--*/ { EFI_STATUS Status; UINT8 SenseDataLength; @@ -1065,26 +1057,26 @@ ScsiDiskTestUnitReady ( &HostAdapterStatus, &TargetStatus ); + // + // no need to check HostAdapterStatus and TargetStatus + // if (Status == EFI_NOT_READY) { - // - // no need to check HostAdapterStatus and TargetStatus - // *NeedRetry = TRUE; return EFI_DEVICE_ERROR; + } else if ((Status == EFI_INVALID_PARAMETER) || (Status == EFI_UNSUPPORTED)) { - // - // no need to check HostAdapterStatus and TargetStatus - // *NeedRetry = FALSE; return EFI_DEVICE_ERROR; } // - // go ahead to check HostAdapterStatus and TargetStatus + // go ahead to check HostAdapterStatus and TargetStatus(in case of EFI_DEVICE_ERROR) // + Status = CheckHostAdapterStatus (HostAdapterStatus); if ((Status == EFI_TIMEOUT) || (Status == EFI_NOT_READY)) { *NeedRetry = TRUE; return EFI_DEVICE_ERROR; + } else if (Status == EFI_DEVICE_ERROR) { // // reset the scsi channel @@ -1102,6 +1094,7 @@ ScsiDiskTestUnitReady ( ScsiDiskDevice->ScsiIo->ResetDevice (ScsiDiskDevice->ScsiIo); *NeedRetry = TRUE; return EFI_DEVICE_ERROR; + } else if (Status == EFI_DEVICE_ERROR) { *NeedRetry = FALSE; return EFI_DEVICE_ERROR; @@ -1109,7 +1102,6 @@ ScsiDiskTestUnitReady ( MaxRetry = 3; for (Index = 0; Index < MaxRetry; Index++) { - Status = ScsiDiskRequestSenseKeys ( ScsiDiskDevice, NeedRetry, @@ -1144,26 +1136,19 @@ DetectMediaParsingSenseKeys ( Routine Description: - TODO: Add function description - + Parsing Sense Keys which got from request sense command. + Arguments: - ScsiDiskDevice - TODO: add argument description - SenseData - TODO: add argument description - NumberOfSenseKeys - TODO: add argument description - Action - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + SenseData - The pointer of EFI_SCSI_SENSE_DATA + NumberOfSenseKeys - The number of sense key + Action - The pointer of action which indicates what is need to do next Returns: - EFI_SUCCESS - TODO: Add description for return value - EFI_SUCCESS - TODO: Add description for return value - EFI_SUCCESS - TODO: Add description for return value - EFI_SUCCESS - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_SUCCESS - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_SUCCESS - TODO: Add description for return value + EFI_DEVICE_ERROR - Indicates that error occurs + EFI_SUCCESS - Successfully to complete the parsing --*/ { @@ -1232,27 +1217,19 @@ ScsiDiskReadCapacity ( Routine Description: - TODO: Add function description + Send read capacity command to device and get the device parameter Arguments: - ScsiDiskDevice - TODO: add argument description - NeedRetry - TODO: add argument description - SenseDataArray - TODO: add argument description - NumberOfSenseKeys - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + NeedRetry - The pointer of flag indicates if need a retry + SenseDataArray - The pointer of an array of sense data + NumberOfSenseKeys - The number of sense key Returns: - EFI_SUCCESS - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value + EFI_DEVICE_ERROR - Indicates that error occurs + EFI_SUCCESS - Successfully to read capacity --*/ { @@ -1286,35 +1263,32 @@ Returns: &DataLength, FALSE ); - if (CommandStatus == EFI_SUCCESS) { - // - // no need to check HostAdapterStatus and TargetStatus - // - GetMediaInfo (ScsiDiskDevice, &CapacityData); - return EFI_SUCCESS; - } else if (CommandStatus == EFI_NOT_READY) { - // - // no need to check HostAdapterStatus and TargetStatus - // - *NeedRetry = TRUE; - return EFI_DEVICE_ERROR; - } else if ((CommandStatus == EFI_INVALID_PARAMETER) || (CommandStatus == EFI_UNSUPPORTED)) { - // + // // no need to check HostAdapterStatus and TargetStatus // - *NeedRetry = FALSE; - return EFI_DEVICE_ERROR; - } - // - // go ahead to check HostAdapterStatus and TargetStatus - // (EFI_TIMEOUT, EFI_DEVICE_ERROR, EFI_WARN_BUFFER_TOO_SMALL) - // - - Status = CheckHostAdapterStatus (HostAdapterStatus); - if ((Status == EFI_TIMEOUT) || (Status == EFI_NOT_READY)) { - *NeedRetry = TRUE; - return EFI_DEVICE_ERROR; - } else if (Status == EFI_DEVICE_ERROR) { + if (CommandStatus == EFI_SUCCESS) { + GetMediaInfo (ScsiDiskDevice, &CapacityData); + return EFI_SUCCESS; + + } else if (CommandStatus == EFI_NOT_READY) { + *NeedRetry = TRUE; + return EFI_DEVICE_ERROR; + + } else if ((CommandStatus == EFI_INVALID_PARAMETER) || (CommandStatus == EFI_UNSUPPORTED)) { + *NeedRetry = FALSE; + return EFI_DEVICE_ERROR; + } + // + // go ahead to check HostAdapterStatus and TargetStatus + // (EFI_TIMEOUT, EFI_DEVICE_ERROR, EFI_WARN_BUFFER_TOO_SMALL) + // + + Status = CheckHostAdapterStatus (HostAdapterStatus); + if ((Status == EFI_TIMEOUT) || (Status == EFI_NOT_READY)) { + *NeedRetry = TRUE; + return EFI_DEVICE_ERROR; + + } else if (Status == EFI_DEVICE_ERROR) { // // reset the scsi channel // @@ -1331,15 +1305,16 @@ Returns: ScsiDiskDevice->ScsiIo->ResetDevice (ScsiDiskDevice->ScsiIo); *NeedRetry = TRUE; return EFI_DEVICE_ERROR; + } else if (Status == EFI_DEVICE_ERROR) { *NeedRetry = FALSE; return EFI_DEVICE_ERROR; } // - // if goes here, meant ScsiReadCapacityCommand() failed. + // if goes here, meant SubmitReadCapacityCommand() failed. // if ScsiDiskRequestSenseKeys() succeeds at last, - // better retry ScsiReadCapacityCommand(). (by setting *NeedRetry = TRUE) + // better retry SubmitReadCapacityCommand(). (by setting *NeedRetry = TRUE) // MaxRetry = 3; for (Index = 0; Index < MaxRetry; Index++) { @@ -1376,40 +1351,39 @@ CheckHostAdapterStatus ( Routine Description: - TODO: Add function description - + Check the HostAdapter status + Arguments: - HostAdapterStatus - TODO: add argument description + HostAdapterStatus - Host Adapter status Returns: - EFI_SUCCESS - TODO: Add description for return value - EFI_TIMEOUT - TODO: Add description for return value - EFI_NOT_READY - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_SUCCESS - TODO: Add description for return value + EFI_SUCCESS + EFI_TIMEOUT + EFI_NOT_READY + EFI_DEVICE_ERROR --*/ { switch (HostAdapterStatus) { - case EFI_SCSI_IO_STATUS_HOST_ADAPTER_OK: + case EFI_EXT_SCSI_STATUS_HOST_ADAPTER_OK: return EFI_SUCCESS; - case EFI_SCSI_IO_STATUS_HOST_ADAPTER_SELECTION_TIMEOUT: - case EFI_SCSI_IO_STATUS_HOST_ADAPTER_TIMEOUT: - case EFI_SCSI_IO_STATUS_HOST_ADAPTER_TIMEOUT_COMMAND: + case EFI_EXT_SCSI_STATUS_HOST_ADAPTER_SELECTION_TIMEOUT: + case EFI_EXT_SCSI_STATUS_HOST_ADAPTER_TIMEOUT: + case EFI_EXT_SCSI_STATUS_HOST_ADAPTER_TIMEOUT_COMMAND: return EFI_TIMEOUT; - case EFI_SCSI_IO_STATUS_HOST_ADAPTER_MESSAGE_REJECT: - case EFI_SCSI_IO_STATUS_HOST_ADAPTER_PARITY_ERROR: - case EFI_SCSI_IO_STATUS_HOST_ADAPTER_REQUEST_SENSE_FAILED: - case EFI_SCSI_IO_STATUS_HOST_ADAPTER_DATA_OVERRUN_UNDERRUN: - case EFI_SCSI_IO_STATUS_HOST_ADAPTER_BUS_RESET: + case EFI_EXT_SCSI_STATUS_HOST_ADAPTER_MESSAGE_REJECT: + case EFI_EXT_SCSI_STATUS_HOST_ADAPTER_PARITY_ERROR: + case EFI_EXT_SCSI_STATUS_HOST_ADAPTER_REQUEST_SENSE_FAILED: + case EFI_EXT_SCSI_STATUS_HOST_ADAPTER_DATA_OVERRUN_UNDERRUN: + case EFI_EXT_SCSI_STATUS_HOST_ADAPTER_BUS_RESET: return EFI_NOT_READY; - case EFI_SCSI_IO_STATUS_HOST_ADAPTER_BUS_FREE: - case EFI_SCSI_IO_STATUS_HOST_ADAPTER_PHASE_ERROR: + case EFI_EXT_SCSI_STATUS_HOST_ADAPTER_BUS_FREE: + case EFI_EXT_SCSI_STATUS_HOST_ADAPTER_PHASE_ERROR: return EFI_DEVICE_ERROR; default: @@ -1425,35 +1399,33 @@ CheckTargetStatus ( Routine Description: - TODO: Add function description - + Check the target status + Arguments: - TargetStatus - TODO: add argument description + TargetStatus - Target status Returns: - EFI_SUCCESS - TODO: Add description for return value - EFI_NOT_READY - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_SUCCESS - TODO: Add description for return value + EFI_NOT_READY + EFI_DEVICE_ERROR + EFI_SUCCESS --*/ { switch (TargetStatus) { - case EFI_SCSI_IO_STATUS_TARGET_GOOD: - case EFI_SCSI_IO_STATUS_TARGET_CHECK_CONDITION: - case EFI_SCSI_IO_STATUS_TARGET_CONDITION_MET: + case EFI_EXT_SCSI_STATUS_TARGET_GOOD: + case EFI_EXT_SCSI_STATUS_TARGET_CHECK_CONDITION: + case EFI_EXT_SCSI_STATUS_TARGET_CONDITION_MET: return EFI_SUCCESS; - case EFI_SCSI_IO_STATUS_TARGET_INTERMEDIATE: - case EFI_SCSI_IO_STATUS_TARGET_INTERMEDIATE_CONDITION_MET: - case EFI_SCSI_IO_STATUS_TARGET_BUSY: - case EFI_SCSI_IO_STATUS_TARGET_COMMOND_TERMINATED: - case EFI_SCSI_IO_STATUS_TARGET_QUEUE_FULL: + case EFI_EXT_SCSI_STATUS_TARGET_INTERMEDIATE: + case EFI_EXT_SCSI_STATUS_TARGET_INTERMEDIATE_CONDITION_MET: + case EFI_EXT_SCSI_STATUS_TARGET_BUSY: + case EFI_EXT_SCSI_STATUS_TARGET_TASK_SET_FULL: return EFI_NOT_READY; - case EFI_SCSI_IO_STATUS_TARGET_RESERVATION_CONFLICT: + case EFI_EXT_SCSI_STATUS_TARGET_RESERVATION_CONFLICT: return EFI_DEVICE_ERROR; break; @@ -1470,27 +1442,31 @@ ScsiDiskRequestSenseKeys ( UINTN *NumberOfSenseKeys, BOOLEAN AskResetIfError ) -// TODO: function comment should start with '/*++' -/* +/*++ + +Routine Description: + Retrieve all sense keys from the device. When encountering error during the process, if retrieve sense keys before error encounterred, return the sense keys with return status set to EFI_SUCCESS, and NeedRetry set to FALSE; otherwize, return the proper return status. -*/ -// TODO: function comment should end with '--*/' -// TODO: function comment is missing 'Routine Description:' -// TODO: function comment is missing 'Arguments:' -// TODO: function comment is missing 'Returns:' -// TODO: ScsiDiskDevice - add argument and description to function comment -// TODO: NeedRetry - add argument and description to function comment -// TODO: SenseDataArray - add argument and description to function comment -// TODO: NumberOfSenseKeys - add argument and description to function comment -// TODO: AskResetIfError - add argument and description to function comment -// TODO: EFI_SUCCESS - add return value to function comment -// TODO: EFI_DEVICE_ERROR - add return value to function comment -// TODO: EFI_SUCCESS - add return value to function comment + +Arguments: + + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + NeedRetry - The pointer of flag indicates if need a retry + SenseDataArray - The pointer of an array of sense data + NumberOfSenseKeys - The number of sense key + AskResetIfError - The flag indicates if need reset when error occurs + +Returns: + + EFI_DEVICE_ERROR - Indicates that error occurs + EFI_SUCCESS - Successfully to request sense key + +--*/ { EFI_SCSI_SENSE_DATA *PtrSenseData; UINT8 SenseDataLength; @@ -1513,7 +1489,6 @@ ScsiDiskRequestSenseKeys ( PtrSenseData = ScsiDiskDevice->SenseData; for (SenseReq = TRUE; SenseReq;) { - Status = ScsiRequestSenseCommand ( ScsiDiskDevice->ScsiIo, EfiScsiStallSeconds (2), @@ -1522,20 +1497,23 @@ ScsiDiskRequestSenseKeys ( &HostAdapterStatus, &TargetStatus ); - if ((Status == EFI_SUCCESS) || (Status == EFI_WARN_BUFFER_TOO_SMALL)) { - FallStatus = EFI_SUCCESS; - } else if ((Status == EFI_TIMEOUT) || (Status == EFI_NOT_READY)) { - *NeedRetry = TRUE; - FallStatus = EFI_DEVICE_ERROR; - } else if ((Status == EFI_INVALID_PARAMETER) || (Status == EFI_UNSUPPORTED)) { - *NeedRetry = FALSE; - FallStatus = EFI_DEVICE_ERROR; - } else if (Status == EFI_DEVICE_ERROR) { - if (AskResetIfError) { - ScsiDiskDevice->ScsiIo->ResetDevice (ScsiDiskDevice->ScsiIo); - } - - FallStatus = EFI_DEVICE_ERROR; + if ((Status == EFI_SUCCESS) || (Status == EFI_WARN_BUFFER_TOO_SMALL)) { + FallStatus = EFI_SUCCESS; + + } else if ((Status == EFI_TIMEOUT) || (Status == EFI_NOT_READY)) { + *NeedRetry = TRUE; + FallStatus = EFI_DEVICE_ERROR; + + } else if ((Status == EFI_INVALID_PARAMETER) || (Status == EFI_UNSUPPORTED)) { + *NeedRetry = FALSE; + FallStatus = EFI_DEVICE_ERROR; + + } else if (Status == EFI_DEVICE_ERROR) { + if (AskResetIfError) { + ScsiDiskDevice->ScsiIo->ResetDevice (ScsiDiskDevice->ScsiIo); + } + + FallStatus = EFI_DEVICE_ERROR; } if (EFI_ERROR (FallStatus)) { @@ -1557,11 +1535,8 @@ ScsiDiskRequestSenseKeys ( (*NumberOfSenseKeys == ScsiDiskDevice->SenseDataNumber)) { SenseReq = FALSE; } - PtrSenseData += 1; - } - return EFI_SUCCESS; } @@ -1574,16 +1549,16 @@ GetMediaInfo ( Routine Description: - TODO: Add function description + Get information from media read capacity command Arguments: - ScsiDiskDevice - TODO: add argument description - Capacity - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + Capacity - The pointer of EFI_SCSI_DISK_CAPACITY_DATA Returns: - TODO: add return values + NONE --*/ { @@ -1614,15 +1589,15 @@ ParseInquiryData ( Routine Description: - TODO: Add function description + Parse Inquiry data Arguments: - ScsiDiskDevice - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV Returns: - TODO: add return values + NONE --*/ { @@ -1631,6 +1606,7 @@ Returns: } EFI_STATUS +EFIAPI ScsiDiskReadSectors ( SCSI_DISK_DEV *ScsiDiskDevice, VOID *Buffer, @@ -1641,20 +1617,19 @@ ScsiDiskReadSectors ( Routine Description: - TODO: Add function description + Read sector from SCSI Disk Arguments: - ScsiDiskDevice - TODO: add argument description - Buffer - TODO: add argument description - Lba - TODO: add argument description - NumberOfBlocks - TODO: add argument description + ScsiDiskDevice - The poiniter of SCSI_DISK_DEV + Buffer - The buffer to fill in the read out data + Lba - Logic block address + NumberOfBlocks - The number of blocks to read Returns: - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_SUCCESS - TODO: Add description for return value + EFI_DEVICE_ERROR + EFI_SUCCESS --*/ { @@ -1753,20 +1728,19 @@ ScsiDiskWriteSectors ( Routine Description: - TODO: Add function description + Write SCSI Disk sectors Arguments: - ScsiDiskDevice - TODO: add argument description - Buffer - TODO: add argument description - Lba - TODO: add argument description - NumberOfBlocks - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + Buffer - The data buffer to write sector + Lba - Logic block address + NumberOfBlocks - The number of blocks to write Returns: - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_DEVICE_ERROR - TODO: Add description for return value - EFI_SUCCESS - TODO: Add description for return value + EFI_DEVICE_ERROR + EFI_SUCCESS --*/ { @@ -1866,24 +1840,24 @@ ScsiDiskRead10 ( Routine Description: - TODO: Add function description + Sumbmit Read command Arguments: - ScsiDiskDevice - TODO: add argument description - NeedRetry - TODO: add argument description - SenseDataArray - TODO: add argument description - NumberOfSenseKeys - TODO: add argument description - Timeout - TODO: add argument description - DataBuffer - TODO: add argument description - DataLength - TODO: add argument description - StartLba - TODO: add argument description - SectorSize - TODO: add argument description + ScsiDiskDevice - The pointer of ScsiDiskDevice + NeedRetry - The pointer of flag indicates if needs retry if error happens + SenseDataArray - The pointer of an array of sense data + NumberOfSenseKeys - The number of sense key + Timeout - The time to complete the command + DataBuffer - The buffer to fill with the read out data + DataLength - The length of buffer + StartLba - The start logic block address + SectorSize - The size of sector Returns: - TODO: add return values - + EFI_STATUS + --*/ { UINT8 SenseDataLength; @@ -1925,23 +1899,23 @@ ScsiDiskWrite10 ( Routine Description: - TODO: Add function description + Submit Write Command Arguments: - ScsiDiskDevice - TODO: add argument description - NeedRetry - TODO: add argument description - SenseDataArray - TODO: add argument description - NumberOfSenseKeys - TODO: add argument description - Timeout - TODO: add argument description - DataBuffer - TODO: add argument description - DataLength - TODO: add argument description - StartLba - TODO: add argument description - SectorSize - TODO: add argument description + ScsiDiskDevice - The pointer of ScsiDiskDevice + NeedRetry - The pointer of flag indicates if needs retry if error happens + SenseDataArray - The pointer of an array of sense data + NumberOfSenseKeys - The number of sense key + Timeout - The time to complete the command + DataBuffer - The buffer to fill with the read out data + DataLength - The length of buffer + StartLba - The start logic block address + SectorSize - The size of sector Returns: - TODO: add return values + EFI_STATUS --*/ { @@ -1977,16 +1951,16 @@ ScsiDiskIsNoMedia ( Routine Description: - TODO: Add function description + Check sense key to find if media presents Arguments: - SenseData - TODO: add argument description - SenseCounts - TODO: add argument description + SenseData - The pointer of EFI_SCSI_SENSE_DATA + SenseCounts - The number of sense key Returns: - TODO: add return values + BOOLEAN --*/ { @@ -1998,7 +1972,6 @@ Returns: SensePtr = SenseData; for (Index = 0; Index < SenseCounts; Index++) { - // // Sense Key is EFI_SCSI_SK_NOT_READY (0x2), // Additional Sense Code is ASC_NO_MEDIA (0x3A) @@ -2007,7 +1980,6 @@ Returns: (SensePtr->Addnl_Sense_Code == EFI_SCSI_ASC_NO_MEDIA)) { IsNoMedia = TRUE; } - SensePtr++; } @@ -2023,16 +1995,16 @@ ScsiDiskIsMediaError ( Routine Description: - TODO: Add function description + Parse sense key Arguments: - SenseData - TODO: add argument description - SenseCounts - TODO: add argument description + SenseData - The pointer of EFI_SCSI_SENSE_DATA + SenseCounts - The number of sense key Returns: - TODO: add return values + BOOLEAN --*/ { @@ -2113,16 +2085,16 @@ ScsiDiskIsHardwareError ( Routine Description: - TODO: Add function description + Check sense key to find if hardware error happens Arguments: - SenseData - TODO: add argument description - SenseCounts - TODO: add argument description + SenseData - The pointer of EFI_SCSI_SENSE_DATA + SenseCounts - The number of sense key Returns: - TODO: add return values + BOOLEAN --*/ { @@ -2157,16 +2129,16 @@ ScsiDiskIsMediaChange ( Routine Description: - TODO: Add function description + Check sense key to find if media has changed Arguments: - SenseData - TODO: add argument description - SenseCounts - TODO: add argument description + SenseData - The pointer of EFI_SCSI_SENSE_DATA + SenseCounts - The number of sense key Returns: - TODO: add return values + BOOLEAN --*/ { @@ -2202,16 +2174,16 @@ ScsiDiskIsResetBefore ( Routine Description: - TODO: Add function description + Check sense key to find if reset happens Arguments: - SenseData - TODO: add argument description - SenseCounts - TODO: add argument description + SenseData - The pointer of EFI_SCSI_SENSE_DATA + SenseCounts - The number of sense key Returns: - TODO: add return values + BOOLEAN --*/ { @@ -2249,17 +2221,17 @@ ScsiDiskIsDriveReady ( Routine Description: - TODO: Add function description + Check sense key to find if the drive is ready Arguments: - SenseData - TODO: add argument description - SenseCounts - TODO: add argument description - RetryLater - TODO: add argument description + SenseData - The pointer of EFI_SCSI_SENSE_DATA + SenseCounts - The number of sense key + RetryLater - The flag means if need a retry Returns: - TODO: add return values + BOOLEAN --*/ { @@ -2325,16 +2297,16 @@ ScsiDiskHaveSenseKey ( Routine Description: - TODO: Add function description + Check sense key to find if it has sense key Arguments: - SenseData - TODO: add argument description - SenseCounts - TODO: add argument description + SenseData - The pointer of EFI_SCSI_SENSE_DATA + SenseCounts - The number of sense key Returns: - TODO: add return values + BOOLEAN --*/ { @@ -2374,15 +2346,15 @@ ReleaseScsiDiskDeviceResources ( Routine Description: - TODO: Add function description + Release resource about disk device Arguments: - ScsiDiskDevice - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV Returns: - TODO: add return values + NONE --*/ { diff --git a/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.h b/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.h index 675c643eff..971df28a2b 100644 --- a/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.h +++ b/MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDisk.h @@ -1,6 +1,6 @@ /*++ -Copyright (c) 2006, Intel Corporation +Copyright (c) 2004 - 2007, Intel Corporation All rights reserved. This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at @@ -119,16 +119,16 @@ ScsiDiskReset ( Routine Description: - TODO: Add function description + Reset SCSI Disk Arguments: - This - TODO: add argument description - ExtendedVerification - TODO: add argument description + This - The pointer of EFI_BLOCK_IO_PROTOCOL + ExtendedVerification - The flag about if extend verificate Returns: - TODO: add return values + EFI_STATUS --*/ ; @@ -146,19 +146,24 @@ ScsiDiskReadBlocks ( Routine Description: - TODO: Add function description + The function is to Read Block from SCSI Disk Arguments: - This - TODO: add argument description - MediaId - TODO: add argument description - LBA - TODO: add argument description - BufferSize - TODO: add argument description - Buffer - TODO: add argument description + This - The pointer of EFI_BLOCK_IO_PROTOCOL + MediaId - The Id of Media detected + LBA - The logic block address + BufferSize - The size of Buffer + Buffer - The buffer to fill the read out data Returns: - TODO: add return values + EFI_INVALID_PARAMETER - Invalid parameter passed in. + EFI_SUCCESS - Successfully to read out block. + EFI_DEVICE_ERROR - Fail to detect media. + EFI_NO_MEDIA - Media is not present. + EFI_MEDIA_CHANGED - Media has changed. + EFI_BAD_BUFFER_SIZE - The buffer size is not multiple of BlockSize. --*/ ; @@ -176,19 +181,24 @@ ScsiDiskWriteBlocks ( Routine Description: - TODO: Add function description + The function is to Write Block to SCSI Disk Arguments: - This - TODO: add argument description - MediaId - TODO: add argument description - LBA - TODO: add argument description - BufferSize - TODO: add argument description - Buffer - TODO: add argument description + This - The pointer of EFI_BLOCK_IO_PROTOCOL + MediaId - The Id of Media detected + LBA - The logic block address + BufferSize - The size of Buffer + Buffer - The buffer to fill the read out data Returns: - TODO: add return values + EFI_INVALID_PARAMETER - Invalid parameter passed in. + EFI_SUCCESS - Successfully to read out block. + EFI_DEVICE_ERROR - Fail to detect media. + EFI_NO_MEDIA - Media is not present. + EFI_MEDIA_CHANGED - Media has changed. + EFI_BAD_BUFFER_SIZE - The buffer size is not multiple of BlockSize. --*/ ; @@ -202,15 +212,15 @@ ScsiDiskFlushBlocks ( Routine Description: - TODO: Add function description + Flush Block to Disk Arguments: - This - TODO: add argument description + This - The pointer of EFI_BLOCK_IO_PROTOCOL Returns: - TODO: add return values + EFI_SUCCESS --*/ ; @@ -225,44 +235,48 @@ ScsiDiskDetectMedia ( Routine Description: - TODO: Add function description + Dectect Device and read out capacity ,if error occurs, parse the sense key. Arguments: - ScsiDiskDevice - TODO: add argument description - MustReadCap - TODO: add argument description - MediaChange - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + MustReadCapacity - The flag about reading device capacity + MediaChange - The pointer of flag indicates if media has changed Returns: - TODO: add return values + EFI_DEVICE_ERROR - Indicates that error occurs + EFI_SUCCESS - Successfully to detect media --*/ ; - + EFI_STATUS ScsiDiskTestUnitReady ( - SCSI_DISK_DEV *ScsiDiskDevice, - BOOLEAN *NeedRetry, - EFI_SCSI_SENSE_DATA **SenseDataArray, - UINTN *NumberOfSenseKeys + SCSI_DISK_DEV *ScsiDiskDevice, + BOOLEAN *NeedRetry, + EFI_SCSI_SENSE_DATA **SenseDataArray, + UINTN *NumberOfSenseKeys ) /*++ Routine Description: - TODO: Add function description - + When Test Unit Ready command succeeds, retrieve Sense Keys via Request Sense; + When Test Unit Ready command encounters any error caused by host adapter or + target, return error without retrieving Sense Keys. + Arguments: - ScsiDiskDevice - TODO: add argument description - NeedRetry - TODO: add argument description - SenseDataArray - TODO: add argument description - NumberOfSenseKeys - TODO: add argument description - + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + NeedRetry - The pointer of flag indicates try again + SenseDataArray - The pointer of an array of sense data + NumberOfSenseKeys - The pointer of the number of sense data array + Returns: - TODO: add return values + EFI_DEVICE_ERROR - Indicates that error occurs + EFI_SUCCESS - Successfully to test unit --*/ ; @@ -278,45 +292,47 @@ DetectMediaParsingSenseKeys ( Routine Description: - TODO: Add function description - + Parsing Sense Keys which got from request sense command. + Arguments: - ScsiDiskDevice - TODO: add argument description - SenseData - TODO: add argument description - NumberOfSenseKeys - TODO: add argument description - Action - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + SenseData - The pointer of EFI_SCSI_SENSE_DATA + NumberOfSenseKeys - The number of sense key + Action - The pointer of action which indicates what is need to do next Returns: - TODO: add return values + EFI_DEVICE_ERROR - Indicates that error occurs + EFI_SUCCESS - Successfully to complete the parsing --*/ ; EFI_STATUS ScsiDiskReadCapacity ( - SCSI_DISK_DEV *ScsiDiskDevice, - BOOLEAN *NeedRetry, - EFI_SCSI_SENSE_DATA **SenseDataArray, - UINTN *NumberOfSenseKeys + SCSI_DISK_DEV *ScsiDiskDevice, + BOOLEAN *NeedRetry, + EFI_SCSI_SENSE_DATA **SenseDataArray, + UINTN *NumberOfSenseKeys ) /*++ Routine Description: - TODO: Add function description + Send read capacity command to device and get the device parameter Arguments: - ScsiDiskDevice - TODO: add argument description - NeedRetry - TODO: add argument description - SenseDataArray - TODO: add argument description - NumberOfSenseKeys - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + NeedRetry - The pointer of flag indicates if need a retry + SenseDataArray - The pointer of an array of sense data + NumberOfSenseKeys - The number of sense key Returns: - TODO: add return values + EFI_DEVICE_ERROR - Indicates that error occurs + EFI_SUCCESS - Successfully to read capacity --*/ ; @@ -329,15 +345,18 @@ CheckHostAdapterStatus ( Routine Description: - TODO: Add function description - + Check the HostAdapter status + Arguments: - HostAdapterStatus - TODO: add argument description + HostAdapterStatus - Host Adapter status Returns: - TODO: add return values + EFI_SUCCESS + EFI_TIMEOUT + EFI_NOT_READY + EFI_DEVICE_ERROR --*/ ; @@ -350,15 +369,17 @@ CheckTargetStatus ( Routine Description: - TODO: Add function description - + Check the target status + Arguments: - TargetStatus - TODO: add argument description + TargetStatus - Target status Returns: - TODO: add return values + EFI_NOT_READY + EFI_DEVICE_ERROR + EFI_SUCCESS --*/ ; @@ -375,19 +396,25 @@ ScsiDiskRequestSenseKeys ( Routine Description: - TODO: Add function description + Retrieve all sense keys from the device. + When encountering error during the process, + if retrieve sense keys before error encounterred, + return the sense keys with return status set to EFI_SUCCESS, + and NeedRetry set to FALSE; otherwize, return the proper return + status. Arguments: - ScsiDiskDevice - TODO: add argument description - NeedRetry - TODO: add argument description - SenseDataArray - TODO: add argument description - NumberOfSenseKeys - TODO: add argument description - AskResetIfError - TODO: add argument description - + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + NeedRetry - The pointer of flag indicates if need a retry + SenseDataArray - The pointer of an array of sense data + NumberOfSenseKeys - The number of sense key + AskResetIfError - The flag indicates if need reset when error occurs + Returns: - TODO: add return values + EFI_DEVICE_ERROR - Indicates that error occurs + EFI_SUCCESS - Successfully to request sense key --*/ ; @@ -401,16 +428,17 @@ ScsiDiskInquiryDevice ( Routine Description: - TODO: Add function description + Send out Inquiry command to Device Arguments: - ScsiDiskDevice - TODO: add argument description - NeedRetry - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + NeedRetry - Indicates if needs try again when error happens Returns: - TODO: add return values + EFI_DEVICE_ERROR - Indicates that error occurs + EFI_SUCCESS - Successfully to detect media --*/ ; @@ -423,15 +451,15 @@ ParseInquiryData ( Routine Description: - TODO: Add function description + Parse Inquiry data Arguments: - ScsiDiskDevice - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV Returns: - TODO: add return values + NONE --*/ ; @@ -447,18 +475,19 @@ ScsiDiskReadSectors ( Routine Description: - TODO: Add function description + Read sector from SCSI Disk Arguments: - ScsiDiskDevice - TODO: add argument description - Buffer - TODO: add argument description - Lba - TODO: add argument description - NumberOfBlocks - TODO: add argument description + ScsiDiskDevice - The poiniter of SCSI_DISK_DEV + Buffer - The buffer to fill in the read out data + Lba - Logic block address + NumberOfBlocks - The number of blocks to read Returns: - TODO: add return values + EFI_DEVICE_ERROR + EFI_SUCCESS --*/ ; @@ -474,18 +503,19 @@ ScsiDiskWriteSectors ( Routine Description: - TODO: Add function description + Write SCSI Disk sectors Arguments: - ScsiDiskDevice - TODO: add argument description - Buffer - TODO: add argument description - Lba - TODO: add argument description - NumberOfBlocks - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + Buffer - The data buffer to write sector + Lba - Logic block address + NumberOfBlocks - The number of blocks to write Returns: - TODO: add return values + EFI_DEVICE_ERROR + EFI_SUCCESS --*/ ; @@ -506,23 +536,23 @@ ScsiDiskRead10 ( Routine Description: - TODO: Add function description + Sumbmit Read command Arguments: - ScsiDiskDevice - TODO: add argument description - NeedRetry - TODO: add argument description - SenseDataArray - TODO: add argument description - NumberOfSenseKeys - TODO: add argument description - Timeout - TODO: add argument description - DataBuffer - TODO: add argument description - DataLength - TODO: add argument description - StartLba - TODO: add argument description - SectorSize - TODO: add argument description + ScsiDiskDevice - The pointer of ScsiDiskDevice + NeedRetry - The pointer of flag indicates if needs retry if error happens + SenseDataArray - The pointer of an array of sense data + NumberOfSenseKeys - The number of sense key + Timeout - The time to complete the command + DataBuffer - The buffer to fill with the read out data + DataLength - The length of buffer + StartLba - The start logic block address + SectorSize - The size of sector Returns: - TODO: add return values + EFI_STATUS --*/ ; @@ -543,23 +573,23 @@ ScsiDiskWrite10 ( Routine Description: - TODO: Add function description + Submit Write Command Arguments: - ScsiDiskDevice - TODO: add argument description - NeedRetry - TODO: add argument description - SenseDataArray - TODO: add argument description - NumberOfSenseKeys - TODO: add argument description - Timeout - TODO: add argument description - DataBuffer - TODO: add argument description - DataLength - TODO: add argument description - StartLba - TODO: add argument description - SectorSize - TODO: add argument description + ScsiDiskDevice - The pointer of ScsiDiskDevice + NeedRetry - The pointer of flag indicates if needs retry if error happens + SenseDataArray - The pointer of an array of sense data + NumberOfSenseKeys - The number of sense key + Timeout - The time to complete the command + DataBuffer - The buffer to fill with the read out data + DataLength - The length of buffer + StartLba - The start logic block address + SectorSize - The size of sector Returns: - TODO: add return values + EFI_STATUS --*/ ; @@ -573,16 +603,16 @@ GetMediaInfo ( Routine Description: - TODO: Add function description + Get information from media read capacity command Arguments: - ScsiDiskDevice - TODO: add argument description - Capacity - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV + Capacity - The pointer of EFI_SCSI_DISK_CAPACITY_DATA Returns: - TODO: add return values + NONE --*/ ; @@ -596,16 +626,16 @@ ScsiDiskIsNoMedia ( Routine Description: - TODO: Add function description + Check sense key to find if media presents Arguments: - SenseData - TODO: add argument description - SenseCounts - TODO: add argument description + SenseData - The pointer of EFI_SCSI_SENSE_DATA + SenseCounts - The number of sense key Returns: - TODO: add return values + BOOLEAN --*/ ; @@ -619,16 +649,16 @@ ScsiDiskIsMediaError ( Routine Description: - TODO: Add function description + Parse sense key Arguments: - SenseData - TODO: add argument description - SenseCounts - TODO: add argument description + SenseData - The pointer of EFI_SCSI_SENSE_DATA + SenseCounts - The number of sense key Returns: - TODO: add return values + BOOLEAN --*/ ; @@ -642,16 +672,16 @@ ScsiDiskIsHardwareError ( Routine Description: - TODO: Add function description + Check sense key to find if hardware error happens Arguments: - SenseData - TODO: add argument description - SenseCounts - TODO: add argument description + SenseData - The pointer of EFI_SCSI_SENSE_DATA + SenseCounts - The number of sense key Returns: - TODO: add return values + BOOLEAN --*/ ; @@ -665,16 +695,18 @@ ScsiDiskIsMediaChange ( Routine Description: - TODO: Add function description +Routine Description: + + Check sense key to find if media has changed Arguments: - SenseData - TODO: add argument description - SenseCounts - TODO: add argument description + SenseData - The pointer of EFI_SCSI_SENSE_DATA + SenseCounts - The number of sense key Returns: - TODO: add return values + BOOLEAN --*/ ; @@ -688,16 +720,16 @@ ScsiDiskIsResetBefore ( Routine Description: - TODO: Add function description + Check sense key to find if reset happens Arguments: - SenseData - TODO: add argument description - SenseCounts - TODO: add argument description + SenseData - The pointer of EFI_SCSI_SENSE_DATA + SenseCounts - The number of sense key Returns: - TODO: add return values + BOOLEAN --*/ ; @@ -712,17 +744,17 @@ ScsiDiskIsDriveReady ( Routine Description: - TODO: Add function description + Check sense key to find if the drive is ready Arguments: - SenseData - TODO: add argument description - SenseCounts - TODO: add argument description - NeedRetry - TODO: add argument description + SenseData - The pointer of EFI_SCSI_SENSE_DATA + SenseCounts - The number of sense key + RetryLater - The flag means if need a retry Returns: - TODO: add return values + BOOLEAN --*/ ; @@ -736,16 +768,16 @@ ScsiDiskHaveSenseKey ( Routine Description: - TODO: Add function description + Check sense key to find if it has sense key Arguments: - SenseData - TODO: add argument description - SenseCounts - TODO: add argument description + SenseData - The pointer of EFI_SCSI_SENSE_DATA + SenseCounts - The number of sense key Returns: - TODO: add return values + BOOLEAN --*/ ; @@ -758,15 +790,15 @@ ReleaseScsiDiskDeviceResources ( Routine Description: - TODO: Add function description + Release resource about disk device Arguments: - ScsiDiskDevice - TODO: add argument description + ScsiDiskDevice - The pointer of SCSI_DISK_DEV Returns: - TODO: add return values + NONE --*/ ;