In Diablo III when the ground is covered in water, it's all messed-up.

I found a rendering error in Diablo III, in a well there ground is covered in water and it's all red and flickers. http://c566453.r53.cf2.rackcdn.com/DiabloIII-well-wine-preloader.trace.xz 3.5G, 234665296 Bytes compressed ~234M. EE r600_shader.c:1605 r600_shader_from_tgsi - GPR limit exceeded -

Include request for SA improvements

On Wed, May 9, 2012 at 3:31 PM, Jerome Glisse wrote: > On Wed, May 9, 2012 at 9:34 AM, Christian K?nig > wrote: >> Hi Dave & Jerome and everybody on the list, >> >> I can't find any more bugs and also I'm out of things to test, so I really >> hope that this is the last incarnation of this

[Bug 43858] DVI of ATI RADEON 9200 AGP don't work

https://bugs.freedesktop.org/show_bug.cgi?id=43858 Alex Deucher changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

[PATCH] drm/radeon/kms: fix warning on 32-bit in atomic fence printing

From: Dave Airlie /ssd/git/drm-core-next/drivers/gpu/drm/radeon/radeon_fence.c: In function ?radeon_debugfs_fence_info?: /ssd/git/drm-core-next/drivers/gpu/drm/radeon/radeon_fence.c:606:7: warning: format ?%lx? expects argument of type ?long unsigned int?, but argument 3

[PATCH 19/19] drm/radeon: make the ib an inline object

From: Jerome Glisse No need to malloc it any more. Signed-off-by: Jerome Glisse Signed-off-by: Christian K?nig --- drivers/gpu/drm/radeon/evergreen_cs.c | 10 +++--- drivers/gpu/drm/radeon/r100.c | 38 ++-- drivers/gpu/drm/radeon/r200.c

[PATCH 18/19] drm/radeon: remove r600 blit mutex v2

If we don't store local data into global variables it isn't necessary to lock anything. v2: rebased on new SA interface Signed-off-by: Christian K?nig --- drivers/gpu/drm/radeon/evergreen_blit_kms.c |1 - drivers/gpu/drm/radeon/r600.c | 13 ++--

[PATCH 17/19] drm/radeon: move the semaphore from the fence into the ib

From: Jerome Glisse It never really belonged there in the first place. Signed-off-by: Christian K?nig --- drivers/gpu/drm/radeon/radeon.h | 16 drivers/gpu/drm/radeon/radeon_cs.c|4 ++-- drivers/gpu/drm/radeon/radeon_fence.c |3 ---

[PATCH 16/19] drm/radeon: immediately free ttm-move semaphore

We can now protected the semaphore ram by a fence, so free it immediately. Signed-off-by: Christian K?nig --- drivers/gpu/drm/radeon/radeon_ttm.c |7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_ttm.c

[PATCH 15/19] drm/radeon: rip out the ib pool

From: Jerome Glisse It isn't necessary any more and the suballocator seems to perform even better. Signed-off-by: Christian K?nig Signed-off-by: Jerome Glisse --- drivers/gpu/drm/radeon/radeon.h | 17 +- drivers/gpu/drm/radeon/radeon_device.c|1 -

[PATCH 14/19] drm/radeon: simplify semaphore handling v2

From: Jerome Glisse Directly use the suballocator to get small chunks of memory. It's equally fast and doesn't crash when we encounter a GPU reset. v2: rebased on new SA interface. Signed-off-by: Christian K?nig Signed-off-by: Jerome Glisse ---

[PATCH 13/19] drm/radeon: multiple ring allocator v3

A startover with a new idea for a multiple ring allocator. Should perform as well as a normal ring allocator as long as only one ring does somthing, but falls back to a more complex algorithm if more complex things start to happen. We store the last allocated bo in last, we always try to allocate

[PATCH 12/19] drm/radeon: use one wait queue for all rings add fence_wait_any v2

From: Jerome Glisse Use one wait queue for all rings. When one ring progress, other likely does to and we are not expecting to have a lot of waiter anyway. Also add a fence_wait_any that will wait until the first fence in the fence array (one fence per ring) is signaled.

[PATCH 11/19] drm/radeon: define new SA interface v3

Define the interface without modifying the allocation algorithm in any way. v2: rebase on top of fence new uint64 patch v3: add ring to debugfs output Signed-off-by: Jerome Glisse Signed-off-by: Christian K?nig --- drivers/gpu/drm/radeon/radeon.h |1 +

[PATCH 10/19] drm/radeon: make sa bo a stand alone object

Allocating and freeing it seperately. Signed-off-by: Christian K?nig --- drivers/gpu/drm/radeon/radeon.h |4 ++-- drivers/gpu/drm/radeon/radeon_cs.c|4 ++-- drivers/gpu/drm/radeon/radeon_gart.c |4 ++-- drivers/gpu/drm/radeon/radeon_object.h|4 ++--

[PATCH 09/19] drm/radeon: keep start and end offset in the SA

Instead of offset + size keep start and end offset directly. Signed-off-by: Christian K?nig --- drivers/gpu/drm/radeon/radeon.h|4 ++-- drivers/gpu/drm/radeon/radeon_cs.c |4 ++-- drivers/gpu/drm/radeon/radeon_object.h |4 ++-- drivers/gpu/drm/radeon/radeon_sa.c |

[PATCH 08/19] drm/radeon: add sub allocator debugfs file

Dumping the current allocations. Signed-off-by: Christian K?nig --- drivers/gpu/drm/radeon/radeon_object.h |5 + drivers/gpu/drm/radeon/radeon_ring.c | 22 ++ drivers/gpu/drm/radeon/radeon_sa.c | 14 ++ 3 files changed, 41 insertions(+) diff

[PATCH 07/19] drm/radeon: add proper locking to the SA v3

Make the suballocator self containing to locking. v2: split the bugfix into a seperate patch. v3: remove some unreleated changes. Sig-off-by: Christian K?nig --- drivers/gpu/drm/radeon/radeon.h|1 + drivers/gpu/drm/radeon/radeon_sa.c |6 ++ 2 files changed, 7 insertions(+)

[PATCH 06/19] drm/radeon: use inline functions to calc sa_bo addr

Instead of hacking the calculation multiple times. Signed-off-by: Christian K?nig --- drivers/gpu/drm/radeon/radeon_gart.c |6 ++ drivers/gpu/drm/radeon/radeon_object.h| 11 +++ drivers/gpu/drm/radeon/radeon_ring.c |6 ++

[PATCH 05/19] drm/radeon: rework locking ring emission mutex in fence deadlock detection v2

Some callers illegal called fence_wait_next/empty while holding the ring emission mutex. So don't relock the mutex in that cases, and move the actual locking into the fence code. v2: Don't try to unlock the mutex if it isn't locked. Signed-off-by: Christian K?nig ---

[PATCH 04/19] drm/radeon: rework fence handling, drop fence list v7

From: Jerome Glisse Using 64bits fence sequence we can directly compare sequence number to know if a fence is signaled or not. Thus the fence list became useless, so does the fence lock that mainly protected the fence list. Things like ring.ready are no longer behind a lock,

[PATCH 03/19] drm/radeon: convert fence to uint64_t v4

From: Jerome Glisse This convert fence to use uint64_t sequence number intention is to use the fact that uin64_t is big enough that we don't need to care about wrap around. Tested with and without writeback using 0xF000 as initial fence sequence and thus allowing to test

[PATCH 02/19] drm/radeon: replace the per ring mutex with a global one

A single global mutex for ring submissions seems sufficient. Signed-off-by: Christian K?nig --- drivers/gpu/drm/radeon/radeon.h |3 ++- drivers/gpu/drm/radeon/radeon_device.c|3 +-- drivers/gpu/drm/radeon/radeon_pm.c| 10 ++-

[PATCH 01/19] drm/radeon: fix possible lack of synchronization btw ttm and other ring

From: Jerome Glisse We need to sync with the GFX ring as ttm might have schedule bo move on it and new command scheduled for other ring need to wait for bo data to be in place. Signed-off-by: Jerome Glisse Reviewed by: Christian K?nig ---

Include request for SA improvements

Hi Dave & Jerome and everybody on the list, I can't find any more bugs and also I'm out of things to test, so I really hope that this is the last incarnation of this patchset, and if Jerome is ok with it it should now be included into drm-next. Cheers, Christian.

[PATCH 2/2 v3] drm/exynos: added userptr feature.

this feature is used to import user space region allocated by malloc() or mmaped into a gem. and to guarantee the pages to user space not to be swapped out, the VMAs within the user space would be locked and then unlocked when the pages are released. but this lock might result in significant

[PATCH 1/2 v3] drm/exynos: added userptr limit ioctl.

this ioctl is used to limit user-desired userptr size as pre-defined and also could be accessed by only root user. with userptr feature, unprivileged user can allocate all the pages on system, so the amount of free physical pages will be very limited. if the VMAs within user address space was

[PATCH 0/2 v3] drm/exynos: added userptr feature

this feature could be used to use memory region allocated by malloc() in user mode and mmaped memory region allocated by other memory allocators. userptr interface can identify memory type through vm_flags value and would get pages or page frame numbers to user space appropriately. changelog v2:

