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, Daniel
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
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:
H
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 t
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
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 printk
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 enab
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 a
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 er
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 re
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.
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
---
drivers/gpu/drm/radeon/radeon_cs.c | 12 ++--
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 ++-
drivers/gpu/drm/radeon/radeon_ring
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 the wrap around from
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 ++
drivers/gpu/drm/radeon/radeon_sema
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
---
drivers/gpu/drm/radeon/rade
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(+)
diff
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 | 1
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, this should
be ok as
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 --g
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 ++--
dri
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 +
drivers/gpu/drm/radeon/rade
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. This allow
to wait on al
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
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 -
drivers/gpu/drm/radeon/ra
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
---
drivers/gpu/drm/radeon/evergreen.c|
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
b/drivers/gpu/drm/radeon/radeon_ttm
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 ---
drivers/gpu/drm/radeon/r
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 ++--
drivers/gpu/drm/radeon/r600_b
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 |2 +-
drivers/g
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 a
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 proble
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:
>> https://bugzilla.redhat.com/show_bug.cgi?
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 int
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 r
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 has
type ‘long long in
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
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 d
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
Stat
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 patchs
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 sp
https://bugs.freedesktop.org/show_bug.cgi?id=43858
Alex Deucher changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
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 -
s
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
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
> Subje
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@sams
> -Original Message-
> From: Minchan Kim [mailto:minc...@kernel.org]
> Sent: Thursday, May 10, 2012 1:58 PM
> To: Inki Dae
> Cc: 'Jerome Glisse'; airl...@linux.ie; dri-devel@lists.freedesktop.org;
> kyungmin.p...@samsung.com; sw0312@samsung.com; linux...@kvack.org
> Subject: Re: [PATC
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
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 needs this done for it's use of NDEBUG, especially if
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:
t
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 lock
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 degra
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.
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, Dan
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
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:
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 t
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 t
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 printk
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 e
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, wha
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 e
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 re
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.
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
---
drivers/gpu/drm/radeon/radeon_cs.c | 12 ++--
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 ++-
drivers/gpu/drm/radeon/radeon_ring
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 the wrap around from
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 ++
drivers/gpu/drm/radeon/radeon_sema
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
---
drivers/gpu/drm/radeon/rade
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(+)
diff
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 | 1
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, this should
be ok as
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 --g
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 ++--
dri
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 +
drivers/gpu/drm/radeon/rade
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. This allow
to wait on al
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
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 -
drivers/gpu/drm/radeon/ra
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
---
drivers/gpu/drm/radeon/evergreen.c|
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
b/drivers/gpu/drm/radeon/radeon_ttm
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 ---
drivers/gpu/drm/radeon/r
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 ++--
drivers/gpu/drm/radeon/r600_b
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 |2 +-
drivers/g
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
--- Yo
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 proble
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:
>> https://bugzilla.redhat.com/show_bug.cgi?
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 in
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 r
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 has
type ?long long in
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
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 d
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 patchs
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 sp
https://bugs.freedesktop.org/show_bug.cgi?id=43858
Alex Deucher changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
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 -
s
95 matches
Mail list logo