Re: [RFC i-g-t 1/3] intel-gpu-top: Rename to intel-legacy-top

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Quoting Tvrtko Ursulin (2018-02-15 09:57:45)
> 
> On 15/02/2018 09:10, Petri Latvala wrote:
> > On Wed, Feb 14, 2018 at 06:52:05PM +0000, Tvrtko Ursulin wrote:
> >> From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
> >>
> >> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
> >> ---
> >>   tools/.gitignore                              | 2 +-
> >>   tools/Makefile.sources                        | 2 +-
> >>   tools/{intel_gpu_top.c => intel_legacy_top.c} | 0
> >>   tools/meson.build                             | 2 +-
> >>   4 files changed, 3 insertions(+), 3 deletions(-)
> >>   rename tools/{intel_gpu_top.c => intel_legacy_top.c} (100%)
> > 
> > 
> > If the old tool is to be kept, the name intel_legacy_top doesn't quite
> > capture what it does. A more important topic though is whether it
> > should be kept at all. What are the features in the old tool that your
> > rewrite doesn't have?
> 
> It's a bit different in target audience and capabilities I think.
> 
> The current one exposes what OA is, I assume at least, able to provide 
> today in a safe way. The new tool doesn't do any of that but just 
> provides basic, more end-user friendly, engine busyness and related stats.
> 
> It may be that when considering gpu-top work, intel-gpu-top rewrite 
> hasn't even got a place. Or it might have as a minimal, easy to use and 
> simple tool. TBD.

I liken it to classic "top", even when we have graphical system
monitors, for a quick assay of what's running, top is unbeatable.
I think each tool has its use. There may be future refactoring to a
single tool with different interfaces, but for now it looks to be a nice
little demonstration vehicle with some practical value.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux