From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1758491AbcIHVT0 (ORCPT <rfc822;w@1wt.eu>);
        Thu, 8 Sep 2016 17:19:26 -0400
Received: from mail-oi0-f53.google.com ([209.85.218.53]:34120 "EHLO
        mail-oi0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1751596AbcIHVTZ (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Thu, 8 Sep 2016 17:19:25 -0400
MIME-Version: 1.0
In-Reply-To: <alpine.LFD.2.20.1609081658570.14769@knanqh.ubzr>
References: <alpine.LFD.2.20.1609081658570.14769@knanqh.ubzr>
From: John Stultz <john.stultz@linaro.org>
Date: Thu, 8 Sep 2016 14:19:24 -0700
Message-ID: <CALAqxLXmbDgcc1oVZezzY=oRV4johPgxFNpWBBfO4Hk9tZiBJg@mail.gmail.com>
Subject: Re: [RFC/PATCH] posix-timers: make them configurable
To: Nicolas Pitre <nico@fluxnic.net>,
        Josh Triplett <josh@joshtriplett.org>
Cc: Thomas Gleixner <tglx@linutronix.de>,
        lkml <linux-kernel@vger.kernel.org>,
        Richard Cochran <richardcochran@gmail.com>
Content-Type: text/plain; charset=UTF-8
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Thu, Sep 8, 2016 at 2:05 PM, Nicolas Pitre <nico@fluxnic.net> wrote:
>
> Small embedded systems typically don't need them.  This removes about
> 16KB from the kernel binary size on ARM when configured out.
> Corresponding syscalls are routed to a stub logging the attempt to
> use those syscalls which should be enough of a clue if they were
> disabled without proper consideration.
>
> Signed-off-by: Nicolas Pitre <nico@linaro.org>

Hrm... So being able to trim down the kernel is important.

Although my sense is that momentum has been moving to clock_gettime()
over gettimeofday(), such that gettimeofday() is mostly a shim over
clock_gettime() logic wise. So this is sort of going the other
direction.

Also given many other syscalls take clockids and the backing logic
isn't really getting removed (probably could cut the dynamic posix
clocks core with the same conditional), I wonder if you could get a
similar size win by taking a slightly more narrow cutting of the
subsystem. That way you could preserve the more useful clock_gettime()
functionality, but maybe stub out some of the less often used
functionality.

Josh (cc'ed) also was talking awhile back about cutting out the core
NTP logic. Having a single minimal-time option might be nice rather
then having a bunch of different conditionals that might be combined.

thanks
-john