On 2021-12-02 7:43 a.m., Mike Lothian
wrote:
On Thu, 25 Nov 2021 at 20:42, Felix Kuehling <felix.kuehling@xxxxxxx> wrote:OK. Dealing with processed timestamp rather than decoded timestamp fixes the race condition where drain would have returned before the last fault was processed. But we're still assuming that each interrupt has a unique timestamp. We haven't seen evidence of the contrary. But I'd like to add a check to be sure, in case that assumption becomes invalid on future hardware. Something like this (before updating ih->processed_timestamp): WARN_ONCE(ih->processed_timestamp == entry.timestamp, "IH timestamps are not unique"); With that fixed, the patch is Reviewed-by: Felix Kuehling <Felix.Kuehling@xxxxxxx>Hi, I'm seeing this on my machine https://nam11.safelinks.protection.outlook.com/?url="">
Thanks for the report, the warning is for chip RENOIR, right after device init done. The assumption of unique timestamp of IH vector is invalid, I will submit another patch to fix the drain fault interrupt logic and remove this WARN_ONCE.
Regards,
Philip
Cheers Mike
