[Nouveau] [PATCH] add missing return in nv50_accel_init failure path

2011-11-06 Thread Marcin Slusarz
--- src/nv50_accel.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/nv50_accel.c b/src/nv50_accel.c index d6c7528..ca06d8a 100644 --- a/src/nv50_accel.c +++ b/src/nv50_accel.c @@ -102,6 +102,7 @@ NVAccelInitNV50TCL(ScrnInfoPtr pScrn)

[Nouveau] [PATCH] nv50/exa: wait for enough space for at least one composite operation

2011-11-06 Thread Marcin Slusarz
--- src/nv50_exa.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/nv50_exa.c b/src/nv50_exa.c index b6094ae..6f753f2 100644 --- a/src/nv50_exa.c +++ b/src/nv50_exa.c @@ -838,7 +838,7 @@ NV50EXAPrepareComposite(int op, NV50EXA_LOCALS(pspix); const

[Nouveau] [PATCH] Default to 16bpp for low memory cards.

2011-11-06 Thread Marcin Slusarz
--- src/nv_driver.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/src/nv_driver.c b/src/nv_driver.c index 3467dbe..651ee0e 100644 --- a/src/nv_driver.c +++ b/src/nv_driver.c @@ -603,6 +603,7 @@ NVPreInit(ScrnInfoPtr pScrn, int flags) const char *reason;

[Nouveau] [PATCH] drm/nouveau: fix acpi related kbuild warnings

2011-11-02 Thread Marcin Slusarz
. To fix this, introduce hidden DRM_NOUVEAU_ACPI variable which selects all acpi/x86 related configs and select it when both X86 and ACPI are enabled. Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/Kconfig |9 +++-- drivers/gpu/drm/nouveau/Makefile

Re: [Nouveau] [PATCH] drm: serialize access to debugs_nodes.list

2011-10-31 Thread Marcin Slusarz
On Mon, Oct 31, 2011 at 09:06:57AM +0100, Daniel Vetter wrote: This is just ugly, you're adding a mutex to every drm_info_node, but only use the one embedded into the minor. On a quick grep we're only ever using the list in there, so I suggest to - replace minor-debugfs_node.list with

[Nouveau] [PATCH] drm: serialize access to list of debugfs files

2011-10-31 Thread Marcin Slusarz
of xterms with looped glxinfo. Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/drm_debugfs.c | 12 +--- drivers/gpu/drm/i915/i915_debugfs.c |5 - include/drm/drmP.h |4 +++- 3 files changed, 16 insertions(+), 5 deletions(-) diff

Re: [Nouveau] enable ctxprog xfer only when we need it to save power introduces big performance regression

2011-10-30 Thread Marcin Slusarz
On Sun, Oct 30, 2011 at 02:25:08AM +0200, Maxim Levitsky wrote: On Sat, 2011-10-29 at 21:09 +0200, Marcin Slusarz wrote: On Sat, Oct 29, 2011 at 07:29:23PM +0200, Marcin Slusarz wrote: On Sat, Oct 29, 2011 at 07:08:01PM +0200, Marcin Slusarz wrote: Hi I've bisected pretty big

[Nouveau] [PATCH] drm: serialize access to debugs_nodes.list

2011-10-30 Thread Marcin Slusarz
of xterms with looped glxinfo. Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/drm_debugfs.c |5 + include/drm/drmP.h|1 + 2 files changed, 6 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/drm_debugfs.c b/drivers/gpu/drm/drm_debugfs.c index

[Nouveau] enable ctxprog xfer only when we need it to save power introduces big performance regression

2011-10-29 Thread Marcin Slusarz
Hi I've bisected pretty big performance regression (nv92): $ git bisect good b2737681d5442f05ab6419e05468c3d2511a5ced is the first bad commit commit b2737681d5442f05ab6419e05468c3d2511a5ced Author: Martin Peres martin.pe...@ensi-bourges.fr Date: Sat Jul 30 23:08:45 2011 +0200 drm/nv50/gr:

Re: [Nouveau] enable ctxprog xfer only when we need it to save power introduces big performance regression

2011-10-29 Thread Marcin Slusarz
On Sat, Oct 29, 2011 at 07:08:01PM +0200, Marcin Slusarz wrote: Hi I've bisected pretty big performance regression (nv92): $ git bisect good b2737681d5442f05ab6419e05468c3d2511a5ced is the first bad commit commit b2737681d5442f05ab6419e05468c3d2511a5ced Author: Martin Peres martin.pe

Re: [Nouveau] enable ctxprog xfer only when we need it to save power introduces big performance regression

2011-10-29 Thread Marcin Slusarz
On Sat, Oct 29, 2011 at 07:29:23PM +0200, Marcin Slusarz wrote: On Sat, Oct 29, 2011 at 07:08:01PM +0200, Marcin Slusarz wrote: Hi I've bisected pretty big performance regression (nv92): $ git bisect good b2737681d5442f05ab6419e05468c3d2511a5ced is the first bad commit commit

Re: [Nouveau] [PATCH] drm/nouveau: initialize chan-fence.lock before use

2011-10-09 Thread Marcin Slusarz
On Fri, Sep 09, 2011 at 02:16:42PM +0200, Marcin Slusarz wrote: Fence lock needs to be initialized before any call to nouveau_channel_put because it calls nouveau_channel_idle-nouveau_fence_update which uses fence lock. BUG: spinlock bad magic on CPU#0, test/24134 lock: 88019f90dba8

Re: [Nouveau] [PATCH 09/10] nouveau: EVO debug iface

2011-10-09 Thread Marcin Slusarz
On Sun, Oct 09, 2011 at 10:58:39PM +0200, Maxim Levitsky wrote: This code adds new debugfs file that allows to send evo commands from userspace. Its usefull for RE. Signed-off-by: Maxim Levitsky maximlevit...@gmail.com Just a minor nit below. ---

[Nouveau] [PATCH] xv: add support for AI44/IA44

2011-10-08 Thread Marcin Slusarz
From: Marcin Slusarz marcin.slus...@gmail.com Date: Sat, 10 Sep 2011 23:14:53 +0200 Subject: [PATCH] xv: support AI44/IA44 It's needed for osd in mplayer's xvmc output. --- src/nouveau_xv.c |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/src/nouveau_xv.c b/src

[Nouveau] [PATCH] drm/ttm/nouveau: add DRM_NOUVEAU_GEM_CPU_PREP_TIMEOUT

2011-09-18 Thread Marcin Slusarz
which eventually manifests in crash with [mi] EQ overflowing. The server is probably stuck in an infinite loop. message instead of being propagated to DDX. The solutions is to add new ioctl NOUVEAU_GEM_CPU_PREP_TIMEOUT with timeout parameter and decrease it on every signal. Signed-off-by: Marcin

[Nouveau] [PATCH] nouveau: add support for DRM_NOUVEAU_GEM_CPU_PREP_TIMEOUT

2011-09-18 Thread Marcin Slusarz
New ioctl is used internally by nouveau_bo_wait to properly handle timeouts with signals. --- include/drm/nouveau_drm.h | 33 ++ nouveau/nouveau_bo.c | 66 ++--- 2 files changed, 77 insertions(+), 22 deletions(-) diff --git

Re: [Nouveau] [PATCH] drm/ttm/nouveau: add DRM_NOUVEAU_GEM_CPU_PREP_TIMEOUT

2011-09-18 Thread Marcin Slusarz
On Sun, Sep 18, 2011 at 03:59:50PM +0200, Daniel Vetter wrote: On Sun, Sep 18, 2011 at 03:18:57PM +0200, Marcin Slusarz wrote: Currently DRM_NOUVEAU_GEM_CPU_PREP ioctl is broken WRT handling of signals. nouveau_gem_ioctl_cpu_prep calls ttm_bo_wait which waits for fence to signal or 3

Re: [Nouveau] [Mesa-dev] [PATCH v2] nouveau: Add support for XvMC acceleration

2011-09-13 Thread Marcin Slusarz
On Sun, Sep 11, 2011 at 12:54:20AM +0200, Marcin Slusarz wrote: On Sat, Sep 10, 2011 at 06:59:39PM +0200, Maarten Lankhorst wrote: Hey ★, On 09/10/2011 06:28 PM, ★ Emeric wrote: Hi Maarten, I tried the v2 patch with my good old geforce 8600M GS, and it worked well. The CPU

Re: [Nouveau] [Mesa-dev] [PATCH v2] nouveau: Add support for XvMC acceleration

2011-09-10 Thread Marcin Slusarz
, and it leads to memory corruption in mplayer. Patch below fixes it, but OSD still doesn't work. --- From 00f7986291b86db9a09c1651eb4450c69761fe27 Mon Sep 17 00:00:00 2001 From: Marcin Slusarz marcin.slus...@gmail.com Date: Sat, 10 Sep 2011 23:14:53 +0200 Subject: [PATCH] xv: support AI44/IA44 --- src

[Nouveau] [PATCH] drm/nouveau: initialize chan-fence.lock before use

2011-09-09 Thread Marcin Slusarz
nouveau_ioctl_fifo_alloc+0x50/0x130 drm_ioctl+0x289/0x361 do_vfs_ioctl+0x4dd/0x52c sys_ioctl+0x42/0x65 system_call_fastpath+0x16/0x1b It's easily triggerable from userspace. Additionally remove double initialization of chan-fence.pending. Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com Cc: sta

[Nouveau] [PATCH] nv50: fix screen-blitctx memory leak

2011-09-06 Thread Marcin Slusarz
--- src/gallium/drivers/nv50/nv50_screen.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/gallium/drivers/nv50/nv50_screen.c b/src/gallium/drivers/nv50/nv50_screen.c index 235cd82..36adb44 100644 --- a/src/gallium/drivers/nv50/nv50_screen.c +++

[Nouveau] [PATCH] nouveau: fix nouveau_fence leak

2011-09-06 Thread Marcin Slusarz
Note: This is a candidate for the 7.11 branch. --- src/gallium/drivers/nouveau/nouveau_fence.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/src/gallium/drivers/nouveau/nouveau_fence.c b/src/gallium/drivers/nouveau/nouveau_fence.c index d8f59dc..ea2038c 100644 ---

[Nouveau] [PATCH] drm/nouveau: properly handle allocation failure in nouveau_sgdma_populate

2011-08-22 Thread Marcin Slusarz
Not cleaning after alloc failure would result in crash on destroy, because nouveau_sgdma_clear assumes ttm_alloced to be not null when pages is not null. Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com Cc: sta...@kernel.org --- drivers/gpu/drm/nouveau/nouveau_sgdma.c |5 - 1 files

[Nouveau] [RFC PATCH] drm/nouveau: fix nv04_sgdma_bind on non-4kB pages archs

2011-08-22 Thread Marcin Slusarz
nv04_sgdma_bind binds the same page multiple times on architectures where PAGE_SIZE != 4096. Let's fix it. Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- I'm not sure about it - other backends seems to not care about page size at all... --- drivers/gpu/drm/nouveau/nouveau_sgdma.c

[Nouveau] [PATCH] drm/nouveau: fix printk typo in ioremap failure path

2011-08-22 Thread Marcin Slusarz
Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/nouveau_state.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_state.c b/drivers/gpu/drm/nouveau/nouveau_state.c index 283dfb4..43cbc81 100644

[Nouveau] [PATCH] drm/nouveau: restore subchannel-object binding on resume

2011-07-17 Thread Marcin Slusarz
-by: Andrew Randrianasulu rand...@mail.ru Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/nouveau_dma.c |3 +-- drivers/gpu/drm/nouveau/nouveau_dma.h | 20 drivers/gpu/drm/nouveau/nouveau_drv.c |8 drivers/gpu/drm

Re: [Nouveau] nouveaufb problems with NV18

2011-07-01 Thread Marcin Slusarz
On Fri, Jul 01, 2011 at 01:16:49AM -0400, George Spelvin wrote: I mentioned these problems back in March, but went back to the nv driver and ignored it. Since I need to reboot for another reason, I gave nouveau a try again, This time, I'm using the kernel git tip, off of 3.0-rc5:

Re: [Nouveau] nouveaufb problems with NV18

2011-07-01 Thread Marcin Slusarz
On Fri, Jul 01, 2011 at 03:22:38PM -0400, George Spelvin wrote: Lowering bpp/resolution for framebuffer and/or X might help. At least temporarily. Bingo! Dropping to 16 bpp produced a working display. Xorg.0.log appensed. Great :) Screenshot might be useful. Will be a few days; I

