-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add metric to cacth the double free error in mmap allocator #8648
Conversation
✅ Deploy Preview for meta-velox canceled.
|
@xiaoxmeng has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
eaf3afe
to
0187320
Compare
@xiaoxmeng has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
velox/common/base/Counters.cpp
Outdated
@@ -143,6 +143,10 @@ void registerVeloxMetrics() { | |||
DEFINE_METRIC( | |||
kMetricMemoryPoolReservationLeakBytes, facebook::velox::StatType::SUM); | |||
|
|||
// Tracks the count of double frees in memory allocator. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: how about: Tracks the count of double frees in memory allocator, indicating the possibility of buffer ownership issues when a buffer is freed more than once.
@xiaoxmeng has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
…incubator#8648) Summary: Currently we only have a warning log for double free in mmap allocator and we shall also add a metric for production alert. Reviewed By: bikramSingh91 Differential Revision: D53341067 Pulled By: xiaoxmeng
This pull request was exported from Phabricator. Differential Revision: D53341067 |
@xiaoxmeng merged this pull request in c4cd265. |
Conbench analyzed the 1 benchmark run on commit There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
…incubator#8648) Summary: Currently we only have a warning log for double free in mmap allocator and we shall also add a metric for production alert. Pull Request resolved: facebookincubator#8648 Reviewed By: bikramSingh91 Differential Revision: D53341067 Pulled By: xiaoxmeng fbshipit-source-id: 3d0979a1621ebae1180b3d59cd98345fd0c4dff8
…incubator#8648) Summary: Currently we only have a warning log for double free in mmap allocator and we shall also add a metric for production alert. Pull Request resolved: facebookincubator#8648 Reviewed By: bikramSingh91 Differential Revision: D53341067 Pulled By: xiaoxmeng fbshipit-source-id: 3d0979a1621ebae1180b3d59cd98345fd0c4dff8
Currently we only have a warning log for double free in mmap allocator
and we shall also add a metric for production alert.