diff options
author | Vladimir Oltean | 2021-10-12 14:40:36 +0300 |
---|---|---|
committer | Jakub Kicinski | 2021-10-12 17:35:17 -0700 |
commit | 52849bcf0029ccc553be304e4f804938a39112e2 (patch) | |
tree | 164fb20ac0bac33fbf83ae0139fc9953db90a897 /arch | |
parent | c57fe0037a4e3863d9b740f8c14df9c51ac31aa1 (diff) |
net: mscc: ocelot: avoid overflowing the PTP timestamp FIFO
PTP packets with 2-step TX timestamp requests are matched to packets
based on the egress port number and a 6-bit timestamp identifier.
All PTP timestamps are held in a common FIFO that is 128 entry deep.
This patch ensures that back-to-back timestamping requests cannot exceed
the hardware FIFO capacity. If that happens, simply send the packets
without requesting a TX timestamp to be taken (in the case of felix,
since the DSA API has a void return code in ds->ops->port_txtstamp) or
drop them (in the case of ocelot).
I've moved the ts_id_lock from a per-port basis to a per-switch basis,
because we need separate accounting for both numbers of PTP frames in
flight. And since we need locking to inc/dec the per-switch counter,
that also offers protection for the per-port counter and hence there is
no reason to have a per-port counter anymore.
Fixes: 4e3b0468e6d7 ("net: mscc: PTP Hardware Clock (PHC) support")
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'arch')
0 files changed, 0 insertions, 0 deletions