[Nouveau] gpu lockup detection and fallback to noaccel

2011-06-14 Thread Marcin Slusarz
Hi I have a very rough patchset which adds support for GPU lockup detection and fallback to (more or less) noaccel to xf86-video-nouveau. As the patches are only a proof of concept and needs a lot of work, I would like to know first if this is a desired feature - I don't want to spend a couple

[Nouveau] [mesa PATCH] nv50: fix nv50_sampler_state_delete array overflow

2011-06-05 Thread Marcin Slusarz
num_samplers is array of 3 elements, not 5 --- src/gallium/drivers/nv50/nv50_state.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/gallium/drivers/nv50/nv50_state.c b/src/gallium/drivers/nv50/nv50_state.c index 799f496..8b54d9f 100644 ---

Re: [Nouveau] [PATCH 1/2] xorg/nouveau: rename to nouveau2

2011-06-05 Thread Marcin Slusarz
On Mon, May 16, 2011 at 09:50:29PM +0200, Marcin Slusarz wrote: --- src/gallium/targets/xorg-nouveau/Makefile |2 +- src/gallium/targets/xorg-nouveau/nouveau_xorg.c | 14 +++--- 2 files changed, 8 insertions(+), 8 deletions(-) ping

Re: [Nouveau] [Mesa-dev] [PATCH v2 2/2] xorg/nouveau: blacklist all pre NV30 cards

