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.

Reply via email to