search

bristlemouth / bm_protocol

Primary Bristlemouth firmware repository
https://www.bristlemouth.org/
Apache License 2.0
10 stars 7 forks source link

Fix/toplogy node position log #96

Closed victorsowa12 closed 6 months ago

victorsowa12 commented 6 months ago

This will only update the node-position in IND logs when we have a large gap between individual readings (indicating a potential unplugging event and replugging in a different order) or when we have the first reading. This change is to reduce the amount we have to take the node_list mutex and potentially wait for it, while maintaining a consistent node_position in the IND logs.

victorsowa12 commented 6 months ago

I will soak overnight and see if any 255 node positions show up and then mark it ready for review.

victorsowa12 commented 6 months ago

There were no instances of a node position of 255 in the overnight soak!