Re: [PATCH 2/2] drm/ttm: Fix ttm_page_alloc.c

2010-05-26 Thread Jerome Glisse
On Wed, May 26, 2010 at 04:18:50PM +0200, Thomas Hellstrom wrote: > Fix a number of typos misspellings and checkpatch.pl warnings. > Replace "[ttm] " with TTM_PFX > > Signed-off-by: Thomas Hellstrom Reviewed-by: Jerome Glisse > --- > drivers/gpu/drm

Re: [PATCH 1/2] drm/ttm: Fix cached TTM page allocation.

2010-05-26 Thread Jerome Glisse
gfx driver on a 64-bit vm. > > Signed-off-by: Thomas Hellstrom Reviewed-by: Jerome Glisse > --- > drivers/gpu/drm/ttm/ttm_page_alloc.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/gpu/drm/ttm/ttm_page_alloc.c > b/drivers/gpu/d

Re: 2.6.33.2 kmalloc-8 slab leaks ~512 objects per second

2010-05-12 Thread Jerome Glisse
On Wed, May 12, 2010 at 11:50:41AM +0100, Tvrtko Ursulin wrote: > On Wednesday 12 May 2010 11:38:05 Tvrtko Ursulin wrote: > > [snip] > > I think it is DRM (radeon) related, leak stopped when I closed all X > > programs. I am compiling 2.6.33.3 right now and will soon reboot into it. > > Leak is

Re: 2.6.34-rc5: Reported regressions from 2.6.33

2010-04-21 Thread Jerome Glisse
On Wed, Apr 21, 2010 at 07:15:38AM +0200, Rafael J. Wysocki wrote: > On Tuesday 20 April 2010, Nick Bowler wrote: > > On 05:15 Tue 20 Apr , Rafael J. Wysocki wrote: > > > If you know of any other unresolved regressions from 2.6.33, please let us > > > know either and we'll add them to the list.

[PATCH] drm/radeon/kms: print GPU family and device id when loading

2010-04-12 Thread Jerome Glisse
This will help figuring out GPU when looking at bugs log. Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_device.c | 53 +++- drivers/gpu/drm/radeon/radeon_family.h |3 +- 2 files changed, 54 insertions(+), 2 deletions(-) diff --git a/drivers

[PATCH 12/13] drm/ttm: remove io_ field from TTM V6

2010-04-09 Thread Jerome Glisse
All TTM driver have been converted to new io_mem_reserve/free interface which allow driver to choose and return proper io base, offset to core TTM for ioremapping if necessary. This patch remove what is now deadcode. V2 adapt to match with change in first patch of the patchset V3 update after io_m

[PATCH 10/13] drm/nouveau/kms: don't initialize TTM io memory manager field

2010-04-09 Thread Jerome Glisse
This isn't needed anymore with the new TTM fault callback Signed-off-by: Jerome Glisse --- drivers/gpu/drm/nouveau/nouveau_bo.c | 11 --- 1 files changed, 0 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c

[PATCH 08/13] drm/vmwgfx: add support for new TTM fault callback V5