2011-06-05 Thread Marcin Slusarz
On Sun, Jun 05, 2011 at 09:15:47PM +0200, Maarten Maathuis wrote: 2011/6/5 Stéphane Marchesin stephane.marche...@gmail.com: On Sun, Jun 5, 2011 at 12:06, Marcin Slusarz marcin.slus...@gmail.com wrote: On Tue, May 17, 2011 at 12:20:14AM +0200, Marcin Slusarz wrote: Bail out early in probe

[Nouveau] [PATCH 1/2] xorg/nouveau: rename to nouveau2

2011-05-16 Thread Marcin Slusarz
--- src/gallium/targets/xorg-nouveau/Makefile |2 +- src/gallium/targets/xorg-nouveau/nouveau_xorg.c | 14 +++--- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/gallium/targets/xorg-nouveau/Makefile b/src/gallium/targets/xorg-nouveau/Makefile index

[Nouveau] [PATCH v2] nv50/nvc0: make transfers aware of PIPE_TRANSFER_MAP_DIRECTLY

2011-05-09 Thread Marcin Slusarz
If state tracker asked us to map resource directly and we can't do it (because of tiling), return NULL instead of doing full transfer - state tracker should handle it and fallback to some other method or repeat transfer without PIPE_TRANSFER_MAP_DIRECTLY. It greatly improves performance of xorg

Re: [Nouveau] [Mesa-dev] [PATCH] gallium/xorg st/nv50/i915: add PIPE_BIND_CURSOR

2011-05-08 Thread Marcin Slusarz
On Tue, May 03, 2011 at 12:06:33PM +0200, Daniel Vetter wrote: On Mon, May 2, 2011 at 8:40 PM, Marcin Slusarz marcin.slus...@gmail.com wrote: On Mon, May 02, 2011 at 03:11:00PM +0200, Daniel Vetter wrote: On Mon, May 2, 2011 at 2:56 PM, Benjamin Franzke benjaminfran...@googlemail.com

[Nouveau] [PATCH] gallium/nouveau: remove unused nouveau_screen_bo_user

2011-05-08 Thread Marcin Slusarz
--- src/gallium/drivers/nouveau/nouveau_screen.c | 14 -- src/gallium/drivers/nouveau/nouveau_screen.h |2 -- 2 files changed, 0 insertions(+), 16 deletions(-) diff --git a/src/gallium/drivers/nouveau/nouveau_screen.c b/src/gallium/drivers/nouveau/nouveau_screen.c index

[Nouveau] [PATCH] nv50/nvc0: make transfers aware of PIPE_TRANSFER_MAP_DIRECTLY

2011-05-08 Thread Marcin Slusarz
If state tracker asked us to map resource directly and we can't do it (because of tiling), return NULL instead of doing full transfer - state tracker should handle it and fallback to some other method or repeat transfer without PIPE_TRANSFER_MAP_DIRECTLY. It greatly improves performance of xorg

Re: [Nouveau] [PATCH] drm/nouveau: printk info about disabled modesetting

2011-05-02 Thread Marcin Slusarz
On Mon, May 02, 2011 at 04:51:14PM +1000, Ben Skeggs wrote: On Mon, 2011-05-02 at 00:03 +0200, Marcin Slusarz wrote: On Mon, Apr 04, 2011 at 07:27:35PM +0200, Marcin Slusarz wrote: ... to make it obvious why nouveau refused to drive the hardware. Signed-off-by: Marcin Slusarz

[Nouveau] [PATCH] drm/nouveau: make cursor_set implementation consistent with other drivers

2011-05-01 Thread Marcin Slusarz
consistent with them. Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/nv04_crtc.c |6 +++--- drivers/gpu/drm/nouveau/nv50_crtc.c |6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nv04_crtc.c b/drivers/gpu/drm

