eelis Roos <mr...@linux.ee>
Cc: Chris Wilson <ch...@chris-wilson.co.uk>
Fixes: ac9b8236551d ("drm/i915: Introduce a gmbus power domain")
Cc: stable@vger.kernel.org
References: http://www.spinics.net/lists/intel-gfx/msg83075.html
Tested-by: Meelis Roos <mr...@linux.ee
: ac9b8236551d ("drm/i915: Introduce a gmbus power domain")
Cc: stable@vger.kernel.org
References: http://www.spinics.net/lists/intel-gfx/msg83075.html
Signed-off-by: Daniel Vetter <daniel.vet...@intel.com>
---
drivers/gpu/drm/i915/i915_dma.c | 6 ++
1 file changed, 2 insertions(+
c: Jani Nikula <jani.nik...@intel.com>
Cc: Meelis Roos <mr...@linux.ee>
Cc: Chris Wilson <ch...@chris-wilson.co.uk>
Fixes: ac9b8236551d ("drm/i915: Introduce a gmbus power domain")
Cc: stable@vger.kernel.org
References: http://www.spinics.net/lists/intel-gfx/msg83075.html
S
On Thu, Jan 7, 2016 at 1:52 PM, Ville Syrjälä
<ville.syrj...@linux.intel.com> wrote:
> On Thu, Jan 07, 2016 at 12:44:21PM +0100, Daniel Vetter wrote:
>> Since
>>
>> commit ac9b8236551d1177fd07b56aef9b565d1864420d
>> Author: Ville Syrjälä <ville.syrj...@linux.in
http://www.spinics.net/lists/intel-gfx/msg83075.html
Tested-by: Meelis Roos <mr...@linux.ee>
Signed-off-by: Daniel Vetter <daniel.vet...@intel.com>
---
drivers/gpu/drm/i915/i915_dma.c | 14 ++
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/i915
intel_connector->encoder->hpd_pin > HPD_NONE)
> connector->polled = DRM_CONNECTOR_POLL_HPD;
> }
>
> --
> 2.5.0
>
> ___
> Intel-gfx mailing list
> intel-...@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
--
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
(!connector->polled && I915_HAS_HOTPLUG(dev) &&
intel_connector->encoder->hpd_pin > HPD_NONE)
connector->polled = DRM_CONNECTOR_POLL_HPD;
}
That yields even tidier code and gives us a place for a comment. Can you
pls respi
sktop.org/show_bug.cgi?id=92826
> Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
Well can't really review with Bspec and I don't expect we can get
confirmation from hw engineers. But does what it says in the commit
message.
Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch&
On Mon, Dec 21, 2015 at 02:49:18PM +0100, Daniel Vetter wrote:
> On Fri, Dec 18, 2015 at 07:24:39PM +0200, ville.syrj...@linux.intel.com wrote:
> > From: Ville Syrjälä <ville.syrj...@linux.intel.com>
> >
> > Turns out CHV pipe C was glued on somewhat poorly, and
; Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
> Cc: stable@vger.kernel.org
> ---
> drivers/gpu/drm/i915/i915_gem_context.c | 29 +
> 1 file changed, 17 insertions(+), 12 deletions(-)
&g
(timing issue).
> Here, we do not need the w/a to force ordering of the read with an
> interrupt.
>
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> Reviewed-by: Tvrtko Ursulin <tvrtko.ursu...@linux.intel.com>
> Cc: "Rogozhkin, Dmitry V" <dmitry.
e.
> > >
> > > v4: Try to simplify the pinning even further.
> > > v5: Leak the VMA (cleaned up by object-free) to avoid complicated error
> > > paths.
> > >
> > > Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> > > Cc:
ration of fb can then be
> changed thus:
>
> - struct drm_framebuffer *fb = NULL;
> + struct drm_framebuffer *fb;
>
> Kind regards,
Yeah there's room for follow-up polish, but this seems good enough at
least for -fixes.
Reviewed-by: Daniel Vetter <daniel.vet...@ffwl
keep
> plane_state->visible set, or a WARN_ON in intel_plane_atomic_calc_changes
> triggers. There are other code points that rely on accurate
> plane_state->visible
> too, so make sure the bool is cleared.
Oh dear is this a mess :(
As a bugfix this is Reviewed-by: Daniel Vette
encoder. And then check this in the atomic helpers. But that's way too
much for -fixes.
On the patch itself, for -fixes: Reviewed-by: Daniel Vetter
<daniel.vet...@ffwll.ch>
> Cc: Ander Conselvan de Oliveira <ander.conselvan.de.olive...@intel.com>
> Fixes: 5448a00d3f06 ("drm/i91
least a duplicate of it was on that list). This way we will
> correctly prune away any mode from the old mode list that didn't
> appear on the probed_modes list.
>
> Cc: stable@vger.kernel.org
> Cc: Adam Jackson <a...@redhat.com>
> Fixes: 05acaec334fc ("drm: Reorganize p
o use of passing a void pointer, I didn't spot one
> > callsite in wait-ioctl was passing the wrong pointer.
> >
> > Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> > Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
> > Cc: stable@vger.kernel.org
>
>
optimisations
>
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> Cc: Lionel Landwerlin <lionel.g.landwer...@linux.intel.com>
> Cc: Tvrtko Ursulin <tvrtko.ursu...@intel.com>
> Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
> Cc: stable@vger.kernel.org
C
to allocate fb_info\n");
>232 ret = PTR_ERR(info);
>233goto out_unpin;
>
> ---
> 0-DAY kernel test infrastructureOpen Source Technology Center
> https://lists.01.org/pipermail/kbuild-all Intel Corporation
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
--
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
nen <c...@tnsp.org>
> References: https://bugs.freedesktop.org/show_bug.cgi?id=90725
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> Cc: Matti Hämäläinen <c...@tnsp.org>
> Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
> Cc: Jani Nikula <j
lse {
> swizzle_x = I915_BIT_6_SWIZZLE_9_10;
> swizzle_y = I915_BIT_6_SWIZZLE_9;
> --
> 2.6.2
>
> _______
> Intel-gfx mailing list
> intel-...@lists.freedesktop.org
> http://lists.freedesk
et gets
> rejected on machines where the primary plane must be fullscreen, and on
> the rest only the first eye would get a visible plane.
>
> Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
> Cc: stable@vger.kernel.org
> Fixes: 042652ed9599 ("drm/atomic-helper: implementatations fo
NTEL_INFO(dev)->cursor_needs_physical)) {
> intel_unpin_fb_obj(old_state->fb, old_state);
> + }
>
> /* prepare_fb aborted? */
> if ((old_obj && (old_obj->frontbuffer_bits &
> intel_plane->frontbuffer_bit)) ||
> --
>
n.lankho...@linux.intel.com>
> Cc: stable@vger.kernel.org #v4.3
Nice catch, but a bit a terse commit message. We should add "Without this
fix the plane->fb refcounting might get out of sync on retries, resulting
in either leaked memory or use-after-free." With that:
Revi
s the struct
> for another addfb2, the ioctl will be rejected since it's now asking for
> some modifiers w/o the flag set.
>
> Fix the problem by making a copy of the user provided structure. We can
> play any games we want with the copy.
>
> Cc: stable@vger.kernel.org
>
using. Deny it.
>
> Can we allow this for unsynchronized userptrs?
I'd like to not add more complexity to a root-only feature.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
--
To unsubscribe from this list: send the line "unsubscribe stable&
+ intel_update_watermarks(crtc);
Does this ever do anything at all? We just killed the crtc completely
above, those watermark writes will get cleansed as soon as the power well
goes down.
-Daniel
> }
> }
>
> --
> 2.1.0
>
> _
well is off.
>
> Signed-off-by: Maarten Lankhorst <maarten.lankho...@linux.intel.com>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92181
> Cc: stable@vger.kernel.org
Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch>
Jani, one for you.
-Daniel
> ---
It shuts up a spurious error in dmesg about a feature which isn't even
implemented and took about 20+ patches to get right in Linus tree. Option
B is to waste reporters time if we leave that DRM_ERROR in there with
filing bugs that we'll close right away. There's no way we can backport
DVFS suppor
On Tue, Oct 13, 2015 at 03:45:58PM +0300, Jani Nikula wrote:
> On Wed, 26 Aug 2015, Chris Wilson <ch...@chris-wilson.co.uk> wrote:
> > On Wed, Aug 26, 2015 at 11:16:34AM +0200, Daniel Vetter wrote:
> >> On Fri, Aug 21, 2015 at 04:08:41PM +0100, Chris Wilson wrote:
On Tue, Oct 13, 2015 at 04:10:19PM +0300, Jani Nikula wrote:
> On Thu, 08 Oct 2015, Ville Syrjälä <ville.syrj...@linux.intel.com> wrote:
> > On Thu, Oct 08, 2015 at 10:17:30AM +0200, Daniel Vetter wrote:
> >> On Wed, Oct 07, 2015 at 10:08:24PM +0300, ville.syrj...@lin
es to unmap its address space. The situation can be even more
> >complicated when the buffer is passed between processes, between a
> >client and display server, where the lifetime and hardware access is
> >even more confusing. Deny it.
>
> Reviewed-by: Tvrtko Ursulin <tv
I915_READ(DPLL(!crtc->pipe)) | DPLL_DVO_2X_MODE);
Don't we also need a POSTING_READ here to make sure the two-step 2x mode
sequence is still followed?
With that addressed Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch>
> }
>
> + I915_WRITE(reg,
On Thu, Oct 01, 2015 at 07:13:11PM +0300, Francisco Jerez wrote:
> Daniel Vetter <daniel.vet...@ffwll.ch> writes:
>
> > In
> >
> > commit 8f0e2b9d95a88ca5d8349deef2375644faf184ae
> > Author: Daniel Vetter <daniel.vet...@ffwll.ch>
> > Date: Tu
In
commit 8f0e2b9d95a88ca5d8349deef2375644faf184ae
Author: Daniel Vetter <daniel.vet...@ffwll.ch>
Date: Tue Dec 2 16:19:07 2014 +0100
drm/i915: Move golden context init into ->init_context
I've shuffled around per-ctx init code a bit for legacy contexts but
accidentally dropped t
tch to just grab the
mode_config.connection_mutex, but we can do a bit better.
Reported-by: Jens Axboe <ax...@fb.com>
Cc: Jens Axboe <ax...@fb.com>
Cc: Rob Clark <robdcl...@gmail.com>
Cc: stable@vger.kernel.org
Signed-off-by: Daniel Vetter <daniel.vet...@intel.com>
---
drivers/gpu
tch to just grab the
mode_config.connection_mutex, but we can do a bit better.
v2: Remove unused variables I failed to git add for real.
Reported-by: Jens Axboe <ax...@fb.com>
Cc: Jens Axboe <ax...@fb.com>
Cc: Rob Clark <robdcl...@gmail.com>
Cc: stable@vger.kernel.org
Signed
are not stable.
> > Just give up after a couple of tries and report whatever we read last.
> >
> > Reported-by: russianneuroman...@ya.ru
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91906
> > Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
>
> v2: Use the most recent values when erring out on an unstable register.
>
> Reported-by: russianneuroman...@ya.ru
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91906
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> Cc: Michał Winiarski <michal.w
tions under which userspace may wish to tell white lies to the
> kernel...
>
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
> Cc: Kristian Høgsberg <k...@bitplanet.net>
> Cc: Jesse Barnes <jbar...@virtuou
://bugs.freedesktop.org/show_bug.cgi?id=89453
Tested-by: Palmer Dabbelt pal...@dabbelt.com
Reviewed-by: Daniel Vetter daniel.vet...@ffwll.ch
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91407
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89461
Bugzilla: https
intel-...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
--
To unsubscribe from this list: send the line unsubscribe stable in
the body of a message to majord...@vger.kernel.org
More
On Wed, Aug 26, 2015 at 03:06:59PM +0200, Daniel Vetter wrote:
On Wed, Aug 26, 2015 at 12:55:57PM +0100, Chris Wilson wrote:
As we mark the preallocated objects as bound, we should also flag them
correctly as being map-and-fenceable (if appropriate!) so that latter
users do not get confused
-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: stable@vger.kernel.org
Reviewed-by: Daniel Vetter daniel.vet...@ffwll.ch
Jani, can you please pick
the driver's connector cleanup
function asynchronously. And that is very unhappy that the port is now
gone.
So perfectly ok regression fix imo to restore the ordering we had before
your patch in the cleanup code.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
this
already. Reviewed-by: Daniel Vetter daniel.vet...@ffwll.ch
-Daniel
---
drivers/gpu/drm/i915/intel_lrc.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/i915/intel_lrc.c
b/drivers/gpu/drm/i915/intel_lrc.c
index 138964afd187..41cfa6fa909d 100644
--- a/drivers/gpu/drm/i915
introduced with
6b8eeca65b18ae77e175cc2b6571731f0ee413bf
drm/dp/mst: close deadlock in connector destruction.
Cc: stable@vger.kernel.org
Cc: Dave Airlie airl...@redhat.com
Signed-off-by: Maarten Lankhorst maarten.lankho...@linux.intel.com
Reviewed-by: Daniel Vetter daniel.vet...@ffwll.ch
Jani
-by: Daniel Vetter daniel.vet...@intel.com
---
drivers/gpu/drm/drm_dp_mst_topology.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c
b/drivers/gpu/drm/drm_dp_mst_topology.c
index 778bbb6425b8..b0487c9f018c 100644
--- a/drivers/gpu/drm/drm_dp_mst_topology.c
+++ b
for an individual page is unknown (it
depends on its physical address only known to the kernel), userspace
cannot correctly swizzle.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91105
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: Daniel Vetter daniel.vet...@ffwll.ch
Cc
___
Intel-gfx mailing list
intel-...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
--
To unsubscribe from this list: send the line unsubscribe stable
On Tue, Jul 21, 2015 at 10:45:45AM +0100, Chris Wilson wrote:
On Tue, Jul 21, 2015 at 08:49:31AM +0200, Daniel Vetter wrote:
On Fri, Jul 17, 2015 at 05:10:25PM +0200, Michał Winiarski wrote:
On Thu, Jul 16, 2015 at 12:37:56PM +0100, Chris Wilson wrote:
Since the hardware sometimes
/gmane.comp.freedesktop.xorg.drivers.intel/62584
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=100641
Cc: Rui Matos tiagoma...@gmail.com
Cc: Julien Wajsberg fel...@gmail.com
Cc: kuddel.m...@gmx.de
Cc: Lennart Poettering mzxre...@0pointer.de
Cc: stable@vger.kernel.org
Signed-off-by: Daniel Vetter
this we need to extend the ioctl with
new paramters that only new userspace looks at.
Cc: Harald Arnesen har...@skogtun.org
Cc: Chris Wilson ch...@chris-wilson.co.uk
Reported-by: Harald Arnesen har...@skogtun.org
Cc: stable@vger.kernel.org
Signed-off-by: Daniel Vetter daniel.vet...@intel.com
the ioctl need explicit
compat handling.
v2: Removed the typedef. (Daniel Vetter)
Signed-off-by: Tvrtko Ursulin tvrtko.ursu...@intel.com
Reviewed-by: Daniel Vetter daniel.vet...@ffwll.ch
Cc: dri-de...@lists.freedesktop.org
Cc: Rob Clark robdcl...@gmail.com
Cc: Daniel Stone dani
known to the kernel), userspace
cannot correctly swizzle objects.
v2: Mark the global swizzling as unknown rather than adjust the value
reported to userspace.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91105
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: Daniel Vetter
When inlining the actual hpd output probing in
commit 69787f7da6b2adc4054357a661aaa1701a9ca76f
Author: Daniel Vetter daniel.vet...@ffwll.ch
Date: Tue Oct 23 18:23:34 2012 +
drm: run the hpd irq event code directly
the check for the drm_kms_hlper.poll module option was lost
djku...@chromium.org
Reviewed-by: Daniel Vetter daniel.vet...@ffwll.ch
Cc: stable@vger.kernel.org
Applied to topic/drm-fixes to make sure it won't get lost, but I expect
rockchip maintainers to pick this one up.
-Daniel
---
drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 67
known to the kernel), userspace
cannot correctly swizzle.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91105
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: Daniel Vetter daniel.vet...@ffwll.ch
Cc: stable@vger.kernel.org
Reviewed-by: Daniel Vetter daniel.vet...@ffwll.ch
://bugs.freedesktop.org/show_bug.cgi?id=91127
References: https://bugzilla.freedesktop.org/show_bug.cgi?id=60391
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: Daniel Vetter daniel.vet...@ffwll.ch
Cc: stable@vger.kernel.org
Oops, can't count how often we've committed this mistake.
Reviewed-by: Daniel
Signed-off-by: Daniel Vetter daniel.vet...@intel.com
---
drivers/gpu/drm/vgem/vgem_drv.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/vgem/vgem_drv.c b/drivers/gpu/drm/vgem/vgem_drv.c
index 7a207ca547be..6394547cf67a 100644
--- a/drivers/gpu/drm/vgem/vgem_drv.c
+++ b/drivers
On Tue, Jun 23, 2015 at 01:05:41PM +0300, Jani Nikula wrote:
On Fri, 19 Jun 2015, Daniel Vetter daniel.vet...@ffwll.ch wrote:
We've never figured out the magic trick to make irq vs. seqno
updates coherent, only tricks to make it work. And since
commit
On Tue, Jun 23, 2015 at 9:12 AM, Jani Nikula jani.nik...@intel.com wrote:
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89600
Cc: stable@vger.kernel.org # 4.0+
Signed-off-by: Peter Antoine peter.anto...@intel.com
Reviewed-by: Daniel Vetter daniel.vet...@ffwll.ch
[Jani: fixed typo
Janes mark.a.ja...@intel.com
Cc: stable@vger.kernel.org
Signed-off-by: Daniel Vetter daniel.vet...@intel.com
---
drivers/gpu/drm/i915/i915_irq.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index e6bb72dca3ff
NOTICE since it might be of value for bug reports (Chris).
Cc: Mark Janes mark.a.ja...@intel.com
Cc: Chris Wilson ch...@chris-wilson.co.uk
Cc: stable@vger.kernel.org
Signed-off-by: Daniel Vetter daniel.vet...@intel.com
---
drivers/gpu/drm/i915/i915_irq.c | 4 ++--
1 file changed, 2 insertions(+), 2
);
+ if (bla)
+ goto retry;
+
goto out;
timeout:
---
Totally untested ofc ;-)
Cheers, Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
--
To unsubscribe from this list: send the line unsubscribe stable in
the body of a message to majord
://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
--
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
On Thu, May 21, 2015 at 5:30 PM, Daniel Vetter dan...@ffwll.ch wrote:
On Thu, May 21, 2015 at 04:22:55PM +0100, Chris Wilson wrote:
On Thu, May 21, 2015 at 04:21:46PM +0200, Daniel Vetter wrote:
Hm right. What about emphasising this a bit more in the comment:
/*
* Empirical
On Tue, May 26, 2015 at 09:49:15AM +0100, Chris Wilson wrote:
On Tue, May 26, 2015 at 10:01:24AM +0200, Daniel Vetter wrote:
On Mon, May 25, 2015 at 06:48:44PM +0100, Chris Wilson wrote:
Since the advent of mmap(wc), where we reused the same cache domain for
WC and GTT paths (oh, how I
On Thu, May 21, 2015 at 02:00:34PM +0100, 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
On Thu, May 21, 2015 at 04:22:55PM +0100, Chris Wilson wrote:
On Thu, May 21, 2015 at 04:21:46PM +0200, Daniel Vetter wrote:
Hm right. What about emphasising this a bit more in the comment:
/*
* Empirical evidence indicates that we need a write barrier to
* make sure
On Thu, May 21, 2015 at 02:13:01PM +0100, Chris Wilson wrote:
On Thu, May 21, 2015 at 03:07:54PM +0200, Daniel Vetter wrote:
On Thu, May 21, 2015 at 02:00:34PM +0100, 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
In
commit f02ad907cd9e7fe3a6405d2d005840912f1ed258
Author: Daniel Vetter daniel.vet...@ffwll.ch
Date: Thu Jan 22 16:36:23 2015 +0100
drm/atomic-helpers: Recover full cursor plane behaviour
we've added a hack to atomic helpers to never to vblank waits for
cursor updates through the legacy
...@chris-wilson.co.uk
Cc: Ville Syrjälä ville.syrj...@linux.intel.com
Cc: Ramalingam C ramalinga...@intel.com
Cc: Rodrigo Vivi rodrigo.v...@intel.com
Cc: Vandana Kannan vandana.kan...@intel.com
Cc: stable@vger.kernel.org (4.0+ only)
Signed-off-by: Daniel Vetter daniel.vet...@intel.com
---
drivers/gpu
, so Reviewed-by: Daniel Vetter daniel.vet...@ffwll.ch
Cheers, Daniel
---
drivers/gpu/drm/drm_edid_load.c| 1 +
drivers/gpu/drm/drm_probe_helper.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/drm_edid_load.c b/drivers/gpu/drm/drm_edid_load.c
index 732cb6f8e653
On Fri, Mar 06, 2015 at 08:54:35AM +, Chris Wilson wrote:
On Thu, Mar 05, 2015 at 01:27:43PM +0100, Daniel Vetter wrote:
On Wed, Mar 04, 2015 at 06:09:26PM +, Chris Wilson wrote:
This fixes a regression from
commit 5ed0bdf21a85d78e04f89f15ccf227562177cbd9
Author: Thomas
timeout return immediately rather than the
previously defined behaviour of waiting indefinitely.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: Daniel Vetter daniel.vet...@ffwll.ch
Cc: Ben Widawsky benjamin.widaw...@intel.com
Cc: Kristian Høgsberg k...@bitplanet.net
Cc: stable
to switch will
be buggy as well ;-)
/me is chicken
Cheers, Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
--
To unsubscribe from this list: send the line unsubscribe stable in
the body of a message to majord...@vger.kernel.org
More majordomo info
, but doesn't add it anywhere.
Signed-off-by: Daniel Vetter daniel.vet...@intel.com (fix commit msg)
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk (provider of
lipstick)
Cc: Ville Syrjälä ville.syrj...@linux.intel.com
Cc: Matt Roper matthew.d.ro...@intel.com
Cc: Rob Clark robdcl
Originally it was impossible to be dropping the last refcount in this
function since there was always one around still from the idr. But in
commit 83f45fc360c8e16a330474860ebda872d1384c8c
Author: Daniel Vetter daniel.vet...@ffwll.ch
Date: Wed Aug 6 09:10:18 2014 +0200
drm: Don't grab an fb
,
--
2.1.0
___
Intel-gfx mailing list
intel-...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
--
To unsubscribe from
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 is a useful tool and it would also be useful to have
)); or similar.
Otherwise when we extended them in upstream (which we're just doing,
the kernel 0-fills the new fields automatically) it'll break because
after a recompile there's now garbage in the new fields.
I just spent a while fixing up libdrm ;-)
-Daniel
--
Daniel Vetter
Software Engineer, Intel
Wilson ch...@chris-wilson.co.uk
Cc: Jani Nikula jani.nik...@linux.intel.com
Reported-by: Johannes W jar...@molb.org
Signed-off-by: Daniel Vetter daniel.vet...@intel.com
--
Johannes, can you please test this patch instead of the one from
Chris?
Thanks, Daniel
---
drivers/gpu/drm/i915
Cc: Johannes W jar...@molb.org
Cc: Chris Wilson ch...@chris-wilson.co.uk
Cc: Jani Nikula jani.nik...@linux.intel.com
Reported-by: Johannes W jar...@molb.org
Signed-off-by: Daniel Vetter daniel.vet...@intel.com
--
Johannes, can you please test this patch instead of the one from
Chris?
Thanks
@vger.kernel.org
Cc: Chris Wilson ch...@chris-wilson.co.uk
Cc: Jani Nikula jani.nik...@intel.com
Signed-off-by: Daniel Vetter daniel.vet...@intel.com (v2)
---
drivers/gpu/drm/i915/i915_gem.c | 49 ++---
1 file changed, 2 insertions(+), 47 deletions(-)
diff --git
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
--
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
On Wed, Jan 21, 2015 at 07:45:01PM +0200, Jani Nikula wrote:
On Wed, 10 Dec 2014, Daniel Vetter dan...@ffwll.ch wrote:
On Wed, Dec 10, 2014 at 08:17:11AM +, Chris Wilson wrote:
This added as a BUG_ON as it considered that no one would ever request
an unaligned object. However, it turns
, or is that too late for
when we change pipe state when sanititizing crtcs?
-Daniel
DRM_DEBUG_KMS([CRTC:%d] hw state readout: %s\n,
crtc-base.base.id,
--
1.9.1
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http
On Tue, Jan 20, 2015 at 11:46:57AM +0200, Ville Syrjälä wrote:
On Tue, Jan 20, 2015 at 11:33:22AM +0200, Ander Conselvan de Oliveira wrote:
On 01/20/2015 11:22 AM, Daniel Vetter wrote:
On Mon, Jan 19, 2015 at 03:51:47PM +0200, Ander Conselvan de Oliveira
wrote:
@@ -13346,6 +13364,7
On Thu, Jan 15, 2015 at 08:44:00PM +, Chris Wilson wrote:
On Thu, Jan 15, 2015 at 08:36:15PM +0100, Daniel Vetter wrote:
On Wed, Jan 14, 2015 at 9:34 PM, Chris Wilson ch...@chris-wilson.co.uk
wrote:
This (partially) reverts
commit 5537252b6b6d71fb1a8ed7395a8e5babf91953fd
pressure
Shouldn't we also revert the hunk in i915_gem_free_objects? Without
the truncate vs. invalidate disdinction it seems to have lost it's
reason for existence ...
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
--
To unsubscribe from
);
+ if (i915_gem_object_is_purgeable(obj))
+ i915_gem_object_truncate(obj);
return 0;
}
--
2.1.4
___
Intel-gfx mailing list
intel-...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Daniel Vetter
Software Engineer, Intel
On Tue, Jan 13, 2015 at 01:24:12PM +0200, Ville Syrjälä wrote:
On Tue, Jan 13, 2015 at 12:34:06AM +0100, Daniel Vetter wrote:
On Mon, Jan 12, 2015 at 05:36:52PM +0200, Ander Conselvan de Oliveira wrote:
Otherwise setting the rotation property will cause the primary plane to
be disabled
On Mon, Jan 12, 2015 at 6:43 PM, Luis Henriques
luis.henriq...@canonical.com wrote:
On Mon, Jan 12, 2015 at 06:20:22PM +0100, Daniel Vetter wrote:
On Sun, Jan 11, 2015 at 10:49 PM, Ben Hutchings b...@decadent.org.uk wrote:
On Mon, 2014-12-15 at 14:24 +, Luis Henriques wrote:
3.16.7-ckt3
Signed-off-by: Ander Conselvan de Oliveira
ander.conselvan.de.olive...@intel.com
Reviewed-by: Daniel Vetter daniel.vet...@ffwll.ch
I guess long term we need to consolidate all our plane state related
readout functions, atm it's splattered all over. But that's maybe
something for after all the atomic
f96de58fc7e7d3d717c7c63975c3b896c906b5e3 upstream.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Reviewed-by: Jani Nikula jani.nik...@intel.com
Signed-off-by: Daniel Vetter daniel.vet...@ffwll.ch
Signed-off-by: Luis Henriques luis.henriq...@canonical.com
Should this also be applied
the stable is what I'm thinking.
Makes sense if we really don't need it any more with Chris' patches in
-fixes. Can you please volunteer someone to cough up an r-b?
Thanks, Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
--
To unsubscribe from
Reviewed-by: Daniel Vetter daniel.vet...@ffwll.ch
---
drivers/gpu/drm/i915/i915_gem.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
index ad55b06a3cb1..4c14db9587a6 100644
--- a/drivers/gpu/drm/i915/i915_gem.c
On Mon, Dec 15, 2014 at 07:03:43PM +0200, Ville Syrjälä wrote:
On Mon, Dec 15, 2014 at 04:24:48PM +, Chris Wilson wrote:
On Mon, Dec 15, 2014 at 10:41:52AM +0100, Daniel Vetter wrote:
On Thu, Dec 11, 2014 at 08:17:01AM +, Chris Wilson wrote:
There exists a current workaround
On Tue, Dec 16, 2014 at 08:26:55AM +, Chris Wilson wrote:
On Mon, Dec 15, 2014 at 10:34:45AM +0100, Daniel Vetter wrote:
On Thu, Dec 11, 2014 at 08:16:59AM +, Chris Wilson wrote:
In the gen7 pipe control there is an extra bit to flush the media
caches, so let's set it during cache
1 - 100 of 411 matches
Mail list logo