Re: [PATCH] drm/ttm: remove fence_lock

2012-10-18 Thread Jerome Glisse
On Thu, Oct 18, 2012 at 06:43:40PM +0200, Thomas Hellstrom wrote: > On 10/18/2012 04:45 PM, Maarten Lankhorst wrote: > >Op 18-10-12 13:55, Thomas Hellstrom schreef: > >>On 10/18/2012 01:38 PM, Maarten Lankhorst wrote: > >>>Op 18-10-12 13:02, Thomas Hellstrom schreef: > On 10/18/2012 10:37 AM, M

Re: [PATCH] drm/ttm: remove fence_lock

2012-10-18 Thread Thomas Hellstrom
On 10/18/2012 04:45 PM, Maarten Lankhorst wrote: Op 18-10-12 13:55, Thomas Hellstrom schreef: On 10/18/2012 01:38 PM, Maarten Lankhorst wrote: Op 18-10-12 13:02, Thomas Hellstrom schreef: On 10/18/2012 10:37 AM, Maarten Lankhorst wrote: Hey, Op 18-10-12 09:59, Thomas Hellstrom schreef: On 1

Re: [PATCH] drm/ttm: remove fence_lock

2012-10-18 Thread Maarten Lankhorst
Op 18-10-12 13:55, Thomas Hellstrom schreef: > On 10/18/2012 01:38 PM, Maarten Lankhorst wrote: >> Op 18-10-12 13:02, Thomas Hellstrom schreef: >>> On 10/18/2012 10:37 AM, Maarten Lankhorst wrote: Hey, Op 18-10-12 09:59, Thomas Hellstrom schreef: > > On 10/18/2012 09:28 AM, T

Re: [PATCH] drm/ttm: remove fence_lock

2012-10-18 Thread Thomas Hellstrom
On 10/18/2012 01:38 PM, Maarten Lankhorst wrote: Op 18-10-12 13:02, Thomas Hellstrom schreef: On 10/18/2012 10:37 AM, Maarten Lankhorst wrote: Hey, Op 18-10-12 09:59, Thomas Hellstrom schreef: On 10/18/2012 09:28 AM, Thomas Hellstrom wrote: Hi, Maarten, As you know I have been having my do

Re: [PATCH] drm/ttm: remove fence_lock

2012-10-18 Thread Maarten Lankhorst
Op 18-10-12 13:02, Thomas Hellstrom schreef: > On 10/18/2012 10:37 AM, Maarten Lankhorst wrote: >> Hey, >> >> Op 18-10-12 09:59, Thomas Hellstrom schreef: >>> >>> >>> On 10/18/2012 09:28 AM, Thomas Hellstrom wrote: Hi, Maarten, As you know I have been having my doubts about this chan

Re: [PATCH] drm/ttm: remove fence_lock

2012-10-18 Thread Thomas Hellstrom
On 10/18/2012 10:37 AM, Maarten Lankhorst wrote: Hey, Op 18-10-12 09:59, Thomas Hellstrom schreef: On 10/18/2012 09:28 AM, Thomas Hellstrom wrote: Hi, Maarten, As you know I have been having my doubts about this change. To me it seems insane to be forced to read the fence pointer under a re

Re: [PATCH] drm/ttm: remove fence_lock

2012-10-18 Thread Maarten Lankhorst
Hey, Op 18-10-12 09:59, Thomas Hellstrom schreef: > > > > On 10/18/2012 09:28 AM, Thomas Hellstrom wrote: >> Hi, Maarten, >> >> As you know I have been having my doubts about this change. >> To me it seems insane to be forced to read the fence pointer under a >> reserved lock, simply because when

Re: [PATCH] drm/ttm: remove fence_lock

2012-10-18 Thread Thomas Hellstrom
On 10/18/2012 09:28 AM, Thomas Hellstrom wrote: Hi, Maarten, As you know I have been having my doubts about this change. To me it seems insane to be forced to read the fence pointer under a reserved lock, simply because when you take the reserve lock, another process may have it and there is

Re: [PATCH] drm/ttm: remove fence_lock

2012-10-18 Thread Thomas Hellstrom
Hi, Maarten, As you know I have been having my doubts about this change. To me it seems insane to be forced to read the fence pointer under a reserved lock, simply because when you take the reserve lock, another process may have it and there is a substantial chance that that process will also be