> > > > Whether XhcCheckUrbResult returns correct error status in case of > > failure? > > > > XhcCheckUrbResult function header indicates it should report URB > > > > transfer state - Urb->Finished (seems to align with EhciDxe) but > > > > the implementation is trying to return EFI_STATUS and it doesn't > > > > get updated based on EvtTrb- > > > > >Completecode. With a failing drive, XhcExecTransfer returns > > > > >EFI_SUCCESS > > > in > > > > case of transaction error. > > > > > > > Looking at the source code for XhcCheckUrbResult, if you pass in a > > > URB Pointer, the > > > Urb->Result is updated with the completion code from the event. > > > Look for the switch on EvtTrb->Completecode. In the case where Urb > > > is passed in, CheckedUrb == Urb. > > > EFI_STATUS that is returned indicates if the function actually > > > checked any new events. > > > > > > Regards, > > > Eric > > > > > If return value of XhcCheckUrbResult is a status of whether it checked > > any new events or not, then XhcBulkTransfer needs to be updated to > > return correct failure status to the caller instead of checking for > EFI_USB_ERR_STALL > > and EFI_USB_NOERROR alone . > > > > > Error status is returned to the caller in TransferResult ( the last parameter > to > XhcBulkTransfer ). > The check you are talking about is used to recover the endpoint to a running > state if it > stalled. > Currently if there is a transaction error, the caller will find > EFI_USB_ERR_TIMEOUT in > TransferResult when XhcBulkTransfer completes. > What do you think should be done differently? > > Regards, > Eric > > > It needs to be consistent between USB controller modules - Ehci/Xhci. XHCI BulkTransfer returns success if event is checked and caller is expected to look for TransferResult. EHCI module passes error back to caller in Status. UsbBot device driver checks for status on bulk transfer call and it looks for TransferResult if status is not success.
-Baranee ------------------------------------------------------------------------------ Monitor 25 network devices or servers for free with OpManager! OpManager is web-based network management software that monitors network devices and physical & virtual servers, alerts via email & sms for fault. Monitor 25 devices for free with no restriction. Download now http://ad.doubleclick.net/ddm/clk/292181274;119417398;o _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel