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

Possible crash and performance issues #1595

Closed
gyl30 opened this issue Sep 29, 2024 · 5 comments
Closed

Possible crash and performance issues #1595

gyl30 opened this issue Sep 29, 2024 · 5 comments
Labels

Comments

@gyl30
Copy link
Contributor

gyl30 commented Sep 29, 2024

Question

I was using PcapPlusPlus and noticed that my program was occasionally crashing, I looked at the core dump and it showed that internalCreateLogStream was sometimes returning null pointers, causing it to crash out on the next log output. In troubleshooting this problem, I modified the PCPP_LOG macro to block the logging. I found that the modification handled more traffic than before, before the modification it handled about 4.2Gbps of traffic per second, after the modification it could handle 4.8Gbps of traffic. Now my program uses the modified PCPP_LOG macro and no longer logs out and everything runs fine. After noticing that internalCreateLogStream would sometimes return a null pointer, I tried running my program with valgrind, which handles very little traffic because valgrind is very slow, and I didn't notice any unusual memory usage when running the program for a long time with valgrind.

This is my modified PCPP_LOG
image

Operating systems

Linux

@gyl30 gyl30 added the question label Sep 29, 2024
Repository owner deleted a comment Sep 29, 2024
@tigercosmos
Copy link
Collaborator

@gyl30 If you think your patch fixes the issue, could you open a PR? It will be easier to review the patch by the PR flow.

@gyl30
Copy link
Contributor Author

gyl30 commented Sep 30, 2024

No, no, no, I just blocked the log output and didn't do anything else. We can open this issue and see if others have encountered the same problem. Forgot to mention, I only observed this phenomenon when the traffic was relatively large. The submission of PcapPlusPlus I use is 4cf8ed4, and the system environment is CentOS 7.9

@egecetin
Copy link
Collaborator

@gyl30 It is a known issue with CentOS, which is currently not supported.

cc @seladb @clementperon

@gyl30
Copy link
Contributor Author

gyl30 commented Sep 30, 2024

I saw the issue #1200, but wasn't sure if it was the same reason, and since CentOS is now out of our support, I'm closing the issue

@gyl30 gyl30 closed this as completed Sep 30, 2024
@egecetin
Copy link
Collaborator

egecetin commented Sep 30, 2024

@gyl30 I forgot to mention it. Actually, there were random crashes in CI for Centos containers because of logger if i remember correctly. Probably your issue corresponds to the CI situation instead of #1200.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants