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


Groups > linux.kernel > #1483840 > unrolled thread

[PATCH v3) posix-timers: make it configurable

Started byNicolas Pitre <nicolas.pitre@linaro.org>
First post2016-09-15 05:50 +0200
Last post2016-09-16 16:40 +0200
Articles 5 on this page of 25 — 5 participants

Back to article view | Back to linux.kernel


Contents

  [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

Page 2 of 2 — ← Prev page 1 [2]


#1485942

FromNicolas Pitre <nicolas.pitre@linaro.org>
Date2016-09-18 22:40 +0200
Message-ID<siR2y-4o4-25@gated-at.bofh.it>
In reply to#1485939
On Sun, 18 Sep 2016, Richard Cochran wrote:

> On Sun, Sep 18, 2016 at 02:49:08PM -0400, Nicolas Pitre wrote:
> > Who should merge this patch?
> 
> John, I guess.  Or do we need acks from the driver maintainers?  In
> that case, please post to netdev, and then davem can merge it.

I don't think the driver changes are significant enough to warrant a ACK 
from individual driver maintainers.

Yours would be welcome though.  ;-)

Then I'll resend the whole thing to John as a series and CC netdev.

What do you think?


Nicolas

[toc] | [prev] | [next] | [standalone]


#1486070

FromRichard Cochran <richardcochran@gmail.com>
Date2016-09-18 23:20 +0200
Message-ID<siRFf-4S5-9@gated-at.bofh.it>
In reply to#1485942
On Sun, Sep 18, 2016 at 04:30:21PM -0400, Nicolas Pitre wrote:
> I don't think the driver changes are significant enough to warrant a ACK 
> from individual driver maintainers.
> 
> Yours would be welcome though.  ;-)

Acked-by: Richard Cochran <richardcochran@gmail.com>

> Then I'll resend the whole thing to John as a series and CC netdev.
> 
> What do you think?

Sounds ok to me.

Thanks,
Richard

[toc] | [prev] | [next] | [standalone]


#1484551

FromRichard Cochran <richardcochran@gmail.com>
Date2016-09-15 23:30 +0200
Message-ID<shMoh-3TA-3@gated-at.bofh.it>
In reply to#1484509
On Thu, Sep 15, 2016 at 12:58:22PM -0700, John Stultz wrote:
> This doesn't look too bad.

Actually, it is worse than ugly.  It is a disaster.  With this change,
registration will succeed and MAC drivers will happily report the PTP
devices in 'ethtool -T', but there won't be anything behind them.

IMHO, it would be better in this case to stub out the whole
subsystem.  That would save you ptp_clock.o, ptp_chardev.o, and
ptp_sysfs.o.

Thanks,
Richard

[toc] | [prev] | [next] | [standalone]


#1484458

FromJohn Stultz <john.stultz@linaro.org>
Date2016-09-15 20:40 +0200
Message-ID<shJJL-28Z-5@gated-at.bofh.it>
In reply to#1484451
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..

thanks
-john

[toc] | [prev] | [next] | [standalone]


#1485060

Fromkbuild test robot <lkp@intel.com>
Date2016-09-16 16:40 +0200
Message-ID<si2t3-5We-15@gated-at.bofh.it>
In reply to#1483840

[Multipart message — attachments visible in raw view] — view raw

Hi Nicolas,

[auto build test WARNING on linus/master]
[also build test WARNING on v4.8-rc6 next-20160916]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
[Suggest to use git(>=2.9.0) format-patch --base=<commit> (or --base=auto for convenience) to record what (public, well-known) commit your patch series was built on]
[Check https://git-scm.com/docs/git-format-patch for more information]

url:    https://github.com/0day-ci/linux/commits/Nicolas-Pitre/PATCH-v3-posix-timers-make-it-configurable/20160915-122808
config: x86_64-randconfig-a0-09161200 (attached as .config)
compiler: gcc-4.4 (Debian 4.4.7-8) 4.4.7
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

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)

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[toc] | [prev] | [standalone]


Page 2 of 2 — ← Prev page 1 [2]

Back to top | Article view | linux.kernel


csiph-web