[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20140905.150315.472751844317741858.davem@davemloft.net>
Date: Fri, 05 Sep 2014 15:03:15 -0700 (PDT)
From: David Miller <davem@...emloft.net>
To: willemb@...gle.com
Cc: netdev@...r.kernel.org
Subject: Re: [PATCH net] net-timestamp: only report sw timestamp if
reporting bit is set
From: Willem de Bruijn <willemb@...gle.com>
Date: Wed, 3 Sep 2014 12:01:18 -0400
> The timestamping API has separate bits for generating and reporting
> timestamps. A software timestamp should only be reported for a packet
> when the packet has the relevant generation flag (SKBTX_..) set
> and the socket has reporting bit SOF_TIMESTAMPING_SOFTWARE set.
>
> The second check was accidentally removed. Reinstitute the original
> behavior.
>
> Tested:
> Without this patch, Documentation/networking/txtimestamp reports
> timestamps regardless of whether SOF_TIMESTAMPING_SOFTWARE is set.
> After the patch, it only reports them when the flag is set.
>
> Fixes: f24b9be5957b ("net-timestamp: extend SCM_TIMESTAMPING ancillary data struct")
>
> Signed-off-by: Willem de Bruijn <willemb@...gle.com>
Applied and queued up for -stable, thanks Willem.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists