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

t/Test2/modules/API/Instance.t leaks a shm resource in blead perl #679

Closed
tonycoz opened this issue Jun 7, 2016 · 0 comments
Closed

t/Test2/modules/API/Instance.t leaks a shm resource in blead perl #679

tonycoz opened this issue Jun 7, 2016 · 0 comments

Comments

@tonycoz
Copy link

tonycoz commented Jun 7, 2016

Found in bleadperl, where this is making daily build testers leak shm blocks:

tony@mars:.../perl/t$ ipcs -m

------ Shared Memory Segments --------
key shmid owner perms bytes nattch status
0x0052e6a9 0 postgres 600 56 5
0x0052ea91 32769 postgres 600 29278208 4
0x0052e2c1 65538 postgres 600 32342016 4

tony@mars:.../perl/t$ ./perl TEST ../cpan/Test-Simple/t/Test2/modules/API/Instance.t
t/../cpan/Test-Simple/t/Test2/modules/API/Instance ... ok
All tests successful.
Elapsed: 1 sec
u=0.01 s=0.00 cu=0.17 cs=0.02 scripts=1 tests=79
tony@mars:.../perl/t$ ipcs -m

------ Shared Memory Segments --------
key shmid owner perms bytes nattch status
0x0052e6a9 0 postgres 600 56 5
0x0052ea91 32769 postgres 600 29278208 4
0x0052e2c1 65538 postgres 600 32342016 4
0x00000000 15466500 tony 666 64 0

Adding:

$one->ipc_free_shm;

to the end of the last block in Instance.t prevents the leak but may not the the correct solution.

p5p pushed a commit to Perl/perl5 that referenced this issue Jun 7, 2016
@exodist exodist closed this as completed in 1168c17 Jun 7, 2016
exodist added a commit that referenced this issue Jun 7, 2016
    - Make sure enabling culling/shm sets pid and tid (Fix #679)
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

No branches or pull requests

1 participant