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

Increase buffer size for reading process groups #306

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

skosachiov
Copy link
Contributor

@skosachiov skosachiov commented May 30, 2024

Increase buffer size for reading process groups from /proc/pid/status. In a corporate environment, the number of groups can be quite large. Let's assume that 8192 will allow us to read 512 groups.

@skosachiov
Copy link
Contributor Author

cat /proc/23548/status
...
Uid: 1532536188 1532536188 1532536188 1532536188
Gid: 53800514 53800514 53800514 53800514
FDSize: 128
Groups: 107 114 123 124 64055 53800514 53972353 53981290 148406938 148408882 148415556 148418815 148419489 148419944 148420627 148421600 148424264 148438895 148451737 148451738 148455828 148456759 148468146 148471238 148471244 148500965 148505765 148514592 148514593 148514594 148516968 148516970 148518402 148518403 148524933 148532538 148535274 148535371 148574653 148591098 148591686 593203385 593207143 593207522 593215512 593227237 593228243 593250151 593273328 593273395 593274144 593281502 593281503 593303214 593303374 593324381 593343147 593345568 593355738 593361660 593373815 593376875 593379143 593388230 1447427059 1447438405 1447452845 1447452846 1447452847 1447452848 1447476528 1447482021 1447492176 1447495332 1447495333 1447502055 1447502056 1447502734 1447504793 1469609140 1469663436 1469730655 1469731436 1469737665 1532427666 1532439582 1532460823 1532500173 1532501444 1532520915 1532521629 1532521630 1532521634 1532558983 1532559732 1532567763 1532568554 1532568556 1532572585 1532583861 1532586853 1532590321 1532594776 1532597887
NStgid: 23548 339 1
NSpid: 23548 339 1
...

@radosroka
Copy link
Member

We are using #define instead of const in other parts of the code could you follow it?

@skosachiov
Copy link
Contributor Author

Added constant #define.

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