Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/i915/intel_fb.c | 10 --
1 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_fb.c b/drivers/gpu/drm/i915/intel_fb.c
index dfbb0c6..c3c5052 100644
--- a/drivers/gpu/drm/i915
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/nouveau/nouveau_fbcon.c |8 +++-
1 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/drivers/gpu/drm/nouveau/nouveau_fbcon.c
b/drivers/gpu/drm/nouveau/nouveau_fbcon.c
index fd4a2df..c9a4a0d 100644
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/radeon/radeon_fb.c | 12 +---
1 files changed, 9 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/radeon/radeon_fb.c
b/drivers/gpu/drm/radeon/radeon_fb.c
index e192acf..dc1634b 100644
--- a/drivers/gpu
This is a resend of Daniel Vetter's drm mm work to provide a basis for
performing fair eviction in i915. I've taken the liberty of attaching the
acks and review comments from the previous round, so please look over and
check that they still hold true.
drivers/gpu/drm/drm_mm.c|
From: Daniel Vetter daniel.vet...@ffwll.ch
Only ever assigned, never used.
Signed-off-by: Daniel Vetter daniel.vet...@ffwll.ch
[glisse: I will re-add if needed for range-restricted allocations]
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/i915/i915_gem.c |4
From: Daniel Vetter daniel.vet...@ffwll.ch
Yeah, I've kinda noticed that fl_entry is the free stack. Still
give it (and the memory node list ml_entry) decent names.
Signed-off-by: Daniel Vetter daniel.vet...@ffwll.ch
Acked-by: Thomas Hellstrom thellst...@vmwgfx.com
Signed-off-by: Chris Wilson ch
-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/drm_mm.c | 71 ++
1 files changed, 34 insertions(+), 37 deletions(-)
diff --git a/drivers/gpu/drm/drm_mm.c b/drivers/gpu/drm/drm_mm.c
index d2267ff..fd86a6c 100644
--- a/drivers/gpu/drm/drm_mm.c
From: Daniel Vetter daniel.vet...@ffwll.ch
This does two little changes:
- Add an alignment parameter for evict_something. It's not really great to
whack a carefully sized hole into the gtt with the wrong alignment.
Especially since the fallback path is a full evict.
- With the inactive
This will be used by the eviction logic to maintain fairness between the
rings.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/i915/i915_drv.h |3 +-
drivers/gpu/drm/i915/i915_gem.c |2 +
drivers/gpu/drm/i915/intel_ringbuffer.c | 46
The eviction code is the gnarly underbelly of memory management, and is
clearer if kept separated from the normal domain management in GEM.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/i915/Makefile |1 +
drivers/gpu/drm/i915/i915_drv.h |6
- Hang when scrolling firefox page with window in front
https://bugs.freedesktop.org/show_bug.cgi?id=24369
Bug 28478 - Intermittent graphics lockups due to overflow/loop
https://bugs.freedesktop.org/show_bug.cgi?id=28478
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu
the CPU objects from casual eviction,
preferring to keep the GPU active for as long as possible.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/i915/i915_gem.c |7 +++
1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_gem.c b
On Fri, 2 Jul 2010 15:02:17 +0100, Chris Wilson ch...@chris-wilson.co.uk
wrote:
From: Daniel Vetter daniel.vet...@ffwll.ch
This does two little changes:
- Add an alignment parameter for evict_something. It's not really great to
whack a carefully sized hole into the gtt with the wrong
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
A side-effect of being able to use custom page allocations with the
sg_table is that it cannot reap the partially constructed scatterlist if
fails to allocate a page. So we need to call sg_free_table() ourselves
if sg_alloc_table() fails.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/drm_ioctl.c | 73 --
1 files changed, 55 insertions(+), 18 deletions(-)
diff --git a/drivers/gpu/drm/drm_ioctl.c b/drivers/gpu/drm/drm_ioctl.c
index 9b9ff46..197267b 100644
[f8316665] drm_setversion+0x11d/0x1b1 [drm]
[f83148d4] drm_ioctl+0x29a/0x356 [drm]
[c04f27c4] vfs_ioctl+0x33/0x91
[c04f31cf] do_vfs_ioctl+0x46b/0x496
[c04f3240] sys_ioctl+0x46/0x66
[c040325f] sysenter_do_call+0x12/0x38
[] 0x
Signed-off-by: Chris Wilson ch
On Sat, 24 Jul 2010 18:29:35 +0100, Chris Wilson ch...@chris-wilson.co.uk
wrote:
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
@@ -323,7 +358,9 @@ int drm_setversion(struct drm_device *dev, void *data,
struct drm_file *file_pri
to do so.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/drm_ioctl.c | 79 --
1 files changed, 60 insertions(+), 19 deletions(-)
diff --git a/drivers/gpu/drm/drm_ioctl.c b/drivers/gpu/drm/drm_ioctl.c
index 9b9ff46..f107fff 100644
is not empty, file a bug. [Of course this means having to
compile kmemleak into the kernel. The DEBUG_KMEMLEAK option is found under
Kernel Hacking.] And since each generation has slightly different
initialization stanzas, we need to repeat the test on all platforms.
--
Chris Wilson, Intel Open Source
/show_bug.cgi?id=16488
Innocuous-looking one-liner is said to have made Milan's X server even
worse than normal.
We go from a random OOPS to a consistent error (and a failing userspace).
It sounds more likely that we have uncovered a real bug, probably in
the ddx.
--
Chris Wilson, Intel Open
to do so.
v3: Rebase to drm-core-next.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/drm_ioctl.c | 85 +--
1 files changed, 66 insertions(+), 19 deletions(-)
diff --git a/drivers/gpu/drm/drm_ioctl.c b/drivers/gpu/drm
-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/drm_gem.c |2 +-
drivers/gpu/drm/i915/i915_gem.c| 26 +-
drivers/gpu/drm/i915/i915_gem_tiling.c |4 ++--
drivers/gpu/drm/nouveau/nouveau_gem.c |8
drivers/gpu/drm
Directly read the GTT mapping for the contents of the batch buffers
rather than relying on possibly stale CPU caches. Also for completeness
scan the flushing/inactive lists for the current buffers - we are
collecting error state after all.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
This is required should we ever attempt to use an io-mapping where
KM_USER0 is verboten, such as inside an IRQ context.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: Eric Anholt e...@anholt.net
---
drivers/gpu/drm/i915/i915_gem.c|9 +
drivers/gpu/drm/i915
!)
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/drm_crtc_helper.c | 10 +-
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/drm_crtc_helper.c
b/drivers/gpu/drm/drm_crtc_helper.c
index 11fe9c8..7b1eb2f 100644
--- a/drivers/gpu/drm
Change the interface to expect a PTR_ERR specifing the real error code
as opposed to assuming a NULL return = -EINVAL. Just once the user may
not be at fault!
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/drm_crtc.c|4 ++--
drivers/gpu/drm/i915
It takes over 0.5s to probe the outputs and configure the i915 console.
This can be performed in the background whilst the rest of the system
is booting.
Based on a patch by Arjan van de Van ar...@linux.intel.com
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: Dave Airlie airl
On Sun, 8 Aug 2010 09:37:24 +0100, Chris Wilson ch...@chris-wilson.co.uk
wrote:
Dave Airlie:
I might be missing something, but what stops the race with something
reopening while we are in lastclose now?
The global_mutex which appears to fill this role is only taken inside
] system_call_fastpath+0x16/0x1b
v2: The spinlock is actually superfluous as access to open_count is
entirely serialised by drm_global_mutex and so can be dropped. The
count_lock spinlock instead appears to be used to protect access to
dev-buf_alloc and dev-buf_use.
Signed-off-by: Chris Wilson ch...@chris
. (The presumption being that all connections are probed in a
single xrandr pass, which is currently valid.)
References:
Bug 29536 - 2.6.35 causes ~600ms latency every 10s
https://bugs.freedesktop.org/show_bug.cgi?id=29536
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Reported-and-tested-by: Bruno
experiencing FIFO underruns, e.g. the screen flickering, then
it is likely that the error is benign (and just wasting power).
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http
Fix link failure without the vga arbitrator.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: Dave Airlie airl...@redhat.com
Cc: Jesse Barnes jbar...@virtuousgeek.org
---
include/linux/vgaarb.h | 15 +--
1 files changed, 13 insertions(+), 2 deletions(-)
diff --git
://bugzilla.kernel.org/show_bug.cgi?id=16265
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Reported-and-tested-by: Bruno Prémont bonb...@linux-vserver.org
---
drivers/gpu/drm/drm_crtc_helper.c | 17 +++--
1 files changed, 15 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm
-by: Meelis Roos mr...@linux.ee
Acked-by: Chris Wilson ch...@chris-wilson.co.uk
I have a pending patch to compile out this code when DEBUG_FS is not used
which is the root cause of the compilation failure. That is a little more
invasive than this simple patch...
--
Chris Wilson, Intel Open Source
failed, ret=%d\n, ret);
+ return false;
}
#define SDVO_CMD_NAME_ENTRY(cmd) {cmd, #cmd}
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo
On Mon, 23 Aug 2010 17:32:25 -0600, Jonathan Corbet cor...@lwn.net wrote:
On Mon, 23 Aug 2010 23:36:55 +0100
Chris Wilson ch...@chris-wilson.co.uk wrote:
Taking the patch at face value, the cause should be a mistake in error
handling. So the first step would be to identify which
is that the change to intel_wait_for_vblank() is actually
causing issues here.
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
On Tue, 24 Aug 2010 09:49:02 +0100, Sitsofe Wheeler sits...@yahoo.com wrote:
On Tue, Aug 24, 2010 at 09:16:50AM +0100, Chris Wilson wrote:
Ok, I'm a little happier that the hangcheck could be just another symptom
of the problem...
I think it is safe to assume that the bug is in i915
to the register
prior to sleeping.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/i915/intel_tv.c |9 -
1 files changed, 8 insertions(+), 1 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_tv.c b/drivers/gpu/drm/i915/intel_tv.c
index d2029ef..19b9739 100644
but is on the right lines. The first bug appears
to be that in the driver we have multiple bits of code to set the panel
backlight with different bugs.
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel
that it was unecessary.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/drm_gem.c |1 -
drivers/gpu/drm/nouveau/nouveau_gem.c |4 +---
drivers/gpu/drm/radeon/radeon_gem.c |3 +--
3 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm
performs the desired
re-initialisation of the polling should the user have toggled the
parameter, without the recursive side-effect.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: Dave Airlie airl...@redhat.com
---
drivers/gpu/drm/drm_crtc_helper.c |2 +-
1 files changed, 1
As the crtc will need to be reconfigured if the framebuffer depth
changes, we need to check for such a change during set_config().
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/drm_crtc_helper.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git
/show_bug.cgi?id=29536
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=16265
Reported-by: Bruno Prémont bonb...@linux-vserver.org
Tested-by: Sitsofe Wheeler sits...@yahoo.com
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: sta...@kernel.org
---
drivers/gpu/drm/drm_crtc_helper.c
instead. Much easier to type as well. ;-)
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=29536
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=16265
Reported-by: Bruno Prémont bonb...@linux-vserver.org
Tested-by: Sitsofe Wheeler sits...@yahoo.com
Signed-off-by: Chris Wilson ch...@chris
v2: Julien Cristau pointed out that @nondestructive results in
double-negatives and confusion when trying to interpret the parameter,
so use @force instead. Much easier to type as well. ;-)
And fix the miscompilation of vmgfx reported by Sedat Dilek.
Signed-off-by: Chris Wilson ch...@chris
;
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
On Sun, 19 Sep 2010 16:22:06 +0800, Zhenyu Wang zhen...@linux.intel.com wrote:
On 2010.09.19 09:14:53 +0100, Chris Wilson wrote:
On Sun, 19 Sep 2010 14:52:09 +0800, Zhenyu Wang zhen...@linux.intel.com
wrote:
Two reasons to add this param, one is for some AV device requiring
HDMI input
Hook the GEM vm open/close ops into the generic drm vm open/close so
that the vma entries are created and destroy appropriately.
Reported-by: Matt Mackall m...@selenic.com
Cc: Dave Airlie airl...@redhat.com
Cc: Jesse Barnes jbar...@virtuousgeek.org
Signed-off-by: Chris Wilson ch...@chris
Barnes jbar...@virtuousgeek.org
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Acked-by: Jesse Barnes jbar...@virtuousgeek.org
Cc: sta...@kernel.org
---
drivers/gpu/drm/drm_gem.c |9 -
drivers/gpu/drm/drm_vm.c | 28 ++--
include/drm/drmP.h|1
the root
cause.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
variant
to take the dev-struct_mutex around the kref_put.
Note that no driver is currently using the free_unlocked vfunc and it is
scheduled for removal.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=30454
Reported-by: Magnus Kessler magnus.kess...@gmx.net
Signed-off-by: Chris Wilson ch
it increase the normal object kref.
Now i915/radeon/nouveau drivers can drop the normal reference on
userspace object creation.
Signed-off-by: Dave Airlie airl...@redhat.com
Speaking as one who was recently confused by the not-a-kref nature of
handlecount:-
Reviewed-by: Chris Wilson ch...@chris
-detection on i8xx/i915).
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
and it is
scheduled for removal, hasten that process.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=30454
Reported-by: Magnus Kessler magnus.kess...@gmx.net
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: Dave Airlie airl...@redhat.com
Cc: sta...@kernel.org
---
drivers/gpu/drm/drm_gem.c | 22
Only drm/i915 does the bookkeeping that makes the information useful,
and the information maintained is driver specific, so move it out of the
core and into its single user.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: Dave Airlie airl...@redhat.com
---
drivers/gpu/drm/drm_debugfs.c
On Thu, 30 Sep 2010 09:10:26 +0100, Chris Wilson ch...@chris-wilson.co.uk
wrote:
In order to be fully threadsafe we need to check that the drm_gem_object
refcount is still 0 after acquiring the mutex in order to call the free
function. Otherwise, we may encounter scenarios like:
[snip
On Sat, 2 Oct 2010 15:59:17 +0200, Julia Lawall ju...@diku.dk wrote:
Extend the error handling code with operations found in other nearby error
handling code
Thanks, your fix was correct. Applied.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
On Sun, 3 Oct 2010 00:33:06 -0700, Keith Packard kei...@keithp.com wrote:
Instead of waiting for the display line value to settle, we can simply
wait for the pipe configuration register 'state' bit to turn off.
Thanks Keith, series applied to -fixes.
-Chris
--
Chris Wilson, Intel Open Source
or disconnected.
The obvious question: why are we doing this in the driver? It looks very
generic and can be shared between any of the drm drivers, so should this
be a facility provided by the server? It would be a good start to a KMS
module...
-Chris
--
Chris Wilson, Intel Open Source Technology
On Sun, 03 Oct 2010 16:06:07 -0700, Keith Packard kei...@keithp.com wrote:
On Sun, 03 Oct 2010 22:05:13 +0100, Chris Wilson ch...@chris-wilson.co.uk
wrote:
It appears that all users (crtc and encoders) are tracking dpms_mode, in
one form or another. Should we move this to core?
Sounds
which neatly pulls the
call before the mmio teardown.
Applied with the amendment to -next.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman
current change.
This fixed incorrect probe for DP monitor's audio. Fengguang helped
to test this set. We got correct DP sound on Eizo EV2333W, and HDMI
audio also worked with no regression found.
Applied. Thanks for the reminder, Zhenyu.
-Chris
--
Chris Wilson, Intel Open Source Technology
On Fri, 15 Oct 2010 17:14:33 +0200, Andrea Gelmini andrea.gelm...@gelma.net
wrote:
userpace - userspace
Applied, I had noticed but it didn't annoy me to actually write a patch.
Thanks,
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
pinned fb. I haven't
gotten to the bottom of this entirely yet, in theory this should be fixed
with the unpin fb added when disabling the connector in -next. I am not
totally convinced that explains everything in this case though:
https://bugs.freedesktop.org/show_bug.cgi?id=29857
-Chris
--
Chris
On Wed, 27 Oct 2010 23:25:34 +0200, Dan Carpenter erro...@gmail.com wrote:
obj_priv-pin_count is unsigned so the BUG_ON(obj_priv-pin_count 0)
will never trigger.
I have the same change pending, but not queued for -fixes since it is
impossible.
Thanks,
-Chris
--
Chris Wilson, Intel Open
.
Applied to fixes, thanks.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
to -fixes, thanks.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
... other mutation of the handle idr is still serialised using spin
locks.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/drm_gem.c | 20
1 files changed, 8 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/drm_gem.c b/drivers/gpu/drm
On Thu, 4 Nov 2010 10:09:53 +, Chris Wilson ch...@chris-wilson.co.uk
wrote:
/* Release reference and decrement refcount. */
idr_remove(filp-object_idr, handle);
spin_unlock(filp-table_lock);
+ synchronize_rcu();
+
This is an absolute performance killer. Looks
for input.
Thanks, and Jean many thanks for the fixes!
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
in your tree? Just cherry-pick and be
dammed?
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
in that it couples the bo much more tightly
with their allocations; to manage the bo is to manage those allocations.
This aligns well with my review of the memory management for i915.
Reviewed-by: Chris Wilson ch...@chris-wilson.co.uk
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
for
this series here).
[ /QUOTE ]
Against which GIT tree or linux-next version are these patchset for?
git://anongit.freedesktop.org/~danvet/drm direct-gtt
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri
length, hence realistically capped at around 4k entries.
However...
Thanks,
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
On Mon, 22 Nov 2010 12:56:42 +0300, Dan Carpenter erro...@gmail.com wrote:
On Sun, Nov 21, 2010 at 09:23:46AM +, Chris Wilson wrote:
Yes, it could. Not through normal use since relocation count can not be
more than buffer length, hence realistically capped at around 4k entries.
However
On Tue, 23 Nov 2010 22:36:42 -0800, Joe Perches j...@perches.com wrote:
Mark it so.
It was my mistake so I've applied this patch and will send upwards.
Thanks,
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
For a deferred-free cache of unreferenced bound objects, a simple
reference count is required without the baggage of kref.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/drm_gem.c | 12 +++-
drivers/gpu/drm/drm_info.c |2 +-
include/drm/drmP.h
On Fri, 26 Nov 2010 08:38:29 +1000, Dave Airlie airl...@redhat.com wrote:
On Thu, 2010-11-25 at 21:40 +, Chris Wilson wrote:
For a deferred-free cache of unreferenced bound objects, a simple
reference count is required without the baggage of kref.
eh?
The issue with kref
and a future mode set
that disables the specified connector will not turn the connector off.
Signed-off-by: Keith Packard kei...@keithp.com
Reviewed-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/drm_crtc_helper.c |7 +++
1 files changed, 7 insertions(+), 0
Ah, having seen the bounce when replying, I now know why Dave didn't see
these...
I've pushed the two patches to
git://people.freedesktop.org/~ickle/linux-2.6 for-airlied
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: Kristian Høgsberg k...@bitplanet.net
Cc: Jesse Barnes jbar...@virtuousgeek.org
---
drivers/gpu/drm/drm_irq.c | 19 ++-
1 files changed, 14 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu
Prevents code that assumes that the encoder is active when asked to be
disabled from dying a horrible death.
Reported-by: Daniel Vetter daniel.vet...@ffwll.ch
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: Dave Airlie airl...@redhat.com
---
drivers/gpu/drm/drm_crtc_helper.c |2
On Wed, 8 Dec 2010 17:34:24 +0100, Florian Mickler flor...@mickler.org wrote:
Does that mean that the kernel regression will not be
fixed/worked-around for old userspace?
I think there is some confusion in that I believe there is more than one
backlight bug at play here.
-Chris
--
Chris
for AUX ACK.
And it didn't break my bog standard DP setup. :)
Applied to -fixes and tagged for stable.
Thanks,
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http
://bugs.freedesktop.org/show_bug.cgi?id=28181
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
Cc: Kristian Høgsberg k...@bitplanet.net
Cc: Michel Dänzer daen...@vmware.com
---
glx/glxcmds.c | 23 +++
glx/glxdrawable.h |3 +++
glx/glxext.c | 15 ++-
3 files changed
On Mon, 13 Dec 2010 20:32:42 -0800, Segovia, Benjamin
benjamin.sego...@intel.com wrote:
To be more explicit, my concern is that I read that Chris Wilson proposed a
patch preventing the VM to swap pages still in GTT. I did not see any trace
of this patch in the main line yet.
No, because
;
if (adjusted_mode-flags DRM_MODE_FLAG_PHSYNC)
adpa |= ADPA_HSYNC_ACTIVE_HIGH;
if (adjusted_mode-flags DRM_MODE_FLAG_PVSYNC)
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http
Always useful to know just which connector was polled and had its
status updated.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/drm_crtc_helper.c |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/drivers/gpu/drm/drm_crtc_helper.c
b/drivers
From: Takashi Iwai ti...@suse.de
When the output is turned off via xrandr --off and re-enabled again
with the same mode, drm doesn't reset DPMS, thus it results in a black
screen. A typical example is something like:
% xrandr --output LVDS1 --mode 1024x768
% xrandr --output VGA1
On Wed, 22 Dec 2010 12:42:32 +, Chris Wilson ch...@chris-wilson.co.uk
wrote:
From: Takashi Iwai ti...@suse.de
This patch adds a new helper function to manage the drm_connector
DPMS so that it can be called commonly in both places.
Signed-off-by: Takashi Iwai ti...@suse.de
FWIW
fixes
the problem.
The question is whose BIOS is wrong? The Lenovo U160's or the
Sandybridge SDV? And why does it work for that other OS? Insert
rhetorical question of the day here.
It's back to the square one for one or the other platform...
-Chris
--
Chris Wilson, Intel Open Source
bounds.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
Otherwise if the connector is left in an unknown status, we leave the
function without any trace leaving me quite confused as to where the
output went.
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk
---
drivers/gpu/drm/drm_crtc_helper.c | 28
1 files changed
and was
confirmed by several people with identical machines. However, it
definitely caused a regression for working panels and therefore it will
be reverted.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel
.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
--
Learn how Oracle Real Application Clusters (RAC) One Node allows customers
to consolidate database storage, standardize their database environment
);
}
--
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
On Thu, 6 Jan 2011 21:55:23 +0100, Alex Riesen raa.l...@gmail.com wrote:
On Thu, Jan 6, 2011 at 18:49, Chris Wilson ch...@chris-wilson.co.uk wrote:
My fear is that some machines have a dependency between the backlight
and panel power status. The patch in question changed the timing between
1 - 100 of 3670 matches
Mail list logo
