On vie, 2007-09-14 at 12:26 +0200, Stephen Hemminger wrote: > On Thu, 13 Sep 2007 14:24:06 +0200 > Eric Dumazet <[EMAIL PROTECTED]> wrote: > > > On Thu, 13 Sep 2007 12:42:53 +0200 > > Stephen Hemminger <[EMAIL PROTECTED]> wrote: > > > > > Currently, af_packet does not allow disabling timestamps. This patch > > > changes > > > that but doesn't force global timestamps on. > > > > > > This shows up in bugzilla as: > > > http://bugzilla.kernel.org/show_bug.cgi?id=4809 > > > > > > Patch against net-2.6.24 tree. > > > > > > > I am not sure I understood this patch. > > > > This means that tcpdump/ethereal wont get precise timestamps > > (gathered when packet is received), but imprecise ones (gathered when the > > sniffer reads the packet) > > > > I added some time ago ktime infrastructure to eventually get nanosecond > > precision in libpcap, so I would prefer a step in the right direction :) > > > > Should'nt we use something like : > > > > [PATCH] af_packet : allow disabling timestamps, or requesting nanosecond > > precision. > > > > Signed-off-by: Eric Dumazet <[EMAIL PROTECTED]> > > > > diff --git a/net/core/sock.c b/net/core/sock.c > > index 5a16e38..1c10b9d 100644 > > --- a/net/core/sock.c > > +++ b/net/core/sock.c > > @@ -563,6 +563,7 @@ set_rcvbuf: > > } else { > > sock_reset_flag(sk, SOCK_RCVTSTAMP); > > sock_reset_flag(sk, SOCK_RCVTSTAMPNS); > > + sock_disable_timestamp(sk); > > } > > break; > > > > diff --git a/net/packet/af_packet.c b/net/packet/af_packet.c > > index 745e2cb..409de44 100644 > > --- a/net/packet/af_packet.c > > +++ b/net/packet/af_packet.c > > @@ -650,12 +650,27 @@ static int tpacket_rcv(struct sk_buff *skb, struct > > net_device *dev, struct packe > > h->tp_snaplen = snaplen; > > h->tp_mac = macoff; > > h->tp_net = netoff; > > - if (skb->tstamp.tv64) > > - tv = ktime_to_timeval(skb->tstamp); > > - else > > - do_gettimeofday(&tv); > > - h->tp_sec = tv.tv_sec; > > - h->tp_usec = tv.tv_usec; > > + h->tp_sec = 0; > > + h->tp_usec = 0; > > + if ((sock_flag(sk, SOCK_TIMESTAMP))) { > > + if (sock_flag(sk, SOCK_RCVTSTAMPNS)) { > > + struct timespec ts; > > + if (skb->tstamp.tv64) > > + ts = ktime_to_timespec(skb->tstamp); > > + else > > + getnstimeofday(&ts); > > + h->tp_sec = ts.tv_sec; > > + h->tp_usec = ts.tv_nsec; /* cheat a litle bit */ > > + } > > + else { > > + if (skb->tstamp.tv64) > > + tv = ktime_to_timeval(skb->tstamp); > > + else > > + do_gettimeofday(&tv); > > + h->tp_sec = tv.tv_sec; > > + h->tp_usec = tv.tv_usec; > > + } > > + } > > > > sll = (struct sockaddr_ll*)((u8*)h + TPACKET_ALIGN(sizeof(*h))); > > sll->sll_halen = 0; > > @@ -1014,6 +1029,7 @@ static int packet_create(struct net *net, struct > > socket *sock, int protocol) > > sock->ops = &packet_ops_spkt; > > > > sock_init_data(sock, sk); > > + sock_enable_timestamp(sk); > > > > po = pkt_sk(sk); > > sk->sk_family = PF_PACKET; > > No, then we end up timestamping all the packets, even if they get dropped by > packet filter. The change in 2.6.24 allows dhclient (and rstp) to only call > hires clock source for packets they want, not all packets. > > Perhaps the timestamping needs to change into a tristate flag? > - > To unsubscribe from this list: send the line "unsubscribe netdev" in > the body of a message to [EMAIL PROTECTED] > More majordomo info at http://vger.kernel.org/majordomo-info.html >
Eric's patch has a feature your previous patch hasn't: a way to disable timestamping from userspace (the changes at net/core/sock.c). But it changes the userspace API. I really think that any developer that sets SO_TIMESTAMP to 0 and still expect to receive valid timestamp is terminally insane and doesn't deserve any mercy. But we should take pity of these poor souls that uses (suffers) closed software and found another way that doesn't changes the API. Bye. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
