From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932309AbdK0Q7w (ORCPT ); Mon, 27 Nov 2017 11:59:52 -0500 Received: from mail-wr0-f175.google.com ([209.85.128.175]:33202 "EHLO mail-wr0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753818AbdK0Q7r (ORCPT ); Mon, 27 Nov 2017 11:59:47 -0500 X-Google-Smtp-Source: AGs4zMZzN+8qROXIjWqvZTpVBaoOw8Ls6VyNUAn+zKuOtt/5NUjU9gzwh1YT2qUjqNDplYJIm5CAYQ== Date: Mon, 27 Nov 2017 17:59:45 +0100 From: Jiri Pirko To: Arnd Bergmann Cc: "David S. Miller" , y2038@lists.linaro.org, Shuah Khan , Willem de Bruijn , Mike Maloney , Eric Dumazet , Kees Cook , "Rosen, Rami" , Andrey Konovalov , "Reshetova, Elena" , Sowmini Varadhan , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org Subject: Re: [PATCH 2/2] [RFC] packet: experimental support for 64-bit timestamps Message-ID: <20171127165945.GC1971@nanopsycho.orion> References: <20171127162001.4055813-1-arnd@arndb.de> <20171127162001.4055813-2-arnd@arndb.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171127162001.4055813-2-arnd@arndb.de> User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Mon, Nov 27, 2017 at 05:19:25PM CET, arnd@arndb.de wrote: >I tried to figure out what it would take to do a version 4 mmap packet >socket interface to completely avoid the y2106 overflow problem. This is >what I came up with, reusing most of the v3 code, except for the parts >where we access the timestamps. > >For kselftest, I'm adding support for testing v4 in addition to v1-v3, >but the test currently does not look at the timestamps, so it won't >check that the timestamp format actually works as intended, only that >I didn't break the parts that worked in the v3 selftest. > >Overall, this is more of a mess than I expected, so it's probably not >worth doing a v4 format just for the timestamp, but the patch can serve >as a reference for anyone that needs a new format for other reasons and >fixes this along with the other changes. > >Signed-off-by: Arnd Bergmann >--- [...] >@@ -250,7 +269,8 @@ struct tpacket_block_desc { > enum tpacket_versions { > TPACKET_V1, > TPACKET_V2, >- TPACKET_V3 >+ TPACKET_V3, >+ TPACKET_V4, I wonder with how many versions are we going to eventually end up with :O