[PATCH 2/2 v3] drm/exynos: added userptr feature.

On Wed, May 9, 2012 at 10:45 AM, Jerome Glisse wrote: > On Wed, May 9, 2012 at 2:17 AM, Inki Dae wrote: >> this feature is used to import user space region allocated by malloc() or >> mmaped into a gem. and to guarantee the pages to user space not to be >> swapped out, the VMAs within the user

[Bug 49632] radeon: The kernel rejected CS,

https://bugs.freedesktop.org/show_bug.cgi?id=49632 --- Comment #9 from execute.method at gmail.com 2012-05-09 06:36:57 PDT --- No, there is nothing else in dmesg. Is there any more info you'd like me to gather? -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email ---

[Bug 49632] radeon: The kernel rejected CS,

https://bugs.freedesktop.org/show_bug.cgi?id=49632 --- Comment #8 from Alex Deucher 2012-05-09 06:14:17 PDT --- (In reply to comment #6) > Ok. I didn't realize that. I have also tried with 3.4rc8, with the same > result. > Did i miss something in building the kernel? Are you getting the same

[PATCH] drm/radeon: don't mess with hot plug detect for eDP or LVDS connector v2

On Wed, May 9, 2012 at 10:23 AM, Jerome Glisse wrote: > On Wed, May 9, 2012 at 9:40 AM, Alex Deucher wrote: >> On Fri, May 4, 2012 at 11:06 AM, ? wrote: >>> From: Jerome Glisse >>> >>> It seems imac pannel doesn't like whe we change the hot plug setup >>> and then refuse to work. This help but

[PATCH] drm/radeon/kms: fix warning on 32-bit in atomic fence printing

On Wed, May 9, 2012 at 12:28 PM, Dave Airlie wrote: > From: Dave Airlie > > /ssd/git/drm-core-next/drivers/gpu/drm/radeon/radeon_fence.c: In function > ?radeon_debugfs_fence_info?: > /ssd/git/drm-core-next/drivers/gpu/drm/radeon/radeon_fence.c:606:7: warning: > format ?%lx? expects argument of

[PATCHv5 08/13] v4l: vb2-dma-contig: add support for scatterlist in userptr mode

Hello Tomasz, Laurent, I have printed some logs during the dmabuf export and attach for the SGT issue below. Please find it in the attachment. I hope it will be useful. Regards, Subash On 05/08/2012 04:45 PM, Subash Patel wrote: > Hi Laurent, > > On 05/08/2012 02:44 PM, Laurent Pinchart wrote:

[PATCHv5 08/13] v4l: vb2-dma-contig: add support for scatterlist in userptr mode

Hi Subash, Could you post the code of vb2_dc_pages_to_sgt with all printk in it. It will help us avoid guessing where and what is debugged in the log. Moreover, I found a line 'size=4294836224' in the log. It means that size is equal to -131072 (!?!) or there are some invalid conversions in

[PATCH 14/20] drm/radeon: multiple ring allocator v2

On 08.05.2012 16:55, Jerome Glisse wrote: > Still i don't want to loop more than necessary, it's bad, i am ok with : > http://people.freedesktop.org/~glisse/reset5/0001-drm-radeon-multiple-ring-allocator-v3.patch > > If there is fence signaled it will retry 2 times at most, otherwise it > will go

[Bug 49632] radeon: The kernel rejected CS,

https://bugs.freedesktop.org/show_bug.cgi?id=49632 --- Comment #7 from execute.method at gmail.com 2012-05-09 04:15:06 PDT --- (In reply to comment #6) > Ok. I didn't realize that. I have also tried with 3.4rc8, with the same > result. > Did i miss something in building the kernel? > > Also,

[PATCH] mm: Work around Intel SNB GTT bug with some physical pages.

On Tue, May 08, 2012 at 02:57:25PM -0700, Hugh Dickins wrote: > On Mon, 7 May 2012, Stephane Marchesin wrote: > > > While investing some Sandy Bridge rendering corruption, I found out > > that all physical memory pages below 1MiB were returning garbage when > > read through the GTT. This has been

[Bug 49632] radeon: The kernel rejected CS,

https://bugs.freedesktop.org/show_bug.cgi?id=49632 --- Comment #6 from execute.method at gmail.com 2012-05-09 03:55:23 PDT --- Ok. I didn't realize that. I have also tried with 3.4rc8, with the same result. Did i miss something in building the kernel? Also, what about projectM without streamout

[PATCH 2/2 v3] drm/exynos: added userptr feature.

