Hi Jake, thanks for all the help and for looking at this!
> > You have two conditionals inside, and I misread where you were doing the > checking of the SKBTX_HW_TSTAMP flag. > > I would do the following : > > if (unlikely(skb_shinfo(skb)->tx_flags & SKBTX_HWTSTAMP) && > gem_ptp_do_txstamp(queue, skb, desc)) { I guess I was thinking the call to gem_ptp_do_txstamp() could be avoided in many cases, but either way is fine with me.. > .... > } > -Paul