ulyssegerkens / philosophers

A simple resolution to the Dining philosophers problem to learn threads usage in C. 42 School project.
0 stars 0 forks source link

Helgrind errors #1

Open h-akbas opened 4 months ago

h-akbas commented 4 months ago

First of all, I really like this repo. I tried to solve issues with Helgrind errors. I run this code on 42 clusters and my own linux machine. 42 machine returns hundreds errors when run with Helgrind and DRD but none of those errors return when I run this on linux version of Valgrind. Do you also use MacOS at your campus? Maybe that's why. There may be an OS-specific false positive errors. There is no error with Thread sanitizer as well.

ulyssegerkens commented 4 months ago

I have heard of similar bugs that depend on the OS. If I remember correctly, there are hellgrind errors, but they only indicate '???' regarding the location of the error. The program was made on MacOS. Unfortunately, I don't have much time and can't investigate further on this. You'll probably find some topics about it on Slack. If you find the source of the problem, don't hesitate to let me know, and I'll include it in the repo's README.