Re: [Nouveau] 2.6.39-rc1 nouveau(?) regression (bisected)

2011-04-19 Thread Marcin Slusarz
On Sun, Apr 17, 2011 at 06:49:20PM +0200, Marcin Slusarz wrote: On Sun, Apr 17, 2011 at 12:24:27PM -0400, Kyle Spaans wrote: On Sun, Apr 17, 2011 at 05:45:57PM +0200, Marcin Slusarz wrote: On Sun, Apr 17, 2011 at 11:12:04AM -0400, Kyle Spaans wrote: On Sat, Apr 16, 2011 at 07:50:28PM

Re: [Nouveau] 2.6.39-rc1 nouveau(?) regression (bisected)

2011-04-19 Thread Marcin Slusarz
On Mon, Apr 18, 2011 at 01:27:10PM -0700, Linus Torvalds wrote: On Mon, Apr 18, 2011 at 1:02 PM, Marcin Slusarz marcin.slus...@gmail.com wrote: It's some nasty corruption: Looks like something wrote 0x to free'd memory. Enabling DEBUG_PAGEALLOC *might* show where it happens

[Nouveau] [PATCH 1/2] drm/nouveau: fix notifier memory corruption bug

2011-04-19 Thread Marcin Slusarz
, so it didn't matter it was a value in bytes). Reported-by: Dominik Brodowski li...@dominikbrodowski.net Reported-by: Nigel Cunningham l...@nigelcunningham.com.au Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com Cc: Ben Skeggs bske...@redhat.com Cc: Pekka Paalanen p...@iki.fi Cc: sta

[Nouveau] [PATCH 2/2] drm/nouveau: fix allocation of notifier object

2011-04-19 Thread Marcin Slusarz
Commit 73412c3854c877e5f37ad944ee8977addde4d35a (drm/nouveau: allocate kernel's notifier object at end of block) intended to align end of notifier block to page boundary, but start of block was miscalculated to be off by -16 bytes. Fix it. Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com Cc

Re: [Nouveau] 2.6.39-rc1 nouveau regression (bisected)

2011-04-18 Thread Marcin Slusarz
On Sun, Apr 17, 2011 at 11:12:04AM -0400, Kyle Spaans wrote: On Sat, Apr 16, 2011 at 07:50:28PM -0400, Kyle Spaans wrote: On Sun, Apr 17, 2011 at 08:12:35AM +1000, Nigel Cunningham wrote: On 15/04/11 16:11, Dominik Brodowski wrote: On Thu, Apr 14, 2011 at 09:02:01PM +0200, Marcin Slusarz

Re: [Nouveau] 2.6.39-rc1 nouveau regression (bisected)

2011-04-18 Thread Marcin Slusarz
, Apr 14, 2011 at 09:02:01PM +0200, Marcin Slusarz wrote: On Thu, Apr 14, 2011 at 07:05:59PM +0200, Dominik Brodowski wrote: Thought about CCing Linus to show him that 2.6.39-rcX isn't as calm to everyone, but then chose to CC Maciej instead: Would you be so kind and add

Re: [Nouveau] 2.6.39-rc1 nouveau regression (bisected)

2011-04-18 Thread Marcin Slusarz
On Sun, Apr 17, 2011 at 12:24:27PM -0400, Kyle Spaans wrote: On Sun, Apr 17, 2011 at 05:45:57PM +0200, Marcin Slusarz wrote: On Sun, Apr 17, 2011 at 11:12:04AM -0400, Kyle Spaans wrote: On Sat, Apr 16, 2011 at 07:50:28PM -0400, Kyle Spaans wrote: On Sun, Apr 17, 2011 at 08:12:35AM +1000

Re: [Nouveau] [PATCH] drm/nouveau: Fix a complier warning in the card init error path

2011-04-17 Thread Marcin Slusarz
On Sun, Apr 17, 2011 at 04:14:53PM -0400, Jimmy Rentz wrote: Fix an unitialized variable complier warning in nouveau_card_init. Signed-off-by: Jimmy Rentz jb17bs...@gmail.com diff --git a/drivers/gpu/drm/nouveau/nouveau_state.c b/drivers/gpu/drm/nouveau/nouveau_state.c index

Re: [Nouveau] 2.6.39-rc1 nouveau regression (bisected)

2011-04-14 Thread Marcin Slusarz
On Thu, Apr 14, 2011 at 07:05:59PM +0200, Dominik Brodowski wrote: Thought about CCing Linus to show him that 2.6.39-rcX isn't as calm to everyone, but then chose to CC Maciej instead: Would you be so kind and add this to your regression list? Thanks! Since commit 38f1cff From: Dave

[Nouveau] [PATCH] drm/nouveau: printk info about disabled modesetting

2011-04-04 Thread Marcin Slusarz
... to make it obvious why nouveau refused to drive the hardware. Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/nouveau_drv.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_drv.c b/drivers/gpu/drm

[Nouveau] [PATCH] drm/nouveau: fix oops on unload with disabled LVDS panel

2011-03-10 Thread Marcin Slusarz
] ? arch_unmap_area_topdown+0x0/0x20 [c15096f4] syscall_call+0x7/0xb Reported-by: Francesco Marella francesco.mare...@gmail.com Tested-by: Francesco Marella francesco.mare...@gmail.com Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/nv04_dfp.c | 20 +++- 1

Re: [Nouveau] [PATCH] drm/nouveau: fix __nouveau_fence_wait performance regression

2011-03-09 Thread Marcin Slusarz
On Wed, Mar 09, 2011 at 06:34:36PM +0100, Francisco Jerez wrote: Marcin Slusarz marcin.slus...@gmail.com writes: On Tue, Mar 08, 2011 at 05:22:52PM +0100, Francisco Jerez wrote: Marcin Slusarz marcin.slus...@gmail.com writes: On Tue, Mar 08, 2011 at 01:58:50AM +0100, Francisco Jerez

Re: [Nouveau] [PATCH] drm/nouveau: fix __nouveau_fence_wait performance regression

2011-03-08 Thread Marcin Slusarz
On Tue, Mar 08, 2011 at 01:58:50AM +0100, Francisco Jerez wrote: Marcin Slusarz marcin.slus...@gmail.com writes: On Tue, Mar 08, 2011 at 08:24:26AM +1000, Ben Skeggs wrote: On Mon, 2011-03-07 at 18:18 +, Maarten Maathuis wrote: On Fri, Mar 4, 2011 at 4:49 PM, Marcin Slusarz

Re: [Nouveau] [PATCH] drm/nouveau: fix __nouveau_fence_wait performance regression

2011-03-08 Thread Marcin Slusarz
On Tue, Mar 08, 2011 at 05:22:52PM +0100, Francisco Jerez wrote: Marcin Slusarz marcin.slus...@gmail.com writes: On Tue, Mar 08, 2011 at 01:58:50AM +0100, Francisco Jerez wrote: Marcin Slusarz marcin.slus...@gmail.com writes: On Tue, Mar 08, 2011 at 08:24:26AM +1000, Ben Skeggs wrote

[Nouveau] [PATCH] drm/nouveau: properly handle pushbuffer check failures

2011-03-07 Thread Marcin Slusarz
] do_raw_spin_lock+0x14/0x13c RSP 8801a6aefb88 ---[ end trace 0014d5d93e6147e1 ]--- Additionally, don't call validate_fini twice in case of validation failure. Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/nouveau_gem.c |6 -- 1 files changed, 4

Re: [Nouveau] [PATCH] gallium/nv50: use 0x8697 class on NVAF

2011-03-07 Thread Marcin Slusarz
On Mon, Mar 07, 2011 at 06:15:46PM +, Maarten Maathuis wrote: On Fri, Mar 4, 2011 at 4:50 PM, Marcin Slusarz marcin.slus...@gmail.com wrote: On Tue, Feb 22, 2011 at 06:32:40PM +0100, Marcin Slusarz wrote: Addresses: https://bugs.launchpad.net/ubuntu/+source/xserver-xorg-video-nouveau

Re: [Nouveau] [PATCH] drm/nouveau: fix __nouveau_fence_wait performance regression

2011-03-07 Thread Marcin Slusarz
On Tue, Mar 08, 2011 at 08:24:26AM +1000, Ben Skeggs wrote: On Mon, 2011-03-07 at 18:18 +, Maarten Maathuis wrote: On Fri, Mar 4, 2011 at 4:49 PM, Marcin Slusarz marcin.slus...@gmail.com wrote: On Sun, Feb 13, 2011 at 09:38:04PM +0100, Marcin Slusarz wrote: Combination of locking

Re: [Nouveau] [PATCH] drm/nouveau: fix __nouveau_fence_wait performance regression

2011-03-07 Thread Marcin Slusarz
On Tue, Mar 08, 2011 at 12:22:56AM +0100, Marcin Slusarz wrote: On Tue, Mar 08, 2011 at 08:24:26AM +1000, Ben Skeggs wrote: On Mon, 2011-03-07 at 18:18 +, Maarten Maathuis wrote: On Fri, Mar 4, 2011 at 4:49 PM, Marcin Slusarz marcin.slus...@gmail.com wrote: On Sun, Feb 13, 2011

Re: [Nouveau] [PATCH] drm/nouveau: fix __nouveau_fence_wait performance regression

2011-03-04 Thread Marcin Slusarz
On Sun, Feb 13, 2011 at 09:38:04PM +0100, Marcin Slusarz wrote: Combination of locking and interchannel synchronization changes uncovered poor behaviour of nouveau_fence_wait, which on HZ=100 configuration could waste up to 10 ms per call. Depending on application, it lead to 10-30% FPS

Re: [Nouveau] [PATCH] gallium/nv50: use 0x8697 class on NVAF

2011-03-04 Thread Marcin Slusarz
On Tue, Feb 22, 2011 at 06:32:40PM +0100, Marcin Slusarz wrote: Addresses: https://bugs.launchpad.net/ubuntu/+source/xserver-xorg-video-nouveau/+bug/723012 Reported-and-tested-by: Alan Pope --- src/gallium/drivers/nv50/nv50_reg.h|4 src/gallium/drivers/nv50/nv50_screen.c

[Nouveau] [PATCH] gallium/nv50: use 0x8697 class on NVAF

2011-02-22 Thread Marcin Slusarz
Addresses: https://bugs.launchpad.net/ubuntu/+source/xserver-xorg-video-nouveau/+bug/723012 Reported-and-tested-by: Alan Pope --- src/gallium/drivers/nv50/nv50_reg.h|4 src/gallium/drivers/nv50/nv50_screen.c |3 +++ 2 files changed, 7 insertions(+), 0 deletions(-) diff --git

[Nouveau] [PATCH] nouveau/gallium: use OUT_RINGp instead of several OUT_RINGs

2011-02-22 Thread Marcin Slusarz
It improves Warsow FPS by ~1.5% --- src/gallium/drivers/nv50/nv50_push.c | 18 +++--- 1 files changed, 3 insertions(+), 15 deletions(-) diff --git a/src/gallium/drivers/nv50/nv50_push.c b/src/gallium/drivers/nv50/nv50_push.c index 380f694..bb429f2 100644 ---

Re: [Nouveau] [PATCH] drm/nouveau: fix __nouveau_fence_wait performance regression

2011-02-21 Thread Marcin Slusarz
On Sun, Feb 13, 2011 at 09:38:04PM +0100, Marcin Slusarz wrote: Combination of locking and interchannel synchronization changes uncovered poor behaviour of nouveau_fence_wait, which on HZ=100 configuration could waste up to 10 ms per call. Depending on application, it lead to 10-30% FPS

[Nouveau] [PATCH 2/2] drm/nv50: fix typos in CCACHE error reporting

