Re: [Intel-gfx] [PATCH] intel_error_decode: use 64b gtt_offset

2014-04-29 Thread Ben Widawsky
On Tue, Apr 29, 2014 at 11:01:10AM +0200, Daniel Vetter wrote: > On Tue, Apr 29, 2014 at 10:52:44AM +0200, Daniel Vetter wrote: > > On Mon, Apr 28, 2014 at 06:45:50PM -0700, Ben Widawsky wrote: > > > See the relevant kernel patch for the details. I guess this breaks > > > support for older error st

Re: [Intel-gfx] [PATCH] intel_error_decode: use 64b gtt_offset

2014-04-29 Thread Daniel Vetter
On Tue, Apr 29, 2014 at 12:48 PM, Chris Wilson wrote: > On Tue, Apr 29, 2014 at 11:01:10AM +0200, Daniel Vetter wrote: >> On Tue, Apr 29, 2014 at 10:52:44AM +0200, Daniel Vetter wrote: >> > On Mon, Apr 28, 2014 at 06:45:50PM -0700, Ben Widawsky wrote: >> > > See the relevant kernel patch for the d

Re: [Intel-gfx] [PATCH] intel_error_decode: use 64b gtt_offset

2014-04-29 Thread Chris Wilson
On Tue, Apr 29, 2014 at 11:01:10AM +0200, Daniel Vetter wrote: > On Tue, Apr 29, 2014 at 10:52:44AM +0200, Daniel Vetter wrote: > > On Mon, Apr 28, 2014 at 06:45:50PM -0700, Ben Widawsky wrote: > > > See the relevant kernel patch for the details. I guess this breaks > > > support for older error st

Re: [Intel-gfx] [PATCH] intel_error_decode: use 64b gtt_offset

2014-04-29 Thread Daniel Vetter
On Tue, Apr 29, 2014 at 10:52:44AM +0200, Daniel Vetter wrote: > On Mon, Apr 28, 2014 at 06:45:50PM -0700, Ben Widawsky wrote: > > See the relevant kernel patch for the details. I guess this breaks > > support for older error state, I am not actually sure. Without > > versioning our error state tho

Re: [Intel-gfx] [PATCH] intel_error_decode: use 64b gtt_offset

2014-04-29 Thread Daniel Vetter
On Mon, Apr 28, 2014 at 06:45:50PM -0700, Ben Widawsky wrote: > See the relevant kernel patch for the details. I guess this breaks > support for older error state, I am not actually sure. Without > versioning our error state though, I cannot think of a better way. > Suggestions are welcome. Just d

[Intel-gfx] [PATCH] intel_error_decode: use 64b gtt_offset

2014-04-28 Thread Ben Widawsky
See the relevant kernel patch for the details. I guess this breaks support for older error state, I am not actually sure. Without versioning our error state though, I cannot think of a better way. Suggestions are welcome. Signed-off-by: Ben Widawsky --- tools/intel_error_decode.c | 14 +++---