netdev - Re: [PATCH net-next v3 1/5] net-timestamp: extend SCM

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <CA+FuTSdVFHVEp0-orPe7KNEbF2-Lsu+myBNdi9bW7kVV3FM6cA@mail.gmail.com>
Date:	Thu, 24 Jul 2014 20:10:55 -0400
From:	Willem de Bruijn <willemb@...gle.com>
To:	David Miller <davem@...emloft.net>
Cc:	netdev@...r.kernel.org, Eric Dumazet <eric.dumazet@...il.com>,
	Richard Cochran <richardcochran@...il.com>,
	Chad Reese <kreese@...iumnetworks.com>,
	David Daney <ddaney@...iumnetworks.com>
Subject: Re: [PATCH net-next v3 1/5] net-timestamp: extend SCM_TIMESTAMPING
 ancillary data struct

>>> The kernel would simply saw that adjusted HW timestamps are not
>>> supported, the application has to accomodate the possibility of
>>> needing to use one of the other timestamping flavors.
>>
>> Cavium's PTP library already programs the device PTP clock,
>> just not through the standard interface. It appears to map the
>> memory addresses CVMX_MIO_.. directly. In any case, it
>> will gracefully handle the loss of syststamp.
>>
>> I doubt that there are other users of this Octeon interface, but
>> if so, they can reasonably be expected to do the same and
>> rely on the same legacy Octeon PTP clock interface +
>> hwtstamp as alternative. Is that enough justification to remove
>> syststamp without patching the driver to implement
>> ptp_clock_info?
>>
>> Cleanup would be two patches: one to revert the relevant
>> code in the octeon driver (mostly, ptp_to_ktime), and a
>> follow-up to remove the core code now that the last user
>> is gone.
>
> I'm not sure I understand, can users still get hardware timestamps
> from Octeon cards after such a change?  Unless I misunderstand, the
> only thing left available will be software timestamps, right?

They will still receive hardware timestamps in raw format. The
only feature that is deprecated is hardware timestamps
converted to system time format (ptp_to_ktime).

By querying the PTP clock source, users can perform  perform
this conversion to system time themselves. The Octeon driver
does not support the standard PTP clock interface, but does
appear to use its own alternative using shared memory. I'm
not exactly sure how that memory is setup.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html