[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-21 Thread Thomas Hellstrom
On 11/20/2011 04:13 PM, Jerome Glisse wrote: > On Sun, Nov 20, 2011 at 4:30 AM, Thomas Hellstrom > wrote: > >> On 11/19/2011 11:54 PM, Jerome Glisse wrote: >> >> As mentioned previously, and in the discussion with Ben, the page tables >> would not need to be rebuilt on each CS. They would be

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-21 Thread Thomas Hellstrom
On 11/20/2011 04:13 PM, Jerome Glisse wrote: On Sun, Nov 20, 2011 at 4:30 AM, Thomas Hellstrom wrote: On 11/19/2011 11:54 PM, Jerome Glisse wrote: As mentioned previously, and in the discussion with Ben, the page tables would not need to be rebuilt on each CS. They would be rebuilt only on

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-20 Thread Thomas Hellstrom
On 11/19/2011 11:54 PM, Jerome Glisse wrote: > >> As mentioned previously, and in the discussion with Ben, the page tables >> would not need to be rebuilt on each CS. They would be rebuilt only on the >> first CS following a move_notify that caused a page table invalidation. >> >> move_notify: >> i

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-20 Thread Jerome Glisse
On Sun, Nov 20, 2011 at 4:30 AM, Thomas Hellstrom wrote: > On 11/19/2011 11:54 PM, Jerome Glisse wrote: > > As mentioned previously, and in the discussion with Ben, the page tables > would not need to be rebuilt on each CS. They would be rebuilt only on the > first CS following a move_notify that

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-20 Thread Jerome Glisse
On Sun, Nov 20, 2011 at 4:30 AM, Thomas Hellstrom wrote: > On 11/19/2011 11:54 PM, Jerome Glisse wrote: > > As mentioned previously, and in the discussion with Ben, the page tables > would not need to be rebuilt on each CS. They would be rebuilt only on the > first CS following a move_notify that

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-20 Thread Thomas Hellstrom
On 11/19/2011 11:54 PM, Jerome Glisse wrote: As mentioned previously, and in the discussion with Ben, the page tables would not need to be rebuilt on each CS. They would be rebuilt only on the first CS following a move_notify that caused a page table invalidation. move_notify: if (is_incompati

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-20 Thread Ben Skeggs
On Sat, 2011-11-19 at 11:07 +0100, Thomas Hellstrom wrote: > On 11/19/2011 01:26 AM, Ben Skeggs wrote: > > On Fri, 2011-11-18 at 23:48 +0100, Thomas Hellstrom wrote: > > > >> On 11/18/2011 06:26 PM, Ben Skeggs wrote: > >> > >>> On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote: >

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Thomas Hellstrom
On 11/19/2011 10:22 PM, Jerome Glisse wrote: > On Sat, Nov 19, 2011 at 4:01 PM, Thomas Hellstrom > wrote: > >> On 11/19/2011 09:37 PM, Jerome Glisse wrote: >> >>> On Sat, Nov 19, 2011 at 2:46 PM, Thomas Hellstrom >>> wrote: >>> >>> On 11/19/2011 07:11 PM, Jerome Glisse wr

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Thomas Hellstrom
On 11/19/2011 09:37 PM, Jerome Glisse wrote: > On Sat, Nov 19, 2011 at 2:46 PM, Thomas Hellstrom > wrote: > >> On 11/19/2011 07:11 PM, Jerome Glisse wrote: >> >> On Sat, Nov 19, 2011 at 12:00 PM, Thomas Hellstrom >> wrote: >> >> >> On 11/19/2011 03:53 PM, Ben Skeggs wrote: >> >> >> On Sat,

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Thomas Hellstrom
On 11/19/2011 07:11 PM, Jerome Glisse wrote: > On Sat, Nov 19, 2011 at 12:00 PM, Thomas Hellstrom > wrote: > >> On 11/19/2011 03:53 PM, Ben Skeggs wrote: >> >>> On Sat, 2011-11-19 at 11:07 +0100, Thomas Hellstrom wrote: >>> >>> On 11/19/2011 01:26 AM, Ben Skeggs wrote:

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Thomas Hellstrom
On 11/19/2011 03:53 PM, Ben Skeggs wrote: > On Sat, 2011-11-19 at 11:07 +0100, Thomas Hellstrom wrote: > >> On 11/19/2011 01:26 AM, Ben Skeggs wrote: >> >>> On Fri, 2011-11-18 at 23:48 +0100, Thomas Hellstrom wrote: >>> >>> On 11/18/2011 06:26 PM, Ben Skeggs wrote:

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Jerome Glisse
On Sat, Nov 19, 2011 at 5:37 PM, Thomas Hellstrom wrote: > On 11/19/2011 10:22 PM, Jerome Glisse wrote: >> >> On Sat, Nov 19, 2011 at 4:01 PM, Thomas Hellstrom >> ?wrote: >> >>> >>> On 11/19/2011 09:37 PM, Jerome Glisse wrote: >>> On Sat, Nov 19, 2011 at 2:46 PM, Thomas Hellstrom ?

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Jerome Glisse
On Sat, Nov 19, 2011 at 4:01 PM, Thomas Hellstrom wrote: > On 11/19/2011 09:37 PM, Jerome Glisse wrote: >> >> On Sat, Nov 19, 2011 at 2:46 PM, Thomas Hellstrom >> ?wrote: >> >>> >>> On 11/19/2011 07:11 PM, Jerome Glisse wrote: >>> >>> On Sat, Nov 19, 2011 at 12:00 PM, Thomas Hellstrom >>> ?wrote

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Jerome Glisse
On Sat, Nov 19, 2011 at 2:46 PM, Thomas Hellstrom wrote: > On 11/19/2011 07:11 PM, Jerome Glisse wrote: > > On Sat, Nov 19, 2011 at 12:00 PM, Thomas Hellstrom > wrote: > > > On 11/19/2011 03:53 PM, Ben Skeggs wrote: > > > On Sat, 2011-11-19 at 11:07 +0100, Thomas Hellstrom wrote: > > > > On 11/1

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Jerome Glisse
On Sat, Nov 19, 2011 at 5:37 PM, Thomas Hellstrom wrote: > On 11/19/2011 10:22 PM, Jerome Glisse wrote: >> >> On Sat, Nov 19, 2011 at 4:01 PM, Thomas Hellstrom >>  wrote: >> >>> >>> On 11/19/2011 09:37 PM, Jerome Glisse wrote: >>> On Sat, Nov 19, 2011 at 2:46 PM, Thomas Hellstrom  w

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Thomas Hellstrom
On 11/19/2011 10:22 PM, Jerome Glisse wrote: On Sat, Nov 19, 2011 at 4:01 PM, Thomas Hellstrom wrote: On 11/19/2011 09:37 PM, Jerome Glisse wrote: On Sat, Nov 19, 2011 at 2:46 PM, Thomas Hellstrom wrote: On 11/19/2011 07:11 PM, Jerome Glisse wrote: On Sat, Nov 19, 2011 a

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Jerome Glisse
On Sat, Nov 19, 2011 at 4:01 PM, Thomas Hellstrom wrote: > On 11/19/2011 09:37 PM, Jerome Glisse wrote: >> >> On Sat, Nov 19, 2011 at 2:46 PM, Thomas Hellstrom >>  wrote: >> >>> >>> On 11/19/2011 07:11 PM, Jerome Glisse wrote: >>> >>> On Sat, Nov 19, 2011 at 12:00 PM, Thomas Hellstrom >>>  wrote:

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Jerome Glisse
On Sat, Nov 19, 2011 at 12:00 PM, Thomas Hellstrom wrote: > On 11/19/2011 03:53 PM, Ben Skeggs wrote: >> >> On Sat, 2011-11-19 at 11:07 +0100, Thomas Hellstrom wrote: >> >>> >>> On 11/19/2011 01:26 AM, Ben Skeggs wrote: >>> On Fri, 2011-11-18 at 23:48 +0100, Thomas Hellstrom wrote:

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Thomas Hellstrom
On 11/19/2011 09:37 PM, Jerome Glisse wrote: On Sat, Nov 19, 2011 at 2:46 PM, Thomas Hellstrom wrote: On 11/19/2011 07:11 PM, Jerome Glisse wrote: On Sat, Nov 19, 2011 at 12:00 PM, Thomas Hellstrom wrote: On 11/19/2011 03:53 PM, Ben Skeggs wrote: On Sat, 2011-11-19 at 11:07 +0100, Th

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Jerome Glisse
On Sat, Nov 19, 2011 at 2:46 PM, Thomas Hellstrom wrote: > On 11/19/2011 07:11 PM, Jerome Glisse wrote: > > On Sat, Nov 19, 2011 at 12:00 PM, Thomas Hellstrom > wrote: > > > On 11/19/2011 03:53 PM, Ben Skeggs wrote: > > > On Sat, 2011-11-19 at 11:07 +0100, Thomas Hellstrom wrote: > > > > On 11/19

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Thomas Hellstrom
On 11/19/2011 07:11 PM, Jerome Glisse wrote: On Sat, Nov 19, 2011 at 12:00 PM, Thomas Hellstrom wrote: On 11/19/2011 03:53 PM, Ben Skeggs wrote: On Sat, 2011-11-19 at 11:07 +0100, Thomas Hellstrom wrote: On 11/19/2011 01:26 AM, Ben Skeggs wrote: On Fri, 2011-11

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Thomas Hellstrom
On 11/19/2011 01:26 AM, Ben Skeggs wrote: > On Fri, 2011-11-18 at 23:48 +0100, Thomas Hellstrom wrote: > >> On 11/18/2011 06:26 PM, Ben Skeggs wrote: >> >>> On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote: >>> >>> On 11/18/2011 02:15 PM, Ben Skeggs wrote:

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Ben Skeggs
On Fri, 2011-11-18 at 23:48 +0100, Thomas Hellstrom wrote: > On 11/18/2011 06:26 PM, Ben Skeggs wrote: > > On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote: > > > >> On 11/18/2011 02:15 PM, Ben Skeggs wrote: > >> > >>> On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: >

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Jerome Glisse
On Sat, Nov 19, 2011 at 12:00 PM, Thomas Hellstrom wrote: > On 11/19/2011 03:53 PM, Ben Skeggs wrote: >> >> On Sat, 2011-11-19 at 11:07 +0100, Thomas Hellstrom wrote: >> >>> >>> On 11/19/2011 01:26 AM, Ben Skeggs wrote: >>> On Fri, 2011-11-18 at 23:48 +0100, Thomas Hellstrom wrote:

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Thomas Hellstrom
On 11/19/2011 03:53 PM, Ben Skeggs wrote: On Sat, 2011-11-19 at 11:07 +0100, Thomas Hellstrom wrote: On 11/19/2011 01:26 AM, Ben Skeggs wrote: On Fri, 2011-11-18 at 23:48 +0100, Thomas Hellstrom wrote: On 11/18/2011 06:26 PM, Ben Skeggs wrote: On Fri, 2011-11-18

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Ben Skeggs
On Sat, 2011-11-19 at 11:07 +0100, Thomas Hellstrom wrote: > On 11/19/2011 01:26 AM, Ben Skeggs wrote: > > On Fri, 2011-11-18 at 23:48 +0100, Thomas Hellstrom wrote: > > > >> On 11/18/2011 06:26 PM, Ben Skeggs wrote: > >> > >>> On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote: >

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Ben Skeggs
On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote: > On 11/18/2011 02:15 PM, Ben Skeggs wrote: > > On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: > > > >> Jerome, > >> > >> I don't like this change for the following reasons > >> > > -snip- > > > > > >>> One can tak

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-19 Thread Thomas Hellstrom
On 11/19/2011 01:26 AM, Ben Skeggs wrote: On Fri, 2011-11-18 at 23:48 +0100, Thomas Hellstrom wrote: On 11/18/2011 06:26 PM, Ben Skeggs wrote: On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote: On 11/18/2011 02:15 PM, Ben Skeggs wrote: On Fri, 2011-11-18

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Thomas Hellstrom
On 11/18/2011 06:26 PM, Ben Skeggs wrote: > On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote: > >> On 11/18/2011 02:15 PM, Ben Skeggs wrote: >> >>> On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: >>> >>> Jerome, I don't like this change for the f

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Ben Skeggs
On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: > Jerome, > > I don't like this change for the following reasons -snip- > > > > One can take advantage of move notify callback but, there are > > corner case where bind/unbind might be call without move notify > > being call (in error pat

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Jerome Glisse
On Fri, Nov 18, 2011 at 6:25 PM, Jerome Glisse wrote: > On Fri, Nov 18, 2011 at 06:14:02PM -0500, Jerome Glisse wrote: >> On Fri, Nov 18, 2011 at 11:48:58PM +0100, Thomas Hellstrom wrote: >> > On 11/18/2011 06:26 PM, Ben Skeggs wrote: >> > >On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Jerome Glisse
On Fri, Nov 18, 2011 at 06:14:02PM -0500, Jerome Glisse wrote: > On Fri, Nov 18, 2011 at 11:48:58PM +0100, Thomas Hellstrom wrote: > > On 11/18/2011 06:26 PM, Ben Skeggs wrote: > > >On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote: > > >>On 11/18/2011 02:15 PM, Ben Skeggs wrote: > > >>>On

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Jerome Glisse
On Fri, Nov 18, 2011 at 11:48:58PM +0100, Thomas Hellstrom wrote: > On 11/18/2011 06:26 PM, Ben Skeggs wrote: > >On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote: > >>On 11/18/2011 02:15 PM, Ben Skeggs wrote: > >>>On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: > >>> > Jerom

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Ben Skeggs
On Fri, 2011-11-18 at 23:48 +0100, Thomas Hellstrom wrote: > On 11/18/2011 06:26 PM, Ben Skeggs wrote: > > On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote: > > > >> On 11/18/2011 02:15 PM, Ben Skeggs wrote: > >> > >>> On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: >

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Thomas Hellstrom
On 11/18/2011 03:56 PM, Jerome Glisse wrote: > On Fri, Nov 18, 2011 at 03:30:03PM +0100, Thomas Hellstrom wrote: > >> On 11/18/2011 02:15 PM, Ben Skeggs wrote: >> >>> On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: >>> Jerome, I don't like this change for

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Jerome Glisse
On Fri, Nov 18, 2011 at 6:25 PM, Jerome Glisse wrote: > On Fri, Nov 18, 2011 at 06:14:02PM -0500, Jerome Glisse wrote: >> On Fri, Nov 18, 2011 at 11:48:58PM +0100, Thomas Hellstrom wrote: >> > On 11/18/2011 06:26 PM, Ben Skeggs wrote: >> > >On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Thomas Hellstrom
On 11/18/2011 02:15 PM, Ben Skeggs wrote: > On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: > >> Jerome, >> >> I don't like this change for the following reasons >> > -snip- > > >>> One can take advantage of move notify callback but, there are >>> corner case where bind/unb

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Jerome Glisse
On Fri, Nov 18, 2011 at 06:14:02PM -0500, Jerome Glisse wrote: > On Fri, Nov 18, 2011 at 11:48:58PM +0100, Thomas Hellstrom wrote: > > On 11/18/2011 06:26 PM, Ben Skeggs wrote: > > >On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote: > > >>On 11/18/2011 02:15 PM, Ben Skeggs wrote: > > >>>On

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Jerome Glisse
On Fri, Nov 18, 2011 at 11:48:58PM +0100, Thomas Hellstrom wrote: > On 11/18/2011 06:26 PM, Ben Skeggs wrote: > >On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote: > >>On 11/18/2011 02:15 PM, Ben Skeggs wrote: > >>>On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: > >>> > Jerom

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Thomas Hellstrom
On 11/18/2011 06:26 PM, Ben Skeggs wrote: On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote: On 11/18/2011 02:15 PM, Ben Skeggs wrote: On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: Jerome, I don't like this change for the following reasons -

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Jerome Glisse
On Fri, Nov 18, 2011 at 04:06:05PM +0100, Thomas Hellstrom wrote: > On 11/18/2011 03:56 PM, Jerome Glisse wrote: > >On Fri, Nov 18, 2011 at 03:30:03PM +0100, Thomas Hellstrom wrote: > >>On 11/18/2011 02:15 PM, Ben Skeggs wrote: > >>>On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: > J

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Jerome Glisse
On Fri, Nov 18, 2011 at 03:30:03PM +0100, Thomas Hellstrom wrote: > On 11/18/2011 02:15 PM, Ben Skeggs wrote: > >On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: > >>Jerome, > >> > >>I don't like this change for the following reasons > >-snip- > > > >>>One can take advantage of move notif

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Ben Skeggs
On Fri, 2011-11-18 at 15:30 +0100, Thomas Hellstrom wrote: > On 11/18/2011 02:15 PM, Ben Skeggs wrote: > > On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: > > > >> Jerome, > >> > >> I don't like this change for the following reasons > >> > > -snip- > > > > > >>> One can tak

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Thomas Hellstrom
Jerome, I don't like this change for the following reasons 1) This is really a layer violation. It's like passing a state tracker object down to the pipe driver i Gallium, so that eventually the winsys can access it. 2) TTM, as you say, doesn't really care about GPU virtual maps. It cares abo

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Jerome Glisse
On Fri, Nov 18, 2011 at 04:06:05PM +0100, Thomas Hellstrom wrote: > On 11/18/2011 03:56 PM, Jerome Glisse wrote: > >On Fri, Nov 18, 2011 at 03:30:03PM +0100, Thomas Hellstrom wrote: > >>On 11/18/2011 02:15 PM, Ben Skeggs wrote: > >>>On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: > J

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Thomas Hellstrom
On 11/18/2011 03:56 PM, Jerome Glisse wrote: On Fri, Nov 18, 2011 at 03:30:03PM +0100, Thomas Hellstrom wrote: On 11/18/2011 02:15 PM, Ben Skeggs wrote: On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: Jerome, I don't like this change for the following reasons

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Jerome Glisse
On Fri, Nov 18, 2011 at 03:30:03PM +0100, Thomas Hellstrom wrote: > On 11/18/2011 02:15 PM, Ben Skeggs wrote: > >On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: > >>Jerome, > >> > >>I don't like this change for the following reasons > >-snip- > > > >>>One can take advantage of move notif

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Thomas Hellstrom
On 11/18/2011 02:15 PM, Ben Skeggs wrote: On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: Jerome, I don't like this change for the following reasons -snip- One can take advantage of move notify callback but, there are corner case where bind/unbind might be call witho

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Ben Skeggs
On Fri, 2011-11-18 at 08:57 +0100, Thomas Hellstrom wrote: > Jerome, > > I don't like this change for the following reasons -snip- > > > > One can take advantage of move notify callback but, there are > > corner case where bind/unbind might be call without move notify > > being call (in error pat

Re: [PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-18 Thread Thomas Hellstrom
Jerome, I don't like this change for the following reasons 1) This is really a layer violation. It's like passing a state tracker object down to the pipe driver i Gallium, so that eventually the winsys can access it. 2) TTM, as you say, doesn't really care about GPU virtual maps. It cares ab

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-17 Thread j.gli...@gmail.com
From: Jerome Glisse ttm_tt is always associated with a buffer object, pass it in bind/unbind callback to make life easier for driver. Main objective is for driver supporting virtual address space. For such driver each buffer object can be several virtual address space but ttm is unaware of this.

[PATCH] drm/ttm: pass buffer object for bind/unbind callback

2011-11-17 Thread j . glisse
From: Jerome Glisse ttm_tt is always associated with a buffer object, pass it in bind/unbind callback to make life easier for driver. Main objective is for driver supporting virtual address space. For such driver each buffer object can be several virtual address space but ttm is unaware of this.