Re: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-22 Thread Christian König
On 22.02.2015 00:18, Ross Zwisler wrote: On Wed, 2015-02-18 at 13:02 +0100, Christian König wrote: Well, what the patch does is just changing where buffers are placed in memory. E.g. now we place the buffer at the end of memory as well. So I can imagine at least three possible causes for the

Re: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-22 Thread Christian König
On 22.02.2015 00:18, Ross Zwisler wrote: On Wed, 2015-02-18 at 13:02 +0100, Christian König wrote: Well, what the patch does is just changing where buffers are placed in memory. E.g. now we place the buffer at the end of memory as well. So I can imagine at least three possible causes for the

Re: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-21 Thread Ross Zwisler
On Wed, 2015-02-18 at 13:02 +0100, Christian König wrote: > Well, what the patch does is just changing where buffers are placed in > memory. E.g. now we place the buffer at the end of memory as well. > > So I can imagine at least three possible causes for the issues you see: > 1. We haven't

Re: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-21 Thread Ross Zwisler
On Wed, 2015-02-18 at 13:02 +0100, Christian König wrote: Well, what the patch does is just changing where buffers are placed in memory. E.g. now we place the buffer at the end of memory as well. So I can imagine at least three possible causes for the issues you see: 1. We haven't

RE: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-18 Thread Deucher, Alexander
ri Kasanen; Koenig, Christian > Subject: Re: [PATCH] drm/radeon: Fix regression with suspend/resume > > On 17.02.2015 18:49, Ross Zwisler wrote: > > On Sat, 2015-02-14 at 06:25 +, Deucher, Alexander wrote: > >>> -Original Message- > >>> From: Ross Zwisler

Re: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-18 Thread Christian König
...@lists.freedesktop.org; Deucher, Alexander; Dave Airlie; Lauri Kasanen; Koenig, Christian Subject: Re: [PATCH] drm/radeon: Fix regression with suspend/resume On Fri, 2015-02-13 at 11:41 +0900, Michel Dänzer wrote: On 13.02.2015 05:30, Ross Zwisler wrote: This patch reverts the changes made

RE: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-18 Thread Deucher, Alexander
Subject: Re: [PATCH] drm/radeon: Fix regression with suspend/resume On 17.02.2015 18:49, Ross Zwisler wrote: On Sat, 2015-02-14 at 06:25 +, Deucher, Alexander wrote: -Original Message- From: Ross Zwisler [mailto:ross.zwis...@linux.intel.com] Sent: Friday, February 13, 2015 10

Re: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-18 Thread Christian König
...@lists.freedesktop.org; Deucher, Alexander; Dave Airlie; Lauri Kasanen; Koenig, Christian Subject: Re: [PATCH] drm/radeon: Fix regression with suspend/resume On Fri, 2015-02-13 at 11:41 +0900, Michel Dänzer wrote: On 13.02.2015 05:30, Ross Zwisler wrote: This patch reverts the changes made

Re: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-17 Thread Ross Zwisler
lists.freedesktop.org; Deucher, > > Alexander; Dave Airlie; Lauri Kasanen; Koenig, Christian > > Subject: Re: [PATCH] drm/radeon: Fix regression with suspend/resume > > > > On Fri, 2015-02-13 at 11:41 +0900, Michel Dänzer wrote: > > > On 13.02.2015 05:30,

Re: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-17 Thread Ross Zwisler
, Alexander; Dave Airlie; Lauri Kasanen; Koenig, Christian Subject: Re: [PATCH] drm/radeon: Fix regression with suspend/resume On Fri, 2015-02-13 at 11:41 +0900, Michel Dänzer wrote: On 13.02.2015 05:30, Ross Zwisler wrote: This patch reverts the changes made in this commit

RE: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-13 Thread Deucher, Alexander
hristian > Subject: Re: [PATCH] drm/radeon: Fix regression with suspend/resume > > On Fri, 2015-02-13 at 11:41 +0900, Michel Dänzer wrote: > > On 13.02.2015 05:30, Ross Zwisler wrote: > > > This patch reverts the changes made in this commit: > > > > > >

Re: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-13 Thread Ross Zwisler
On Fri, 2015-02-13 at 11:41 +0900, Michel Dänzer wrote: > On 13.02.2015 05:30, Ross Zwisler wrote: > > This patch reverts the changes made in this commit: > > > > deadcb36f49b ("drm/radeon: Use two-ended allocation by size, v2") > > > > That patch caused a regression on my system where the

Re: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-13 Thread Ross Zwisler
On Fri, 2015-02-13 at 11:41 +0900, Michel Dänzer wrote: On 13.02.2015 05:30, Ross Zwisler wrote: This patch reverts the changes made in this commit: deadcb36f49b (drm/radeon: Use two-ended allocation by size, v2) That patch caused a regression on my system where the bottom of the

RE: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-13 Thread Deucher, Alexander
: [PATCH] drm/radeon: Fix regression with suspend/resume On Fri, 2015-02-13 at 11:41 +0900, Michel Dänzer wrote: On 13.02.2015 05:30, Ross Zwisler wrote: This patch reverts the changes made in this commit: deadcb36f49b (drm/radeon: Use two-ended allocation by size, v2) That patch

Re: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-12 Thread Michel Dänzer
On 13.02.2015 05:30, Ross Zwisler wrote: > This patch reverts the changes made in this commit: > > deadcb36f49b ("drm/radeon: Use two-ended allocation by size, v2") > > That patch caused a regression on my system where the bottom of the > screen flickers after my laptop goes thorough a suspend

[PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-12 Thread Ross Zwisler
This patch reverts the changes made in this commit: deadcb36f49b ("drm/radeon: Use two-ended allocation by size, v2") That patch caused a regression on my system where the bottom of the screen flickers after my laptop goes thorough a suspend and resume. This is reproducible 100% of the time.

[PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-12 Thread Ross Zwisler
This patch reverts the changes made in this commit: deadcb36f49b (drm/radeon: Use two-ended allocation by size, v2) That patch caused a regression on my system where the bottom of the screen flickers after my laptop goes thorough a suspend and resume. This is reproducible 100% of the time.

Re: [PATCH] drm/radeon: Fix regression with suspend/resume

2015-02-12 Thread Michel Dänzer
On 13.02.2015 05:30, Ross Zwisler wrote: This patch reverts the changes made in this commit: deadcb36f49b (drm/radeon: Use two-ended allocation by size, v2) That patch caused a regression on my system where the bottom of the screen flickers after my laptop goes thorough a suspend and