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

pmem2_map_from_existing/TEST[0-2] failed memcheck #5594

Closed
szadam opened this issue Apr 21, 2023 · 1 comment
Closed

pmem2_map_from_existing/TEST[0-2] failed memcheck #5594

szadam opened this issue Apr 21, 2023 · 1 comment
Assignees
Labels
libpmem2 libpmem- and libpmem2-related miniasync async related issue Type: Bug A previously unknown bug in PMDK
Milestone

Comments

@szadam
Copy link

szadam commented Apr 21, 2023

ISSUE:

Environment Information

  • PMDK package version(s): 50693c2
  • OS(es) version(s): openSUSE15_3
  • ndctl version(s): 71.1
  • kernel version(s): 5.3.18-150300.59.106

Please provide a reproduction of the bug:

[pmdk_thread_1] <libpmem2>: <1> [out.c:215 out_init] src version: 1.12.1+git172.g50693c2c2
[pmdk_thread_1] <libpmem2>: <1> [out.c:223 out_init] compiled with support for Valgrind pmemcheck
[pmdk_thread_1] <libpmem2>: <1> [out.c:228 out_init] compiled with support for Valgrind helgrind
[pmdk_thread_1] <libpmem2>: <1> [out.c:233 out_init] compiled with support for Valgrind memcheck
[pmdk_thread_1] <libpmem2>: <1> [out.c:238 out_init] compiled with support for Valgrind drd
[pmdk_thread_1] <libpmem2>: <1> [out.c:243 out_init] compiled with support for shutdown state
[pmdk_thread_1] <libpmem2>: <1> [out.c:248 out_init] compiled with libndctl 63+
[pmdk_thread_1] <libpmem2>: <3> [libpmem2.c:29 libpmem2_init] 
[pmdk_thread_1] <libpmem2>: <3> [mover.c:181 mover_new] map 0x731cda0, vdm 0x731ce28
[pmdk_thread_1] <libpmem2>: <3> [mover.c:181 mover_new] map 0x731d010, vdm 0x731d098
[pmdk_thread_1] <libpmem2>: <1> [map.c:289 pmem2_map_from_existing] Provided mapping(addr 0xfff0 len 255) is already registered by libpmem2
[pmdk_thread_1] <libpmem2>: <3> [map_posix.c:580 pmem2_map_delete] map_ptr 0x1ffefff380
[pmdk_thread_1] <libpmem2>: <3> [libpmem2.c:44 libpmem2_fini] 
[pmdk_thread_1] Last 30 lines of /home/jenkins-slave/workspace/PMDK-unittests-linux-py/pmdk_1/src/test/pmem2_map_from_existing/memcheck1.log below (whole file has 39 lines):
[pmdk_thread_1] ==61806==   total heap usage: 2,554 allocs, 2,549 frees, 177,559 bytes allocated
[pmdk_thread_1] ==61806== 
[pmdk_thread_1] ==61806== 40 bytes in 1 blocks are definitely lost in loss record 1 of 5
[pmdk_thread_1] ==61806==    at 0x4C32735: malloc (vg_replace_malloc.c:393)
[pmdk_thread_1] ==61806==    by 0x453724: _Malloc (alloc.c:72)
[pmdk_thread_1] ==61806==    by 0x40EC3F: pmem2_malloc (pmem2_utils.c:21)
[pmdk_thread_1] ==61806==    by 0x40F3F1: pmem2_source_from_fd (source_posix.c:69)
[pmdk_thread_1] ==61806==    by 0x404D28: test_mapping_overlap_bottom (pmem2_map_from_existing.c:56)
[pmdk_thread_1] ==61806==    by 0x404AC6: TEST_CASE_PROCESS (unittest.h:700)
[pmdk_thread_1] ==61806==    by 0x40534E: main (pmem2_map_from_existing.c:182)
[pmdk_thread_1] ==61806== 
[pmdk_thread_1] ==61806== 144 (64 direct, 80 indirect) bytes in 1 blocks are definitely lost in loss record 5 of 5
[pmdk_thread_1] ==61806==    at 0x4C32735: malloc (vg_replace_malloc.c:393)
[pmdk_thread_1] ==61806==    by 0x453724: _Malloc (alloc.c:72)
[pmdk_thread_1] ==61806==    by 0x40EC3F: pmem2_malloc (pmem2_utils.c:21)
[pmdk_thread_1] ==61806==    by 0x408B4C: mover_new (mover.c:183)
[pmdk_thread_1] ==61806==    by 0x40A01D: pmem2_map_from_existing (map.c:274)
[pmdk_thread_1] ==61806==    by 0x404D84: test_mapping_overlap_bottom (pmem2_map_from_existing.c:59)
[pmdk_thread_1] ==61806==    by 0x404AC6: TEST_CASE_PROCESS (unittest.h:700)
[pmdk_thread_1] ==61806==    by 0x40534E: main (pmem2_map_from_existing.c:182)
[pmdk_thread_1] ==61806== 
[pmdk_thread_1] ==61806== LEAK SUMMARY:
[pmdk_thread_1] ==61806==    definitely lost: 104 bytes in 2 blocks
[pmdk_thread_1] ==61806==    indirectly lost: 80 bytes in 1 blocks
[pmdk_thread_1] ==61806==      possibly lost: 0 bytes in 0 blocks
[pmdk_thread_1] ==61806==    still reachable: 0 bytes in 0 blocks
[pmdk_thread_1] ==61806==         suppressed: 112 bytes in 2 blocks
[pmdk_thread_1] ==61806== 
[pmdk_thread_1] ==61806== For lists of detected and suppressed errors, rerun with: -s
[pmdk_thread_1] ==61806== ERROR SUMMARY: 2 errors from 2 contexts (suppressed: 0 from 0)

