search

martinkersner / gtop

CPU, GPU and memory viewer utilizing information provided by tegrastats
GNU General Public License v3.0
91 stars 49 forks source link

(TX2 Fix) Updating gtop to use command-line arguments #5

Open zlacelle opened 6 years ago

zlacelle commented 6 years ago

to specify the board type, not guess based on fields out of tegrastats. Added EMC plot.

martinkersner commented 6 years ago

Hi @zlacelle!

Awesome work! Could you check if it is working on TX2?

Cheers, Martin

zlacelle commented 6 years ago

@confusedrobot if you can send me a few lines of tegrastats output, I will run it through the code today and make sure it's good. I'll try to add in some sanity checks as well.

There are a few other changes I made in my repo which I'll also push, once I clean them up.

confusedrobot commented 6 years ago

Here's my tegrastats output: tegrastats.txt

@zlacelle

confusedrobot commented 6 years ago

@zlacelle let me know if there's anything else I can provide to help debug this.

zlacelle commented 6 years ago

@confusedrobot Just didn't have time today, I'll hopefully have some time to take a look tomorrow.

gustavz commented 6 years ago

Do you get it working till tomorrow? Would be super awesome and important for me!

zlacelle commented 6 years ago

I created a new pull request here: https://github.com/martinkersner/gtop/pull/6

I switched to what should be a more robust parsing method. @confusedrobot I checked your tegrastats output, and it matches mine, so it should work.

If it doesn't work, compile with debug flags and send a core dump.

@GustavZ If it doesn't work for you, send me your tegrastats output. Right now, the only testing I can do is on my TX2 running JetPack 3.2

gustavz commented 6 years ago

@zlacelle It works!! Thank you!

But would it be possible that the tool averages the usage over a range of 30 secs or something?

confusedrobot commented 6 years ago

@zlacelle sorry it took awhile to respond, I am going to try the new fix now.

martinkersner commented 5 years ago

Hello @confusedrobot, @gustavz and @zlacelle !

I am planning to add test suite for supported devices so new pull requests can be tested against it. For this I would need an example output from tegrastats. Could you provide example output for any device you have?

Thank you, Martin

zlacelle commented 5 years ago

@martinkersner Will do. I'll try to get it from a Xavier and a TX2, but might take a few days.

martinkersner commented 5 years ago

@zlacelle Great! Thank you.