On Wed, May 9, 2012 at 2:17 AM, Inki Dae wrote: > this feature is used to import user space region allocated by malloc() or > mmaped into a gem. and to guarantee the pages to user space not to be > swapped out, the VMAs within the user space would be locked and then unlocked > when the pages are

Include request for SA improvements

On Wed, May 9, 2012 at 9:34 AM, Christian K?nig wrote: > Hi Dave & Jerome and everybody on the list, > > I can't find any more bugs and also I'm out of things to test, so I really > hope that this is the last incarnation of this patchset, and if Jerome is > ok with it it should now be included

[PATCH] drm/radeon: don't mess with hot plug detect for eDP or LVDS connector v2

On Wed, May 9, 2012 at 9:40 AM, Alex Deucher wrote: > On Fri, May 4, 2012 at 11:06 AM, ? wrote: >> From: Jerome Glisse >> >> It seems imac pannel doesn't like whe we change the hot plug setup >> and then refuse to work. This help but doesn't fully fix: >>

[PATCH] drm/radeon: don't mess with hot plug detect for eDP or LVDS connector v2

On Fri, May 4, 2012 at 11:06 AM, wrote: > From: Jerome Glisse > > It seems imac pannel doesn't like whe we change the hot plug setup > and then refuse to work. This help but doesn't fully fix: > https://bugzilla.redhat.com/show_bug.cgi?id=726143 How does it help? Does it fix the the aux

[Bug 49603] [regression] Fullscreen video no longer smooth with GPU in low power mode

https://bugs.freedesktop.org/show_bug.cgi?id=49603 --- Comment #5 from Simon Farnsworth 2012-05-09 02:30:15 PDT --- Created attachment 61271 --> https://bugs.freedesktop.org/attachment.cgi?id=61271 A program to stop the CPU entering low power states Trying a different route. Can you compile

[PATCH] drm/radeon: eliminate redundant connector_names table

On Fri, May 4, 2012 at 11:25 AM, Ilija Hadzic wrote: > connector_names table is just a repeat of information that > already exists in drm_connector_enum_list and the same string > can be retrieved using drm_get_connector_name function. > > Nuke the redundant table and use the proper function to

[PATCH] MAINTAINERS: switch drm/i915 to Daniel Vetter

On Wed, May 09, 2012 at 07:51:35AM +0100, Dave Airlie wrote: > On Tue, May 8, 2012 at 12:19 PM, Daniel Vetter > wrote: > > ... given that I essentially run the show already, let's make this > > official. > > Acked-by: Dave Airlie > > probably just push it via your -next. Will do. Thanks,

[PATCH] MAINTAINERS: switch drm/i915 to Daniel Vetter

On Tue, May 8, 2012 at 12:19 PM, Daniel Vetter wrote: > ... given that I essentially run the show already, let's make this > official. Acked-by: Dave Airlie probably just push it via your -next. Dave.

[Bug 49110] debug build: AMDILCFGStructurizer.cpp:1751:3: error: 'isCurrentDebugType' was not declared in this scope

https://bugs.freedesktop.org/show_bug.cgi?id=49110 --- Comment #6 from Mike Mestnik 2012-05-08 20:25:23 PDT --- I've added 3 patches to http://llvm.org/bugs/show_bug.cgi?id=12759 and did my best to describe what/why. I believe that mesa also

[Bug 49603] [regression] Fullscreen video no longer smooth with GPU in low power mode

https://bugs.freedesktop.org/show_bug.cgi?id=49603 --- Comment #4 from Alex Deucher 2012-05-08 19:24:48 PDT --- Does forcing the CPU into the highest power state fix the issue? I suspect that the patch reduces CPU usage which in turn means the CPU power state stays lower longer. -- Configure

[PATCH 0/2 v3] drm/exynos: added userptr feature

this feature could be used to use memory region allocated by malloc() in user mode and mmaped memory region allocated by other memory allocators. userptr interface can identify memory type through vm_flags value and would get pages or page frame numbers to user space appropriately. changelog v2:

[PATCH 1/2 v3] drm/exynos: added userptr limit ioctl.

this ioctl is used to limit user-desired userptr size as pre-defined and also could be accessed by only root user. with userptr feature, unprivileged user can allocate all the pages on system, so the amount of free physical pages will be very limited. if the VMAs within user address space was

[PATCH 2/2 v3] drm/exynos: added userptr feature.

this feature is used to import user space region allocated by malloc() or mmaped into a gem. and to guarantee the pages to user space not to be swapped out, the VMAs within the user space would be locked and then unlocked when the pages are released. but this lock might result in significant

Re: [PATCHv5 08/13] v4l: vb2-dma-contig: add support for scatterlist in userptr mode

