On 8/6/21 1:34 PM, Thomas Hellström (Intel) wrote:
Hi,
On 8/3/21 7:26 PM, Matthew Brost wrote:
On Tue, Aug 03, 2021 at 02:15:13PM +0200, Daniel Vetter wrote:
On Tue, Aug 3, 2021 at 6:53 AM Matthew Brost
<matthew.brost@xxxxxxxxx> wrote:
Minimum set of patches to enable GuC submission on DG1 and enable
it by
default.
A little difficult to test as IGTs do not work with DG1 due to a bunch
of uAPI features being disabled (e.g. relocations, caching memory
options, etc...).
Matt Auld has an igt series which fixes a lot of this stuff, would be
good to do at least a Test-With run with that.
It looks like Maarten now merged Matt's series to IGT.
There is a series on IGT trybot with pending work to have some igt
tests support relocations,
https://patchwork.freedesktop.org/series/92043/
One of the tests that have WIP fixes is gem_exec_whisper, and that
particular test has historically shown occasional hangs with GuC
submission on DG1 so it would be very desirable if we could make that
test in particular work (I haven't verified that that's the case)
reliably.
Also the following series:
https://patchwork.freedesktop.org/series/93455/
tries a bit harder to get some more tests running, squashing the above
series on top of latest IGT.
Thanks,
/Thomas
And also while we're working on getting igt adapted to uapi changes and
to get more LMEM coverage in there, an IMO relevant test case to run
manually is "piglit quick"on top of DG1-enabled OpenGL checking for
regressions and significant changes in execution time.
/Thomas