Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1484459

Re: [PATCH v3) posix-timers: make it configurable

From John Stultz <john.stultz@linaro.org>
Newsgroups linux.kernel
Subject Re: [PATCH v3) posix-timers: make it configurable
Date 2016-09-15 20:50 +0200
Message-ID <shJTr-2cl-1@gated-at.bofh.it> (permalink)
References <shvQt-1BE-7@gated-at.bofh.it> <shJqp-22F-5@gated-at.bofh.it> <shJA5-25R-3@gated-at.bofh.it> <shJJL-28Z-5@gated-at.bofh.it> <shJJL-28Z-3@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Thu, Sep 15, 2016 at 11:37 AM, Nicolas Pitre
<nicolas.pitre@linaro.org> wrote:
> On Thu, 15 Sep 2016, John Stultz wrote:
>
>> On Thu, Sep 15, 2016 at 11:28 AM, Nicolas Pitre
>> <nicolas.pitre@linaro.org> wrote:
>> > On Thu, 15 Sep 2016, John Stultz wrote:
>> >
>> >> > diff --git a/kernel/time/Kconfig b/kernel/time/Kconfig
>> >> > index 62824f2fe4..62504a2c9f 100644
>> >> > --- a/kernel/time/Kconfig
>> >> > +++ b/kernel/time/Kconfig
>> >> > @@ -195,3 +195,21 @@ config HIGH_RES_TIMERS
>> >> >
>> >> >  endmenu
>> >> >  endif
>> >> > +
>> >> > +config POSIX_TIMERS
>> >> > +       bool "Posix Clocks & timers" if EMBEDDED
>> >> > +       default y
>> >> > +       help
>> >> > +         This includes native support for POSIX timers to the kernel.
>> >> > +         Most embedded systems may have no use for them and therefore they
>> >> > +         can be configured out to reduce the size of the kernel image.
>> >> > +
>> >> > +         When this option is disabled, the following syscalls won't be
>> >> > +         available: timer_create, timer_gettime: timer_getoverrun,
>> >> > +         timer_settime, timer_delete, clock_adjtime. Furthermore, the
>> >> > +         clock_settime, clock_gettime, clock_getres and clock_nanosleep
>> >> > +         syscalls will be limited to CLOCK_REALTIME and CLOCK_MONOTONIC
>> >> > +         only.
>> >> > +
>> >> > +         If unsure say y.
>> >> >
>> >>
>> >> One thought.. Should this go under:
>> >>     Configure standard kernel features (expert users)
>> >> rather then a top level item under  General Setup ?
>> >
>> > Hmmm... probably yes.
>> >
>> > Do you need that I repost the patch?
>>
>> I can see about moving it..
>
> OK.
>
> Making it conditional on EXPERT rather than EMBEDDED would also be more
> inline with the other similar options there.

Ack.

Although I'm also seeing some Kconfig noise when I disable it:

warning: (SFC && TILE_NET && AMD_XGBE && BFIN_MAC_USE_HWSTAMP &&
TIGON3 && BNX2X && LIQUIDIO && FEC && E1000E && IGB && IXGBE && I40E
&& FM10K && MLX4_EN && MLX5_CORE_EN && RAVB && SXGBE_ETH && STMMAC_ETH
&& TI_CPTS && PTP_1588_CLOCK_GIANFAR && PTP_1588_CLOCK_IXP46X &&
DP83640_PHY && PTP_1588_CLOCK_PCH) selects PTP_1588_CLOCK which has
unmet direct dependencies (NET && POSIX_TIMERS)
warning: (SFC && TILE_NET && AMD_XGBE && BFIN_MAC_USE_HWSTAMP &&
TIGON3 && BNX2X && LIQUIDIO && FEC && E1000E && IGB && IXGBE && I40E
&& FM10K && MLX4_EN && MLX5_CORE_EN && RAVB && SXGBE_ETH && STMMAC_ETH
&& TI_CPTS && PTP_1588_CLOCK_GIANFAR && PTP_1588_CLOCK_IXP46X &&
DP83640_PHY && PTP_1588_CLOCK_PCH) selects PTP_1588_CLOCK which has
unmet direct dependencies (NET && POSIX_TIMERS)

Not sure if this is just expected given we can't do reverse dependency
checking on select...

Maybe PTP_1588_CLOCK needs to select POSIX_TIMERS instead of just depend on it?

thanks
-john

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v3) posix-timers: make it configurable Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-09-15 05:50 +0200
  Re: [PATCH v3) posix-timers: make it configurable John Stultz <john.stultz@linaro.org> - 2016-09-15 19:50 +0200
    Re: [PATCH v3) posix-timers: make it configurable Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-09-15 20:00 +0200
  Re: [PATCH v3) posix-timers: make it configurable John Stultz <john.stultz@linaro.org> - 2016-09-15 20:20 +0200
    Re: [PATCH v3) posix-timers: make it configurable Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-09-15 20:30 +0200
      Re: [PATCH v3) posix-timers: make it configurable Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-09-15 20:40 +0200
        Re: [PATCH v3) posix-timers: make it configurable John Stultz <john.stultz@linaro.org> - 2016-09-15 20:50 +0200
          Re: [PATCH v3) posix-timers: make it configurable Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-09-15 21:40 +0200
            Re: [PATCH v3) posix-timers: make it configurable John Stultz <john.stultz@linaro.org> - 2016-09-15 22:00 +0200
              Re: [PATCH v3) posix-timers: make it configurable Richard Cochran <richardcochran@gmail.com> - 2016-09-15 23:10 +0200
                Re: [PATCH v3) posix-timers: make it configurable Josh Triplett <josh@joshtriplett.org> - 2016-09-15 23:20 +0200
                Re: [PATCH v3) posix-timers: make it configurable Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-09-15 23:40 +0200
                Re: [PATCH v3) posix-timers: make it configurable Josh Triplett <josh@joshtriplett.org> - 2016-09-16 00:00 +0200
                Re: [PATCH v3) posix-timers: make it configurable Richard Cochran <richardcochran@gmail.com> - 2016-09-16 09:30 +0200
                Re: [PATCH v3) posix-timers: make it configurable Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-09-17 05:00 +0200
                Re: [PATCH v3) posix-timers: make it configurable Richard Cochran <richardcochran@gmail.com> - 2016-09-18 16:40 +0200
                Re: [PATCH v3) posix-timers: make it configurable Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-09-18 19:00 +0200
                Re: [PATCH v3) posix-timers: make it configurable Richard Cochran <richardcochran@gmail.com> - 2016-09-18 20:30 +0200
                Re: [PATCH v3) posix-timers: make it configurable Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-09-18 21:00 +0200
                Re: [PATCH v3) posix-timers: make it configurable Richard Cochran <richardcochran@gmail.com> - 2016-09-18 22:30 +0200
                Re: [PATCH v3) posix-timers: make it configurable Nicolas Pitre <nicolas.pitre@linaro.org> - 2016-09-18 22:40 +0200
                Re: [PATCH v3) posix-timers: make it configurable Richard Cochran <richardcochran@gmail.com> - 2016-09-18 23:20 +0200
              Re: [PATCH v3) posix-timers: make it configurable Richard Cochran <richardcochran@gmail.com> - 2016-09-15 23:30 +0200
      Re: [PATCH v3) posix-timers: make it configurable John Stultz <john.stultz@linaro.org> - 2016-09-15 20:40 +0200
  Re: [PATCH v3) posix-timers: make it configurable kbuild test robot <lkp@intel.com> - 2016-09-16 16:40 +0200

csiph-web