Commit b6561f84 authored by Martin KaFai Lau's avatar Martin KaFai Lau Committed by David S. Miller

net: ipv6: Get rcv timestamp if needed when handling hop-by-hop IOAM option

IOAM is a hop-by-hop option with a temporary iana allocation (49).
Since it is hop-by-hop, it is done before the input routing decision.
One of the traced data field is the (rcv) timestamp.

When the locally generated skb is looping from egress to ingress over
a virtual interface (e.g. veth, loopback...), skb->tstamp may have the
delivery time before it is known that it will be delivered locally
and received by another sk.

Like handling the network tapping (tcpdump) in the earlier patch,
this patch gets the timestamp if needed without over-writing the
delivery_time in the skb->tstamp.  skb_tstamp_cond() is added to do the
ktime_get_real() with an extra cond arg to check on top of the
netstamp_needed_key static key.  skb_tstamp_cond() will also be used in
a latter patch and it needs the netstamp_needed_key check.
Signed-off-by: default avatarMartin KaFai Lau <kafai@fb.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 335c8cf3
...@@ -4028,6 +4028,17 @@ static inline ktime_t skb_tstamp(const struct sk_buff *skb) ...@@ -4028,6 +4028,17 @@ static inline ktime_t skb_tstamp(const struct sk_buff *skb)
return skb->tstamp; return skb->tstamp;
} }
static inline ktime_t skb_tstamp_cond(const struct sk_buff *skb, bool cond)
{
if (!skb->mono_delivery_time && skb->tstamp)
return skb->tstamp;
if (static_branch_unlikely(&netstamp_needed_key) || cond)
return ktime_get_real();
return 0;
}
static inline u8 skb_metadata_len(const struct sk_buff *skb) static inline u8 skb_metadata_len(const struct sk_buff *skb)
{ {
return skb_shinfo(skb)->meta_len; return skb_shinfo(skb)->meta_len;
......
...@@ -635,7 +635,8 @@ static void __ioam6_fill_trace_data(struct sk_buff *skb, ...@@ -635,7 +635,8 @@ static void __ioam6_fill_trace_data(struct sk_buff *skb,
struct ioam6_schema *sc, struct ioam6_schema *sc,
u8 sclen, bool is_input) u8 sclen, bool is_input)
{ {
struct __kernel_sock_timeval ts; struct timespec64 ts;
ktime_t tstamp;
u64 raw64; u64 raw64;
u32 raw32; u32 raw32;
u16 raw16; u16 raw16;
...@@ -680,10 +681,9 @@ static void __ioam6_fill_trace_data(struct sk_buff *skb, ...@@ -680,10 +681,9 @@ static void __ioam6_fill_trace_data(struct sk_buff *skb,
if (!skb->dev) { if (!skb->dev) {
*(__be32 *)data = cpu_to_be32(IOAM6_U32_UNAVAILABLE); *(__be32 *)data = cpu_to_be32(IOAM6_U32_UNAVAILABLE);
} else { } else {
if (!skb->tstamp) tstamp = skb_tstamp_cond(skb, true);
__net_timestamp(skb); ts = ktime_to_timespec64(tstamp);
skb_get_new_timestamp(skb, &ts);
*(__be32 *)data = cpu_to_be32((u32)ts.tv_sec); *(__be32 *)data = cpu_to_be32((u32)ts.tv_sec);
} }
data += sizeof(__be32); data += sizeof(__be32);
...@@ -694,13 +694,12 @@ static void __ioam6_fill_trace_data(struct sk_buff *skb, ...@@ -694,13 +694,12 @@ static void __ioam6_fill_trace_data(struct sk_buff *skb,
if (!skb->dev) { if (!skb->dev) {
*(__be32 *)data = cpu_to_be32(IOAM6_U32_UNAVAILABLE); *(__be32 *)data = cpu_to_be32(IOAM6_U32_UNAVAILABLE);
} else { } else {
if (!skb->tstamp) if (!trace->type.bit2) {
__net_timestamp(skb); tstamp = skb_tstamp_cond(skb, true);
ts = ktime_to_timespec64(tstamp);
}
if (!trace->type.bit2) *(__be32 *)data = cpu_to_be32((u32)(ts.tv_nsec / NSEC_PER_USEC));
skb_get_new_timestamp(skb, &ts);
*(__be32 *)data = cpu_to_be32((u32)ts.tv_usec);
} }
data += sizeof(__be32); data += sizeof(__be32);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment