MdeModulePkg ScsiBusDxe: Only signal caller event when PassThru() succeeds

In ScsiExecuteSCSICommand(), when SCSI devices do not support non-blocking
I/O but an event is passed from caller (UefiScsiLib), the function will
execute the SCSI I/O command in a blocking manner and signal the caller
event when the command completes.

Originally, caller event from UefiScsiLib will be signaled if the SCSI
command fails. UefiScsiLib will continue to signal its caller (BlockIO2
request from ScsiDiskDxe driver), which is not aligned with the UEFI spec
that event will not be signaled when BlockIO2 request returns with error.

This commit will signal ScsiExecuteSCSICommand()'s caller event only when
the SCSI command succeeds.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Hao Wu <hao.a.wu@intel.com>
Reviewed-by: Feng Tian <feng.tian@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@19447 6f19259b-4bc3-4df7-8a09-765794883524
This commit is contained in:
Hao Wu 2015-12-22 13:55:34 +00:00 committed by hwu1225
parent 66ca22d05c
commit 9f840a9ea3
1 changed files with 3 additions and 2 deletions

View File

@ -1007,9 +1007,10 @@ ScsiExecuteSCSICommand (
ExtRequestPacket, ExtRequestPacket,
NULL NULL
); );
if (Event != NULL) { if ((!EFI_ERROR(Status)) && (Event != NULL)) {
// //
// Signal Event to tell caller to pick up the SCSI IO Packet. // Signal Event to tell caller to pick up the SCSI IO packet if the
// PassThru() succeeds.
// //
gBS->SignalEvent (Event); gBS->SignalEvent (Event);
} }