> > > On 03/06/2020 12.51, Lorenzo Bianconi wrote: > > > > > > > > > > Hmm, suspend again for 15s. > > > > > > > > > > https://gist.github.com/sknsean/911ae4ae2e74ebe1e7eca1405c645ff1 > > > > > > > > > > [ 105.520634] PM: suspend exit > > > > > [ 105.540206] ts_ref 1591097307780181385, ts 2861700000, sample_time > > > > > 1591097310641881385 > > > > > > > > > > [ 105.548416] ktime_get_real_ns: 1591097322928175385 > > > > > > > > > > CLOCK_REALTIME isn't ready in the resume function. I think we need to link > > > > > it to CLOCK_BOOTTIME (Which ticking under suspend) instead. > > > > > > > > With latest patch it seems to me the time reported is now monotonic so it seems > > > > correct. What is the clocktype you set? you can check it in: > > > > /sys/bus/iio/devices/iio:device<x>/current_timestamp_clock > > > > > > default: realtime > > > > > > In the dump above the ktime_get_real_ns(first fifo dump after suspend) - > > > ts_ref + ts is = 12,286294 sec... > > > > > > /Sean > > > > right. Could you please add to the dump log the ts_ref value in st_lsm6dsx_resume() > > just after iio_get_time_ns()? > > > Sure, > > [ 173.849649] ktime_get_real_ns: 1591097391339190269 > > [ 173.855244] ts_ref 1591097386868224810, ts 4488650000, sample_time > 1591097391356874810 > > [ 173.855272] ktime_get_real_ns: 1591097391344812894 > > [ 173.861256] PM: suspend devices took 0.080 seconds > > [ 173.875214] Disabling non-boot CPUs ... > [ 173.887482] sensor->ts_ref[1] = 1591097391377130644 > > [ 173.906546] PM: resume devices took 0.020 seconds > > [ 174.028152] OOM killer enabled. > [ 174.031331] Restarting tasks ... done. > [ 174.078636] PM: suspend exit > [ 174.127877] ts_ref 1591097391377130644, ts 4531750000, sample_time > 1591097395908880644 > > [ 174.136383] ktime_get_real_ns: 1591097405112347968 > > [ 174.145841] ts_ref 1591097391377130644, ts 4570200000, sample_time > 1591097395947330644 > > [ 174.154031] ktime_get_real_ns: 1591097405129995760 > > Missing 13,73 sec :( > > If the CLOCK_REALTIME was updated in resume(), I think the first patch > proposal was better. Can you please try to use CLOCK_BOOTTIME instead? Regards, Lorenzo > > /Sean
Attachment:
signature.asc
Description: PGP signature
