[Bug 20211] New: Runing Kubrick game maximized freezes X and causes kernel panic

2009-02-19 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=20211 Summary: Runing Kubrick game maximized freezes X and causes kernel panic Product: DRI Version: XOrg CVS Platform: x86-64 (AMD64) OS/Version: Linux (All) Status: NEW

[Bug 20211] Runing Kubrick game maximized freezes X and causes kernel panic

2009-02-19 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=20211 --- Comment #1 from Jure Repinc 2009-02-19 01:30:09 PST --- Created an attachment (id=23111) --> (http://bugs.freedesktop.org/attachment.cgi?id=23111) Xorg.0.log -- Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email -

[Bug 12613] [Suspend regression][DRM, RADEON]

2009-02-19 Thread bugme-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=12613 mmvi...@yahoo.com changed: What|Removed |Added CC||mmvi...@yahoo.com --- Comment #

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

2009-02-19 Thread Peter Zijlstra
On Wed, 2009-02-18 at 11:38 -0500, k...@bitplanet.net wrote: > From: Kristian Høgsberg > > A number of GEM operations (and legacy drm ones) want to copy data to > or from userspace while holding the struct_mutex lock. However, the > fault handler calls us with the mmap_sem held and thus enforces

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

2009-02-19 Thread Peter Zijlstra
On Thu, 2009-02-19 at 10:19 +0100, Peter Zijlstra wrote: > On Wed, 2009-02-18 at 11:38 -0500, k...@bitplanet.net wrote: > > From: Kristian Høgsberg > > > > A number of GEM operations (and legacy drm ones) want to copy data to > > or from userspace while holding the struct_mutex lock. However, th

Re: r200 lockup in radeon_cp_texture/radeon_freelist_get

2009-02-19 Thread Roland Scheidegger
On 19.02.2009 12:23, Arkadi Shishlov wrote: > Roland Scheidegger wrote: >> I suspect you're hitting a r200 asic bug, which isn't present in rv250 >> and other r200 family members. There are workarounds in the driver for >> this (see r200UpdateTextureState), but to my knowledge they are >> insuffici

Re: r200 lockup in radeon_cp_texture/radeon_freelist_get

2009-02-19 Thread Arkadi Shishlov
Roland Scheidegger wrote: > I suspect you're hitting a r200 asic bug, which isn't present in rv250 > and other r200 family members. There are workarounds in the driver for > this (see r200UpdateTextureState), but to my knowledge they are > insufficient for two-pass ATI_fragment_shader shaders. Ther

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

2009-02-19 Thread Nick Piggin
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 > > > > A number of GEM operations (and legacy drm ones) want to copy data to > > or from userspace while holding the struct_mutex lock. Howe

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

2009-02-19 Thread Arnd Bergmann
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. > > Therefore this compat translation is only correct, and only needed, >

Re: r200 lockup in radeon_cp_texture/radeon_freelist_get

