On Mon, 29 Jun 2020 17:07:00 -0700 Andre Guedes wrote: > > What if timeout happens, igc_ptp_tx_hang() starts cleaning up and then > > irq gets delivered half way through? Perhaps we should just add a spin > > lock around the ptp_tx_s* fields? > > Yep, I think this other scenario is possible indeed, and we should probably > protect ptp_tx_s* with a lock. Thanks for pointing that out. In fact, it seems > this issue can happen even with current net-next code. > > Since that issue is not introduced by this patch, would it be OK we move > forward > with it, and fix the issue in a separate patch?
Fine by me.