Skip to content
This repository was archived by the owner on Feb 4, 2025. It is now read-only.

Conversation

@TomekIdczak97
Copy link

@TomekIdczak97 TomekIdczak97 commented Jul 17, 2019

Simple slab metrics unit test for pmem: test_metrics_delete_basic

Fail message:

Assertion 'm1_buf == m2_buf' failed: m1_buf == "STATS item_alloc 1", m2_buf == "STATS item_alloc 0"


This change is Reviewable

@michalbiesek
Copy link

item_alloc is one of stats that is only incremented so this inconsistency is expected.
After reset when slabs/items are processed by recreate method item_alloc is incremented only if item was inserted (when it is linked)

We should wait for upstream repository maintainers to decide how to handle this situation.

@TomekIdczak97 TomekIdczak97 force-pushed the test_metrics_delete_basic branch 2 times, most recently from be9e255 to 21a5dd0 Compare July 22, 2019 09:35
@TomekIdczak97
Copy link
Author

TomekIdczak97 commented Jul 22, 2019

Failure - assertion message:
Saved:

STATS item_alloc 1
STATS item_dealloc 1
STATS item_link 1
STATS item_unlink 1

After restart:

STATS item_alloc 0
STATS item_dealloc 0
STATS item_link 0
STATS item_unlink 0

@TomekIdczak97 TomekIdczak97 force-pushed the test_metrics_delete_basic branch from 21a5dd0 to c1e0837 Compare July 24, 2019 13:55
@TomekIdczak97 TomekIdczak97 force-pushed the test_metrics_delete_basic branch 2 times, most recently from 0e2db64 to 0032f5d Compare August 12, 2019 06:42
@TomekIdczak97 TomekIdczak97 force-pushed the test_metrics_delete_basic branch 3 times, most recently from 5db9d0f to d468066 Compare August 20, 2019 09:09
@TomekIdczak97 TomekIdczak97 force-pushed the test_metrics_delete_basic branch from d468066 to 2ee7f1e Compare August 20, 2019 12:35
@TomekIdczak97 TomekIdczak97 force-pushed the test_metrics_delete_basic branch from 2ee7f1e to 9627aba Compare August 20, 2019 15:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants