Re: [PATCH 00/15] Batch submission via GuC

Chris Wilson
2015-06-24T12:57:30Z

On Wed, Jun 24, 2015 at 02:16:05PM +0200, Daniel Vetter wrote:
> - For async gem init it's tempting to repurpose the gpu reset code with
> all the existing synchronization: We'd only need to set the
> RESET_IN_PROGRESS flag synchronously and then run a gpu reset after the
> firmware is loaded, all from the async worker. The problem with that is
> that we still have issues with rogue -EIO escaping into the modeset code
> while a reset is pending, and that would not be good since we want
> modesets to work right away ofc. Also most modeset paths need to stall
> for gpu resets synchronously, which again isn't what we want.

We already have async GEM init! It is the source of a bug I keep
reporting.

async GEM init is trivial, the memory management code is independent of
request submission. Request submission can be delayed indefinitely until
the submission port is ready. The only effect is that we have to be
careful when doing hangcheck to be sure that the submission port is
running before declaring it hung.
-Chris

--
Chris Wilson, Intel Open Source Technology Centre