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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 26 May 2020 16:28:25 -0700 (PDT) From: David Miller <davem@...emloft.net> To: fugang.duan@....com Cc: peppe.cavallaro@...com, alexandre.torgue@...com, joabreu@...opsys.com, kuba@...nel.org, netdev@...r.kernel.org, mcoquelin.stm32@...il.com, p.zabel@...gutronix.de, linux-stm32@...md-mailman.stormreply.com, linux-arm-kernel@...ts.infradead.org Subject: Re: [PATCH net 1/1] net: stmmac: enable timestamp snapshot for required PTP packets in dwmac v5.10a From: Fugang Duan <fugang.duan@....com> Date: Mon, 25 May 2020 16:18:14 +0800 > For rx filter 'HWTSTAMP_FILTER_PTP_V2_EVENT', it should be > PTP v2/802.AS1, any layer, any kind of event packet, but HW only > take timestamp snapshot for below PTP message: sync, Pdelay_req, > Pdelay_resp. > > Then it causes below issue when test E2E case: > ptp4l[2479.534]: port 1: received DELAY_REQ without timestamp > ptp4l[2481.423]: port 1: received DELAY_REQ without timestamp > ptp4l[2481.758]: port 1: received DELAY_REQ without timestamp > ptp4l[2483.524]: port 1: received DELAY_REQ without timestamp > ptp4l[2484.233]: port 1: received DELAY_REQ without timestamp > ptp4l[2485.750]: port 1: received DELAY_REQ without timestamp > ptp4l[2486.888]: port 1: received DELAY_REQ without timestamp > ptp4l[2487.265]: port 1: received DELAY_REQ without timestamp > ptp4l[2487.316]: port 1: received DELAY_REQ without timestamp > > Timestamp snapshot dependency on register bits in received path: > SNAPTYPSEL TSMSTRENA TSEVNTENA PTP_Messages > 01 x 0 SYNC, Follow_Up, Delay_Req, > Delay_Resp, Pdelay_Req, Pdelay_Resp, > Pdelay_Resp_Follow_Up > 01 0 1 SYNC, Pdelay_Req, Pdelay_Resp > > For dwmac v5.10a, enabling all events by setting register > DWC_EQOS_TIME_STAMPING[SNAPTYPSEL] to 2’b01, clearing bit [TSEVNTENA] > to 0’b0, which can support all required events. > > Signed-off-by: Fugang Duan <fugang.duan@....com> Applied and queued up for -stable, thanks.
Powered by blists - more mailing lists