Re: [PATCH 0/6] drm: trace: Introduce drm_trace() and instrument drm_atomic.c

2019-11-08 Thread Daniel Vetter
On Fri, Nov 08, 2019 at 11:23:27AM -0500, Sean Paul wrote: > On Fri, Nov 08, 2019 at 09:59:19AM +0100, Daniel Vetter wrote: > > On Fri, Nov 08, 2019 at 09:46:48AM +0100, Daniel Vetter wrote: > > > On Fri, Nov 08, 2019 at 10:16:59AM +0200, Pekka Paalanen wrote: > > > > On Thu, 7 Nov 2019 16:02:59

Re: [PATCH 0/6] drm: trace: Introduce drm_trace() and instrument drm_atomic.c

2019-11-08 Thread Sean Paul
On Fri, Nov 08, 2019 at 09:59:19AM +0100, Daniel Vetter wrote: > On Fri, Nov 08, 2019 at 09:46:48AM +0100, Daniel Vetter wrote: > > On Fri, Nov 08, 2019 at 10:16:59AM +0200, Pekka Paalanen wrote: > > > On Thu, 7 Nov 2019 16:02:59 -0500 > > > Sean Paul wrote: > > > > > > > From: Sean Paul > > >

Re: [PATCH 0/6] drm: trace: Introduce drm_trace() and instrument drm_atomic.c

2019-11-08 Thread Pekka Paalanen
On Fri, 8 Nov 2019 09:50:30 +0100 Daniel Vetter wrote: > On Fri, Nov 08, 2019 at 10:16:59AM +0200, Pekka Paalanen wrote: > > Is it ok to build userspace to rely on these trace events during normal > > operations, e.g. for continuous adjustment of timings/timers? > > Aside discussion on this:

Re: [PATCH 0/6] drm: trace: Introduce drm_trace() and instrument drm_atomic.c

2019-11-08 Thread Daniel Vetter
On Fri, Nov 08, 2019 at 09:46:48AM +0100, Daniel Vetter wrote: > On Fri, Nov 08, 2019 at 10:16:59AM +0200, Pekka Paalanen wrote: > > On Thu, 7 Nov 2019 16:02:59 -0500 > > Sean Paul wrote: > > > > > From: Sean Paul > > > > > > Hey all, > > > I'm back with another trace events patchset. My

Re: [PATCH 0/6] drm: trace: Introduce drm_trace() and instrument drm_atomic.c

2019-11-08 Thread Daniel Vetter
On Fri, Nov 08, 2019 at 10:16:59AM +0200, Pekka Paalanen wrote: > Is it ok to build userspace to rely on these trace events during normal > operations, e.g. for continuous adjustment of timings/timers? Aside discussion on this: If we add this (I think userspace might want some information about

Re: [PATCH 0/6] drm: trace: Introduce drm_trace() and instrument drm_atomic.c

2019-11-08 Thread Daniel Vetter
On Fri, Nov 08, 2019 at 10:16:59AM +0200, Pekka Paalanen wrote: > On Thu, 7 Nov 2019 16:02:59 -0500 > Sean Paul wrote: > > > From: Sean Paul > > > > Hey all, > > I'm back with another trace events patchset. My first attempt [1] went > > better than expected, with enthusiasm for the idea and

Re: [PATCH 0/6] drm: trace: Introduce drm_trace() and instrument drm_atomic.c

2019-11-08 Thread Pekka Paalanen
On Thu, 7 Nov 2019 16:02:59 -0500 Sean Paul wrote: > From: Sean Paul > > Hey all, > I'm back with another trace events patchset. My first attempt [1] went > better than expected, with enthusiasm for the idea and distain for the > implementation. > > As promised, I went through and added