Re: [PATCH v2 1/2] drm/i915: Set the map-and-fenceable flag for preallocated objects

2015-11-20 Thread Jesse Barnes
able binding. > > Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk> > Cc: "Goel, Akash" <akash.g...@intel.com> > Cc: Daniel Vetter <daniel.vet...@ffwll.ch> > Cc: Jesse Barnes <jbar...@virtuousgeek.org> > Cc: stable@vger.kernel.org > --- >

Re: [PATCH v4] drm/i915: Pin the ifbdev for the info->system_base GGTT mmapping

2015-11-20 Thread Jesse Barnes
ing even further. > > Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk> > Cc: "Goel, Akash" <akash.g...@intel.com> > Cc: Daniel Vetter <daniel.vet...@ffwll.ch> > Cc: Jesse Barnes <jbar...@virtuousgeek.org> > Cc: stable@vger.kernel.org &

Re: [PATCH v3] drm/i915: Pin the ifbdev for the info->system_base GGTT mmapping

2015-11-20 Thread Jesse Barnes
Wilson <ch...@chris-wilson.co.uk> > Cc: "Goel, Akash" <akash.g...@intel.com> > Cc: Daniel Vetter <daniel.vet...@ffwll.ch> > Cc: Jesse Barnes <jbar...@virtuousgeek.org> > Cc: stable@vger.kernel.org > --- > drivers/gpu/drm/i915/intel_fbdev.c | 23 ++

Re: [PATCH] drm/i915: Pin the ifbdev for the info->system_base GGTT mmapping

2015-10-08 Thread Jesse Barnes
t; The effect is most pronounced across suspend/resume as >> intel_fbdev_set_suspend() does a full clear over the whole scanout. >> >> v2: rebased on latest nightly (Wayne) >> >> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk> >> Cc: "Goel, Akash

Re: [Intel-gfx] [PATCH] drm/i915: Set the map-and-fenceable flag for preallocated objects

2015-08-27 Thread Jesse Barnes
that latter users do not get confused and try and rebind the pinned vma in order to get a map-and-fenceable binding. Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk Cc: Goel, Akash akash.g...@intel.com Cc: Daniel Vetter daniel.vet...@ffwll.ch Cc: Jesse Barnes jbar...@virtuousgeek.org Cc

Re: [Intel-gfx] [PATCH] drm/i915: Unconditionally flush writes before execbuffer

2015-05-21 Thread Jesse Barnes
On 05/21/2015 06:00 AM, Chris Wilson wrote: On Tue, May 19, 2015 at 03:41:48PM +0100, Chris Wilson wrote: On Mon, May 11, 2015 at 04:25:52PM +0100, Chris Wilson wrote: On Mon, May 11, 2015 at 12:34:37PM +0200, Daniel Vetter wrote: On Mon, May 11, 2015 at 08:51:36AM +0100, Chris Wilson wrote:

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Disallow pin ioctl completely for kms drivers

2015-02-23 Thread Jesse Barnes
On 11/24/2014 06:13 AM, Chris Wilson wrote: On Mon, Nov 24, 2014 at 03:10:05PM +0100, Daniel Vetter wrote: On Mon, Nov 24, 2014 at 10:35:29AM +, Chris Wilson wrote: Pinning is a useful tool and it would also be useful to have again on gen6+. I think softpin or similar is doable with

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Disallow pin ioctl completely for kms drivers

2015-02-23 Thread Jesse Barnes
On 02/23/2015 03:40 PM, Daniel Vetter wrote: On Mon, Feb 23, 2015 at 01:29:57PM -0800, Jesse Barnes wrote: On 11/24/2014 06:13 AM, Chris Wilson wrote: On Mon, Nov 24, 2014 at 03:10:05PM +0100, Daniel Vetter wrote: On Mon, Nov 24, 2014 at 10:35:29AM +, Chris Wilson wrote: Pinning

Re: [Intel-gfx] [PATCH] drm/i915: Don't complain about stolen conflicts on gen3

2014-12-05 Thread Jesse Barnes
) but does make things messier for us. Reviewed-by: Jesse Barnes jbar...@virtuousgeek.org Looks like the reporter gave a t-b too. Jesse -- To unsubscribe from this list: send the line unsubscribe stable in the body of a message to majord...@vger.kernel.org More majordomo info at http

Re: [Intel-gfx] [PATCH] [v3] drm/i915/bdw: Only use 2g GGTT for 32b platforms

2014-05-19 Thread Jesse Barnes
a new patch with Tested-by on it. BR, Jani. [1] https://bugs.freedesktop.org/show_bug.cgi?id=76619#c16 Ben, will you re-post this or can you find someone else who can? -- Jesse Barnes, Intel Open Source Technology Center -- To unsubscribe from this list: send the line unsubscribe stable

Re: [Intel-gfx] [PATCH] drm/i915: Use the first mode if there is no preferred mode in the EDID

2014-05-13 Thread Jesse Barnes
)); + modes[i] = list_first_entry(struct drm_display_mode, + connector-modes, head); Please imagine that I wrote this correctly. Imagining you wrote it correctly: Reviewed-by: Jesse Barnes jbar...@virtuousgeek.org -- Jesse Barnes, Intel Open Source

Re: [Intel-gfx] [PATCH] drm/i915: correctly restore fences with objects attached

