Re: [Intel-gfx] [PATCH 06/24] drm/i915: Convert the dmabuf object to use the new i915_gem_object_ops

2012-09-14 Thread Daniel Vetter
On Tue, Sep 04, 2012 at 09:02:58PM +0100, Chris Wilson wrote: > By providing a callback for when we need to bind the pages, and then > release them again later, we can shorten the amount of time we hold the > foreign pages mapped and pinned, and importantly the dmabuf objects then > behave as any o

Re: [Intel-gfx] [PATCH 4/4] drm/i915: update dpms property in set_mode

2012-09-14 Thread Daniel Vetter
On Thu, Sep 13, 2012 at 01:42:01PM -0700, Jesse Barnes wrote: > On Thu, 6 Sep 2012 22:08:35 +0200 > Daniel Vetter wrote: > > > Hopefully this makes userspace slightly less confused about us > > frobbing the dpms state behind its back. Yeah, it would be better > > to be more careful with not chan

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Limit the ioremap of the PCI bar to the registers

2012-09-14 Thread Daniel Vetter
On Fri, Sep 14, 2012 at 11:50:30AM -0700, Ben Widawsky wrote: > On Fri, 14 Sep 2012 11:57:46 +0100 > Chris Wilson wrote: > > > In the future we may like to experiment with using a WC map of the GTT > > portion. However, that will conflict with i915.ko mapping the entire bar > > as UC in order to

Re: [Intel-gfx] [pull] drm-intel-next

2012-09-14 Thread Paulo Zanoni
Hi 2012/9/14 Daniel Vetter : > On Fri, Sep 14, 2012 at 09:55:58AM -0400, Bobby Powers wrote: >> This tree gives me recursive dependency problems, which ends up >> removing a big (& important) part of my .config: >> >> [bpowers@fina linux]$ git reset --hard drm-intel-next-2012-09-09 >> HEAD is now

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Limit the ioremap of the PCI bar to the registers

2012-09-14 Thread Ben Widawsky
On Fri, 14 Sep 2012 11:57:46 +0100 Chris Wilson wrote: > In the future we may like to experiment with using a WC map of the GTT > portion. However, that will conflict with i915.ko mapping the entire bar > as UC in order to access the GPU registers. Instead we can shrink the > register ioremap to

Re: [Intel-gfx] [PATCH] sysfs rps test added

2012-09-14 Thread Daniel Vetter
On Fri, Sep 07, 2012 at 07:43:45PM -0700, Ben Widawsky wrote: > Signed-off-by: Ben Widawsky > --- One small improvement would be nice: [snip] > +static void writeval(FILE *filp, int val) > +{ > + rewind(filp); > + fprintf(filp, "%d", val); > + fflush(filp); > +} Adding some assert

Re: [Intel-gfx] [PATCH 06/24] drm/i915: Convert the dmabuf object to use the new i915_gem_object_ops

2012-09-14 Thread Chris Wilson
On Fri, 14 Sep 2012 11:02:02 -0700, Ben Widawsky wrote: > On Tue, 4 Sep 2012 21:02:58 +0100 > Chris Wilson wrote: > > @@ -3731,9 +3731,6 @@ void i915_gem_free_object(struct drm_gem_object > > *gem_obj) > > > > trace_i915_gem_object_destroy(obj); > > > > - if (gem_obj->import_attach) >

Re: [Intel-gfx] i915/drm: SNB gpu won't throttle up with latest forcewake/gpufreq tracepoint drm patches

2012-09-14 Thread Nicolas Kalkhof
Hi Daniel,ok fixing cpu freq static to max (2700MHz) on both cores does increase performance a little bit however gpu freq reported by /sys/class/drm/card0/gt_cur_freq_mhz stays fixed at 650 MHz. -Nic Gesendet: Freitag, 14. September 2012 um 18:44 Uhr Von: "Daniel Vetter"

Re: [Intel-gfx] [PATCH 06/24] drm/i915: Convert the dmabuf object to use the new i915_gem_object_ops

2012-09-14 Thread Ben Widawsky
On Tue, 4 Sep 2012 21:02:58 +0100 Chris Wilson wrote: > By providing a callback for when we need to bind the pages, and then > release them again later, we can shorten the amount of time we hold the > foreign pages mapped and pinned, and importantly the dmabuf objects then > behave as any other

Re: [Intel-gfx] [REPOST PATCH 7/7] drm/i915: Show render P state thresholds in sysfs

2012-09-14 Thread Daniel Vetter
On Fri, Sep 07, 2012 at 07:43:44PM -0700, Ben Widawsky wrote: > This is useful for userspace utilities which wish to use the previous > interface, specifically for micromanaging the increase/decrease steps by > setting min == max. > > Reviewed-by: Chris Wilson > Signed-off-by: Ben Widawsky I've

Re: [Intel-gfx] i915/drm: SNB gpu won't throttle up with latest forcewake/gpufreq tracepoint drm patches

2012-09-14 Thread Daniel Vetter
On Fri, Sep 14, 2012 at 06:23:59PM +0200, Nicolas Kalkhof wrote: > Hello, > > the patchset from ~danvet/drm-intel git from earlier september concerning > forcewake and gpu frequency made my SNB much calmer during idle times. However > the GPU won't hit the turbo throttle when the load goes up res

