[PATCH 4/7] anv: Support wait for heterogeneous list of fences [v2]

Jason Ekstrand jason at jlekstrand.net
Wed Jun 20 02:22:26 UTC 2018


On Tue, Jun 19, 2018 at 6:22 PM, Keith Packard <keithp at keithp.com> wrote:

> Jason Ekstrand <jason at jlekstrand.net> writes:
>
> > I suppose we probably shouldn't worry about current_time being greater
> than
> > INT64_MAX?  I guess if that happens we have pretty big problems...
>
> Nope, we've already given up on that working -- it's a couple hundred
> years of system uptime. Neither of us have any concerns in this area.
>
> >>            timeout = MIN2(max_timeout, timeout);
> >>
> >>            return (current_time + timeout);
> >>         }
> >>
> >
> > Yeah, I think that's better.
>
> Cool. I'll wait for further review :-)
>

I don't think I have any more comments on this patch.  It's gross but I
think it should work.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20180619/6e02d6f7/attachment.html>


More information about the dri-devel mailing list