Re: [Intel-gfx] [PATCH 1/2] drm/i915/gem: Replace reloc chain with terminator on error unwind

2020-09-08 Thread Pavel Machek
Hi! > > > > Thanks for the patches. I assume this should fix problem from > > > > "5.9-rc1: graphics regression moved from -next to mainline" thread. > > > > > > > > I have applied them over current -next, and my machine seems to be > > > > working so far (but uptime is less than 30 minutes). >

Re: [Intel-gfx] [PATCH 1/2] drm/i915/gem: Replace reloc chain with terminator on error unwind

2020-08-20 Thread Chris Wilson
Quoting Pavel Machek (2020-08-19 20:33:26) > Hi! > > > > > If we hit an error during construction of the reloc chain, we need to > > > > replace the chain into the next batch with the terminator so that upon > > > > flushing the relocations so far, we do not execute a hanging batch. > > > > > >

Re: [Intel-gfx] [PATCH 1/2] drm/i915/gem: Replace reloc chain with terminator on error unwind

2020-08-19 Thread Chris Wilson
Quoting Pavel Machek (2020-08-19 20:47:23) > Hi! > > > > Yep, my machines are low on memory. > > > > > > But ... test did not work that well. I have dead X and blinking > > > screen. Machine still works reasonably well over ssh, so I guess > > > that's an improvement. > > > > > [ 7744.718473]

Re: [Intel-gfx] [PATCH 1/2] drm/i915/gem: Replace reloc chain with terminator on error unwind

2020-08-19 Thread Pavel Machek
Hi! > > Yep, my machines are low on memory. > > > > But ... test did not work that well. I have dead X and blinking > > screen. Machine still works reasonably well over ssh, so I guess > > that's an improvement. > > > [ 7744.718473] BUG: unable to handle page fault for address: f8c0 > > [

Re: [Intel-gfx] [PATCH 1/2] drm/i915/gem: Replace reloc chain with terminator on error unwind

2020-08-19 Thread Chris Wilson
Quoting Pavel Machek (2020-08-19 20:33:26) > Hi! > > > > > If we hit an error during construction of the reloc chain, we need to > > > > replace the chain into the next batch with the terminator so that upon > > > > flushing the relocations so far, we do not execute a hanging batch. > > > > > >

Re: [Intel-gfx] [PATCH 1/2] drm/i915/gem: Replace reloc chain with terminator on error unwind

2020-08-19 Thread Pavel Machek
Hi! > > > If we hit an error during construction of the reloc chain, we need to > > > replace the chain into the next batch with the terminator so that upon > > > flushing the relocations so far, we do not execute a hanging batch. > > > > Thanks for the patches. I assume this should fix problem

Re: [Intel-gfx] [PATCH 1/2] drm/i915/gem: Replace reloc chain with terminator on error unwind

2020-08-19 Thread Chris Wilson
Quoting Pavel Machek (2020-08-19 18:23:31) > Hi! > > > If we hit an error during construction of the reloc chain, we need to > > replace the chain into the next batch with the terminator so that upon > > flushing the relocations so far, we do not execute a hanging batch. > > Thanks for the

Re: [Intel-gfx] [PATCH 1/2] drm/i915/gem: Replace reloc chain with terminator on error unwind

2020-08-19 Thread Pavel Machek
Hi! > If we hit an error during construction of the reloc chain, we need to > replace the chain into the next batch with the terminator so that upon > flushing the relocations so far, we do not execute a hanging batch. Thanks for the patches. I assume this should fix problem from "5.9-rc1:

[Intel-gfx] [PATCH 1/2] drm/i915/gem: Replace reloc chain with terminator on error unwind

2020-08-19 Thread Chris Wilson
If we hit an error during construction of the reloc chain, we need to replace the chain into the next batch with the terminator so that upon flushing the relocations so far, we do not execute a hanging batch. Reported-by: Pavel Machek Fixes: 964a9b0f611e ("drm/i915/gem: Use chained reloc