Hi, I think this patch was reviewed and no objection now, right? (I see status is " Changes Requested ".) Thanks.
> -----Original Message----- > From: Yangbo Lu <yangbo...@nxp.com> > Sent: 2021年4月13日 11:48 > To: netdev@vger.kernel.org > Cc: Y.b. Lu <yangbo...@nxp.com>; David S . Miller <da...@davemloft.net>; > Richard Cochran <richardcoch...@gmail.com>; Claudiu Manoil > <claudiu.man...@nxp.com>; Jakub Kicinski <k...@kernel.org>; Vladimir > Oltean <vladimir.olt...@nxp.com>; Russell King <li...@armlinux.org.uk> > Subject: [net-next] enetc: fix locking for one-step timestamping packet > transfer > > The previous patch to support PTP Sync packet one-step timestamping > described one-step timestamping packet handling logic as below in commit > message: > > - Trasmit packet immediately if no other one in transfer, or queue to > skb queue if there is already one in transfer. > The test_and_set_bit_lock() is used here to lock and check state. > - Start a work when complete transfer on hardware, to release the bit > lock and to send one skb in skb queue if has. > > There was not problem of the description, but there was a mistake in > implementation. The locking/test_and_set_bit_lock() should be put in > enetc_start_xmit() which may be called by worker, rather than in enetc_xmit(). > Otherwise, the worker calling enetc_start_xmit() after bit lock released is > not > able to lock again for transfer. > > Fixes: 7294380c5211 ("enetc: support PTP Sync packet one-step > timestamping") > Signed-off-by: Yangbo Lu <yangbo...@nxp.com> [...]