Skip to content

create patches to improve jtop feature - #175

Open
limeng-linux wants to merge 3 commits into
OE4T:masterfrom
limeng-linux:master
Open

create patches to improve jtop feature#175
limeng-linux wants to merge 3 commits into
OE4T:masterfrom
limeng-linux:master

Conversation

@limeng-linux

Copy link
Copy Markdown
Contributor

No description provided.

@ichergui

Copy link
Copy Markdown
Member

Thanks @limeng-linux for the PR.

@ichergui

Copy link
Copy Markdown
Member

FYI. The official version that supports L4T R36.5.0 will be available next week.
I assume you tested these changes on Orin platform right ? If so, could you please confirm.

@limeng-linux

limeng-linux commented Mar 12, 2026

Copy link
Copy Markdown
Contributor Author

yes. I tested jop command with NVIDIA Orin AGX platform.
without the 0001 patch, jtop command can not work.
root@p3737-0000-p3701-0005:# jtop
The jtop.service is not active. Please run:
systemctl restart jtop.service
root@p3737-0000-p3701-0005:
# systemctl status jtop.service
● jtop.service - jtop service
Loaded: loaded (/usr/lib/systemd/system/jtop.service; enabled; preset: enabled)
Active: activating (auto-restart) (Result: exit-code) since Thu 2026-03-12 02:32:44 UTC; 5s ago
Invocation: 8fa37a0eac214cd3aca681cc656f0c79
Process: 1987 ExecStart=/usr/bin/jtop --force (code=exited, status=1/FAILURE)
Main PID: 1987 (code=exited, status=1/FAILURE)
Mem peak: 35.5M
CPU: 579ms
root@p3737-0000-p3701-0005:~#

without the 0002 patch, jtop can not display correct version at the top-left of screen.

thanks,
LImeng

@limeng-linux

Copy link
Copy Markdown
Contributor Author

BTW, you mean that The official version is NVIDIA yocto release(36.5.), not NVIDAI Jetson Linux 36.5. Is it right?

thanks,
Limeng

@madisongh

Copy link
Copy Markdown
Member

@limeng-linux I don't know what @ichergui means by "official version," but the latest code in the source repo appears to be up-to-date with L4T/JetPack versions. It doesn't look like it has the Python 3.14 compatibility fix, though.

I also don't see any recent releases tagged in the repo, and, rather concerningly, the latest version available from pypi (4.3.2) is a year old and doesn't have a corresponding tag in the source repo. So I'm not sure what state this package is in.

@ichergui

Copy link
Copy Markdown
Member

@limeng-linux @madisongh
By the official version. I meant the upcoming release of jetson-stats
I discussed with the maintainer and he mentioned that he will provide new version next week that supports L4T R36.5.0.

@mdcedgx

mdcedgx commented Mar 17, 2026

Copy link
Copy Markdown

I've also contributed two PRs upstream to jetson_stats for Python 3.14 compatibility and Thor tegrastats support, of which the Python 3.14 one is related to your fix:

Since the maintainer hasn't published to PyPI in a year, I've been writing my own recipe that pulls directly from the source repo. Shouldn't we consider moving away from python3-jetson-stats_4.3.1.bb in favor of something like this?

# recipes-devtools/python/python3-jetson-stats_7.1.0.bb
SRC_URI += "\
    git://github.com/rbonghi/jetson_stats.git;branch=master;protocol=https \
    ..."
SRCREV = "4a43d735dd45a1eab1b5ffd880cd40ee11f92e2b"

@madisongh

Copy link
Copy Markdown
Member

@mdcedgx That sounds reasonable to me. @ichergui What do you think? I'm still not seeing any activity in either pypi or even the source repo that looks anything like preparation for a new release.

@ichergui

Copy link
Copy Markdown
Member

@mdcedgx @madisongh it is reasonable.
@mdcedgx @limeng-linux Please update your branch then I will move forward with the PR review then merge.

@limeng-linux

limeng-linux commented Mar 24, 2026

Copy link
Copy Markdown
Contributor Author

With the latest code in https://github.com/rbonghi/jetson_stats, it has support 36.5.0 version, So my second patch can be ignored. @mdcedgx please create a PR to submit your patch firstly, and then I will check whether my first patch is still necessary after jetson_stats has been updated to latest code.

thanks,
Limeng

Create a patch to fix Python 3.14 multiprocessing compatibility

Signed-off-by: Meng Li <Meng.Li@windriver.com>
Create a patch to add L4T 36.5.0 support for jtop feature.

Signed-off-by: Meng Li <Meng.Li@windriver.com>
@limeng-linux limeng-linux reopened this Mar 24, 2026
@limeng-linux

Copy link
Copy Markdown
Contributor Author

Is there any update? How can I process this pull request, keep waiting, or close it?

thanks,
Limeng

@ichergui

Copy link
Copy Markdown
Member

Is there any update? How can I process this pull request, keep waiting, or close it?

thanks,

Limeng

Hey @limeng-linux
I was expecting an update to 7.1.5
So rebase your branch then update the jetson_stats to 7.1.5 and push your changes
I will review then merge if all good.

@limeng-linux

Copy link
Copy Markdown
Contributor Author

OK, got it. Because this ideal is raised up by @mdcedgx, I didn't upgrade jetson_stats to 7.1.5.
I will do it in later, and improve my patch.

thanks,
Limeng

@limeng-linux

Copy link
Copy Markdown
Contributor Author

@mdcedgx
I found out this patch (rbonghi/jetson_stats#834) is not compatible with NVIDIA Orin SoC, with this patch, GPU can not be detected, and only display black screen with just the top bar on HDMI monitor. After revert this patch, jetson_stats 7.1.5 works fine on NVIDIA Orin SoC.

thanks,
Limeng

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants