Re: [UMR] Add GPU_POWER (v3) + refactored cleanup

2017-02-11 Thread Kai Wasserbäch
Hey Tom, Tom St Denis wrote on 11.02.2017 20:05: > On 02/11/2017 01:57 PM, Kai Wasserbäch wrote: >> Tom St Denis wrote on 11.02.2017 18:52: >>> I split out the deletion of the unused bits in --top. >> >> thank you so much for bearing with me (and Edward, though I can't speak for >> him, >> obvious

Re: [UMR] Add GPU_POWER (v3) + refactored cleanup

2017-02-11 Thread Tom St Denis
On 02/11/2017 01:57 PM, Kai Wasserbäch wrote: Hey Tom, Tom St Denis wrote on 11.02.2017 18:52: I split out the deletion of the unused bits in --top. thank you so much for bearing with me (and Edward, though I can't speak for him, obviously)! You can still have my R-b. ;-) Cheers, Kai No wo

Re: [UMR] Add GPU_POWER (v3) + refactored cleanup

2017-02-11 Thread Kai Wasserbäch
Hey Tom, Tom St Denis wrote on 11.02.2017 18:52: > I split out the deletion of the unused bits in --top. thank you so much for bearing with me (and Edward, though I can't speak for him, obviously)! You can still have my R-b. ;-) Cheers, Kai signature.asc Description: OpenPGP digital signature

Re: [PATCH] Add GPU_POWER sensors (v2)

2017-02-11 Thread Tom St Denis
On 02/11/2017 08:29 AM, Kai Wasserbäch wrote: Hey Tom, Tom St Denis wrote on 11.02.2017 12:26: Add the ability to sample GPU_POWER sensors. Because the sensors have a high latency we read them from a background thread which means we've added the requirement for pthreads. Signed-off-by: Tom St

[PATCH 1/2] Remove unused bits from --top

2017-02-11 Thread Tom St Denis
Signed-off-by: Tom St Denis --- src/app/top.c | 5 - 1 file changed, 5 deletions(-) diff --git a/src/app/top.c b/src/app/top.c index b081515a5b40..d9a474d78cf7 100644 --- a/src/app/top.c +++ b/src/app/top.c @@ -171,19 +171,14 @@ static struct umr_bitfield stat_uvd_pgfsm7_bits[] = { static s

[PATCH 2/2] Add GPU_POWER sensors (v3)

2017-02-11 Thread Tom St Denis
Add the ability to sample GPU_POWER sensors. Because the sensors have a high latency we read them from a background thread which means we've added the requirement for pthreads. Signed-off-by: Tom St Denis (v2) Cleaned up CMake around pthreads (v3) Update readme, factor out some deletions, clean

[UMR] Add GPU_POWER (v3) + refactored cleanup

2017-02-11 Thread Tom St Denis
I split out the deletion of the unused bits in --top. ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Re: [PATCH] Add GPU_POWER sensors (v2)

2017-02-11 Thread Kai Wasserbäch
Hey Tom, Tom St Denis wrote on 11.02.2017 12:26: > Add the ability to sample GPU_POWER sensors. Because > the sensors have a high latency we read them from a background > thread which means we've added the requirement for pthreads. > > Signed-off-by: Tom St Denis > > (v2) Cleaned up CMake aroun

Re: [PATCH] Add GPU_POWER sensors

2017-02-11 Thread Tom St Denis
On 11/02/17 05:56 AM, Kai Wasserbäch wrote: Hey Tom, Tom St Denis wrote on 11.02.2017 02:02: On 02/10/2017 07:25 PM, Edward O'Callaghan wrote: Hey Tom, On 02/11/2017 05:10 AM, Tom St Denis wrote: Add the ability to sample GPU_POWER sensors. Because the sensors have a high latency we read the

[PATCH] Add GPU_POWER sensors (v2)

2017-02-11 Thread Tom St Denis
Add the ability to sample GPU_POWER sensors. Because the sensors have a high latency we read them from a background thread which means we've added the requirement for pthreads. Signed-off-by: Tom St Denis (v2) Cleaned up CMake around pthreads --- CMakeLists.txt | 4 +++ README

Re: [PATCH] Add GPU_POWER sensors

2017-02-11 Thread Kai Wasserbäch
Hey Tom, Tom St Denis wrote on 11.02.2017 02:02: > On 02/10/2017 07:25 PM, Edward O'Callaghan wrote: >> Hey Tom, >> >> On 02/11/2017 05:10 AM, Tom St Denis wrote: >>> Add the ability to sample GPU_POWER sensors. Because >>> the sensors have a high latency we read them from a background >>> thread