On Wed, Feb 5, 2025 at 11:14 AM Jakub Kicinski <kuba@xxxxxxxxxx> wrote: > > On Wed, 5 Feb 2025 10:40:42 +0800 Jason Xing wrote: > > I wonder if we need a separate cleanup after this series about moving > > this kind of functions into net/core/timestamping.c, say, > > __skb_tstamp_tx()? > > IMHO no need to go too far, just move the one function as part of this > series. The only motivation is to avoid adding includes to > linux/skbuff.h since skbuff.h is included in something like 8k objects. Thanks for clarifying. Will do it in the re-spin. Thanks, Jason