[Bug 4016] SCT StorageSecurityCommandProtocolTest ReceiveData_Conf failed with latest edk2 code base


bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4016

--- Comment #2 from Mateusz Albecki <mateusz.albecki@...> ---
I think it is a bug in SATA driver. The bug is that the Status variable is used
to store ErrorRecovery data and overrides packet transfer status on last
iteration of the retry loop:

in
https://github.com/tianocore/edk2/commit/64e25d4b062c907dab2dd30b686de9219d8e372c

AhciMode.c:976

The correct flow should be to return packet transfer/operation status. Bug is
present in both PIO and DMA transfer.

--
You are receiving this mail because:
You are on the CC list for the bug.