Patch Set 1:

Also, the CDS timeout would be announced through either a an actual timeout or through FCERR aborting the loop. This cosmetic incorrectness is there with and without this patch.

Overall I think the error message should distinguish between those two exit criteria, but (the rest of) this patch should be abandoned.

And I should have kept reading. The code is actually already doing this correctly without any patch. So it should be fine to just abandon this, as it is a no-op as far as I can tell

Good point, thanks for noticing.

Patch set 1:Code-Review -1

View Change

To view, visit change 46325. To unsubscribe, or for help writing mail filters, visit settings.