2010-04-09 Thread Jerome Glisse
x27;s not NULL only for VRAM or GTT V3 update after io_mem_reserve/io_mem_free callback balancing V4 callback has to ioremap V5 ioremap is done by TTM Signed-off-by: Jerome Glisse --- drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c | 44 +-- 1 files changed, 41 insertions(+

[PATCH 06/13] drm/radeon/kms: add support for new fault callback V7

2010-04-09 Thread Jerome Glisse
new balanced io_mem_reserve/free change V6 callback is responsible for iomapping memory V7 move back iomapping to ttm Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_object.c | 26 +++- drivers/gpu/drm/radeon/radeon_object.h |2 +- drivers/gpu/drm/radeon/radeon_tt

[PATCH 07/13] drm/nouveau/kms: add support for new TTM fault callback V5

2010-04-09 Thread Jerome Glisse
x27;t been tested on any hw. V2 don't derefence bo->mem.mm_node as it's not NULL only for VRAM or GTT V3 update after io_mem_reserve/io_mem_free callback balancing V4 callback has to ioremap V5 ioremap is done by ttm Signed-off-by: Jerome Glisse --- drivers/gpu/drm/nouv

[PATCH 05/13] drm/ttm: ttm_fault callback to allow driver to handle bo placement V6

2010-04-09 Thread Jerome Glisse
is isn't needed but we don't necesarily need to ioremap in the callback but still allow driver to use static mapping Signed-off-by: Jerome Glisse --- drivers/gpu/drm/ttm/ttm_bo.c |7 ++- drivers/gpu/drm/ttm/ttm_bo_util.c | 124 ++-- d

[PATCH 13/13] drm/radeon/kms: enable use of unmappable VRAM V2

2010-04-09 Thread Jerome Glisse
This patch enable the use of unmappable VRAM thanks to previous TTM infrastructure change. V2 update after io_mem_reserve/io_mem_free callback balancing Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/evergreen.c |5 - drivers/gpu/drm/radeon/r100.c |5 - drivers

[PATCH 11/13] drm/vmwgfx: don't initialize TTM io memory manager field

2010-04-09 Thread Jerome Glisse
This isn't needed anymore with the new TTM fault callback Signed-off-by: Jerome Glisse --- drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c |6 -- 1 files changed, 0 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c b/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c

unmappable vram v8

2010-04-09 Thread Jerome Glisse
Ok so here again this time ioremap is kept inside ttm so we don't waste ioremap when faulting page. Cheers, Jerome -- Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bu

[PATCH 02/13] drm/radeon/kms: update to TTM no_wait splitted argument

2010-04-09 Thread Jerome Glisse
This patch update radeon to the new no_wait splitted argument TTM functionality. Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_object.c |6 ++-- drivers/gpu/drm/radeon/radeon_ttm.c| 39 +-- 2 files changed, 24 insertions(+), 21 deletions

[PATCH 04/13] drm/vmwgfx: update to TTM no_wait splitted argument

2010-04-09 Thread Jerome Glisse
This patch update radeon to the new no_wait splitted argument TTM functionality. Compile tested only (but thing should run as there is no operating change from driver point of view) Signed-off-by: Jerome Glisse --- drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c |4 ++-- drivers/gpu/drm/vmwgfx

[PATCH 09/13] drm/radeon/kms: don't initialize TTM io memory manager field

2010-04-09 Thread Jerome Glisse
This isn't needed anymore with the new TTM fault callback Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_ttm.c | 13 + 1 files changed, 1 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_ttm.c b/drivers/gpu/drm/radeon/radeon_ttm.c

[PATCH 01/13] drm/ttm: split no_wait argument in 2 GPU or reserve wait

2010-04-09 Thread Jerome Glisse
, update to others driver are following in separate patches. Signed-off-by: Jerome Glisse --- drivers/gpu/drm/ttm/ttm_bo.c | 57 drivers/gpu/drm/ttm/ttm_bo_util.c |9 -- include/drm/ttm/ttm_bo_api.h |6 ++- include/drm/ttm/ttm_bo_driver.h

[PATCH 03/13] drm/nouveau: update to TTM no_wait splitted argument

2010-04-09 Thread Jerome Glisse
This patch update radeon to the new no_wait splitted argument TTM functionality. Compile tested only (but thing should run as there is no operating change from driver point of view) Signed-off-by: Jerome Glisse --- drivers/gpu/drm/nouveau/nouveau_bo.c | 45

Re: radeondb capturing & replaying GPU command stream

2010-04-08 Thread Jerome Glisse
On Thu, Apr 08, 2010 at 11:09:33PM +0300, Pauli Nieminen wrote: > On Thu, Apr 8, 2010 at 7:10 PM, Jerome Glisse wrote: > > Hi all, > > > > So i pushed a new dump facility in libdrm, it will dump > > everything needed to replay a command stream you just need > &g

radeondb capturing & replaying GPU command stream

2010-04-08 Thread Jerome Glisse
dump format to kernel to capture cs responsible of GPU lockup. Idea/comments are welcome Cheers, Jerome radeondb should show up at: http://cgit.freedesktop.org/~glisse/radeondb -- Download Intel® Parallel Studio Eval Try t

[PATCH 09/13] drm/radeon/kms: don't initialize TTM io memory manager field

2010-04-07 Thread Jerome Glisse
This isn't needed anymore with the new TTM fault callback Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_ttm.c | 13 + 1 files changed, 1 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_ttm.c b/drivers/gpu/drm/radeon/radeon_ttm.c

[PATCH 13/13] drm/radeon/kms: enable use of unmappable VRAM V2

2010-04-07 Thread Jerome Glisse
This patch enable the use of unmappable VRAM thanks to previous TTM infrastructure change. V2 update after io_mem_reserve/io_mem_free callback balancing Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/evergreen.c |5 - drivers/gpu/drm/radeon/r100.c |5 - drivers

[PATCH 05/13] drm/ttm: ttm_fault callback to allow driver to handle bo placement V5

2010-04-07 Thread Jerome Glisse
gt; bool as member is protected by reserve mecanism from concurent access V5 the callback is now responsible for iomapping the bo and providing a virtual address this simplify TTM and will allow to get rid of TTM_MEMTYPE_FLAG_NEEDS_IOREMAP Signed-off-by: Jerome Glisse --- drivers/gpu

[PATCH 08/13] drm/vmwgfx: add support for new TTM fault callback V4

2010-04-07 Thread Jerome Glisse
x27;s not NULL only for VRAM or GTT V3 update after io_mem_reserve/io_mem_free callback balancing V4 callback has to ioremap Signed-off-by: Jerome Glisse --- drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c | 49 ++-- 1 files changed, 46 insertions(+), 3 deletions(-) diff

[PATCH 03/13] drm/nouveau: update to TTM no_wait splitted argument

2010-04-07 Thread Jerome Glisse
This patch update radeon to the new no_wait splitted argument TTM functionality. Compile tested only (but thing should run as there is no operating change from driver point of view) Signed-off-by: Jerome Glisse --- drivers/gpu/drm/nouveau/nouveau_bo.c | 45

[PATCH 01/13] drm/ttm: split no_wait argument in 2 GPU or reserve wait

2010-04-07 Thread Jerome Glisse
, update to others driver are following in separate patches. Signed-off-by: Jerome Glisse --- drivers/gpu/drm/ttm/ttm_bo.c | 57 drivers/gpu/drm/ttm/ttm_bo_util.c |9 -- include/drm/ttm/ttm_bo_api.h |6 ++- include/drm/ttm/ttm_bo_driver.h

[PATCH 07/13] drm/nouveau/kms: add support for new TTM fault callback V4

2010-04-07 Thread Jerome Glisse
x27;t been tested on any hw. V2 don't derefence bo->mem.mm_node as it's not NULL only for VRAM or GTT V3 update after io_mem_reserve/io_mem_free callback balancing V4 callback has to ioremap Signed-off-by: Jerome Glisse --- drivers/gpu/drm/nouveau/nouveau_bo.c | 66 +

[PATCH 04/13] drm/vmwgfx: update to TTM no_wait splitted argument

2010-04-07 Thread Jerome Glisse
This patch update radeon to the new no_wait splitted argument TTM functionality. Compile tested only (but thing should run as there is no operating change from driver point of view) Signed-off-by: Jerome Glisse --- drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c |4 ++-- drivers/gpu/drm/vmwgfx

[PATCH 02/13] drm/radeon/kms: update to TTM no_wait splitted argument

2010-04-07 Thread Jerome Glisse
This patch update radeon to the new no_wait splitted argument TTM functionality. Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_object.c |6 ++-- drivers/gpu/drm/radeon/radeon_ttm.c| 39 +-- 2 files changed, 24 insertions(+), 21 deletions

unmappable vram V7

2010-04-07 Thread Jerome Glisse
So here is i hope the last spawn of the patch serie, i drop the needs ioremap flag and callback is responsible for ioremapping memory and providing a valid virtual address this simplify TTM code. I haven't change the value of enum after removing dead flag, Thomas maybe you prefer to change the enum

[PATCH 06/13] drm/radeon/kms: add support for new fault callback V6

2010-04-07 Thread Jerome Glisse
new balanced io_mem_reserve/free change V6 callback is responsible for iomapping memory Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_object.c | 26 +- drivers/gpu/drm/radeon/radeon_object.h |2 +- drivers/gpu/drm/radeon/radeon_ttm.c| 59 +++

[PATCH 12/13] drm/ttm: remove io_ field from TTM V5

2010-04-07 Thread Jerome Glisse
io_mem_reserve/io_mem_free callback balancing V4 adjust to minor cleanup V5 remove the needs ioremap flag Signed-off-by: Jerome Glisse --- drivers/gpu/drm/ttm/ttm_bo.c | 22 -- drivers/gpu/drm/ttm/ttm_bo_util.c | 80 ++--- include/drm/ttm/ttm_bo_api.h

[PATCH 10/13] drm/nouveau/kms: don't initialize TTM io memory manager field

2010-04-07 Thread Jerome Glisse
This isn't needed anymore with the new TTM fault callback Signed-off-by: Jerome Glisse --- drivers/gpu/drm/nouveau/nouveau_bo.c | 11 --- 1 files changed, 0 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c

[PATCH 11/13] drm/vmwgfx: don't initialize TTM io memory manager field

2010-04-07 Thread Jerome Glisse
This isn't needed anymore with the new TTM fault callback Signed-off-by: Jerome Glisse --- drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c |6 -- 1 files changed, 0 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c b/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c

Re: unmappable vram V6

2010-04-05 Thread Jerome Glisse
On Mon, Apr 05, 2010 at 02:23:58PM +0200, Thomas Hellstrom wrote: > Jerome Glisse wrote: > > So in these patchset i use bool instead of atomic remove empty line > > removal, and i hope addressed standing issues. Again only compile > > tested for nouveau & vmwgfx. Tested

AGP force use of scratch page

2010-04-01 Thread Jerome Glisse
gt;From 5b2469a7149280fb6866345cb6d3d395ee400cd8 Mon Sep 17 00:00:00 2001 From: Jerome Glisse Date: Thu, 1 Apr 2010 16:46:15 +0200 Subject: [PATCH] agp: force use of scratch page It seems that some AGP hardware can induce activity on bus after reporting being idle with the memory (preteching maybe). As new infrastruct

Re: [PATCH 1/7] drm/ttm: add pool wc/uc page allocator V3

2010-04-01 Thread Jerome Glisse
rome Glisse's and Dave Airlie's pool allocator. > > Signed-off-by: Jerome Glisse > Signed-off-by: Dave Airlie > Signed-off-by: Pauli Nieminen Reviewed-by: Jerome Glisse > --- > drivers/gpu/drm/ttm/Makefile |2 +- > drivers/gpu/drm/ttm/ttm_memory.c |

Re: [PATCH 1/7] drm/ttm: add pool wc/uc page allocator V2

2010-04-01 Thread Jerome Glisse
fe to normal in pool fill error path. > * Initialize sum variable and make the loop prettier in get_num_unused_pages > > Based on Jerome Glisse's and Dave Airlie's pool allocator. > > Signed-off-by: Jerome Glisse > Signed-off-by: Dave Airlie > Signed-off-by: Paul

Re: [git pull] drm fixes

2010-03-30 Thread Jerome Glisse
On Tue, Mar 30, 2010 at 07:24:42AM -0700, Linus Torvalds wrote: > > > On Tue, 30 Mar 2010, Dave Airlie wrote: > > > > Actually Linus, don't bother, consider this revoked, I'm going to kill > > the GPU reset code and re-send this tomorrow, its just a mess to get it > > back out of the tree at t

Re: [PATCH 1/7] drm/ttm: add pool wc/uc page allocator

2010-03-26 Thread Jerome Glisse
allows kernel to claim back pages when they are required for something else. > > Based on Jerome Glisse's and Dave Airlie's pool allocator. > > Signed-off-by: Jerome Glisse > Signed-off-by: Dave Airlie > Signed-off-by: Pauli Nieminen I think using array rather than lis

[PATCH 05/13] drm/ttm: ttm_fault callback to allow driver to handle bo placement V4

2010-03-25 Thread Jerome Glisse
gt; bool as member is protected by reserve mecanism from concurent access Signed-off-by: Jerome Glisse --- drivers/gpu/drm/ttm/ttm_bo.c |7 ++- drivers/gpu/drm/ttm/ttm_bo_util.c | 98 ++-- drivers/gpu/drm/ttm/ttm_bo_vm.c | 41 inc

[PATCH 01/13] drm/ttm: split no_wait argument in 2 GPU or reserve wait

2010-03-25 Thread Jerome Glisse
, update to others driver are following in separate patches. Signed-off-by: Jerome Glisse --- drivers/gpu/drm/ttm/ttm_bo.c | 57 drivers/gpu/drm/ttm/ttm_bo_util.c |9 -- include/drm/ttm/ttm_bo_api.h |6 ++- include/drm/ttm/ttm_bo_driver.h

[PATCH 09/13] drm/radeon/kms: don't initialize TTM io memory manager field

2010-03-25 Thread Jerome Glisse
This isn't needed anymore with the new TTM fault callback Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_ttm.c | 13 + 1 files changed, 1 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_ttm.c b/drivers/gpu/drm/radeon/radeon_ttm.c

[PATCH 06/13] drm/radeon/kms: add support for new fault callback V5

2010-03-25 Thread Jerome Glisse
new balanced io_mem_reserve/free change Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_object.c | 26 - drivers/gpu/drm/radeon/radeon_object.h |2 +- drivers/gpu/drm/radeon/radeon_ttm.c| 47 +++- 3 files changed, 71 insertions(+)

[PATCH 02/13] drm/radeon/kms: update to TTM no_wait splitted argument

2010-03-25 Thread Jerome Glisse
This patch update radeon to the new no_wait splitted argument TTM functionality. Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_object.c |6 ++-- drivers/gpu/drm/radeon/radeon_ttm.c| 39 +-- 2 files changed, 24 insertions(+), 21 deletions

[PATCH 03/13] drm/nouveau: update to TTM no_wait splitted argument

2010-03-25 Thread Jerome Glisse
This patch update radeon to the new no_wait splitted argument TTM functionality. Compile tested only (but thing should run as there is no operating change from driver point of view) Signed-off-by: Jerome Glisse --- drivers/gpu/drm/nouveau/nouveau_bo.c | 45

[PATCH 10/13] drm/nouveau/kms: don't initialize TTM io memory manager field

2010-03-25 Thread Jerome Glisse
This isn't needed anymore with the new TTM fault callback Signed-off-by: Jerome Glisse --- drivers/gpu/drm/nouveau/nouveau_bo.c | 11 --- 1 files changed, 0 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c

[PATCH 08/13] drm/vmwgfx: add support for new TTM fault callback V3

2010-03-25 Thread Jerome Glisse
x27;s not NULL only for VRAM or GTT V3 update after io_mem_reserve/io_mem_free callback balancing Signed-off-by: Jerome Glisse --- drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c | 40 +++- 1 files changed, 39 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/vm

[PATCH 13/13] drm/radeon/kms: enable use of unmappable VRAM V2

2010-03-25 Thread Jerome Glisse
This patch enable the use of unmappable VRAM thanks to previous TTM infrastructure change. V2 update after io_mem_reserve/io_mem_free callback balancing Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/evergreen.c |5 - drivers/gpu/drm/radeon/r100.c |5 - drivers

[PATCH 04/13] drm/vmwgfx: update to TTM no_wait splitted argument

2010-03-25 Thread Jerome Glisse
This patch update radeon to the new no_wait splitted argument TTM functionality. Compile tested only (but thing should run as there is no operating change from driver point of view) Signed-off-by: Jerome Glisse --- drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c |4 ++-- drivers/gpu/drm/vmwgfx

[PATCH 11/13] drm/vmwgfx: don't initialize TTM io memory manager field

2010-03-25 Thread Jerome Glisse
This isn't needed anymore with the new TTM fault callback Signed-off-by: Jerome Glisse --- drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c |6 -- 1 files changed, 0 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c b/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c

unmappable vram V6

2010-03-25 Thread Jerome Glisse
So in these patchset i use bool instead of atomic remove empty line removal, and i hope addressed standing issues. Again only compile tested for nouveau & vmwgfx. Tested this time only tested on RV710 with special patch to force unmappable vram use. http://people.freedesktop.org/~glisse/0014-

[PATCH 12/13] drm/ttm: remove io_ field from TTM V4

2010-03-25 Thread Jerome Glisse
io_mem_reserve/io_mem_free callback balancing V4 adjust to minor cleanup Signed-off-by: Jerome Glisse --- drivers/gpu/drm/ttm/ttm_bo.c | 22 -- drivers/gpu/drm/ttm/ttm_bo_util.c | 29 + include/drm/ttm/ttm_bo_driver.h | 10 -- 3

[PATCH 07/13] drm/nouveau/kms: add support for new TTM fault callback V3

2010-03-25 Thread Jerome Glisse
x27;t been tested on any hw. V2 don't derefence bo->mem.mm_node as it's not NULL only for VRAM or GTT V3 update after io_mem_reserve/io_mem_free callback balancing Signed-off-by: Jerome Glisse --- drivers/gpu/drm/nouveau/nouveau_bo.c | 51 ++ 1 files

Re: [PATCH] drm/ttm: ttm_fault callback to allow driver to handle bo placement V3

2010-03-24 Thread Jerome Glisse
On Wed, Mar 24, 2010 at 09:08:08PM +0100, Thomas Hellstrom wrote: > Jerome Glisse wrote: > >On Wed, Mar 24, 2010 at 07:27:57PM +0100, Thomas Hellstrom wrote: > >>Jerome Glisse wrote: > >>>On fault the driver is given the opportunity to perform any operation > >

Re: [PATCH] drm/ttm: ttm_fault callback to allow driver to handle bo placement V3

2010-03-24 Thread Jerome Glisse
On Wed, Mar 24, 2010 at 07:27:57PM +0100, Thomas Hellstrom wrote: > Jerome Glisse wrote: > > On fault the driver is given the opportunity to perform any operation > > it sees fit in order to place the buffer into a CPU visible area of > > memory. This patch doesn't break

[PATCH] drm/ttm: ttm_fault callback to allow driver to handle bo placement V3

2010-03-24 Thread Jerome Glisse
e and remove the old path from TTM once driver are converted. V2 return VM_FAULT_NOPAGE if callback return -EBUSY or -ERESTARTSYS V3 balance io_mem_reserve and io_mem_free call, fault_reserve_notify is responsible to perform any necessary task for mapping to succeed Signed-off-by: Jerome G

[PATCH 12/14] drm/ttm: remove io_ field from TTM V3

2010-03-24 Thread Jerome Glisse
io_mem_reserve/io_mem_free callback balancing Signed-off-by: Jerome Glisse --- drivers/gpu/drm/ttm/ttm_bo.c | 22 -- drivers/gpu/drm/ttm/ttm_bo_util.c | 26 +++--- include/drm/ttm/ttm_bo_driver.h | 10 -- 3 files changed, 7 insertions

[PATCH 01/14] drm/ttm: split no_wait argument in 2 GPU or reserve wait

2010-03-24 Thread Jerome Glisse
, update to others driver are following in separate patches. Signed-off-by: Jerome Glisse --- drivers/gpu/drm/ttm/ttm_bo.c | 57 drivers/gpu/drm/ttm/ttm_bo_util.c |9 -- include/drm/ttm/ttm_bo_api.h |6 ++- include/drm/ttm/ttm_bo_driver.h

[PATCH 07/14] drm/nouveau/kms: add support for new TTM fault callback V3

2010-03-24 Thread Jerome Glisse
x27;t been tested on any hw. V2 don't derefence bo->mem.mm_node as it's not NULL only for VRAM or GTT V3 update after io_mem_reserve/io_mem_free callback balancing Signed-off-by: Jerome Glisse --- drivers/gpu/drm/nouveau/nouveau_bo.c | 51 ++ 1 files

[PATCH 03/14] drm/nouveau: update to TTM no_wait splitted argument

2010-03-24 Thread Jerome Glisse
This patch update radeon to the new no_wait splitted argument TTM functionality. Compile tested only (but thing should run as there is no operating change from driver point of view) Signed-off-by: Jerome Glisse --- drivers/gpu/drm/nouveau/nouveau_bo.c | 45

[PATCH 08/14] drm/vmwgfx: add support for new TTM fault callback V3

2010-03-24 Thread Jerome Glisse
x27;s not NULL only for VRAM or GTT V3 update after io_mem_reserve/io_mem_free callback balancing Signed-off-by: Jerome Glisse --- drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c | 40 +++- 1 files changed, 39 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/vm

[PATCH 11/14] drm/vmwgfx: don't initialize TTM io memory manager field

2010-03-24 Thread Jerome Glisse
This isn't needed anymore with the new TTM fault callback Signed-off-by: Jerome Glisse --- drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c |6 -- 1 files changed, 0 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c b/drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c

[PATCH 13/14] drm/radeon/kms: enable use of unmappable VRAM V2

2010-03-24 Thread Jerome Glisse
This patch enable the use of unmappable VRAM thanks to previous TTM infrastructure change. V2 update after io_mem_reserve/io_mem_free callback balancing Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/evergreen.c |5 - drivers/gpu/drm/radeon/r100.c |5 - drivers

[PATCH 10/14] drm/nouveau/kms: don't initialize TTM io memory manager field

2010-03-24 Thread Jerome Glisse
This isn't needed anymore with the new TTM fault callback Signed-off-by: Jerome Glisse --- drivers/gpu/drm/nouveau/nouveau_bo.c | 11 --- 1 files changed, 0 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c

[PATCH 09/14] drm/radeon/kms: don't initialize TTM io memory manager field

2010-03-24 Thread Jerome Glisse
This isn't needed anymore with the new TTM fault callback Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_ttm.c | 13 + 1 files changed, 1 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_ttm.c b/drivers/gpu/drm/radeon/radeon_ttm.c

[PATCH 02/14] drm/radeon/kms: update to TTM no_wait splitted argument

2010-03-24 Thread Jerome Glisse
This patch update radeon to the new no_wait splitted argument TTM functionality. Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_object.c |6 ++-- drivers/gpu/drm/radeon/radeon_ttm.c| 39 +-- 2 files changed, 24 insertions(+), 21 deletions

[PATCH 06/14] drm/radeon/kms: add support for new fault callback V5

2010-03-24 Thread Jerome Glisse
new balanced io_mem_reserve/free change Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_object.c | 26 - drivers/gpu/drm/radeon/radeon_object.h |2 +- drivers/gpu/drm/radeon/radeon_ttm.c| 47 +++- 3 files changed, 71 insertions(+)

[PATCH 04/14] drm/vmwgfx: update to TTM no_wait splitted argument

2010-03-24 Thread Jerome Glisse
This patch update radeon to the new no_wait splitted argument TTM functionality. Compile tested only (but thing should run as there is no operating change from driver point of view) Signed-off-by: Jerome Glisse --- drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c |4 ++-- drivers/gpu/drm/vmwgfx

unmappable vram V5

2010-03-24 Thread Jerome Glisse
I think i have addressed all concern. Patch doesn't follow 80colons limit as their was discussion about dropping that on lkml. only compile tested on nouveau & vmwgfx. Sorry for short mail but i wrote a lot longer one which got lost ... don't feel like rewriting it. Cheers, Jerome ---

Re: A tiny problem regarding with the mesa source layout and DRI driver development

2010-03-22 Thread Jerome Glisse
On Mon, Mar 22, 2010 at 08:29:11PM +0800, LiYe wrote: > Hi devs, > > I have no idea if this is the right place to ask. But I've searched > around and didn't find the answer. So I am sorry if I made a noise. > > I'm interested in openGL implementation and the DRI driver development. > Specificall

Re: Unmappable VRAM patchset V4

2010-03-17 Thread Jerome Glisse
On Wed, Mar 17, 2010 at 02:01:47PM +0100, Thomas Hellstrom wrote: > Jerome Glisse wrote: > >On Mon, Mar 01, 2010 at 01:03:38PM +0100, Thomas Hellstrom wrote: > >>Dave Airlie wrote: > >>>On Fri, Feb 26, 2010 at 3:01 AM, Jerome Glisse wrote: > >>>>Upd

[PATCH] drm/radeon/kms: avoid possible oops (call gart_fini before gart_disable)

2010-03-17 Thread Jerome Glisse
radeon_gart_fini might call GART unbind callback function which might try to access GART table but if gart_disable is call first the GART table will be unmapped so any access to it will oops. Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/r100.c |2 +- drivers/gpu/drm/radeon/r300

Re: Unmappable VRAM patchset V4

2010-03-17 Thread Jerome Glisse
On Mon, Mar 01, 2010 at 01:03:38PM +0100, Thomas Hellstrom wrote: > Dave Airlie wrote: > > On Fri, Feb 26, 2010 at 3:01 AM, Jerome Glisse wrote: > > > >> Updated patchset, to apply cleanly on top of TTM split no_wait argument. > >> Compile tested for nouveau+vm

[PATCH] drm/radeon/kms: fix typo in r520 asic functions

2010-03-16 Thread Jerome Glisse
This will fix suspend/resume on r520 asic. Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon_asic.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_asic.c b/drivers/gpu/drm/radeon/radeon_asic.c index 3e40bc4..a5be6e0 100644

Re: [PATCH 0/5] clean up radeon_asic.h v2

2010-03-12 Thread Jerome Glisse
On Thu, Mar 11, 2010 at 10:19:13PM +0100, Daniel Vetter wrote: > Hi all, > > All new patch pile to make radeon_asic.h into a real header file. Now all > the asic structs are gathered in the new radeon_asic.c file. > > Tested on my rv570. > > I've also added a new patch that gathers all r100 spec

Re: [PATCH 00/14] cleanup radeon_asic.h

2010-03-11 Thread Jerome Glisse
On Thu, Mar 11, 2010 at 05:24:22PM +0100, Rafał Miłecki wrote: > 2010/3/11 Alex Deucher : > > I like keeping all the asic definitions in one file as you tend to > > need to update them all at one time and having them spread across all > > the asic files increases the likelihood of one or more of th

Re: [PATCH 00/14] cleanup radeon_asic.h

2010-03-11 Thread Jerome Glisse
On Thu, Mar 11, 2010 at 02:06:02PM +0100, Daniel Vetter wrote: > Hi all, > > This patch pile moves the "static struct radeon_asic " definitions > form radeon_asic.h into the asic-specific files, where I think they belong. > This way radeon_asic.h becomes a real header file that can be #included. >

Improved GPU reset

2010-03-09 Thread Jerome Glisse
This serie of patch fix the shortcoming of the previous one, their shouldn't be any more false positive and resume should lead to infinite look or other reinitialization issue on r6xx/r7xx. Cheers, Jerome -- Download Int

[PATCH 2/3] drm/radeon/kms: rename gpu_reset to asic_reset

2010-03-09 Thread Jerome Glisse
Patch rename gpu_reset to asic_reset in prevision of having gpu_reset doing more stuff than just basic asic reset. Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/evergreen.c |2 +- drivers/gpu/drm/radeon/r100.c |6 ++-- drivers/gpu/drm/radeon/r300.c |6

[PATCH 3/3] drm/radeon/kms: simplify & improve GPU reset V2

2010-03-09 Thread Jerome Glisse
and back to X will restore cursor. Next step is to record the bogus command that leaded to the lockup. V2 Fix r6xx resume path to avoid reinitializing blit module, use the gpu_lockup boolean to avoid entering inifinite waiting loop on fence while reiniting the GPU Signed-off-by: Jerome Glisse --- dri

[PATCH 1/3] drm/radeon/kms: fence cleanup + more reliable GPU lockup detection V4

2010-03-09 Thread Jerome Glisse
don't have false positive Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/evergreen.c|6 ++ drivers/gpu/drm/radeon/r100.c | 86 +++ drivers/gpu/drm/radeon/r300.c | 28 - drivers/gpu/drm/radeon/r600.c | 34 ++-

Re: Improved GPU reset

2010-03-08 Thread Jerome Glisse
On Sun, Mar 07, 2010 at 10:41:32AM +0100, Ladislav Kunc wrote: > On Friday 05 March 2010 11:30:02 Jerome Glisse wrote: > > This patches improve the GPU reset, many time i able to successfully > > reset the GPU and carry on operation, note that after a reset you > > will lik

Re: [git pull] drm request 3

2010-03-05 Thread Jerome Glisse
On Fri, Mar 05, 2010 at 04:31:29PM +, Alan Cox wrote: > On Fri, 05 Mar 2010 08:06:26 -0800 (PST) > David Miller wrote: > > > From: Daniel Stone > > Date: Fri, 5 Mar 2010 18:04:34 +0200 > > > > > So you're saying that there's no way to develop any reasonable body of > > > code for the Linux

[PATCH 3/3] drm/radeon/kms: simplify & improve GPU reset

2010-03-05 Thread Jerome Glisse
and back to X will restore cursor. Next step is to record the bogus command that leaded to the lockup. Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/r100.c | 180 +++ drivers/gpu/drm/radeon/r100d.h | 128 ++ drivers/gp

[PATCH 2/3] drm/radeon/kms: rename gpu_reset to asic_reset

2010-03-05 Thread Jerome Glisse
Patch rename gpu_reset to asic_reset in prevision of having gpu_reset doing more stuff than just basic asic reset. Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/evergreen.c |2 +- drivers/gpu/drm/radeon/r100.c |6 ++-- drivers/gpu/drm/radeon/r300.c |6

Improved GPU reset

2010-03-05 Thread Jerome Glisse
This patches improve the GPU reset, many time i able to successfully reset the GPU and carry on operation, note that after a reset you will likely see corrpution on the screen. Hope is that we should now be able to capture faulty command stream. I still need to do a full retesting with this patch

Re: TTM split no_wait argument in 2 (no_wait_reserve, no_wait_gpu)

2010-03-04 Thread Jerome Glisse
On Thu, Feb 25, 2010 at 05:50:10PM +0100, Jerome Glisse wrote: > This patch change the TTM API to allow driver to select btw choosing > to wait or not either for bo reserve or GPU wait separately. This is > needed for the unmappabled VRAM work. > > Comments ? > > Cheers,

Re: [RFC] drm/ttm: add pool wc/uc page allocator

2010-03-03 Thread Jerome Glisse
On Wed, Mar 03, 2010 at 05:46:43PM +0200, Pauli Nieminen wrote: > On Wed, Mar 3, 2010 at 4:50 PM, Jerome Glisse wrote: > > On Wed, Mar 03, 2010 at 04:23:08PM +0200, Pauli Nieminen wrote: > >> On Wed, Mar 3, 2010 at 3:33 PM, Jerome Glisse > >> wrote: > >>

[PATCH 1/3] drm/radeon/kms: fence cleanup + more reliable GPU lockup detection V3

2010-03-02 Thread Jerome Glisse
the 1s CP activity timeout. V2 switch to 500ms timeout so GPU lockup get call at least 2 times in less than 2sec. V3 store last jiffies in fence struct so on ERESTART, EBUSY we keep track of how long we already wait for a given fence Signed-off-by: Jerome Glisse --- drivers/gpu/

[PATCH] drm/radeon/kms: catch atombios infinite loop and break out of it

2010-03-02 Thread Jerome Glisse
kernel context which is bad. Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/atom.c | 59 drivers/gpu/drm/radeon/atom.h |2 +- 2 files changed, 48 insertions(+), 13 deletions(-) diff --git a/drivers/gpu/drm/radeon/atom.c b/drivers/gpu/drm/radeon

Re: [SPAM] [PATCH 1/3] drm/radeon/kms: fence cleanup + more reliable GPU lockup detection V2

2010-03-02 Thread Jerome Glisse
On Tue, Mar 02, 2010 at 10:13:19AM +0100, Erik Andrén wrote: > 2010/3/1 : > > From: Jerome Glisse > > > > This patch cleanup the fence code, it drops the timeout field of > > fence as the time to complete each IB is unpredictable and shouldn't > > be bound.

Re: [PATCH 1/2] drm/ttm: remove some bo->mutex remains

2010-03-01 Thread Jerome Glisse
On Mon, Mar 01, 2010 at 07:34:39PM +0100, Maarten Maathuis wrote: > - A few comments existed here and there that referred to a bo->mutex. > > Signed-off-by: Maarten Maathuis Reviewed-by: Jerome Glisse > --- > drivers/gpu/drm/ttm/ttm_bo.c|6 +- > drivers/gp

Re: [PATCH 2/2] drm/ttm: don't write to bo->reserved without holding glob->lru_lock

2010-03-01 Thread Jerome Glisse
On Mon, Mar 01, 2010 at 07:34:40PM +0100, Maarten Maathuis wrote: > - The headerfile says you can't write to it without holding the lock. > > Signed-off-by: Maarten Maathuis NAK, from my POV as we always use atomic_* on reserved it's useless to protect it with spinlock. Cheers, Jerome > --- >

Re: Convert DRM_INFO/DRM_ERROR to dev_info/dev_err

2010-03-01 Thread Jerome Glisse
On Mon, Mar 01, 2010 at 03:47:35PM +1000, Dave Airlie wrote: > On Fri, Feb 26, 2010 at 4:56 AM, Jerome Glisse wrote: > > Attached is conversion from DRM_INFO/DRM_ERROR to dev_info/dev_err > > to apply it copy all doconv* file into the radeon subfolder of the > > kernel run

Re: [PATCH] drm/radeon/kms: fence cleanup + more reliable GPU lockup detection

2010-03-01 Thread Jerome Glisse
On Mon, Mar 01, 2010 at 04:00:02PM +1000, Dave Airlie wrote: > On Mon, Mar 1, 2010 at 2:47 AM, Jerome Glisse wrote: > > On Sun, Feb 28, 2010 at 12:22:52PM +, Alan Swanson wrote: > >> On Fri, 2010-02-26 at 15:49 +0100, Jerome Glisse wrote: > >> > This patch cleanu

Re: [PATCH] drm/radeon/kms: fence cleanup + more reliable GPU lockup detection

2010-02-28 Thread Jerome Glisse
On Sun, Feb 28, 2010 at 12:22:52PM +, Alan Swanson wrote: > On Fri, 2010-02-26 at 15:49 +0100, Jerome Glisse wrote: > > This patch cleanup the fence code, it drops the timeout field of > > fence as the time to complete each IB is unpredictable and shouldn't > > be

Re: [PATCH 1/2] drm/radeon/kms: rename gpu_reset to asic_reset V2

2010-02-27 Thread Jerome Glisse
On Fri, Feb 26, 2010 at 10:33:34PM +0100, Jerome Glisse wrote: > Patch rename gpu_reset to asic_reset in prevision of having > gpu_reset doing more stuff than just basic asic reset. > > V2 store the last time we had new fence in the fence driver > so on EBUSY/ERESTART we stil

[PATCH 1/2] drm/radeon/kms: rename gpu_reset to asic_reset V2

2010-02-26 Thread Jerome Glisse
-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/evergreen.c |2 +- drivers/gpu/drm/radeon/r100.c |6 ++-- drivers/gpu/drm/radeon/r300.c |6 ++-- drivers/gpu/drm/radeon/r420.c |4 +- drivers/gpu/drm/radeon/r520.c |4 +- drivers/gpu/drm

  1   2   3   4   5   6   7   8   9   >