search

commaai / msgq

MSGQ: A lock free single producer multi consumer message queue
184 stars 251 forks source link

fix logMonoTime, as c++ use boottime #572

Closed Smartype closed 9 months ago

Smartype commented 9 months ago

This causes error in locationd on my Google Pixel 4. as calibrationd use mono time, which differs from others.

Observation timestamp is older than the max rewind threshold of the filter.

adeebshihadeh commented 9 months ago

I assume this is because your Pixel will actually sleep? This will have to be fixed in quite a few more places.

Smartype commented 9 months ago

I assume this is because your Pixel will actually sleep? This will have to be fixed in quite a few more places.

Yes. As it is stock android 13.

Kernel rebuilt to enable USB host.

Rooted.

adeebshihadeh commented 9 months ago

I'm supportive of fixing this, but I don't want to merge this change alone. It doesn't completely fix the problem and there is nothing preventing this from breaking again.