netdev - Re: [PATCH net-next 0/3] Enable PPS and PTP clocks automatically

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20121101081924.GB2637@netboy.at.omicron.at>
Date:	Thu, 1 Nov 2012 09:19:24 +0100
From:	Richard Cochran <richardcochran@...il.com>
To:	Ben Hutchings <bhutchings@...arflare.com>
Cc:	David Miller <davem@...emloft.net>,
	Rodolfo Giometti <giometti@...eenne.com>,
	netdev@...r.kernel.org, linux-net-drivers@...arflare.com
Subject: Re: [PATCH net-next 0/3] Enable PPS and PTP clocks automatically

Ben,

I really like this idea, but I have a couple of questions, below.

On Thu, Nov 01, 2012 at 01:29:57AM +0000, Ben Hutchings wrote:

> Ben Hutchings (3):
>   pps, ptp: Remove dependencies on EXPERIMENTAL

Kees Cook just posted a massive series to remove EXPERIMENTAL
everywhere. Maybe it would make sense to have this series follow that
one?

>   ptp: Make PTP_1588_CLOCK select rather than depend on PPS
>   ptp: Enable clock drivers along with associated net/PHY drivers
> 
>  drivers/net/ethernet/intel/Kconfig                 |   28 +------------------
>  drivers/net/ethernet/intel/igb/Makefile            |    4 +--
>  drivers/net/ethernet/intel/igb/igb.h               |    6 ----
>  drivers/net/ethernet/intel/igb/igb_ethtool.c       |    2 -
>  drivers/net/ethernet/intel/igb/igb_main.c          |   28 --------------------

The #ifdefs in the Intel and PCH drivers are meant to streamline the
code for users who are not interested in time stamping. Although the
overhead of the additional is minimal (it really just a bunch of
tests), and I wouldn't mind removing the conditionals, I suspect that
some people won't like it.

>  drivers/net/ethernet/intel/ixgbe/Makefile          |    3 +-
>  drivers/net/ethernet/intel/ixgbe/ixgbe.h           |    6 ----
>  drivers/net/ethernet/intel/ixgbe/ixgbe_ethtool.c   |    2 -
>  drivers/net/ethernet/intel/ixgbe/ixgbe_main.c      |   28 --------------------
>  drivers/net/ethernet/oki-semi/pch_gbe/Kconfig      |   16 -----------
>  drivers/net/ethernet/oki-semi/pch_gbe/pch_gbe.h    |    2 -
>  .../net/ethernet/oki-semi/pch_gbe/pch_gbe_main.c   |   14 ----------

Thanks,
Richard

--
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