Re: [Nouveau] [PATCH v2] drm/nouveau: Add getparam for current PTIMER time.

2010-05-24 Thread Ben Skeggs
On Mon, 2010-05-24 at 07:50 +, Marcin Kościelnicki wrote: > > On Sun, 2010-05-23 at 11:36 +, Marcin Kościelnicki wrote: > > > This will be useful for computing GPU-CPU latency, including > > > GL_ARB_timer_query extension. > > > > Looks fine, just a quick question as to whether this is nece

Re: [Nouveau] [PATCH v2] drm/nouveau: Add getparam for current PTIMER time.

2010-05-24 Thread Marcin Kościelnicki
> On Sun, 2010-05-23 at 11:36 +, Marcin Kościelnicki wrote: > > This will be useful for computing GPU-CPU latency, including > > GL_ARB_timer_query extension. > > Looks fine, just a quick question as to whether this is necessary? I > seem to recall that on nv40, the QUERY methods actually retu

Re: [Nouveau] [PATCH v2] drm/nouveau: Add getparam for current PTIMER time.

2010-05-23 Thread Ben Skeggs
On Sun, 2010-05-23 at 11:36 +, Marcin Kościelnicki wrote: > This will be useful for computing GPU-CPU latency, including > GL_ARB_timer_query extension. Looks fine, just a quick question as to whether this is necessary? I seem to recall that on nv40, the QUERY methods actually return timestamp

[Nouveau] [PATCH v2] drm/nouveau: Add getparam for current PTIMER time.

2010-05-23 Thread Marcin Kościelnicki
This will be useful for computing GPU-CPU latency, including GL_ARB_timer_query extension. Signed-off-by: Marcin Kościelnicki --- drivers/gpu/drm/nouveau/nouveau_state.c |3 +++ include/drm/nouveau_drm.h |1 + 2 files changed, 4 insertions(+), 0 deletions(-) diff --git a/d