Re: [Intel-gfx] [PATCH 12/12] README: Add quick_dump info

2013-02-07 Thread Ben Widawsky
On Fri, Feb 08, 2013 at 12:18:31AM +0100, Paul Menzel wrote: > Am Donnerstag, den 07.02.2013, 10:39 -0800 schrieb Ben Widawsky: > > Thank you for updating documentation! > > > Signed-off-by: Ben Widawsky > > --- > > README | 15 +++ > > 1 file changed, 15 insertions(+) > > > > diff

Re: [Intel-gfx] [PATCH 12/12] README: Add quick_dump info

2013-02-07 Thread Paul Menzel
Am Donnerstag, den 07.02.2013, 10:39 -0800 schrieb Ben Widawsky: Thank you for updating documentation! > Signed-off-by: Ben Widawsky > --- > README | 15 +++ > 1 file changed, 15 insertions(+) > > diff --git a/README b/README > index 0ec53fb..5ede794 100644 > --- a/README > +++ b/R

Re: [Intel-gfx] [PATCH] Build: Add --disable-tests configure flag to avoid tests build.

2013-02-07 Thread Ben Widawsky
On Thu, Feb 07, 2013 at 09:52:32PM +, Damien Lespiau wrote: > On Mon, Feb 04, 2013 at 07:52:27PM -0200, Rodrigo Vivi wrote: > > Tests are still being built by default. However this request > > came from OSVs in order to allow them to include i-g-t in their > > distributions by default avoiding

Re: [Intel-gfx] [PATCH] Build: Add --disable-tests configure flag to avoid tests build.

2013-02-07 Thread Damien Lespiau
On Mon, Feb 04, 2013 at 07:52:27PM -0200, Rodrigo Vivi wrote: > Tests are still being built by default. However this request > came from OSVs in order to allow them to include i-g-t in their > distributions by default avoiding adding more and more dependencies > since we are improving and adding mo

[Intel-gfx] [PATCH] drm/i915: Clarify HW context size logic

2013-02-07 Thread Ben Widawsky
This was a rebase error from when the patches originally landed. Since the context size is unsigned, there is also no use in checking if it's less than 0. The existing code is not really wrong, but it's not simple as it should be. Signed-off-by: Ben Widawsky --- drivers/gpu/drm/i915/i915_gem_co

[Intel-gfx] [PATCH 12/12] README: Add quick_dump info

2013-02-07 Thread Ben Widawsky
Signed-off-by: Ben Widawsky --- README | 15 +++ 1 file changed, 15 insertions(+) diff --git a/README b/README index 0ec53fb..5ede794 100644 --- a/README +++ b/README @@ -75,6 +75,21 @@ tools/ These tools generally must be run as root, safe for the ones that just deco

[Intel-gfx] [PATCH 11/12] quick_dump: Give dumper an AM_CONDITIONAL

2013-02-07 Thread Ben Widawsky
It may sometimes be undesirable to build or install the quick dumper. This was requested by Damien. Cc: Damien Lespiau Signed-off-by: Ben Widawsky --- configure.ac | 19 ++- tools/Makefile.am | 2 ++ 2 files changed, 16 insertions(+), 5 deletions(-) diff --git a/configure

[Intel-gfx] [PATCH 10/12] quick_dump: Use the register access library

2013-02-07 Thread Ben Widawsky
Signed-off-by: Ben Widawsky --- tools/quick_dump/quick_dump.py | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/tools/quick_dump/quick_dump.py b/tools/quick_dump/quick_dump.py index 44aa2ba..5c88ef1 100755 --- a/tools/quick_dump/quick_dump.py +++ b/tools/quick_dump/

[Intel-gfx] [PATCH 09/12] quick_dump: Connect libpciaccess and other utils

2013-02-07 Thread Ben Widawsky
Make a register access library with sample to do register reads Signed-off-by: Ben Widawsky --- tools/quick_dump/Makefile.am | 14 +- tools/quick_dump/chipset.i | 16 ++-- tools/quick_dump/intel_chipset.c | 7 +++ tools/quick_dump/quick_dump.py | 5 ++---

[Intel-gfx] [PATCH 08/12] quick_dump: SWIG chipset interface

2013-02-07 Thread Ben Widawsky
This isn't strictly necessary it would have been easy enough to simply convert intel_chipset.h but this should be nice prep work for directly doing MMIO. It also serves as a nice review point. It's demonstrated with an autodetect function in the script. That autodetect has a hardcoded path that sh

[Intel-gfx] [PATCH 06/12] quick_dump: vlv support

