Skip to content

Add hugetop command#392

Closed
eduardorittner wants to merge 0 commit into
uutils:mainfrom
eduardorittner:main
Closed

Add hugetop command#392
eduardorittner wants to merge 0 commit into
uutils:mainfrom
eduardorittner:main

Conversation

@eduardorittner

Copy link
Copy Markdown

The hugetop command displays system-wide hugepage informantion as well as per-process hugepage information.

It has the flags --human (for human-readable size output), --numa (to display info per numa core), --delay <N> (for display refresh) and --once (to only display once then exit). For now none of these flags are supported, and we currently only print once and then exit.

The info shown is also very minimal, but it's the same one which is displayed by the original hugetop. Since we get the info from /proc/meminfo and /sys/kernel/mm/hugepages/hugepages-<size>/... this is not portable at all and only works on linux.

@eduardorittner

Copy link
Copy Markdown
Author

I wanted to reuse some functionality from top, specifically the header formatting with time and things, and for that I made the header module public, if you have any suggestions they are welcome.

@cakebaker cakebaker mentioned this pull request Apr 29, 2025
Comment thread Cargo.toml Outdated
Comment thread Cargo.toml Outdated
Comment thread src/uu/hugetop/hugetop.md Outdated
Comment thread src/uu/hugetop/Cargo.toml Outdated

@Krysztal112233 Krysztal112233 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I apologize for the late review of this PR. Since Debian's procps-ng does not currently include hugetop, I will continue the review later. I'm very sorry!

@Krysztal112233

Copy link
Copy Markdown
Collaborator

And since there were some conflicts due to the commits that were previously merged, could you please help fix them?

@Franklin-Qi

Copy link
Copy Markdown
Contributor

I apologize for the late review of this PR. Since Debian's procps-ng does not currently include hugetop, I will continue the review later. I'm very sorry!

I checked and found that the hugetop command is only available in procps-ng version 4.0.5. As of now, the latest version of procps on Debian Daily is 4.0.4-9+b1.

Krysztal112233 added a commit to Franklin-Qi/procps that referenced this pull request Mar 26, 2026
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.

3 participants