2011-02-13 Thread Marcin Slusarz
The code was supposed to print registers around 0x405018 (which is read earlier), not 0x405818. Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/nv50_graph.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/nouveau

[Nouveau] [PATCH] fix drmSetMaster error reporting

2011-02-13 Thread Marcin Slusarz
On error, drmSetMaster returns -1 and sets errno. --- src/nv_driver.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/nv_driver.c b/src/nv_driver.c index 3e009fc..754739c 100644 --- a/src/nv_driver.c +++ b/src/nv_driver.c @@ -334,7 +334,7 @@ NVEnterVT(int scrnIndex,

[Nouveau] [PATCH] nv50: fix query assertion

2011-02-13 Thread Marcin Slusarz
--- src/gallium/drivers/nv50/nv50_query.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/gallium/drivers/nv50/nv50_query.c b/src/gallium/drivers/nv50/nv50_query.c index 53f9482..f3418df 100644 --- a/src/gallium/drivers/nv50/nv50_query.c +++

[Nouveau] [PATCH] drm/nouveau: fix __nouveau_fence_wait performance regression

2011-02-13 Thread Marcin Slusarz
spinning happens for at least one *full* tick. Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/nouveau_fence.c | 10 -- 1 files changed, 8 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_fence.c b/drivers/gpu/drm/nouveau

[Nouveau] [PATCH] drm/nouveau: kill tasks causing the GPU to raise errors

2011-01-23 Thread Marcin Slusarz
nouveau.error_action=0 to kernel command line. Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/nouveau_channel.c | 23 ++ drivers/gpu/drm/nouveau/nouveau_drv.c |7 ++ drivers/gpu/drm/nouveau/nouveau_drv.h | 16 drivers/gpu/drm/nouveau

[Nouveau] [ddx PATCH v2] convert OUT_RELOCh + OUT_RELOCl to OUT_RELOC2

2011-01-12 Thread Marcin Slusarz
--- src/nv50_accel.c | 32 src/nv50_exa.c | 24 src/nv50_xv.c| 25 - 3 files changed, 24 insertions(+), 57 deletions(-) v2: s/OUT_RELOChl/OUT_RELOC2/g as requested diff --git a/src/nv50_accel.c

[Nouveau] [mesa PATCH 2/2 v2] gallium/nouveau: convert OUT_RELOCh + OUT_RELOCl to OUT_RELOC2

2011-01-12 Thread Marcin Slusarz
--- src/gallium/drivers/nv50/nv50_query.c |6 +-- src/gallium/drivers/nv50/nv50_screen.c | 39 src/gallium/drivers/nv50/nv50_shader_state.c |3 +- src/gallium/drivers/nv50/nv50_surface.c| 12 ++-

[Nouveau] [mesa PATCH 3/2 ; )] gallium/nouveau: introduce so_reloc2 as so_reloc(high) + so_reloc(low)

2011-01-12 Thread Marcin Slusarz
--- src/gallium/drivers/nouveau/nouveau_stateobj.h |8 src/gallium/drivers/nv50/nv50_shader_state.c | 15 +++ src/gallium/drivers/nv50/nv50_state_validate.c | 15 +-- src/gallium/drivers/nv50/nv50_vbo.c| 15 --- 4 files

[Nouveau] [libdrm PATCH 2/3 v2] nouveau: add new OUT_RELOC2 (OUT_RELOCh + OUT_RELOCl)

2011-01-12 Thread Marcin Slusarz
--- nouveau/nouveau_pushbuf.h | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) v2: s/OUT_RELOChl/OUT_RELOC2/g as requested diff --git a/nouveau/nouveau_pushbuf.h b/nouveau/nouveau_pushbuf.h index 70d0f28..9273272 100644 --- a/nouveau/nouveau_pushbuf.h +++

[Nouveau] [kernel PATCH] drm/nouveau: add NOUVEAU_GEM_RELOC_HILO

2011-01-12 Thread Marcin Slusarz
add new relocation flag which allows to halving number of relocation entries on = nv50 cards Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/nouveau_gem.c | 22 +++--- include/drm/nouveau_drm.h |1 + 2 files changed, 16

[Nouveau] [libdrm PATCH] nouveau: optimize OUT_RELOC2

2011-01-12 Thread Marcin Slusarz
use one relocation with newly introduced flag unfortunately we can't enable it unconditionally, because we don't know if the kernel supports it... --- include/drm/nouveau_drm.h |1 + nouveau/nouveau_bo.h |1 + nouveau/nouveau_pushbuf.h |9 + nouveau/nouveau_reloc.c |

[Nouveau] [mesa PATCH] gallium/nouveau: optimize so_reloc2

2011-01-12 Thread Marcin Slusarz
use one relocation with newly introduced flag unfortunately we can't enable it unconditionally, because we don't know if the kernel supports it... --- src/gallium/drivers/nouveau/nouveau_stateobj.h | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git

Re: [Nouveau] [kernel PATCH] drm/nouveau: add NOUVEAU_GEM_RELOC_HILO

2011-01-12 Thread Marcin Slusarz
On Wed, Jan 12, 2011 at 10:08:55PM +0100, Francisco Jerez wrote: Marcin Slusarz marcin.slus...@gmail.com writes: add new relocation flag which allows to halving number of relocation entries on = nv50 cards I suspect that this change is unnecessary. If you're actually seeing

[Nouveau] [libdrm PATCH 1/3] nouveau: remove unused OUT_RELOC2 and OUT_RELOCr

2011-01-09 Thread Marcin Slusarz
--- nouveau/nouveau_pushbuf.h | 17 - 1 files changed, 0 insertions(+), 17 deletions(-) diff --git a/nouveau/nouveau_pushbuf.h b/nouveau/nouveau_pushbuf.h index 2a98789..70d0f28 100644 --- a/nouveau/nouveau_pushbuf.h +++ b/nouveau/nouveau_pushbuf.h @@ -109,15 +109,6 @@

