Re: [PATCH] drm: Fix lock order reversal between mmap_sem and struct_mutex.

2009-02-20 Thread Thomas Hellstrom
Peter Zijlstra wrote: On Thu, 2009-02-19 at 22:02 +0100, Thomas Hellstrom wrote: It looks to me like the driver preferred locking order is object_mutex (which happens to be the device global struct_mutex) mmap_sem offset_mutex. So if one could avoid using the struct_mutex for

Re: [PATCH] drm: Only use DRM_IOCTL_UPDATE_DRAW compat wrapper for compat X86.

2009-02-20 Thread David Miller
From: Arnd Bergmann a...@arndb.de Date: Thu, 19 Feb 2009 15:19:01 +0100 On Wednesday 18 February 2009, David Miller wrote: drm: Only use DRM_IOCTL_UPDATE_DRAW compat wrapper for compat X86. Only X86 32-bit uses a different alignment for unsigned long long than it's 64-bit counterpart.

Re: [PATCH] drm: Preserve SHMLBA bits in hash key for _DRM_SHM mappings.

2009-02-20 Thread David Miller
From: Andrew Morton a...@linux-foundation.org Date: Thu, 19 Feb 2009 15:27:26 -0800 eg: arch/xtensa/include/asm/shmparam.h #define SHMLBA ((PAGE_SIZE DCACHE_WAY_SIZE)? PAGE_SIZE : DCACHE_WAY_SIZE) But including linux/shm.h here seems a bit silly. We'll see.. If DRM even builds on

Re: [PATCH] drm: Fix lock order reversal between mmap_sem and struct_mutex.

2009-02-20 Thread Peter Zijlstra
On Fri, 2009-02-20 at 09:31 +0100, Thomas Hellstrom wrote: Peter Zijlstra wrote: On Thu, 2009-02-19 at 22:02 +0100, Thomas Hellstrom wrote: It looks to me like the driver preferred locking order is object_mutex (which happens to be the device global struct_mutex) mmap_sem

Re: [PATCH] drm: Preserve SHMLBA bits in hash key for _DRM_SHM mappings.

2009-02-20 Thread Andrew Morton
On Fri, 20 Feb 2009 00:54:14 -0800 (PST) David Miller da...@davemloft.net wrote: From: Andrew Morton a...@linux-foundation.org Date: Thu, 19 Feb 2009 15:27:26 -0800 eg: arch/xtensa/include/asm/shmparam.h #define SHMLBA ((PAGE_SIZE DCACHE_WAY_SIZE)? PAGE_SIZE : DCACHE_WAY_SIZE)

Re: [PATCH] i915: add page flipping ioctl

2009-02-20 Thread Chris Wilson
On Thu, 2009-02-19 at 16:43 -0800, Jesse Barnes wrote: On Thursday 19 February 2009 11:37:01 Chris Wilson wrote: With a few additional suggestions by Jesse, I've managed to get tear-free compositing working on i915. Here's the diff on top of the original patch (though obviously this is just

Re: [Intel-gfx] [PATCH] i915: add page flipping ioctl

2009-02-20 Thread Chris Wilson
On Fri, 2009-02-20 at 13:46 +0800, Zou, Nanhai wrote: +struct drm_i915_gem_page_flip { +/** Handle of new front buffer */ Should this be handle of new front buffer or handle of the execbuf? I can't see how this can be an execbuf here. Do you mind elaborating? Anyway this reminded

Acid-like effect in dark regions on key press

2009-02-20 Thread Chris Wilson
Hello all, I'm seeing a bizarre problem whilst running wayland/cairo-drm/i915 under KMS. Occasionally after pressing a key (which is handled by wayland through the input layer) the dark regions (I'm estimating where the value is less than ~4) become garbage. I've managed to capture a screenshot

Re: [Intel-gfx] [PATCH] i915: add page flipping ioctl

2009-02-20 Thread Barry Scott
I saw this and thought it was wrong. int i915_seqno_passed(uint32_t seq1, uint32_t seq2) { return (int32_t)(seq1 - seq2) = 0; After a bit of thinking I realized that this is doing modulas arithmetic to deal with the seqno wrapping around. Given its not obvious at first glance you might

Re: [Intel-gfx] [PATCH] i915: add page flipping ioctl

2009-02-20 Thread Jesse Barnes
On Friday 20 February 2009 07:31:43 Barry Scott wrote: I saw this and thought it was wrong. int i915_seqno_passed(uint32_t seq1, uint32_t seq2) { return (int32_t)(seq1 - seq2) = 0; After a bit of thinking I realized that this is doing modulas arithmetic to deal with the seqno

[Bug 20235] New: [i915] [Regression] VT switch and suspend to disk broken

2009-02-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=20235 Summary: [i915] [Regression] VT switch and suspend to disk broken Product: DRI Version: unspecified Platform: x86 (IA32) OS/Version: Linux (All) Status: NEW Severity: normal

[Bug 20235] [i915] [Regression] VT switch and suspend to disk broken

2009-02-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=20235 --- Comment #1 from Stefano Avallone stava...@unina.it 2009-02-20 09:21:39 PST --- Created an attachment (id=23140) -- (http://bugs.freedesktop.org/attachment.cgi?id=23140) Xorg log -- Configure bugmail:

[PATCH] drm/i915: Fix regression in 95ca9d

2009-02-20 Thread Chris Wilson
The object is dereferenced before the NULL check. Oops. Fixes http://bugs.freedesktop.org/show_bug.cgi?id=20235 Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk --- drivers/gpu/drm/i915/i915_gem.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git

[Bug 20235] [i915] [Regression] VT switch and suspend to disk broken

2009-02-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=20235 Stefano Avallone stava...@unina.it changed: What|Removed |Added Status|NEW |RESOLVED

Re: [Mesa3d-dev] DRI2 flush extension

2009-02-20 Thread Kristian Høgsberg
On Thu, Feb 19, 2009 at 8:46 PM, Alan Hourihane al...@fairlite.co.uk wrote: Attached is a new DRI2 flush extension that allows the driver to perform a real flush before dispatching a swap or Xserver copy operation. Currently we do this before a DRI2CopyRegion() call. This allows drivers a

Re: [PATCH] drm: Take mmap_sem up front to avoid lock order violations.

2009-02-20 Thread Eric Anholt
On Thu, 2009-02-19 at 13:57 +0100, Nick Piggin wrote: On Thu, Feb 19, 2009 at 10:19:05AM +0100, Peter Zijlstra wrote: On Wed, 2009-02-18 at 11:38 -0500, k...@bitplanet.net wrote: From: Kristian Høgsberg k...@redhat.com A number of GEM operations (and legacy drm ones) want to copy data