How often bug is revealed: (always, often, rare):

always

Actual behavior:

Tests failed.

Expected behavior:

Tests should pass.

Details

Additional information about Priority and Help Requested:

@szadam szadam added the Type: Bug A previously unknown bug in PMDK label Apr 21, 2023
@janekmi janekmi added libpmem2 libpmem- and libpmem2-related Exposure: Medium miniasync async related issue labels Apr 24, 2023
@grom72 grom72 modified the milestones: 1.14, 1.13 on GHA May 15, 2023
@grom72 grom72 self-assigned this May 19, 2023
grom72 added a commit to grom72/pmdk that referenced this issue May 19, 2023
Disable tests TEST[0-2] for memcheck until pmem#5594 is fixed.

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@intel.com>
@grom72 grom72 added the CI: Disabled Temporarily disabled from testing label May 19, 2023
grom72 added a commit to grom72/pmdk that referenced this issue May 19, 2023
Disable tests TEST[0-2] for memcheck until pmem#5594 is fixed.

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@intel.com>
grom72 added a commit to grom72/pmdk that referenced this issue May 19, 2023
Disable tests pmem2_map_from_existing TEST[0-2] for memcheck
until pmem#5594 is fixed.

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@intel.com>
grom72 added a commit to grom72/pmdk that referenced this issue May 19, 2023
Disable tests pmem2_map_from_existing TEST[0-2] for memcheck
until pmem#5594 is fixed.

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@intel.com>
janekmi added a commit that referenced this issue May 19, 2023
test: disable pmem2_map_from_existing TEST[0-2] until #5594 is fixed.
grom72 added a commit to grom72/pmdk that referenced this issue May 23, 2023
Fixes: pmem#5594

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@intel.com>
@grom72 grom72 removed the CI: Disabled Temporarily disabled from testing label May 23, 2023
grom72 added a commit to grom72/pmdk that referenced this issue May 23, 2023
Fixes: pmem#5594

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@intel.com>
@grom72
Copy link
Contributor

grom72 commented May 25, 2023

@grom72 grom72 closed this as completed May 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libpmem2 libpmem- and libpmem2-related miniasync async related issue Type: Bug A previously unknown bug in PMDK
Projects
None yet
Development

No branches or pull requests

3 participants