[Nouveau] [libdrm PATCH 3/3] nouveau: remove unused data2 argument from reloc functions

2011-01-09 Thread Marcin Slusarz
Remove data2 arg from nouveau_pushbuf_emit_reloc and nouveau_reloc_emit. Note: it changes the API ABI of libdrm_nouveau. --- nouveau/nouveau_pushbuf.c |4 ++-- nouveau/nouveau_pushbuf.h |4 ++-- nouveau/nouveau_reloc.c |2 +- nouveau/nouveau_reloc.h |2 +- 4 files changed, 6

[Nouveau] [libdrm PATCH 2/3] nouveau: add OUT_RELOChl

2011-01-09 Thread Marcin Slusarz
--- nouveau/nouveau_pushbuf.h | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/nouveau/nouveau_pushbuf.h b/nouveau/nouveau_pushbuf.h index 70d0f28..5151e01 100644 --- a/nouveau/nouveau_pushbuf.h +++ b/nouveau/nouveau_pushbuf.h @@ -142,4 +142,15 @@

[Nouveau] [ddx PATCH] convert OUT_RELOCh + OUT_RELOCl to OUT_RELOChl

2011-01-09 Thread Marcin Slusarz
--- src/nv50_accel.c | 32 src/nv50_exa.c | 24 src/nv50_xv.c| 25 - 3 files changed, 24 insertions(+), 57 deletions(-) diff --git a/src/nv50_accel.c b/src/nv50_accel.c index d6c7528..3ba2898 100644 ---

[Nouveau] [mesa PATCH 2/2] gallium/nouveau: convert OUT_RELOCh + OUT_RELOCl to OUT_RELOChl

2011-01-09 Thread Marcin Slusarz
--- src/gallium/drivers/nv50/nv50_query.c |6 +-- src/gallium/drivers/nv50/nv50_screen.c | 39 src/gallium/drivers/nv50/nv50_shader_state.c |3 +- src/gallium/drivers/nv50/nv50_surface.c| 12 ++-

Re: [Nouveau] GeForce FX 5200: no kms

2010-11-14 Thread Marcin Slusarz
On Sun, Nov 14, 2010 at 10:18:10AM +, Paolo Milani wrote: --- Dom 14/11/10, Marcin Slusarz marcin.slus...@gmail.com wrote: It should work. Please post your kernel logs from a failed boot (they are somewhere in /var/log). when boot fails (and system crashes), it write nothing in /var

Re: [Nouveau] GeForce FX 5200: no kms

2010-11-13 Thread Marcin Slusarz
On Sat, Nov 13, 2010 at 10:25:45PM +, Paolo Milani wrote: Hi all! According to feature matrix http://nouveau.freedesktop.org/wiki/FeatureMatrix kms on GeForce FX 5200 is supported, but on my system it's not working. I've tried with fedora 14 and k/ubuntu 10.10, but the screen is blank and

Re: [Nouveau] nouveau/ttm: BUG in ttm_bo_release_list

2010-10-09 Thread Marcin Slusarz
On Sat, Oct 09, 2010 at 03:53:58PM +0200, Kai Ruhnau wrote: On 09/18/2010 01:18 PM, Marcin Slusarz wrote: On Sat, Sep 18, 2010 at 09:39:21AM +1000, Ben Skeggs wrote: On Fri, 2010-09-17 at 19:43 +0200, Marcin Slusarz wrote: Hi Since upgrade from 2.6.35 to 2.6.36-rc3 (nouveau tree) I'm

Re: [Nouveau] GeForce FX5200 dual DVI Samsung 204b

2010-10-05 Thread Marcin Slusarz
On Wed, Oct 06, 2010 at 12:33:25AM +0300, Pekka Paalanen wrote: On Tue, 05 Oct 2010 16:53:16 +0200 Grzesiek Sójka p...@pfu.pl wrote: On 10/05/10 14:55, Francisco Jerez wrote: PS. I'm afraid that my system is not very stable when the AGP support is turned on both using the nouveau

Re: [Nouveau] Problems with the driver

2010-09-26 Thread Marcin Slusarz
On Sun, Sep 26, 2010 at 04:13:06PM +0200, Piotr Hosowicz wrote: Hello, I can build the driver, it loads, but it does not work. Please find attached my local data, sorry for the form, I made it a few days ago and would have to build a new kernel which would also not work. My card is

Re: [Nouveau] nouveau/ttm: BUG in ttm_bo_release_list

2010-09-18 Thread Marcin Slusarz
On Sat, Sep 18, 2010 at 09:39:21AM +1000, Ben Skeggs wrote: On Fri, 2010-09-17 at 19:43 +0200, Marcin Slusarz wrote: Hi Since upgrade from 2.6.35 to 2.6.36-rc3 (nouveau tree) I'm hitting this bug a couple of times a day: [ 2869.618504] [ cut here

[Nouveau] nouveau/ttm: BUG in ttm_bo_release_list

2010-09-17 Thread Marcin Slusarz
Hi Since upgrade from 2.6.35 to 2.6.36-rc3 (nouveau tree) I'm hitting this bug a couple of times a day: [ 2869.618504] [ cut here ] [ 2869.618532] kernel BUG at drivers/gpu/drm/ttm/ttm_bo.c:153! [ 2869.618560] invalid opcode: [#1] PREEMPT SMP [ 2869.618600] last

Re: [Nouveau] [PATCH] drm/nv50: initialize ramht_refs list for faked 0 channel

2010-09-02 Thread Marcin Slusarz
On Wed, Sep 01, 2010 at 07:52:58AM +1000, Ben Skeggs wrote: On Tue, 2010-08-31 at 23:22 +0200, Marcin Slusarz wrote: On Sun, Aug 22, 2010 at 08:54:08PM +0200, Marcin Slusarz wrote: We need it for PFIFO_INTR_CACHE_ERROR interrupt handling, because nouveau_fifo_swmthd looks for matching

