Breakage in "track dev_mapping in more robust and flexible way"

2012-10-29 Thread Thomas Hellstrom
On 10/26/2012 03:14 PM, Ilija Hadzic wrote: > > > On Fri, 26 Oct 2012, Thomas Hellstrom wrote: > >> Hi, >> >> On 10/25/2012 11:27 PM, Ilija Hadzic wrote: >>> >>> Can you give the attached patch a whirl and let me know if it fixes >>> the problem? >>> >>> As I indicated in my previous note, vmwgfx

Re: Breakage in "track dev_mapping in more robust and flexible way"

2012-10-29 Thread Thomas Hellstrom
On 10/26/2012 03:14 PM, Ilija Hadzic wrote: On Fri, 26 Oct 2012, Thomas Hellstrom wrote: Hi, On 10/25/2012 11:27 PM, Ilija Hadzic wrote: Can you give the attached patch a whirl and let me know if it fixes the problem? As I indicated in my previous note, vmwgfx should be the only affect

Breakage in "track dev_mapping in more robust and flexible way"

2012-10-26 Thread Thomas Hellstrom
Hi, On 10/25/2012 11:27 PM, Ilija Hadzic wrote: > > Can you give the attached patch a whirl and let me know if it fixes > the problem? > > As I indicated in my previous note, vmwgfx should be the only affected > driver because it looks at dev_mapping in the open hook (others do it > when they c

Breakage in "track dev_mapping in more robust and flexible way"

2012-10-26 Thread Ilija Hadzic
On Fri, 26 Oct 2012, Thomas Hellstrom wrote: > Hi, > > On 10/25/2012 11:27 PM, Ilija Hadzic wrote: >> >> Can you give the attached patch a whirl and let me know if it fixes the >> problem? >> >> As I indicated in my previous note, vmwgfx should be the only affected >> driver because it looks

Re: Breakage in "track dev_mapping in more robust and flexible way"

2012-10-26 Thread Ilija Hadzic
On Fri, 26 Oct 2012, Thomas Hellstrom wrote: Hi, On 10/25/2012 11:27 PM, Ilija Hadzic wrote: Can you give the attached patch a whirl and let me know if it fixes the problem? As I indicated in my previous note, vmwgfx should be the only affected driver because it looks at dev_mapping in

Re: Breakage in "track dev_mapping in more robust and flexible way"

2012-10-26 Thread Thomas Hellstrom
Hi, On 10/25/2012 11:27 PM, Ilija Hadzic wrote: Can you give the attached patch a whirl and let me know if it fixes the problem? As I indicated in my previous note, vmwgfx should be the only affected driver because it looks at dev_mapping in the open hook (others do it when they create an

Breakage in "track dev_mapping in more robust and flexible way"

2012-10-25 Thread Thomas Hellström
On 10/25/12 7:12 PM, Ilija Hadzic wrote: > On Thu, Oct 25, 2012 at 11:10 AM, Thomas Hellstr?m > wrote: >> On 10/25/12 4:41 PM, Jerome Glisse wrote: >>> On Thu, Oct 25, 2012 at 04:02:25PM +0200, Thomas Hellstrom wrote: Hi, This commit From 949c4a34afacfe800fc442afac117aba

Breakage in "track dev_mapping in more robust and flexible way"

2012-10-25 Thread Thomas Hellström
On 10/25/12 4:41 PM, Jerome Glisse wrote: > On Thu, Oct 25, 2012 at 04:02:25PM +0200, Thomas Hellstrom wrote: >> Hi, >> >> This commit >> >> From 949c4a34afacfe800fc442afac117aba15284962 Mon Sep 17 00:00:00 2001 >> From: Ilija Hadzic >> Date: Tue, 15 May 2012 16:40:10 -0400 >> Subject: [PATCH] dr

Breakage in "track dev_mapping in more robust and flexible way"

2012-10-25 Thread Ilija Hadzic
Can you give the attached patch a whirl and let me know if it fixes the problem? As I indicated in my previous note, vmwgfx should be the only affected driver because it looks at dev_mapping in the open hook (others do it when they create an object, so they are not affected). I'll probably re

Breakage in "track dev_mapping in more robust and flexible way"

2012-10-25 Thread Thomas Hellstrom
Hi, This commit From 949c4a34afacfe800fc442afac117aba15284962 Mon Sep 17 00:00:00 2001 From: Ilija Hadzic Date: Tue, 15 May 2012 16:40:10 -0400 Subject: [PATCH] drm: track dev_mapping in more robust and flexible way Setting dev_mapping (pointer to the address_space structure used for memory ma

Re: Breakage in "track dev_mapping in more robust and flexible way"

2012-10-25 Thread Ilija Hadzic
Can you give the attached patch a whirl and let me know if it fixes the problem? As I indicated in my previous note, vmwgfx should be the only affected driver because it looks at dev_mapping in the open hook (others do it when they create an object, so they are not affected). I'll probably

Breakage in "track dev_mapping in more robust and flexible way"

2012-10-25 Thread Ilija Hadzic
On Thu, Oct 25, 2012 at 11:10 AM, Thomas Hellstr?m wrote: > On 10/25/12 4:41 PM, Jerome Glisse wrote: >> >> On Thu, Oct 25, 2012 at 04:02:25PM +0200, Thomas Hellstrom wrote: >>> >>> Hi, >>> >>> This commit >>> >>> From 949c4a34afacfe800fc442afac117aba15284962 Mon Sep 17 00:00:00 2001 >>> From: Il

Breakage in "track dev_mapping in more robust and flexible way"

2012-10-25 Thread Ilija Hadzic
Or could the driver that causes the problem be vmwgfx ? I now looked at the code and I notice that indeed vmwgfx sets its private dev_mapping in the open hook, while all other TTM-based drivers (radeon and nouveau) do it when they create an object. -- Ilija

Re: Breakage in "track dev_mapping in more robust and flexible way"

2012-10-25 Thread Thomas Hellström
On 10/25/12 7:12 PM, Ilija Hadzic wrote: On Thu, Oct 25, 2012 at 11:10 AM, Thomas Hellström wrote: On 10/25/12 4:41 PM, Jerome Glisse wrote: On Thu, Oct 25, 2012 at 04:02:25PM +0200, Thomas Hellstrom wrote: Hi, This commit From 949c4a34afacfe800fc442afac117aba15284962 Mon Sep 17 00:00:00

Breakage in "track dev_mapping in more robust and flexible way"

2012-10-25 Thread Jerome Glisse
On Thu, Oct 25, 2012 at 04:02:25PM +0200, Thomas Hellstrom wrote: > Hi, > > This commit > > From 949c4a34afacfe800fc442afac117aba15284962 Mon Sep 17 00:00:00 2001 > From: Ilija Hadzic > Date: Tue, 15 May 2012 16:40:10 -0400 > Subject: [PATCH] drm: track dev_mapping in more robust and flexible wa

Re: Breakage in "track dev_mapping in more robust and flexible way"

2012-10-25 Thread Ilija Hadzic
Or could the driver that causes the problem be vmwgfx ? I now looked at the code and I notice that indeed vmwgfx sets its private dev_mapping in the open hook, while all other TTM-based drivers (radeon and nouveau) do it when they create an object. -- Ilija

Re: Breakage in "track dev_mapping in more robust and flexible way"

2012-10-25 Thread Ilija Hadzic
On Thu, Oct 25, 2012 at 11:10 AM, Thomas Hellström wrote: > On 10/25/12 4:41 PM, Jerome Glisse wrote: >> >> On Thu, Oct 25, 2012 at 04:02:25PM +0200, Thomas Hellstrom wrote: >>> >>> Hi, >>> >>> This commit >>> >>> From 949c4a34afacfe800fc442afac117aba15284962 Mon Sep 17 00:00:00 2001 >>> From: Il

Re: Breakage in "track dev_mapping in more robust and flexible way"

2012-10-25 Thread Thomas Hellström
On 10/25/12 4:41 PM, Jerome Glisse wrote: On Thu, Oct 25, 2012 at 04:02:25PM +0200, Thomas Hellstrom wrote: Hi, This commit From 949c4a34afacfe800fc442afac117aba15284962 Mon Sep 17 00:00:00 2001 From: Ilija Hadzic Date: Tue, 15 May 2012 16:40:10 -0400 Subject: [PATCH] drm: track dev_mapping

Re: Breakage in "track dev_mapping in more robust and flexible way"

2012-10-25 Thread Jerome Glisse
On Thu, Oct 25, 2012 at 04:02:25PM +0200, Thomas Hellstrom wrote: > Hi, > > This commit > > From 949c4a34afacfe800fc442afac117aba15284962 Mon Sep 17 00:00:00 2001 > From: Ilija Hadzic > Date: Tue, 15 May 2012 16:40:10 -0400 > Subject: [PATCH] drm: track dev_mapping in more robust and flexible wa

Breakage in "track dev_mapping in more robust and flexible way"

2012-10-25 Thread Thomas Hellstrom
Hi, This commit From 949c4a34afacfe800fc442afac117aba15284962 Mon Sep 17 00:00:00 2001 From: Ilija Hadzic Date: Tue, 15 May 2012 16:40:10 -0400 Subject: [PATCH] drm: track dev_mapping in more robust and flexible way Setting dev_mapping (pointer to the address_space structure used for memory ma