Hi Laurent, On 05/08/2012 02:44 PM, Laurent Pinchart wrote: Hi Subash, On Monday 07 May 2012 20:08:25 Subash Patel wrote: Hello Thomasz, Laurent, I found an issue in the function vb2_dc_pages_to_sgt() below. I saw that during the attach, the size of the SGT and size requested mis-matched (by

Re: [PATCH] mm: Work around Intel SNB GTT bug with some physical pages.

On Mon, 7 May 2012, Stephane Marchesin wrote: While investing some Sandy Bridge rendering corruption, I found out that all physical memory pages below 1MiB were returning garbage when read through the GTT. This has been causing graphics corruption (when it's used for textures, render targets

Re: [PATCH] MAINTAINERS: switch drm/i915 to Daniel Vetter

On Tue, May 8, 2012 at 12:19 PM, Daniel Vetter daniel.vet...@ffwll.ch wrote: ... given that I essentially run the show already, let's make this official. Acked-by: Dave Airlie airl...@redhat.com probably just push it via your -next. Dave. ___

Re: [PATCH] MAINTAINERS: switch drm/i915 to Daniel Vetter

On Wed, May 09, 2012 at 07:51:35AM +0100, Dave Airlie wrote: On Tue, May 8, 2012 at 12:19 PM, Daniel Vetter daniel.vet...@ffwll.ch wrote: ... given that I essentially run the show already, let's make this official. Acked-by: Dave Airlie airl...@redhat.com probably just push it via your

Re: [PATCH] mm: Work around Intel SNB GTT bug with some physical pages.

On Tue, May 08, 2012 at 02:57:25PM -0700, Hugh Dickins wrote: On Mon, 7 May 2012, Stephane Marchesin wrote: While investing some Sandy Bridge rendering corruption, I found out that all physical memory pages below 1MiB were returning garbage when read through the GTT. This has been causing

Re: [PATCHv5 08/13] v4l: vb2-dma-contig: add support for scatterlist in userptr mode

Hello Tomasz, Laurent, I have printed some logs during the dmabuf export and attach for the SGT issue below. Please find it in the attachment. I hope it will be useful. Regards, Subash On 05/08/2012 04:45 PM, Subash Patel wrote: Hi Laurent, On 05/08/2012 02:44 PM, Laurent Pinchart wrote:

[Bug 49603] [regression] Fullscreen video no longer smooth with GPU in low power mode

https://bugs.freedesktop.org/show_bug.cgi?id=49603 --- Comment #5 from Simon Farnsworth simon.farnswo...@onelan.co.uk 2012-05-09 02:30:15 PDT --- Created attachment 61271 -- https://bugs.freedesktop.org/attachment.cgi?id=61271 A program to stop the CPU entering low power states Trying a

Re: [PATCH 14/20] drm/radeon: multiple ring allocator v2

On 08.05.2012 16:55, Jerome Glisse wrote: Still i don't want to loop more than necessary, it's bad, i am ok with : http://people.freedesktop.org/~glisse/reset5/0001-drm-radeon-multiple-ring-allocator-v3.patch If there is fence signaled it will retry 2 times at most, otherwise it will go to wait

Re: [PATCHv5 08/13] v4l: vb2-dma-contig: add support for scatterlist in userptr mode

Hi Subash, Could you post the code of vb2_dc_pages_to_sgt with all printk in it. It will help us avoid guessing where and what is debugged in the log. Moreover, I found a line 'size=4294836224' in the log. It means that size is equal to -131072 (!?!) or there are some invalid conversions in

[Bug 49632] radeon: The kernel rejected CS,

https://bugs.freedesktop.org/show_bug.cgi?id=49632 --- Comment #6 from execute.met...@gmail.com 2012-05-09 03:55:23 PDT --- Ok. I didn't realize that. I have also tried with 3.4rc8, with the same result. Did i miss something in building the kernel? Also, what about projectM without streamout

[Bug 49632] radeon: The kernel rejected CS,

https://bugs.freedesktop.org/show_bug.cgi?id=49632 --- Comment #7 from execute.met...@gmail.com 2012-05-09 04:15:06 PDT --- (In reply to comment #6) Ok. I didn't realize that. I have also tried with 3.4rc8, with the same result. Did i miss something in building the kernel? Also, what about

[Bug 49632] radeon: The kernel rejected CS,

https://bugs.freedesktop.org/show_bug.cgi?id=49632 --- Comment #8 from Alex Deucher ag...@yahoo.com 2012-05-09 06:14:17 PDT --- (In reply to comment #6) Ok. I didn't realize that. I have also tried with 3.4rc8, with the same result. Did i miss something in building the kernel? Are you

Re: [PATCH] drm/radeon: eliminate redundant connector_names table

On Fri, May 4, 2012 at 11:25 AM, Ilija Hadzic ihad...@research.bell-labs.com wrote: connector_names table is just a repeat of information that already exists in drm_connector_enum_list and the same string can be retrieved using drm_get_connector_name function. Nuke the redundant table and use

Include request for SA improvements

Hi Dave Jerome and everybody on the list, I can't find any more bugs and also I'm out of things to test, so I really hope that this is the last incarnation of this patchset, and if Jerome is ok with it it should now be included into drm-next. Cheers, Christian.

[PATCH 01/19] drm/radeon: fix possible lack of synchronization btw ttm and other ring

From: Jerome Glisse jgli...@redhat.com We need to sync with the GFX ring as ttm might have schedule bo move on it and new command scheduled for other ring need to wait for bo data to be in place. Signed-off-by: Jerome Glisse jgli...@redhat.com Reviewed by: Christian König

[PATCH 02/19] drm/radeon: replace the per ring mutex with a global one

A single global mutex for ring submissions seems sufficient. Signed-off-by: Christian König deathsim...@vodafone.de --- drivers/gpu/drm/radeon/radeon.h |3 ++- drivers/gpu/drm/radeon/radeon_device.c|3 +-- drivers/gpu/drm/radeon/radeon_pm.c| 10 ++-

[PATCH 03/19] drm/radeon: convert fence to uint64_t v4

From: Jerome Glisse jgli...@redhat.com This convert fence to use uint64_t sequence number intention is to use the fact that uin64_t is big enough that we don't need to care about wrap around. Tested with and without writeback using 0xF000 as initial fence sequence and thus allowing to test

[PATCH 06/19] drm/radeon: use inline functions to calc sa_bo addr

Instead of hacking the calculation multiple times. Signed-off-by: Christian König deathsim...@vodafone.de --- drivers/gpu/drm/radeon/radeon_gart.c |6 ++ drivers/gpu/drm/radeon/radeon_object.h| 11 +++ drivers/gpu/drm/radeon/radeon_ring.c |6 ++

[PATCH 05/19] drm/radeon: rework locking ring emission mutex in fence deadlock detection v2

Some callers illegal called fence_wait_next/empty while holding the ring emission mutex. So don't relock the mutex in that cases, and move the actual locking into the fence code. v2: Don't try to unlock the mutex if it isn't locked. Signed-off-by: Christian König deathsim...@vodafone.de ---

[PATCH 07/19] drm/radeon: add proper locking to the SA v3

Make the suballocator self containing to locking. v2: split the bugfix into a seperate patch. v3: remove some unreleated changes. Sig-off-by: Christian König deathsim...@vodafone.de --- drivers/gpu/drm/radeon/radeon.h|1 + drivers/gpu/drm/radeon/radeon_sa.c |6 ++ 2 files

[PATCH 09/19] drm/radeon: keep start and end offset in the SA

Instead of offset + size keep start and end offset directly. Signed-off-by: Christian König deathsim...@vodafone.de --- drivers/gpu/drm/radeon/radeon.h|4 ++-- drivers/gpu/drm/radeon/radeon_cs.c |4 ++-- drivers/gpu/drm/radeon/radeon_object.h |4 ++--

[PATCH 04/19] drm/radeon: rework fence handling, drop fence list v7

From: Jerome Glisse jgli...@redhat.com Using 64bits fence sequence we can directly compare sequence number to know if a fence is signaled or not. Thus the fence list became useless, so does the fence lock that mainly protected the fence list. Things like ring.ready are no longer behind a lock,

[PATCH 08/19] drm/radeon: add sub allocator debugfs file

Dumping the current allocations. Signed-off-by: Christian König deathsim...@vodafone.de --- drivers/gpu/drm/radeon/radeon_object.h |5 + drivers/gpu/drm/radeon/radeon_ring.c | 22 ++ drivers/gpu/drm/radeon/radeon_sa.c | 14 ++ 3 files changed, 41

[PATCH 10/19] drm/radeon: make sa bo a stand alone object

Allocating and freeing it seperately. Signed-off-by: Christian König deathsim...@vodafone.de --- drivers/gpu/drm/radeon/radeon.h |4 ++-- drivers/gpu/drm/radeon/radeon_cs.c|4 ++-- drivers/gpu/drm/radeon/radeon_gart.c |4 ++--

[PATCH 11/19] drm/radeon: define new SA interface v3

Define the interface without modifying the allocation algorithm in any way. v2: rebase on top of fence new uint64 patch v3: add ring to debugfs output Signed-off-by: Jerome Glisse jgli...@redhat.com Signed-off-by: Christian König deathsim...@vodafone.de --- drivers/gpu/drm/radeon/radeon.h

[PATCH 12/19] drm/radeon: use one wait queue for all rings add fence_wait_any v2

From: Jerome Glisse jgli...@redhat.com Use one wait queue for all rings. When one ring progress, other likely does to and we are not expecting to have a lot of waiter anyway. Also add a fence_wait_any that will wait until the first fence in the fence array (one fence per ring) is signaled. This

[PATCH 13/19] drm/radeon: multiple ring allocator v3

A startover with a new idea for a multiple ring allocator. Should perform as well as a normal ring allocator as long as only one ring does somthing, but falls back to a more complex algorithm if more complex things start to happen. We store the last allocated bo in last, we always try to allocate

[PATCH 15/19] drm/radeon: rip out the ib pool

From: Jerome Glisse jgli...@redhat.com It isn't necessary any more and the suballocator seems to perform even better. Signed-off-by: Christian König deathsim...@vodafone.de Signed-off-by: Jerome Glisse jgli...@redhat.com --- drivers/gpu/drm/radeon/radeon.h | 17 +-

[PATCH 14/19] drm/radeon: simplify semaphore handling v2

From: Jerome Glisse jgli...@redhat.com Directly use the suballocator to get small chunks of memory. It's equally fast and doesn't crash when we encounter a GPU reset. v2: rebased on new SA interface. Signed-off-by: Christian König deathsim...@vodafone.de Signed-off-by: Jerome Glisse

[PATCH 16/19] drm/radeon: immediately free ttm-move semaphore

We can now protected the semaphore ram by a fence, so free it immediately. Signed-off-by: Christian König deathsim...@vodafone.de --- drivers/gpu/drm/radeon/radeon_ttm.c |7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_ttm.c

[PATCH 17/19] drm/radeon: move the semaphore from the fence into the ib

From: Jerome Glisse jgli...@redhat.com It never really belonged there in the first place. Signed-off-by: Christian König deathsim...@vodafone.de --- drivers/gpu/drm/radeon/radeon.h | 16 drivers/gpu/drm/radeon/radeon_cs.c|4 ++--

[PATCH 18/19] drm/radeon: remove r600 blit mutex v2

If we don't store local data into global variables it isn't necessary to lock anything. v2: rebased on new SA interface Signed-off-by: Christian König deathsim...@vodafone.de --- drivers/gpu/drm/radeon/evergreen_blit_kms.c |1 - drivers/gpu/drm/radeon/r600.c | 13 ++--

[PATCH 19/19] drm/radeon: make the ib an inline object

From: Jerome Glisse jgli...@redhat.com No need to malloc it any more. Signed-off-by: Jerome Glisse jgli...@redhat.com Signed-off-by: Christian König deathsim...@vodafone.de --- drivers/gpu/drm/radeon/evergreen_cs.c | 10 +++--- drivers/gpu/drm/radeon/r100.c | 38 ++--

[Bug 49632] radeon: The kernel rejected CS,

https://bugs.freedesktop.org/show_bug.cgi?id=49632 --- Comment #9 from execute.met...@gmail.com 2012-05-09 06:36:57 PDT --- No, there is nothing else in dmesg. Is there any more info you'd like me to gather? -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You

Re: [PATCH] drm/radeon: don't mess with hot plug detect for eDP or LVDS connector v2

On Fri, May 4, 2012 at 11:06 AM, j.gli...@gmail.com wrote: From: Jerome Glisse jgli...@redhat.com It seems imac pannel doesn't like whe we change the hot plug setup and then refuse to work. This help but doesn't fully fix: https://bugzilla.redhat.com/show_bug.cgi?id=726143 How does it help?

Re: [PATCH] drm/radeon: don't mess with hot plug detect for eDP or LVDS connector v2

On Wed, May 9, 2012 at 9:40 AM, Alex Deucher alexdeuc...@gmail.com wrote: On Fri, May 4, 2012 at 11:06 AM,  j.gli...@gmail.com wrote: From: Jerome Glisse jgli...@redhat.com It seems imac pannel doesn't like whe we change the hot plug setup and then refuse to work. This help but doesn't fully

Re: Include request for SA improvements

On Wed, May 9, 2012 at 9:34 AM, Christian König deathsim...@vodafone.de wrote: Hi Dave Jerome and everybody on the list, I can't find any more bugs and also I'm out of things to test, so I really hope that this is the last incarnation of this patchset, and if Jerome is ok with it it should

[PATCH] drm/radeon/kms: fix warning on 32-bit in atomic fence printing

From: Dave Airlie airl...@redhat.com /ssd/git/drm-core-next/drivers/gpu/drm/radeon/radeon_fence.c: In function ‘radeon_debugfs_fence_info’: /ssd/git/drm-core-next/drivers/gpu/drm/radeon/radeon_fence.c:606:7: warning: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has

Re: [PATCH] drm/radeon/kms: fix warning on 32-bit in atomic fence printing

On Wed, May 9, 2012 at 12:28 PM, Dave Airlie airl...@gmail.com wrote: From: Dave Airlie airl...@redhat.com /ssd/git/drm-core-next/drivers/gpu/drm/radeon/radeon_fence.c: In function ‘radeon_debugfs_fence_info’: /ssd/git/drm-core-next/drivers/gpu/drm/radeon/radeon_fence.c:606:7: warning:

Re: [PATCH] drm/radeon: don't mess with hot plug detect for eDP or LVDS connector v2

On Wed, May 9, 2012 at 10:23 AM, Jerome Glisse j.gli...@gmail.com wrote: On Wed, May 9, 2012 at 9:40 AM, Alex Deucher alexdeuc...@gmail.com wrote: On Fri, May 4, 2012 at 11:06 AM,  j.gli...@gmail.com wrote: From: Jerome Glisse jgli...@redhat.com It seems imac pannel doesn't like whe we change

[Bug 43215] New: Nouveau: Resume from s2disk fails.

https://bugzilla.kernel.org/show_bug.cgi?id=43215 Summary: Nouveau: Resume from s2disk fails. Product: Drivers Version: 2.5 Kernel Version: 3.3.5, 3.4-rc4+, 3.4-rc6+ Platform: All OS/Version: Linux Tree: Mainline

Re: Include request for SA improvements

On Wed, May 9, 2012 at 3:31 PM, Jerome Glisse j.gli...@gmail.com wrote: On Wed, May 9, 2012 at 9:34 AM, Christian König deathsim...@vodafone.de wrote: Hi Dave Jerome and everybody on the list, I can't find any more bugs and also I'm out of things to test, so I really hope that this is the

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

On Wed, May 9, 2012 at 10:45 AM, Jerome Glisse j.gli...@gmail.com wrote: On Wed, May 9, 2012 at 2:17 AM, Inki Dae inki@samsung.com wrote: this feature is used to import user space region allocated by malloc() or mmaped into a gem. and to guarantee the pages to user space not to be swapped

[Bug 43858] DVI of ATI RADEON 9200 AGP don't work

https://bugs.freedesktop.org/show_bug.cgi?id=43858 Alex Deucher ag...@yahoo.com changed: What|Removed |Added Status|NEW |RESOLVED

In Diablo III when the ground is covered in water, it's all messed-up.

I found a rendering error in Diablo III, in a well there ground is covered in water and it's all red and flickers. http://c566453.r53.cf2.rackcdn.com/DiabloIII-well-wine-preloader.trace.xz 3.5G, 234665296 Bytes compressed ~234M. EE r600_shader.c:1605 r600_shader_from_tgsi - GPR limit exceeded -

RE: [PATCH 2/2 v3] drm/exynos: added userptr feature.

Hi Jerome, -Original Message- From: Jerome Glisse [mailto:j.gli...@gmail.com] Sent: Wednesday, May 09, 2012 11:46 PM To: Inki Dae Cc: airl...@linux.ie; dri-devel@lists.freedesktop.org; kyungmin.p...@samsung.com; sw0312@samsung.com; linux...@kvack.org Subject: Re: [PATCH 2/2 v3]

RE: [PATCH 2/2 v3] drm/exynos: added userptr feature.

Hi Jerome, Thank you again. -Original Message- From: Jerome Glisse [mailto:j.gli...@gmail.com] Sent: Thursday, May 10, 2012 3:33 AM To: Inki Dae Cc: airl...@linux.ie; dri-devel@lists.freedesktop.org; kyungmin.p...@samsung.com; sw0312@samsung.com; linux...@kvack.org Subject:

Re: [PATCH 2/2 v3] drm/exynos: added userptr feature.

On 05/10/2012 10:39 AM, Inki Dae wrote: Hi Jerome, -Original Message- From: Jerome Glisse [mailto:j.gli...@gmail.com] Sent: Wednesday, May 09, 2012 11:46 PM To: Inki Dae Cc: airl...@linux.ie; dri-devel@lists.freedesktop.org; kyungmin.p...@samsung.com; sw0312@samsung.com;