Re: CLOCK_MONOTONIC datagram timestamps by the kernel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



John wrote:

I know it's possible to have Linux timestamp incoming datagrams as soon
as they are received, then for one to retrieve this timestamp later with
an ioctl command or a recvmsg call.

Has it ever been proposed to modify struct skb_timeval to hold nanosecond stamps instead of just microsecond stamps? Then make the improved precision somehow available to user space.

On a related note, the comment for skb_set_timestamp() states:

/**
 * skb_set_timestamp - set timestamp of a skb
 * @skb: skb to set stamp of
 * @stamp: pointer to struct timeval to get stamp from
 *
 * Timestamps are stored in the skb as offsets to a base timestamp.
 * This function converts a struct timeval to an offset and stores
 * it in the skb.
 */

But there is no mention of an offset in the code:

static inline void skb_set_timestamp(
  struct sk_buff *skb, const struct timeval *stamp)
{
  skb->tstamp.off_sec  = stamp->tv_sec;
  skb->tstamp.off_usec = stamp->tv_usec;
}

Likewise for skb_get_timestamp:

/**
 * skb_get_timestamp - get timestamp from a skb
 * @skb: skb to get stamp from
 * @stamp: pointer to struct timeval to store stamp in
 *
 * Timestamps are stored in the skb as offsets to a base timestamp.
 * This function converts the offset back to a struct timeval and stores
 * it in stamp.
 */

static inline void skb_get_timestamp(
  const struct sk_buff *skb, struct timeval *stamp)
{
  stamp->tv_sec  = skb->tstamp.off_sec;
  stamp->tv_usec = skb->tstamp.off_usec;
}

Are the comments related to code that has since been modified?

Regards.
-
To unsubscribe from this list: send the line "unsubscribe linux-net" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux 802.1Q VLAN]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Git]     [Bugtraq]     [Yosemite News and Information]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux PCI]     [Linux Admin]     [Samba]

  Powered by Linux