linux-kernel - Re: [PATCH v5] [PATCH v5] pps: Add elapsed realtime timestamping

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [day] [month] [year] [list]

Date:   Mon, 10 Apr 2023 17:51:27 -0700
From:   Alex Komrakov <alexander.komrakov@...adcom.com>
To:     Greg KH <gregkh@...uxfoundation.org>
Cc:     linux-kernel@...r.kernel.org, giometti@...eenne.com
Subject: Re: [PATCH v5] [PATCH v5] pps: Add elapsed realtime timestamping

 greg k-h> Where is the Android code that needs this?  Have a pointer
to it anywhere?
 [AK]  https://cs.android.com/android/platform/superproject/+/master:cts/tests/location/location_gnss/src/android/location/cts/gnss/GnssLocationUpdateIntervalTest.java;drc=4788a776bc579841b0093c8f29215a1ce0686403;l=218]
 that requires 100ms accuracy. But you may challenge that 100ms
accuracy wouldn't require PPS.

greg k-h> My bot will remind you that you dropped the version information off :(
[AK] Done in [PATCH v6]:

greg k-h> That should be "ppsX" right?
 [AK]Done in [PATCH v6]:

 greg k-h> Why are your new entries above the main pps entry?
Shouldn't they be  below it?
 [AK] Done in [PATCH v6]:


>>
>> >
>> > Signed-off-by: Alexander Komrakov <alexander.komrakov@...adcom.com>
>> > ---
>> >  Documentation/ABI/testing/sysfs-pps | 27 +++++++++++++++++++++++
>> >  Documentation/driver-api/pps.rst    | 19 ++++++++++++++++
>> >  drivers/pps/kapi.c                  | 24 +++++++++++++++++---
>> >  drivers/pps/sysfs.c                 | 34 +++++++++++++++++++++++++++++
>> >  include/linux/pps_kernel.h          |  2 ++
>> >  5 files changed, 103 insertions(+), 3 deletions(-)
>>
>> My bot will remind you that you dropped the version information off :(
>>
>> > diff --git a/Documentation/ABI/testing/sysfs-pps b/Documentation/ABI/testing/sysfs-pps
>> > index 25028c7bc37d..031ec89e1ed6 100644
>> > --- a/Documentation/ABI/testing/sysfs-pps
>> > +++ b/Documentation/ABI/testing/sysfs-pps
>> > @@ -1,3 +1,30 @@
>> > +What:                /sys/class/pps/pps0/assert_elapsed
>>
>> That should be "ppsX" right?
>>
>> > +Date:                October 2021
>> > +Contact:     Alexander Komrakov <alexander.komrakov@...adcom.com>
>> > +Description:
>> > +             The /sys/class/pps/ppsX/assert_elapsed file reports the
>> > +             elapsed real-time assert events and the elapsed
>> > +             real-time assert sequence number of the X-th source
>> > +             in the form:
>> > +
>> > +                     <secs>.<nsec>#<sequence>
>> > +
>> > +             If the source has no elapsed real-time assert events
>> > +             the content of this file is empty.
>> > +
>> > +What:                /sys/class/pps/ppsX/clear_elapsed
>> > +Date:                October 2021
>> > +Contact:     Alexander Komrakov <alexander.komrakov@...adcom.com>
>> > +Description:
>> > +             The /sys/class/pps/ppsX/clear_elapsed file reports the elapsed
>> > +             real-time clear events and the elapsed real-time clear
>> > +             sequence number of the X-th source in the form:
>> > +
>> > +                     <secs>.<nsec>#<sequence>
>> > +
>> > +             If the source has no elapsed real-time clear events the
>> > +             content of this file is empty.
>> > +
>> >  What:                /sys/class/pps/
>>
>> Why are your new entries above the main pps entry?  Shouldn't they be
>> below it?
>>
>> thanks,
>>
>> greg k-h

Download attachment "smime.p7s" of type "application/pkcs7-signature" (4222 bytes)