Hi
On Thu, Jul 18, 2013 at 1:24 PM, Thomas Hellstrom
wrote:
> On 07/18/2013 01:07 PM, David Herrmann wrote:
>>
>> Hi
>>
>> On Thu, Jul 18, 2013 at 10:53 AM, Thomas Hellstrom
>> wrote:
>>>
>>> A quick look, but not a full review:
>>>
>>> Looks mostly good, but it looks like the TTM vm lock
https://bugzilla.kernel.org/show_bug.cgi?id=60381
--- Comment #27 from Arek Ru?niak ---
Created attachment 106939
--> https://bugzilla.kernel.org/attachment.cgi?id=106939=edit
boot with radeon.aspm=0
aspm=0 didn't help, 3d apps hang my pc.
But finally UVD is working, doesn't mater with or
ash.
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/6231877b/attachment.html>
https://bugzilla.kernel.org/show_bug.cgi?id=60381
Arek Ru?niak changed:
What|Removed |Added
Attachment #106895|0 |1
is obsolete|
On Thu, Jul 18, 2013 at 4:54 PM, David Herrmann
wrote:
> Hi
>
> On Thu, Jul 18, 2013 at 1:24 PM, Thomas Hellstrom
> wrote:
>> On 07/18/2013 01:07 PM, David Herrmann wrote:
>>>
>>> Hi
>>>
>>> On Thu, Jul 18, 2013 at 10:53 AM, Thomas Hellstrom
>>> wrote:
A quick look, but not a full
Hey Linus,
you'll be terribly disappointed in this, I'm not trying to sneak any
features in or anything, its mostly radeon and intel fixes, a couple of
ARM driver fixes.
Dave.
The following changes since commit ad81f0545ef01ea651886dddac4bef6cec930092:
Linux 3.11-rc1 (2013-07-14 15:18:27
--
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/9c5442b8/attachment.html>
this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/f81b5b9d/attachment.html>
On Wed, Jul 17, 2013 at 8:14 PM, David Herrmann
wrote:
> Use the new vma manager instead of the old hashtable. Also convert all
> drivers to use the new convenience helpers. This drops all the
> (map_list.hash.key << PAGE_SHIFT) non-sense.
>
> Locking and access-management is exactly the same as
alignment.
Yes that patch fixes the problem.
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/f2dad0de/attachment.html>
ent.
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/837ef67b/attachment.html>
that will reproduce this
bug?
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/3d36ddb4/attachment.html>
till doesn't work?
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/626d3393/attachment-0001.html>
.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/6756ede8/attachment.html>
attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/4f85489d/attachment.html>
nee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/7493e0a4/attachment.html>
ttp://lists.freedesktop.org/archives/dri-devel/attachments/20130718/6b66400f/attachment.html>
ttp://lists.freedesktop.org/archives/dri-devel/attachments/20130718/8c3707ba/attachment.html>
was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/4d19cd2a/attachment.html>
https://bugzilla.kernel.org/show_bug.cgi?id=60381
--- Comment #25 from rafael castillo ---
well i tried drm-fixes-3.11 with both gcc 4.8.1 and 4.7.2 and both reset the
GPU when reclock, i attached the dmesg.
i meant that with your recent changes the GPU recover and allow you to close
the game
bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/7ff52a58/attachment.html>
ze >> PAGE_SHIFT;
+ count += obj->base.size >> PAGE_SHIFT;
if (unlock)
mutex_unlock(>struct_mutex);
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 byte
art --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/01a9c07e/attachment.html>
I have just discovered that WAIT_UNTIL=WAIT_3D_IDLE must be set before using
CP DMA.
Marek
On Thu, Jul 11, 2013 at 9:35 PM, wrote:
> From: Alex Deucher
>
> Lighter weight than using the 3D engine.
>
> v2: fix ring count
>
> Signed-off-by: Alex Deucher
> ---
> drivers/gpu/drm/radeon/r600.c
ttp://lists.freedesktop.org/archives/dri-devel/attachments/20130718/70c0066a/attachment.html>
ttp://lists.freedesktop.org/archives/dri-devel/attachments/20130718/a3526780/attachment.html>
TML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/6da95ff9/attachment.html>
are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/c73ef813/attachment.html>
ttp://lists.freedesktop.org/archives/dri-devel/attachments/20130718/dbdde3be/attachment.html>
nee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/920de9df/attachment-0001.html>
.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/f100541d/attachment.html>
On 07/18/2013 01:07 PM, David Herrmann wrote:
> Hi
>
> On Thu, Jul 18, 2013 at 10:53 AM, Thomas Hellstrom
> wrote:
>> A quick look, but not a full review:
>>
>> Looks mostly good, but it looks like the TTM vm lock isn't needed at all
>> anymore (provided the vma offset manager is properly
0 times without fail with that kernel commit reverted and
RADEON_VA=1.
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments
https://bugzilla.kernel.org/show_bug.cgi?id=60381
--- Comment #24 from Alex Deucher ---
You might try the latest drm-fixes branch if you were using gcc 4.8. See:
https://bugs.freedesktop.org/show_bug.cgi?id=66932
--
You are receiving this mail because:
You are watching the assignee of the
for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/5c626d50/attachment.html>
Hi
On Thu, Jul 18, 2013 at 10:53 AM, Thomas Hellstrom
wrote:
> A quick look, but not a full review:
>
> Looks mostly good, but it looks like the TTM vm lock isn't needed at all
> anymore (provided the vma offset manager is properly protected), since
> kref_get_unless_zero() is used when a
Hi Dave,
I know you usually don't open -next this early, but since this here
contains a few changes to drm_mm I've figured I should unblock David
Herrmann. Highlights:
- follow-up refactoring after the shared dpll rework that landed in 3.11
- oddball prep cleanups from Ben for ppgtt
-
ttp://lists.freedesktop.org/archives/dri-devel/attachments/20130718/4346a709/attachment.html>
On Thu, Jul 18, 2013 at 10:53 AM, Thomas Hellstrom
wrote:
> A quick look, but not a full review:
>
> Looks mostly good, but it looks like the TTM vm lock isn't needed at all
> anymore (provided the vma offset manager is properly protected), since
> kref_get_unless_zero() is used when a reference
On Fri, Jul 12, 2013 at 5:51 PM, wrote:
> From: Alex Deucher
>
> Covers requirements of all current asics.
>
> Signed-off-by: Alex Deucher
> ---
> drivers/gpu/drm/radeon/radeon.h |5 +
> drivers/gpu/drm/radeon/radeon_gart.c | 12 ++--
> 2 files changed, 11
t += obj->base.size >> PAGE_SHIFT;
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/5df47da0/attachment.pgp>
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/00425be8/attachment.pgp>
:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/c0e6c6dd/attachment-0001.html>
A quick look, but not a full review:
Looks mostly good, but it looks like the TTM vm lock isn't needed at all
anymore (provided the vma offset manager is properly protected), since
kref_get_unless_zero() is used when a reference after lookup is taken.
(please see the kref_get_unless_zero
From: Matthew Garrett
We have to call acpi_video_init_brightness() even if we're not going
to initialise the backlight - Thinkpads seem to use this as the
trigger for enabling ACPI notifications rather than handling it in
firmware.
[rjw: Drop the brightness object
From: Aaron Lu
Expose acpi_gbl_osi_data so that code outside of ACPICA can check
the value of the last successfull _OSI call. The definitions for
OSI versions are moved to actypes.h so that other components can
access them too.
Based on a patch from Matthew Garrett which in
this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/28fb7a40/attachment.html>
(I took time to have dinner).
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/4f6641d2/attachment.html>
From: Rafael J. Wysocki
According to Matthew Garrett, "Windows 8 leaves backlight control up
to individual graphics drivers rather than making ACPI calls itself.
There's plenty of evidence to suggest that the Intel driver for
Windows [8] doesn't use the ACPI
From: Matthew Garrett
We have to call acpi_video_init_brightness() even if we're not going
to initialise the backlight - Thinkpads seem to use this as the
trigger for enabling ACPI notifications rather than handling it in
firmware.
[rjw: Drop the brightness object
From: Aaron Lu
Expose acpi_gbl_osi_data so that code outside of ACPICA can check
the value of the last successfull _OSI call. The definitions for
OSI versions are moved to actypes.h so that other components can
access them too.
Based on a patch from Matthew Garrett which in
On Sunday, June 09, 2013 07:01:36 PM Matthew Garrett wrote:
> Windows 8 introduced new policy for backlight control by pushing it out to
> graphics drivers. This appears to have coincided with a range of vendors
> adding Windows 8 checks to their backlight control code which trigger either
>
https://bugzilla.kernel.org/show_bug.cgi?id=60381
--- Comment #23 from rafael castillo ---
ok xonotic still crash play hell with the GPU but now it can resume after
failure and for things like gpu accel in browser or normal kwin usage seems
stable enough.
it seems only real 3d apps like games
https://bugzilla.kernel.org/show_bug.cgi?id=60381
--- Comment #22 from rafael castillo ---
Created attachment 106917
--> https://bugzilla.kernel.org/attachment.cgi?id=106917=edit
latest drm-fixes-3.11 dmesg
--
You are receiving this mail because:
You are watching the assignee of the bug.
this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20130718/4cdbb879/attachment.html>
On Wed, Jul 17, 2013 at 09:44:00PM +, Deucher, Alexander wrote:
>> -Original Message-
>> From: Anthoine Bourgeois [mailto:anthoine.bourgeois at gmail.com]
>> Sent: Wednesday, July 17, 2013 5:23 PM
>> To: Deucher, Alexander; Koenig, Christian; Jerome Glisse
>> Cc: dri-devel at
On Wed, Jul 17, 2013 at 08:54:33PM +, Deucher, Alexander wrote:
>Good catch, I screwed up the ref divider calculation. How about the attached
>patch?
It works too. But here was my logic, tell me if I'm wrong:
The output frequency of the PLL is equal to the VCO frequency (FVCO)
divided by
https://bugs.freedesktop.org/show_bug.cgi?id=66963
--- Comment #1 from Sergey orbitg...@ukr.net ---
I can see same behavior on
Graphics: RV620/M82 Mobility Radeon HD 3450/3470
Kernel: 3.11-rc1
Mesa: 9.1.2-rc1, 9.2_pre20130619
OS: Gentoo
ARCH: x86_64
Other: fluxbox-1.3.2, xdm-1.1.11-r1,
A quick look, but not a full review:
Looks mostly good, but it looks like the TTM vm lock isn't needed at all
anymore (provided the vma offset manager is properly protected), since
kref_get_unless_zero() is used when a reference after lookup is taken.
(please see the kref_get_unless_zero
On Thu, Jul 18, 2013 at 10:53 AM, Thomas Hellstrom
thellst...@vmware.com wrote:
A quick look, but not a full review:
Looks mostly good, but it looks like the TTM vm lock isn't needed at all
anymore (provided the vma offset manager is properly protected), since
kref_get_unless_zero() is used
Hi Dave,
I know you usually don't open -next this early, but since this here
contains a few changes to drm_mm I've figured I should unblock David
Herrmann. Highlights:
- follow-up refactoring after the shared dpll rework that landed in 3.11
- oddball prep cleanups from Ben for ppgtt
-
Hi
On Thu, Jul 18, 2013 at 10:53 AM, Thomas Hellstrom
thellst...@vmware.com wrote:
A quick look, but not a full review:
Looks mostly good, but it looks like the TTM vm lock isn't needed at all
anymore (provided the vma offset manager is properly protected), since
kref_get_unless_zero() is
On 07/18/2013 01:07 PM, David Herrmann wrote:
Hi
On Thu, Jul 18, 2013 at 10:53 AM, Thomas Hellstrom
thellst...@vmware.com wrote:
A quick look, but not a full review:
Looks mostly good, but it looks like the TTM vm lock isn't needed at all
anymore (provided the vma offset manager is properly
https://bugs.freedesktop.org/show_bug.cgi?id=67016
--- Comment #1 from Martin Andersson g02ma...@gmail.com ---
Perhaps mesa needs to be updated after this commit. I have tried to change mesa
to align to 32768 instead of 4096. With some changes I can make the test pass a
few times before it hangs.
https://bugs.freedesktop.org/show_bug.cgi?id=67043
Priority: medium
Bug ID: 67043
Assignee: dri-devel@lists.freedesktop.org
Summary: Atombios stuck in a look during resume
Severity: normal
Classification: Unclassified
OS:
https://bugs.freedesktop.org/show_bug.cgi?id=67016
--- Comment #3 from Martin Andersson g02ma...@gmail.com ---
(In reply to comment #2)
Does the test work reliably with that kernel commit reverted? You don't
need to adjust the alignment of anything in mesa. I don't see how that
commit would
https://bugs.freedesktop.org/show_bug.cgi?id=67016
--- Comment #4 from Martin Andersson g02ma...@gmail.com ---
Created attachment 82599
-- https://bugs.freedesktop.org/attachment.cgi?id=82599action=edit
dmesg
I managed to get a dmesg without my patches.
--
You are receiving this mail
https://bugs.freedesktop.org/show_bug.cgi?id=67016
--- Comment #5 from Alex Deucher ag...@yahoo.com ---
Do you still get the issue with dpm disabled?
--
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
https://bugs.freedesktop.org/show_bug.cgi?id=67016
--- Comment #6 from Michel Dänzer mic...@daenzer.net ---
That commit causes problems (VM faults, lockups) on my Cape Verde card as well.
I suspect other kernel code needs to be adjusted for the increased buffer
sizes, or something like that.
--
https://bugs.freedesktop.org/show_bug.cgi?id=67016
--- Comment #7 from Martin Andersson g02ma...@gmail.com ---
(In reply to comment #5)
Do you still get the issue with dpm disabled?
yes
--
You are receiving this mail because:
You are the assignee for the bug.
https://bugs.freedesktop.org/show_bug.cgi?id=67016
--- Comment #8 from Alex Deucher ag...@yahoo.com ---
Ok. I'll go ahead and revert it for now.
--
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
https://bugs.freedesktop.org/show_bug.cgi?id=67046
Priority: medium
Bug ID: 67046
Assignee: dri-devel@lists.freedesktop.org
Summary: [radeonsi] pixmark_piano benchmark from geeks3D
gputest fail with a llvm error output
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #2 from Parag parag.warud...@gmail.com ---
Created attachment 82603
-- https://bugs.freedesktop.org/attachment.cgi?id=82603action=edit
dmesg with dpm enabled
--
You are receiving this mail because:
You are the assignee for the
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #3 from Parag parag.warud...@gmail.com ---
Created attachment 82613
-- https://bugs.freedesktop.org/attachment.cgi?id=82613action=edit
dmesg from 3.10.1 kernel without the timeout issue
--
You are receiving this mail because:
You
https://bugzilla.kernel.org/show_bug.cgi?id=60381
--- Comment #25 from rafael castillo jrch2...@gmail.com ---
well i tried drm-fixes-3.11 with both gcc 4.8.1 and 4.7.2 and both reset the
GPU when reclock, i attached the dmesg.
i meant that with your recent changes the GPU recover and allow you
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #5 from Alex Deucher ag...@yahoo.com ---
If it started in 3.11 can you bisect?
--
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #6 from Parag parag.warud...@gmail.com ---
I suck at bisecting, but yes, I will give it a shot.
--
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel
https://bugs.freedesktop.org/show_bug.cgi?id=67016
--- Comment #9 from Martin Andersson g02ma...@gmail.com ---
Created attachment 82616
-- https://bugs.freedesktop.org/attachment.cgi?id=82616action=edit
possible fix
This patch fixes the issue for me.
--
You are receiving this mail because:
https://bugs.freedesktop.org/show_bug.cgi?id=67016
--- Comment #10 from Alex Deucher ag...@yahoo.com ---
I think that patch is correct. We have to align the PTE block size as well.
--
You are receiving this mail because:
You are the assignee for the bug.
https://bugs.freedesktop.org/show_bug.cgi?id=67016
--- Comment #11 from Martin Andersson g02ma...@gmail.com ---
At least it works for me, I have run a complete piglit test(quick.tests) with
that patch without issues, with dpm enabled.
--
You are receiving this mail because:
You are the assignee
https://bugs.freedesktop.org/show_bug.cgi?id=67016
--- Comment #12 from Christian König deathsim...@vodafone.de ---
(In reply to comment #9)
Created attachment 82616 [details] [review]
possible fix
This patch fixes the issue for me.
The patch itself is correct, but the problem is it
https://bugs.freedesktop.org/show_bug.cgi?id=60929
--- Comment #9 from Tom Stellard tstel...@gmail.com ---
I'm not really sure what's happening here, but I don't think these closed
source games are good enough tests cases to diagnose the problem. Could you
try to find a very simple Open Source
https://bugs.freedesktop.org/show_bug.cgi?id=67016
--- Comment #13 from Alex Deucher ag...@yahoo.com ---
Created attachment 82622
-- https://bugs.freedesktop.org/attachment.cgi?id=82622action=edit
only align the pt base to 32k
Does this patch help? Only the page table base address should need
https://bugs.freedesktop.org/show_bug.cgi?id=67016
--- Comment #14 from Martin Andersson g02ma...@gmail.com ---
(In reply to comment #13)
Created attachment 82622 [details] [review]
only align the pt base to 32k
Does this patch help? Only the page table base address should need the 32k
On Wed, Jul 17, 2013 at 8:14 PM, David Herrmann dh.herrm...@gmail.com wrote:
Use the new vma manager instead of the old hashtable. Also convert all
drivers to use the new convenience helpers. This drops all the
(map_list.hash.key PAGE_SHIFT) non-sense.
Locking and access-management is
Hi
On Thu, Jul 18, 2013 at 1:24 PM, Thomas Hellstrom thellst...@vmware.com wrote:
On 07/18/2013 01:07 PM, David Herrmann wrote:
Hi
On Thu, Jul 18, 2013 at 10:53 AM, Thomas Hellstrom
thellst...@vmware.com wrote:
A quick look, but not a full review:
Looks mostly good, but it looks like
Hey Linus,
you'll be terribly disappointed in this, I'm not trying to sneak any
features in or anything, its mostly radeon and intel fixes, a couple of
ARM driver fixes.
Dave.
The following changes since commit ad81f0545ef01ea651886dddac4bef6cec930092:
Linux 3.11-rc1 (2013-07-14 15:18:27
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #7 from Parag parag.warud...@gmail.com ---
After a not-so-clean git bisect (compilation failures in vgacon.c, some TI
driver I didn't need) this was the result - at least it is related to drm.
git bisect good
https://bugs.freedesktop.org/show_bug.cgi?id=67043
--- Comment #8 from Parag parag.warud...@gmail.com ---
Verified : Reverting 372835a8527f85b3eff20a18c2c339e827dfd4e4 from HEAD does
make the problem go away.
--
You are receiving this mail because:
You are the assignee for the bug.
https://bugzilla.kernel.org/show_bug.cgi?id=60381
Arek Ruśniak arek.r...@gmail.com changed:
What|Removed |Added
Attachment #106895|0 |1
is obsolete|
https://bugs.freedesktop.org/show_bug.cgi?id=66805
--- Comment #5 from Tom Stellard tstel...@gmail.com ---
Created attachment 82639
-- https://bugs.freedesktop.org/attachment.cgi?id=82639action=edit
Possible Work Around
I'm working on a proper fix for this, but in the mean time this patch
https://bugzilla.kernel.org/show_bug.cgi?id=60381
--- Comment #27 from Arek Ruśniak arek.r...@gmail.com ---
Created attachment 106939
-- https://bugzilla.kernel.org/attachment.cgi?id=106939action=edit
boot with radeon.aspm=0
aspm=0 didn't help, 3d apps hang my pc.
But finally UVD is working,
On Thu, Jul 18, 2013 at 4:54 PM, David Herrmann dh.herrm...@gmail.com wrote:
Hi
On Thu, Jul 18, 2013 at 1:24 PM, Thomas Hellstrom thellst...@vmware.com
wrote:
On 07/18/2013 01:07 PM, David Herrmann wrote:
Hi
On Thu, Jul 18, 2013 at 10:53 AM, Thomas Hellstrom
thellst...@vmware.com wrote:
https://bugs.freedesktop.org/show_bug.cgi?id=66942
Alexandre Demers alexandre.f.dem...@gmail.com changed:
What|Removed |Added
See Also|
https://bugs.freedesktop.org/show_bug.cgi?id=66963
Alexandre Demers alexandre.f.dem...@gmail.com changed:
What|Removed |Added
See Also|
i915_gem_vma_create() returns and ERR_PTR() or a valid pointer, it never
returns NULL.
Signed-off-by: Dan Carpenter dan.carpen...@oracle.com
diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
index 40c2fc6..9a9a77a 100644
--- a/drivers/gpu/drm/i915/i915_gem.c
+++
i915_gem_vma_destroy() frees its argument so we have to move the
drm_mm_remove_node() call up a few lines.
Signed-off-by: Dan Carpenter dan.carpen...@oracle.com
diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
index 9a9a77a..f347ad5 100644
---
97 matches
Mail list logo