2013-07-17 Thread Jesse Barnes
{ + i915_gem_write_fence(dev, i, NULL); + } Why do we look at reg-obj here? Can it be NULL? Or would reg-obj-tiling_mode != I915_TILING_NONE do the same thing? -- Jesse Barnes, Intel Open Source Technology Center -- To unsubscribe from this list: send the line unsubscribe stable

Re: [Intel-gfx] [PATCH] [v2] drm/i915: Fix context sizes on HSW

2013-06-25 Thread Jesse Barnes
this information is kept, but other than that: Reviewed-by: Jesse Barnes jbar...@virtuousgeek.org -- Jesse Barnes, Intel Open Source Technology Center -- To unsubscribe from this list: send the line unsubscribe stable in the body of a message to majord...@vger.kernel.org More majordomo info

Re: [Intel-gfx] [PATCH] drm/i915: Workaround incoherence between fences and LLC across multiple CPUs

2013-04-05 Thread Jesse Barnes
-mm.fence_list); } else { I almost wish x86 just gave up and went fully weakly ordered. At least then we'd know we need barriers everywhere, rather than just in mystical places. Reviewed-by: Jesse Barnes jbar...@virtuousgeek.org -- Jesse Barnes, Intel Open Source Technology Center

Re: [PATCH] x86/Sandy Bridge: reserve pages when integrated graphics is present

2013-01-17 Thread Jesse Barnes
On Thu, 17 Jan 2013 02:23:01 -0500 (EST) CAI Qian caiq...@redhat.com wrote: Hello Peter and Jesse, a9acc5365dbda29f7be2884efb63771dc24bd815 Should this be marked as stable? It is applied and built successfully for 3.0.x, 3.4.x and 3.7.x. If needed, we have a couple of Sandy Bridge with

Re: [Intel-gfx] [PATCH] drm/i915: Revert RPS UP_EI value for SandyBridge and IvyBridge

2012-12-20 Thread Jesse Barnes
is probably a bad idea in general. -- Jesse Barnes, Intel Open Source Technology Center -- To unsubscribe from this list: send the line unsubscribe stable in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [Intel-gfx] [PATCH] drm/i915: don't disable disconnected outputs

2012-12-18 Thread Jesse Barnes
this logic comes from X, when we weren't sure about the initial config so we just shut everything off pretty aggressively. We clearly weren't thinking of atomic mode sets back then either... Reviewed-by: Jesse Barnes jbar...@virtuousgeek.org -- Jesse Barnes, Intel Open Source Technology Center

Re: [Intel-gfx] [PATCH] drm/i915: disable cpt phase pointer fdi rx workaround

2012-12-10 Thread Jesse Barnes
On Sat, 8 Dec 2012 12:58:33 +0100 Daniel Vetter daniel.vet...@ffwll.ch wrote: We've originally added this in commit 291427f5fdadec6e4be2924172e8350e1539 Author: Jesse Barnes jbar...@virtuousgeek.org Date: Fri Jul 29 12:42:37 2011 -0700 drm/i915: apply phase pointer override

Re: [Intel-gfx] [PATCH] drm/i915: Initialize obj-pages before use by i915_gem_object_do_bit17_swizzle()

2012-10-19 Thread Jesse Barnes
drm_i915_gem_object *obj) sg_set_page(sg, page, PAGE_SIZE, 0); } + obj-pages = st; + if (i915_gem_object_needs_bit17_swizzle(obj)) i915_gem_object_do_bit_17_swizzle(obj); - obj-pages = st; return 0; err_pages: Reviewed-by: Jesse Barnes

Re: [PATCH] drm: Destroy the planes prior to destroying the associated CRTC

2012-09-17 Thread Jesse Barnes
/show_bug.cgi?id=54101 Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk Cc: Jesse Barnes jbar...@virtuousgeek.org Cc: stable@vger.kernel.org --- drivers/gpu/drm/drm_crtc.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm

Re: [PATCH] drm/i915: enable lvds pin pairs before dpll on gen2

2012-09-12 Thread Jesse Barnes
/i915: split PLL update code out of i9xx_crtc_mode_set Cc: Jesse Barnes jbar...@virtuousgeek.org Cc: stable@vger.kernel.org (for 3.5 only) Signed-off-by: Daniel Vetter daniel.vet...@ffwll.ch --- drivers/gpu/drm/i915/intel_display.c | 12 ++-- 1 file changed, 6 insertions(+), 6

Re: [Intel-gfx] [PATCH] drm/i915: [GEN7] Use HW scheduler for fixed function shaders

2012-05-21 Thread Jesse Barnes
. -- Jesse Barnes, Intel Open Source Technology Center -- To unsubscribe from this list: send the line unsubscribe stable in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH] drm/i915: make rc6 module parameter read-only

2012-04-11 Thread Jesse Barnes
People have been getting confused and thinking this is a runtime control. Cc: stable@vger.kernel.org Signed-off-by: Jesse Barnes jbar...@virtuousgeek.org --- drivers/gpu/drm/i915/i915_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_drv.c

[PATCH] drm/i915: treat src w h as fixed point in sprite handling code

2012-03-20 Thread Jesse Barnes
This was missed when we converted the source values to 16.16 fixed point. Cc: stable@vger.kernel.org Signed-off-by: Jesse Barnes jbar...@virtuousgeek.org --- drivers/gpu/drm/i915/intel_sprite.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/i915