Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/kthreads #446

Merged
merged 8 commits into from
Mar 20, 2023
Merged

Feature/kthreads #446

merged 8 commits into from
Mar 20, 2023

Conversation

cristiano-prato
Copy link
Contributor

I often need to remove kthreads from the output of top/ps commands, to only check what is running in userspace.
For this reason I would like an option to filter out them.
I also prefer to print them out surrounded by square brackets to easily eyeball them when not filtered out.

output of tools like ps usually put square brackets around kernel
threads to easily distinguish those from userspace processes
by default the output is unchanged, but this option allows the user to
hide kernel threads
@dalance
Copy link
Owner

dalance commented Mar 14, 2023

Looks good!
I'm wondering whether --no-kthreads option or configuration like below.

[display]
show_kthreads = false

How about it?

@cristiano-prato
Copy link
Contributor Author

I think it depends on how often you want to switch between the two options (with and without kthreads). If most of the time users are not interested in kthreads, having this in the configuration is probably the best option. 90% of the time I am not interested in kthreads for example, so this would work well for me.

@dalance
Copy link
Owner

dalance commented Mar 15, 2023

I think so too. Could you change to configuration?

@cristiano-prato
Copy link
Contributor Author

Sure!

@dalance
Copy link
Owner

dalance commented Mar 20, 2023

Thanks!
I'll merge this PR.

@dalance dalance merged commit aa72697 into dalance:master Mar 20, 2023
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.

2 participants