Add a reset interface

2011-01-31 Thread Dave Airlie
On Sun, Jan 30, 2011 at 9:50 PM, Chris Wilson wrote: > On Mon, 24 Jan 2011 15:55:27 +, Chris Wilson chris-wilson.co.uk> wrote: >> For i915 there is a need to invalidate some cached state after resuming or >> reseting the GPU. This is not quite the same as simply restoring saved >> state (i.e

Add a reset interface

2011-01-30 Thread Chris Wilson
On Mon, 31 Jan 2011 06:03:25 +1000, Dave Airlie wrote: > It seems like an interface we could use, I just wasn't sure how a regression > fix > for something in. 38 isn't a revert but requires a whole new interface > in the core > drm to fix. Perhaps if you can enlighten me on that I'd be happier.

Re: Add a reset interface

2011-01-30 Thread Chris Wilson
On Mon, 31 Jan 2011 06:03:25 +1000, Dave Airlie wrote: > It seems like an interface we could use, I just wasn't sure how a regression > fix > for something in. 38 isn't a revert but requires a whole new interface > in the core > drm to fix. Perhaps if you can enlighten me on that I'd be happier.

Re: Add a reset interface

2011-01-30 Thread Dave Airlie
On Sun, Jan 30, 2011 at 9:50 PM, Chris Wilson wrote: > On Mon, 24 Jan 2011 15:55:27 +, Chris Wilson > wrote: >> For i915 there is a need to invalidate some cached state after resuming or >> reseting the GPU. This is not quite the same as simply restoring saved >> state (i.e. the standard sus

Add a reset interface

2011-01-30 Thread Chris Wilson
On Mon, 24 Jan 2011 15:55:27 +, Chris Wilson wrote: > For i915 there is a need to invalidate some cached state after resuming or > reseting the GPU. This is not quite the same as simply restoring saved > state (i.e. the standard suspend resume method), so do not seem to merit > reusing the sa

Re: Add a reset interface

2011-01-30 Thread Chris Wilson
On Mon, 24 Jan 2011 15:55:27 +, Chris Wilson wrote: > For i915 there is a need to invalidate some cached state after resuming or > reseting the GPU. This is not quite the same as simply restoring saved > state (i.e. the standard suspend resume method), so do not seem to merit > reusing the sa

Add a reset interface

2011-01-24 Thread Takashi Iwai
At Mon, 24 Jan 2011 15:55:27 +, Chris Wilson wrote: > > For i915 there is a need to invalidate some cached state after resuming or > reseting the GPU. This is not quite the same as simply restoring saved > state (i.e. the standard suspend resume method), so do not seem to merit > reusing the s

Add a reset interface

2011-01-24 Thread Chris Wilson
For i915 there is a need to invalidate some cached state after resuming or reseting the GPU. This is not quite the same as simply restoring saved state (i.e. the standard suspend resume method), so do not seem to merit reusing the save|restore vfuncs. Instead I propose a drm_mode_config_reset(

Re: Add a reset interface

2011-01-24 Thread Takashi Iwai
At Mon, 24 Jan 2011 15:55:27 +, Chris Wilson wrote: > > For i915 there is a need to invalidate some cached state after resuming or > reseting the GPU. This is not quite the same as simply restoring saved > state (i.e. the standard suspend resume method), so do not seem to merit > reusing the s

Add a reset interface

2011-01-24 Thread Chris Wilson
For i915 there is a need to invalidate some cached state after resuming or reseting the GPU. This is not quite the same as simply restoring saved state (i.e. the standard suspend resume method), so do not seem to merit reusing the save|restore vfuncs. Instead I propose a drm_mode_config_reset(