2009-02-19 Thread Alex Deucher
On Thu, Feb 19, 2009 at 7:26 AM, Roland Scheidegger wrote: > On 19.02.2009 12:23, Arkadi Shishlov wrote: >> Roland Scheidegger wrote: >>> I suspect you're hitting a r200 asic bug, which isn't present in rv250 >>> and other r200 family members. There are workarounds in the driver for >>> this (see

[Bug 20211] Runing Kubrick game maximized freezes X and causes kernel panic

2009-02-19 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=20211 Alex Deucher changed: What|Removed |Added Attachment #23111|application/x-trash |text/plain mime type|

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

2009-02-19 Thread Kristian Høgsberg
On Thu, 2009-02-19 at 11:33 +0100, Peter Zijlstra wrote: > On Thu, 2009-02-19 at 10:19 +0100, Peter Zijlstra wrote: > > On Wed, 2009-02-18 at 11:38 -0500, k...@bitplanet.net wrote: > > > From: Kristian Høgsberg > > > > > > A number of GEM operations (and legacy drm ones) want to copy data to > >

Re: r200 lockup in radeon_cp_texture/radeon_freelist_get

2009-02-19 Thread Stephane Marchesin
On Thu, Feb 19, 2009 at 15:46, Alex Deucher wrote: > On Thu, Feb 19, 2009 at 7:26 AM, Roland Scheidegger > wrote: >> On 19.02.2009 12:23, Arkadi Shishlov wrote: >>> Roland Scheidegger wrote: I suspect you're hitting a r200 asic bug, which isn't present in rv250 and other r200 family mem

Re: r200 lockup in radeon_cp_texture/radeon_freelist_get

2009-02-19 Thread Alex Deucher
On Thu, Feb 19, 2009 at 9:51 AM, Stephane Marchesin wrote: > On Thu, Feb 19, 2009 at 15:46, Alex Deucher wrote: >> On Thu, Feb 19, 2009 at 7:26 AM, Roland Scheidegger >> wrote: >>> On 19.02.2009 12:23, Arkadi Shishlov wrote: Roland Scheidegger wrote: > I suspect you're hitting a r200 as

Re: [PATCH] i915: add page flipping ioctl

2009-02-19 Thread Chris Wilson
On Mon, 2009-02-16 at 10:55 -0800, Jesse Barnes wrote: > On Sunday, February 15, 2009 4:02 pm Chris Wilson wrote: > > On my i915, the flip never occurs and we wait forever on the the vblank. > > So I presume the command we sent the chip is invalid, or we miss the > > irq? (I double-checked with loc

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

2009-02-19 Thread Kristian Høgsberg
On Thu, 2009-02-19 at 16:17 +0100, Nick Piggin wrote: > On Thu, Feb 19, 2009 at 09:49:40AM -0500, Kristian Høgsberg wrote: > > > > > Secondly, mmap_sem is not a recursive lock (very few kernel locks are, > > > and we generally frown upon recursive locking schemes), this means that > > > the fault

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

2009-02-19 Thread Nick Piggin
On Thu, Feb 19, 2009 at 09:49:40AM -0500, Kristian Høgsberg wrote: > > > Secondly, mmap_sem is not a recursive lock (very few kernel locks are, > > and we generally frown upon recursive locking schemes), this means that > > the fault handler still cannot function properly. > > I understand, but w

Re: [PATCH] i915: add page flipping ioctl

2009-02-19 Thread Chris Wilson
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 a suggestion, still need to prevent multiple pending flips to the same plane and ensure that the old buffer is eventuall

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

2009-02-19 Thread Thomas Hellstrom
Peter Zijlstra wrote: > On Tue, 2009-02-17 at 16:59 -0800, Eric Anholt wrote: > >> The basic problem was >> mmap_sem (do_mmap()) -> struct_mutex (drm_gem_mmap(), i915_gem_fault()) >> struct_mutex (i915_gem_execbuffer()) -> mmap_sem (copy_from/to_user()) >> > > That's not the only problem, t

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

2009-02-19 Thread Andrew Morton
On Wed, 18 Feb 2009 15:41:02 -0800 (PST) David Miller wrote: > > Platforms such as sparc64 have D-cache aliasing issues. We > cannot allow virtual mappings in different contexts to be such > that two cache lines can be loaded for the same backing data. > Updates to one cache line won't be seen

Re: [PATCH] i915: add page flipping ioctl

2009-02-19 Thread Jesse Barnes
On Thursday 19 February 2009 06:55:28 Chris Wilson wrote: > diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c > index 3795dbc..93e677a 100644 > --- a/drivers/gpu/drm/drm_irq.c > +++ b/drivers/gpu/drm/drm_irq.c > @@ -435,6 +435,8 @@ EXPORT_SYMBOL(drm_vblank_get); > */ > void drm_

Re: [PATCH] i915: add page flipping ioctl

2009-02-19 Thread Jesse Barnes
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 a suggestion, still need > to prevent multiple pending f

Re: [PATCH] i915: add page flipping ioctl

2009-02-19 Thread Jesse Barnes
On Thursday 19 February 2009 16:43:54 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 thi

[git pull] drm fixes tree.

2009-02-19 Thread Dave Airlie
Hi Linus, Please pull the 'drm-fixes' branch from ssh://master.kernel.org/pub/scm/linux/kernel/git/airlied/drm-2.6.git drm-fixes Intel: This contains one lockdep fix, the other is still under discussion along with a few locking and error path fixes. It also contains some minor kms updates for

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

2009-02-19 Thread Eric Anholt
On Thu, 2009-02-19 at 23:26 +0100, 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.

DRI2 flush extension

2009-02-19 Thread Alan Hourihane
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 real "end of scene" flush to ensure rendering is complete prior to a swap. I've

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

2009-02-19 Thread Peter Zijlstra
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 object bookkeepin

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

2009-02-19 Thread Zou, Nanhai
>-Original Message- >From: intel-gfx-boun...@lists.freedesktop.org >[mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of Jesse Barnes >Sent: 2009年2月20日 8:48 >To: dri-devel@lists.sourceforge.net >Cc: intel-...@lists.freedesktop.org >Subject: Re: [Intel-gfx] [PATCH] i915: add page fl

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

2009-02-19 Thread Peter Zijlstra
On Thu, 2009-02-19 at 18:04 -0800, Eric Anholt wrote: > On Thu, 2009-02-19 at 23:26 +0100, 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