Re: [Nouveau] [PATCH] drm/nv50: initialize ramht_refs list for faked 0 channel

2010-09-02 Thread Marcin Slusarz
On Thu, Sep 02, 2010 at 10:31:55PM +1000, Ben Skeggs wrote: On Thu, 2010-09-02 at 12:45 +0200, Marcin Slusarz wrote: On Wed, Sep 01, 2010 at 07:52:58AM +1000, Ben Skeggs wrote: On Tue, 2010-08-31 at 23:22 +0200, Marcin Slusarz wrote: On Sun, Aug 22, 2010 at 08:54:08PM +0200, Marcin

Re: [Nouveau] [PATCH] drm/nv50: initialize ramht_refs list for faked 0 channel

2010-08-31 Thread Marcin Slusarz
On Sun, Aug 22, 2010 at 08:54:08PM +0200, Marcin Slusarz wrote: We need it for PFIFO_INTR_CACHE_ERROR interrupt handling, because nouveau_fifo_swmthd looks for matching gpuobj in ramht_refs list. It fixes kernel panic in nouveau_gpuobj_ref_find. Signed-off-by: Marcin Slusarz marcin.slus

[Nouveau] [PATCH] drm/nouveau: try to load fbcon

2010-08-25 Thread Marcin Slusarz
. Let's fix it. --- From: Marcin Slusarz marcin.slus...@gmail.com Subject: [PATCH] drm/nouveau: try to load fbcon Currently users have to load fbcon manually or build it into the kernel to have any console output. Let's make it easy for them and load fbcon automatically at init time. Nouveau already

[Nouveau] [mesa PATCH] nouveau: handle early initialization errors

2010-08-23 Thread Marcin Slusarz
handle very early errors in pipe_screen creation (failure of nouveau_screen_init in nv50_screen_create) --- src/gallium/drivers/nouveau/nouveau_screen.c |3 ++- .../winsys/nouveau/drm/nouveau_drm_winsys.c|3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git

[Nouveau] [PATCH] drm/nv50: initialize ramht_refs list for faked 0 channel

2010-08-22 Thread Marcin Slusarz
We need it for PFIFO_INTR_CACHE_ERROR interrupt handling, because nouveau_fifo_swmthd looks for matching gpuobj in ramht_refs list. It fixes kernel panic in nouveau_gpuobj_ref_find. Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com --- drivers/gpu/drm/nouveau/nv50_instmem.c |2 ++ 1

Re: [Nouveau] Macbook Pro 5, 1 - nouveau works if proprietary drivers are loaded first, then unloaded

2010-08-14 Thread Marcin Slusarz
On Sat, Aug 14, 2010 at 11:13:11AM -0600, Jesse Johnson wrote: Hello, I have a Macbook Pro 5,1 and decided that I wanted to play around with Hardened Gentoo for fun. I have another Gentoo install on the same machine along side Mac OS X (I keep an extra partition free for when I want

Re: [Nouveau] 2 cards in, but work only separately

2010-08-01 Thread Marcin Slusarz
On Sun, Aug 01, 2010 at 08:59:28AM +, Kārlis Repsons wrote: On Sunday 01 August 2010 08:41:04 Pekka Paalanen wrote: On Sat, 31 Jul 2010 22:08:23 + Kārlis Repsons k...@11.lv wrote: Hi list, my thanks to developers of nouveau first: I wonder how you put it together this far,

[Nouveau] [PATCH] x86, mmiotrace: add support for tracing STOS instruction

2010-07-31 Thread Marcin Slusarz
Signed-off-by: Marcin Slusarz marcin.slus...@gmail.com Cc: Pekka Paalanen p...@iki.fi --- arch/x86/mm/pf_in.c | 30 +- 1 files changed, 17 insertions(+), 13 deletions(-) diff --git a/arch/x86/mm/pf_in.c b/arch/x86/mm/pf_in.c index 308e325..38e6d17 100644 --- a/arch

Re: [Nouveau] [patch v2] nouveau: unwind on load errors

2010-07-30 Thread Marcin Slusarz
. Signed-off-by: Dan Carpenter erro...@gmail.com --- V2: updated to the nouveau git tree. Thanks. Reviewed-by: Marcin Slusarz marcin.slus...@gmail.com diff --git a/drivers/gpu/drm/nouveau/nouveau_state.c b/drivers/gpu/drm/nouveau/nouveau_state.c index

Re: [Nouveau] [patch] nouveau: unwind on load errors

2010-07-29 Thread Marcin Slusarz
On Thu, Jul 29, 2010 at 08:02:31PM +0200, Dan Carpenter wrote: nouveau_load() just returned directly if there was an error instead of releasing resources. Signed-off-by: Dan Carpenter erro...@gmail.com Patch looks good, but could you base it on top of nouveau tree?

Re: [Nouveau] deadlock possiblity introduced by drm/nouveau: use drm_mm in preference to custom code doing the same thing

2010-07-26 Thread Marcin Slusarz
On Sun, Jul 11, 2010 at 11:02:12AM +1000, Ben Skeggs wrote: On Sun, 2010-07-11 at 01:24 +0200, Marcin Slusarz wrote: Hi Patch drm/nouveau: use drm_mm in preference to custom code doing the same thing in nouveau tree introduced new deadlock possibility, for which lockdep complains

Re: [Nouveau] deadlock possiblity introduced by drm/nouveau: use drm_mm in preference to custom code doing the same thing

2010-07-26 Thread Marcin Slusarz
On Mon, Jul 26, 2010 at 07:15:47PM +0200, Francisco Jerez wrote: Marcin Slusarz marcin.slus...@gmail.com writes: On Sun, Jul 11, 2010 at 11:02:12AM +1000, Ben Skeggs wrote: On Sun, 2010-07-11 at 01:24 +0200, Marcin Slusarz wrote: Hi Patch drm/nouveau: use drm_mm in preference

<    1   2   3   4   >