>> + /* pull rx packet timestamp if available */ >> + if (igb_test_staterr(rx_desc, E1000_RXDADV_STAT_TSIP)) >> { >> + timestamp = igb_ptp_rx_pktstamp(rx_ring- >> >q_vector, >> + pktbuf); > > The timestamp should be checked for failure and not adjust these values > if the timestamp was invalid. OK. I'll adjust it. Thanks, Kurt