On 9/9/20 5:34 AM, Dmitry Osipenko wrote:
09.09.2020 05:10, Dmitry Osipenko пишет:
05.09.2020 13:34, Mikko Perttunen пишет:
+       job->timeout = min(args->timeout_us / 1000, 10000U);
+       if (job->timeout == 0)
+               job->timeout = 1;

clamp()


Will fix.


Does it make sense to have timeout in microseconds?


Not sure, but better have it a bit more fine-grained rather than coarse-grained. This still gives a maximum timeout of 71 minutes so I don't think it has any negatives compared to milliseconds.

Mikko
_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to