From: Ethan Zhao <haifeng.zhao@xxxxxxxxxxxxxxx>if in the end the agreement is to remove the restart logic, then do it.
Sent: Thursday, December 28, 2023 9:10 PM
On 12/28/2023 4:38 PM, Tian, Kevin wrote:
This change is intended to break the restar login when device-TLBFrom: Ethan Zhao <haifeng.zhao@xxxxxxxxxxxxxxx>this change is moot.
Sent: Thursday, December 28, 2023 8:17 AM
- if (rc == -EAGAIN)
+ if (rc == -EAGAIN && type !=QI_DIOTLB_TYPE && type !=
QI_DEIOTLB_TYPE)
goto restart;
-EAGAIN is set only when hardware detects a ATS invalidation completion
timeout in qi_check_fault(). so above just essentially kills the restart logic.
invalidation timeout happens, we don't know how long the ITE took
if the device is just no reponse.
it's not good to introduce a change which essentially kills the restart
logic but still keeps the related code.