| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <1427105790.2577.157.camel@jtkirshe-mobl>
Date: Mon, 23 Mar 2015 03:16:30 -0700
From: Jeff Kirsher <jeffrey.t.kirsher@...el.com>
To: Richard Cochran <richardcochran@...il.com>
Cc: netdev@...r.kernel.org, linux-kernel@...r.kernel.org,
Amir Vadai <amirv@...lanox.com>,
Ariel Elior <ariel.elior@...gic.com>,
Arnd Bergmann <arnd@...aro.org>,
Baolin Wang <baolin.wang@...aro.org>,
Ben Hutchings <ben@...adent.org.uk>,
Bruce Allan <bruce.w.allan@...el.com>,
Carolyn Wyborny <carolyn.wyborny@...el.com>,
Chris Metcalf <cmetcalf@...hip.com>,
David Miller <davem@...emloft.net>,
Frank Li <Frank.Li@...escale.com>,
Giuseppe Cavallaro <peppe.cavallaro@...com>,
John Stultz <john.stultz@...aro.org>,
Luwei Zhou <b45643@...escale.com>,
Matthew Vick <matthew.vick@...el.com>,
Michael Chan <mchan@...adcom.com>,
Prashant Sreedharan <prashant@...adcom.com>,
Shradha Shah <sshah@...arflare.com>,
Solarflare linux maintainers <linux-net-drivers@...arflare.com>,
Sonic Zhang <sonic.zhang@...log.com>,
Stefan Sørensen
<stefan.sorensen@...ctralink.com>,
Thomas Gleixner <tglx@...utronix.de>,
Tom Lendacky <thomas.lendacky@....com>
Subject: Re: [PATCH net-next V2 00/23] ptp: get ready for 2038
On Sat, 2015-03-21 at 22:39 +0100, Richard Cochran wrote:
> This series converts the core driver methods of the PTP Hardware Clock
> (PHC) subsystem to use the 64 bit version of the timespec structure,
> making the core API ready for the year 2038.
>
> In addition, I reviewed how each driver and device represents the time
> value at the hardware register level. Most of the drivers are ready,
> but a few will need some work before the year 2038, as shown:
>
> Patch Driver
> ------------------------------------------------
> 12 drivers/net/ethernet/intel/igb/igb_ptp.c
> 15 ? drivers/net/ethernet/sfc/ptp.c
> 16 drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c
> 18 ? drivers/net/ethernet/tile/tilegx.c
> 19 drivers/net/phy/dp83640.c
>
> The commit log message documents how each driver is ready or why it is
> not ready. For patches 15 and 18, I could not easily find out the
> hardware representation of the time value, so I would ask the
> maintainers for a review.
>
> Thanks,
> Richard
>
> * ChangeLog
> ** V2
> - use the new methods in the posix clock code right away (patch #3)
The Intel driver changes look fine to me and as noted, more work appears
to be needed for igb. It does not appear that Jake Jacobs got copied on
the patches, so I will make sure he takes a look at the changes for our
Intel drivers, since he did much of the work.
You get my ACK for the Intel changes.
Acked-by: Jeff Kirsher <jeffrey.t.kirsher@...el.com>
>
> Richard Cochran (23):
> ptp: introduce get/set time methods with explicit 64 bit seconds.
> ptp: use the 64 bit gettime method for the SYS_OFFSET ioctl.
> ptp: use the 64 bit get/set time methods for the posix clock.
> ptp: blackfin: convert to the 64 bit get/set time methods.
> ptp: xgbe: convert to the 64 bit get/set time methods.
> ptp: bnx2x: convert to the 64 bit get/set time methods.
> ptp: tg3: convert to the 64 bit get/set time methods.
> ptp: fec: convert to the 64 bit get/set time methods.
> ptp: gianfar: convert to the 64 bit get/set time methods.
> ptp: e1000e: convert to the 64 bit get/set time methods.
> ptp: fm10k: convert to the 64 bit get/set time methods.
> ptp: i40e: convert to the 64 bit get/set time methods.
> ptp: igb: convert to the 64 bit get/set time methods.
> ptp: ixgbe: convert to the 64 bit get/set time methods.
> ptp: mlx4: convert to the 64 bit get/set time methods.
> ptp: sfc: convert to the 64 bit get/set time methods.
> ptp: stmmac: convert to the 64 bit get/set time methods.
> ptp: cpts: convert to the 64 bit get/set time methods.
> ptp: tilegx: convert to the 64 bit get/set time methods.
> ptp: dp83640: convert to the 64 bit get/set time methods.
> ptp: ixp46x: convert to the 64 bit get/set time methods.
> ptp: pch: convert to the 64 bit get/set time methods.
> ptp: remove 32 bit get/set methods.
>
> drivers/net/ethernet/adi/bfin_mac.c | 8 ++---
> drivers/net/ethernet/amd/xgbe/xgbe-ptp.c | 13 +++----
> drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c | 8 ++---
> drivers/net/ethernet/broadcom/tg3.c | 10 +++---
> drivers/net/ethernet/freescale/fec_ptp.c | 8 ++---
> drivers/net/ethernet/freescale/gianfar_ptp.c | 13 +++----
> drivers/net/ethernet/intel/e1000e/ptp.c | 16 ++++-----
> drivers/net/ethernet/intel/fm10k/fm10k_ptp.c | 12 +++----
> drivers/net/ethernet/intel/i40e/i40e_ptp.c | 26 +++++++-------
> drivers/net/ethernet/intel/igb/igb_ptp.c | 41 +++++++++++-----------
> drivers/net/ethernet/intel/ixgbe/ixgbe_ptp.c | 14 ++++----
> drivers/net/ethernet/mellanox/mlx4/en_clock.c | 11 +++---
> drivers/net/ethernet/sfc/ptp.c | 22 ++++++------
> drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c | 8 ++---
> drivers/net/ethernet/ti/cpts.c | 12 +++----
> drivers/net/ethernet/tile/tilegx.c | 16 +++++----
> drivers/net/phy/dp83640.c | 12 ++++---
> drivers/ptp/ptp_chardev.c | 8 ++---
> drivers/ptp/ptp_clock.c | 12 +++++--
> drivers/ptp/ptp_ixp46x.c | 8 ++---
> drivers/ptp/ptp_pch.c | 8 ++---
> include/linux/ptp_clock_kernel.h | 12 +++----
> 22 files changed, 158 insertions(+), 140 deletions(-)
>
Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)
Powered by blists - more mailing lists