ivanseidel / ivanseidel/ArduinoThread
Using memory address as ThreadID sometimes gives different threads the same address
- Vorherrschende Sprache
- C++
- Sterne
- 974
- Forks
- 197
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
In my program, I've created 3 threads and then added them all to a controller thread. However, one thread was never executed (I had print-statements that should have run) and after some debugging I noticed that the size of the controller thread showed 2, when it should have been 3. I did not use thread names. I am only using lambda functions as callback functions for the threads.
Looking at the thread-controllers add function, I saw that a thread is not added when the controller already has a thread with the same ID. I printed the thread-ids of all my threads and it turned out that the 1st and the 3rd thread got the same ID, so the 3rd thread was never added. But this should not be possible as different threads should get different ids?
When I added Serial.println() before and Serial.println(controller.size()) after adding the 3rd thread to the controller, all 3 threads got different IDs. I can not explain this behaviour.
Thus, my conclusion is that using memory address as ThreadID feels unpredictable. Maybe thread id could be substituted by a random number instead? (I did this as a workaround).
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Beginne mit der add-Funktion von thread-controllers und der ThreadID-Generierung, die Speicheradressen verwendet. Reproduziere den Fall mit drei auf Lambdas basierenden Threads und vergleiche ihre IDs sowie die Größe des Controllers mit und ohne Serial.println()-Aufrufe. Fertig ist die Arbeit, wenn unterschiedliche Threads beibehalten werden und der Controller alle drei konsistent meldet.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- arduino, cpp
- Bereich
- embedded-iot
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100