2013-02-07 Thread Ben Widawsky
This patch includes a patch from Jesse which removed a bunch of VLV registers which were useless in my original RFC. Cc: Jesse Barnes Signed-off-by: Ben Widawsky --- tools/quick_dump/Makefile.am | 1 + tools/quick_dump/valleyview | 6 +++ tools/quick_dump/vlv_display.txt | 84 +++

[Intel-gfx] [PATCH 07/12] configure.ac: Add swig dependency

2013-02-07 Thread Ben Widawsky
As far as I can tell (and recommended to me by Matt) taking these m4 extension macros from http://www.gnu.org/software/autoconf-archive/ doesn't require the project distribute GPL. I am a bit confused from reading the license. I'd really hope someone can comment. The only other solution would be t

[Intel-gfx] [PATCH 05/12] quick_dump: gen7 support

2013-02-07 Thread Ben Widawsky
Signed-off-by: Ben Widawsky --- tools/quick_dump/Makefile.am| 1 + tools/quick_dump/gen7_other.txt | 1 + tools/quick_dump/ivybridge | 1 + 3 files changed, 3 insertions(+) create mode 100644 tools/quick_dump/gen7_other.txt create mode 100644 tools/quick_dump/ivybridge diff --git a/to

[Intel-gfx] [PATCH 04/12] quick_dump: gen6 support

2013-02-07 Thread Ben Widawsky
Signed-off-by: Ben Widawsky --- tools/quick_dump/Makefile.am| 1 + tools/quick_dump/gen6_other.txt | 1 + tools/quick_dump/sandybridge| 1 + 3 files changed, 3 insertions(+) create mode 100644 tools/quick_dump/gen6_other.txt create mode 100644 tools/quick_dump/sandybridge diff --git a/

[Intel-gfx] [PATCH 03/12] quick_dump: A dump utility different than reg_dumper

2013-02-07 Thread Ben Widawsky
This is the base tool for quick dump. At it's heart, quick dump is simply a basic text parsing thingie which plugs into intel-gpu-tools to do something similar to intel_reg_dumper. The format for the register definition files is very open, so it's just something simple for now. Signed-off-by: Ben

[Intel-gfx] [PATCH 02/12] configure.ac: Move prime stuff to where it belongs

2013-02-07 Thread Ben Widawsky
Signed-off-by: Ben Widawsky --- configure.ac | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/configure.ac b/configure.ac index d66c9f9..1381279 100644 --- a/configure.ac +++ b/configure.ac @@ -59,16 +59,6 @@ XORG_DEFAULT_OPTIONS PKG_CHECK_MODULES(DRM, [

[Intel-gfx] [PATCH 01/12] configure.ac: slightly more standard --disable-tests

2013-02-07 Thread Ben Widawsky
This partially reverts commit 7c170724003f42cd4ceea9f6c805292bbe2e81b2 Author: Rodrigo Vivi Date: Tue Feb 5 16:17:54 2013 -0200 Build: Add --disable-tests configure flag to avoid tests build I was still in the middle of arguing this when Daniel merged it. Cc: Rodrigo Vivi Cc: Daniel Vett

Re: [Intel-gfx] [PATCH 2/8] drm/i915: Introduce i915_gem_object_create_stolen_for_preallocated

2013-02-07 Thread Ben Widawsky
On Thu, Feb 07, 2013 at 02:32:58PM +, Chris Wilson wrote: > On Wed, Feb 06, 2013 at 09:09:34PM -0800, Ben Widawsky wrote: > > On Wed, Feb 06, 2013 at 11:10:22AM +, Chris Wilson wrote: > > > Wrap a preallocated region of stolen memory within an ordinary GEM > > > object, for example the BIOS

Re: [Intel-gfx] [PATCH] drm/i915: implement MCH pipe underrun prevention for snb/ivb

2013-02-07 Thread Ben Widawsky
On Thu, Feb 07, 2013 at 03:23:24PM +0100, Daniel Vetter wrote: > Some early bios versions seem to ship with the wrong tuning values for > the MCH, possible resulting in pipe underruns under load. Especially > on DP outputs this can lead to black screen, since DP really doesn't > like an occasional

[Intel-gfx] [PATCH] drm/i915: implement MCH pipe underrun prevention for snb/ivb

2013-02-07 Thread Daniel Vetter
Some early bios versions seem to ship with the wrong tuning values for the MCH, possible resulting in pipe underruns under load. Especially on DP outputs this can lead to black screen, since DP really doesn't like an occasional whack from an underrun. Unfortunately the registers seem to be locked

Re: [Intel-gfx] [PATCH 2/8] drm/i915: Introduce i915_gem_object_create_stolen_for_preallocated

2013-02-07 Thread Chris Wilson
On Wed, Feb 06, 2013 at 09:09:34PM -0800, Ben Widawsky wrote: > On Wed, Feb 06, 2013 at 11:10:22AM +, Chris Wilson wrote: > > Wrap a preallocated region of stolen memory within an ordinary GEM > > object, for example the BIOS framebuffer. > > > > Signed-off-by: Chris Wilson > > --- > > drive

Re: [Intel-gfx] [RFC] [PATCH 7/7] drm/i915: find guilty batch buffer on ring resets

2013-02-07 Thread Ville Syrjälä
On Mon, Feb 04, 2013 at 04:04:43PM +0200, Mika Kuoppala wrote: > After hang check timer has declared gpu to be hang, > rings are reset. In ring reset, when clearing > request list, do post mortem analysis to find out > the guilty batch buffer. > > Select requests for further analysis by inspecting

Re: [Intel-gfx] [PATCH 1/8] drm/i915: Skip modifying PCH DREF if not changing clock sources

2013-02-07 Thread Daniel Vetter
On Thu, Feb 7, 2013 at 1:45 PM, Paulo Zanoni wrote: > 2013/2/6 Chris Wilson : >> Modifying the clock sources (via the DREF control on the PCH) is a slow >> multi-stage process as we need to let the clocks stabilise between each >> stage. If we are not actually changing the clock sources, then we c

Re: [Intel-gfx] [PATCH] drm/i915: unify HDMI/DP hpd definitions

2013-02-07 Thread Paulo Zanoni
2013/2/7 Daniel Vetter : > They're physically the same pins and also the same bits, duplicating > only confuses the reader. This also makes it a bit obvious that we > have quite some code duplication going on here. Squashing that is for > a larger rework in our hpd handling though. > > Signed-off-b

Re: [Intel-gfx] [PATCH 6/8] drm/i915: Retrieve the current mode upon KMS takeover

2013-02-07 Thread Paulo Zanoni
Hi 2013/2/6 Chris Wilson : > Read the current hardware state to retrieve the active mode and populate > our CRTC config if that mode matches our presumptions. On Haswell you need to read TRANS_DDI_FUNC_CTL(cpu_transcoder), check for bits TRANS_DDI_PVSYNC and TRANS_DDI_PHSYNC. > > Signed-off-by:

Re: [Intel-gfx] [PATCH 1/8] drm/i915: Skip modifying PCH DREF if not changing clock sources

2013-02-07 Thread Paulo Zanoni
Hi 2013/2/6 Chris Wilson : > Modifying the clock sources (via the DREF control on the PCH) is a slow > multi-stage process as we need to let the clocks stabilise between each > stage. If we are not actually changing the clock sources, then we can > return early. > > Signed-off-by: Chris Wilson T

[Intel-gfx] [PATCH] drm/i915: unify HDMI/DP hpd definitions

2013-02-07 Thread Daniel Vetter
They're physically the same pins and also the same bits, duplicating only confuses the reader. This also makes it a bit obvious that we have quite some code duplication going on here. Squashing that is for a larger rework in our hpd handling though. Signed-off-by: Daniel Vetter --- drivers/gpu/d

[Intel-gfx] [PATCH] drm/i915: After hibernation, discard the unbound list

2013-02-07 Thread Chris Wilson
The unbound list is an optimisation to track objects that have been evicted from the GTT but remain untouched by the CPU. After hibernation, all memory is in the CPU domain (having been restored from a disk image) and so we need to restore the domain tracking upon the objects. However, for the unbo

Re: [Intel-gfx] [Alsa-user] intel-hda: sound via HDMI only when using interlaced modes

2013-02-07 Thread Daniel Vetter
On Wed, Feb 06, 2013 at 10:35:33PM +0100, David Härdeman wrote: > I'll break etiquette here and include the entire original message below > (and top-post!) since I'm sending this to intel-gfx as well. > > Since the previous mail I've tested a more recent kernel (3.8-rc6), > swapping HDMI cables an

[Intel-gfx] [PATCH] drm/i915: After hibernation, discard the unbound list

2013-02-07 Thread Chris Wilson
The unbound list is an optimisation to track objects that have been evicted from the GTT but remain untouched by the CPU. After hibernation, all memory is in the CPU domain (having been restored from a disk image) and so we need to restore the domain tracking upon the objects. However, for the unbo