starpu.suppr 4.0 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156
  1. {
  2. don't care about cache hit stats
  3. Helgrind:Race
  4. fun:_starpu_msi_cache_hit
  5. ...
  6. }
  7. {
  8. don't care about cache miss stats
  9. Helgrind:Race
  10. fun:_starpu_msi_cache_miss
  11. ...
  12. }
  13. {
  14. known race, but not problematic in practice, see comment in _starpu_tag_clear
  15. Helgrind:LockOrder
  16. ...
  17. fun:_starpu_tag_free
  18. fun:_starpu_htbl_clear_tags
  19. ...
  20. fun:_starpu_tag_clear
  21. fun:starpu_shutdown
  22. ...
  23. }
  24. {
  25. There is actually no race on current_mode, because the mode can not change unexpectedly, until _starpu_notify_data_dependencies() is called further down. Valgrind can not know about such software rwlock.
  26. Helgrind:Race
  27. fun:_starpu_release_data_on_node
  28. fun:_starpu_push_task_output
  29. ...
  30. }
  31. {
  32. We do not care about races on profiling statistics
  33. Helgrind:Race
  34. fun:_starpu_worker_get_status
  35. fun:_starpu_worker_reset_profiling_info_with_lock
  36. ...
  37. }
  38. {
  39. This is racy, but since we'll always put the same values, this is not a problem.
  40. Helgrind:Race
  41. fun:_starpu_codelet_check_deprecated_fields
  42. ...
  43. }
  44. {
  45. This is racy, but we don't care, it's only a statistic
  46. Helgrind:Race
  47. fun:starpu_task_nsubmitted
  48. ...
  49. }
  50. {
  51. This is racy, but we don't care, it's only a statistic
  52. Helgrind:Race
  53. fun:starpu_task_nready
  54. ...
  55. }
  56. {
  57. This is racy, but we don't care, it's only a statistic
  58. Helgrind:Race
  59. fun:_starpu_bus_update_profiling_info
  60. ...
  61. }
  62. {
  63. fscanf error
  64. Memcheck:Cond
  65. ...
  66. fun:fscanf
  67. fun:_starpu_load_bus_performance_files
  68. ...
  69. }
  70. {
  71. mc / handle locking order. We always take handle before mc, except when we tidy memory, where we use trylock, but helgrind doesn't handle that :/ https://bugs.kde.org/show_bug.cgi?id=243232
  72. Helgrind:LockOrder
  73. ...
  74. fun:_starpu_spin_trylock
  75. fun:lock_all_subtree
  76. ...
  77. }
  78. {
  79. mc / handle locking order2
  80. Helgrind:LockOrder
  81. ...
  82. fun:_starpu_spin_lock
  83. fun:_starpu_memchunk_recently_used
  84. ...
  85. }
  86. {
  87. mc / handle locking order3
  88. Helgrind:LockOrder
  89. ...
  90. fun:_starpu_spin_lock
  91. fun:_starpu_request_mem_chunk_removal
  92. ...
  93. }
  94. {
  95. mc / handle locking order4
  96. Helgrind:LockOrder
  97. ...
  98. fun:_starpu_spin_lock
  99. fun:_starpu_allocate_interface
  100. ...
  101. }
  102. {
  103. TODO1: This is temporary. It perhaps does not pose problem because only the worker takes this mutex. Fixing this will require changing the scheduler interface, so that the schedulers themselves take the scheduling lock, not the caller. Filter it out for now, so we can see other races more easily.
  104. Helgrind:LockOrder
  105. fun:pthread_mutex_lock
  106. fun:starpu_pthread_mutex_lock
  107. fun:simple_worker_pull_task
  108. ...
  109. }
  110. {
  111. TODO1: This is temporary. It perhaps does not pose problem because only the worker takes this mutex. Fixing this will require changing the scheduler interface, so that the schedulers themselves take the scheduling lock, not the caller. Filter it out for now, so we can see other races more easily.
  112. Helgrind:LockOrder
  113. fun:pthread_mutex_lock
  114. fun:starpu_pthread_mutex_lock
  115. fun:_starpu_sched_component_worker_lock_scheduling
  116. fun:simple_worker_pull_task
  117. ...
  118. }
  119. {
  120. TODO2: This is temporary. It perhaps does not pose problem because only the worker takes this mutex. Fixing this will require changing the scheduler interface, so that the schedulers themselves take the scheduling lock, not the caller. Filter it out for now, so we can see other races more easily.
  121. Helgrind:LockOrder
  122. fun:pthread_mutex_lock
  123. fun:starpu_pthread_mutex_lock
  124. fun:_starpu_sched_component_lock_worker
  125. fun:simple_worker_pull_task
  126. ...
  127. }
  128. {
  129. TODO2: This is temporary. It perhaps does not pose problem because only the worker takes this mutex. Fixing this will require changing the scheduler interface, so that the schedulers themselves take the scheduling lock, not the caller. Filter it out for now, so we can see other races more easily.
  130. Helgrind:LockOrder
  131. fun:pthread_mutex_lock
  132. fun:starpu_pthread_mutex_lock
  133. fun:_starpu_sched_component_lock_worker
  134. fun:_starpu_sched_component_worker_lock_scheduling
  135. fun:simple_worker_pull_task
  136. ...
  137. }