Re: [Intel-gfx] [PATCH] drm/i915: rip out edp special case from dp_link_down

2012-09-14 Thread Daniel Vetter
On Thu, Sep 13, 2012 at 07:17:55PM -0300, Paulo Zanoni wrote: > Hi > > 2012/9/9 Daniel Vetter : > > This has been tons of fun to figure out with git blame. The first > > notion of this code block goes back to the original cpu edp enabling > > for ilk in > > > > commit 32f9d658aee5be09ebdd28fc73063

Re: [Intel-gfx] [PATCH] drm/i915: Drop the misleading cast to the wrong user pointer type

2012-09-14 Thread Daniel Vetter
On Fri, Sep 14, 2012 at 04:24:46PM +0300, Jani Nikula wrote: > On Fri, 14 Sep 2012, Chris Wilson wrote: > > The exec_list is of type drm_i915_gem_exec_object2 and so casting it to > > a drm_i915_gem_relocation_entry is very confusing! > > Reviewed-by: Jani Nikula Queued for -next, thanks for the

[Intel-gfx] i915/drm: SNB gpu won't throttle up with latest forcewake/gpufreq tracepoint drm patches

2012-09-14 Thread Nicolas Kalkhof
Hello,the patchset from ~danvet/drm-intel git  from earlier september concerning forcewake and gpu frequency made my SNB much calmer during idle times. However the GPU won't hit the turbo throttle when the load goes up resulting in very low framerates during gameplay. Most times the frequency read

Re: [Intel-gfx] [pull] drm-intel-next

2012-09-14 Thread Daniel Vetter
On Fri, Sep 14, 2012 at 09:55:58AM -0400, Bobby Powers wrote: > This tree gives me recursive dependency problems, which ends up > removing a big (& important) part of my .config: > > [bpowers@fina linux]$ git reset --hard drm-intel-next-2012-09-09 > HEAD is now at e04190e drm/fb helper: don't call

Re: [Intel-gfx] Valid DP connection without EDID?

2012-09-14 Thread Adam Jackson
On 9/14/12 10:19 AM, Takashi Iwai wrote: Hi, we've got a machine showing a ghost DP2 output on a docking station. The docking station has only one DP port and it's connected to DP1. As a result, we get an DP2 active output containing the bogus VESA standard modes 1024x768, 800x600, 640x480 altho

[Intel-gfx] macbook pro retina and intel

2012-09-14 Thread Benoit Gschwind
Hello, I get an issue with mac book pro retina with git snapshot of kernel[1]. the screen flickering like this http://www.youtube.com/watch?v=Oq0l-bHmSYE anyone has an idea where the issue can come ? if not this could be a bug report ^^ the flickering is more visible on screen update, like refr

[Intel-gfx] Valid DP connection without EDID?

2012-09-14 Thread Takashi Iwai
Hi, we've got a machine showing a ghost DP2 output on a docking station. The docking station has only one DP port and it's connected to DP1. As a result, we get an DP2 active output containing the bogus VESA standard modes 1024x768, 800x600, 640x480 although it's not connected at all. Looking a b

Re: [Intel-gfx] [PATCH] drm/i915: Drop the misleading cast to the wrong user pointer type

2012-09-14 Thread Jani Nikula
On Fri, 14 Sep 2012, Chris Wilson wrote: > The exec_list is of type drm_i915_gem_exec_object2 and so casting it to > a drm_i915_gem_relocation_entry is very confusing! Reviewed-by: Jani Nikula > > Signed-off-by: Chris Wilson > --- > drivers/gpu/drm/i915/i915_gem_execbuffer.c |9 +++--

Re: [Intel-gfx] [PATCH 2/2] agp/intel: Use a write-combining map for updating PTEs

2012-09-14 Thread Chris Wilson
On Fri, 14 Sep 2012 11:57:47 +0100, Chris Wilson wrote: > v2: Limit the WC mapping to older generations as we should the TLB > invalidation on SandyBridge+ unreliable. /me fires his editor and proof-reader v2: Limit the WC mapping to older generations as we have observed that the TLB invalidati

[Intel-gfx] [PATCH 2/2] agp/intel: Use a write-combining map for updating PTEs

2012-09-14 Thread Chris Wilson
Rewriting the PTE entries using an WC mapping is roughly an order of magnitude faster than through the uncached mapping. This makes an observable difference on workloads that cycle through large numbers of buffers, for example Chromium using ShmPixmaps where virtually all the CPU time is currently

[Intel-gfx] [PATCH 1/2] drm/i915: Limit the ioremap of the PCI bar to the registers

2012-09-14 Thread Chris Wilson
In the future we may like to experiment with using a WC map of the GTT portion. However, that will conflict with i915.ko mapping the entire bar as UC in order to access the GPU registers. Instead we can shrink the register ioremap to only map the register block. Signed-off-by: Chris Wilson --- d

[Intel-gfx] [PATCH] drm/i915: Drop the misleading cast to the wrong user pointer type

2012-09-14 Thread Chris Wilson
The exec_list is of type drm_i915_gem_exec_object2 and so casting it to a drm_i915_gem_relocation_entry is very confusing! Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i915_gem_execbuffer.c |9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/i9