Received: by 2002:a25:31c3:0:0:0:0:0 with SMTP id x186csp6043532ybx; Mon, 11 Nov 2019 02:59:08 -0800 (PST) X-Google-Smtp-Source: APXvYqwfVm3fpW7wRATRF42XqzOENFsF6UWxa5Aj3XYxrgh0WblRsoTnqT7UXkDwvlaIeWaGmDwd X-Received: by 2002:aa7:c453:: with SMTP id n19mr26413076edr.103.1573469947960; Mon, 11 Nov 2019 02:59:07 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1573469947; cv=none; d=google.com; s=arc-20160816; b=iYD49UYddb60RvfWXXlqXMubYaCZkPG7X85rX+lkYxMwX0n5rvtwnPmh4ZWZWQ9xqw C4jmnvBY1G8a/FiAtaNE3RWkV6JfFxdCf1LnXIzOy2t7/q+Cve39U6ADs3++yN77h6MK 9yZizxJWvaRfOR6u4OAi1hrGPHFT7JkyGnKDlTp3SeSj6fXFR2nFekcJtCgmeS7sSxjd b7wR46LcxA7/60hV1GNqwYAp1Ern6HBYwLSwuACvK8C0TpETNp7HkmlAFpY4lRVM59xK A63ZRRZxU5lPyEj0dQnwqNuJf5LXkMvu2W6vYD2RkkD8oHydTzyP/5DOOMnXzTFrAJzS zo8Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version; bh=Brxah728uXZ7wZM4sH0lke+1WIS5UruZar5yCsFRYMg=; b=HXlNyRZ5t+QkzYirZwd4jPPSzwMyWkATm+mQsbGSVBArP8v6g+Yu5hoIdutykFJ6h+ ikrWCU3O6AgVivNZD1M6RNlDougrY3bjb36ktGgJCkb/JDauHQz8qp7hrjaVI85u0FV2 MDhJt42oNV0VhwsyosXR9rwBV/3euhU1K52nfGRNuMnlUwRR1zbHMk2YZK7cOF/01lIY wmmaWkXgeDfC9tzr++EzTw/H0YxiFZOj/LROO/VgJlTWsh2aCiupIt/hvS0EFAOISnRP YViC4WGFP7VTXMISCqi0uRqLlu1x/TMp2zuUNybo70658uO6nu5LLZ+zXCZiMVj1tnTc KAPg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id z13si8710350ejp.16.2019.11.11.02.58.43; Mon, 11 Nov 2019 02:59:07 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726888AbfKKK6M (ORCPT + 99 others); Mon, 11 Nov 2019 05:58:12 -0500 Received: from mout.kundenserver.de ([212.227.126.135]:50599 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726819AbfKKK6L (ORCPT ); Mon, 11 Nov 2019 05:58:11 -0500 Received: from mail-qk1-f182.google.com ([209.85.222.182]) by mrelayeu.kundenserver.de (mreue012 [212.227.15.129]) with ESMTPSA (Nemesis) id 1MzCMN-1hiBRi44ah-00wA8T; Mon, 11 Nov 2019 11:58:10 +0100 Received: by mail-qk1-f182.google.com with SMTP id q70so10718563qke.12; Mon, 11 Nov 2019 02:58:09 -0800 (PST) X-Gm-Message-State: APjAAAVGs/HLw5w5QN7PdknxvBbPPa+/ga1tchoz59C0U9Znw6TnUbO3 UsyHt/OO8nyJKMh2suz6LlEXAm9EtYZ0RAhH2AE= X-Received: by 2002:a37:4f0a:: with SMTP id d10mr9793076qkb.286.1573469888741; Mon, 11 Nov 2019 02:58:08 -0800 (PST) MIME-Version: 1.0 References: <20191108210236.1296047-1-arnd@arndb.de> <20191108211323.1806194-11-arnd@arndb.de> In-Reply-To: From: Arnd Bergmann Date: Mon, 11 Nov 2019 11:57:52 +0100 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 20/23] y2038: move itimer reset into itimer.c To: Ondrej Mosnacek Cc: y2038 Mailman List , John Stultz , Thomas Gleixner , Paul Moore , Stephen Smalley , Eric Paris , Linux kernel mailing list , Stephen Boyd , Sebastian Andrzej Siewior , Ingo Molnar , Anna-Maria Gleixner , Al Viro , SElinux list Content-Type: text/plain; charset="UTF-8" X-Provags-ID: V03:K1:talTFI+q4JcUPtrySBmEjOPKUX09D5LJWAeHUZ9hJvNb739cZt9 beNlgaGDcvAG6HfQekZh9C1WXKFXZVY+PnMf1cp+rBsPF8iCawUpVm/+B2VsDIcvrmW9DoX S+TTeCJIJP+aLnKW6OXhOQJ1UlI5VO1bs1T97GHY/AAVcxY5xwsuYi3cCkEP3kwKO6yJXOd ppjy5hRI3OLgEMEj8GOqw== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1;V03:K0:kVY/Hkf8KzU=:pIeNep2ZsiaPrEk1sTX9vl UgWYRr7g53x76Rco7DZ7BQVwqeNBhsZsmtWe++MfRbgJ55SxQOf7QGjcLlMDMEwuzRFJ1d7bm ioY3q7iNCYVBPzdyXgG5wGf9446ceDEr9m3AGZj26nCfckAfr8Pcf58WRom4V352d3A7SJ5J+ R1PwWR29vwIxV3PnNVYjGfo8ao9i3oOkQDOHJFc3bK3zVkoxmUmq8S5u8kUvTuUIGxAsO6WGR UyP/uhYziM87FY8YflxQNBQBSAu/nhHO/um3mUQHN3ACDoZ2FAEJx+RkaKLDgiOxWseMUBxFQ X14OWzQ1k6M2sFPFkiUjvOvjXFvhmw5CaZHV2Y0MWIMO+5KQ+qa3C1azxaT4BMAWAWBda84sf GN1qbWtJd1/fagJxkA+7p2fDf22TbK0gZRHh4a5sTP/UD+q+IK6b9Sr2eXIRIfTFkHplvPfbu j224YhRg+sabtqCQKv60QVYT2XmKnp//Qq9+TMCDWUktsO/l1IkUCp0FR8MABhLz3YZnCtYMI bQ31NS2SVhS3ShQkE8wVDJVo3Wrj/O9oJUpJ9jdP6cVb1pARgTAQCBn2OOKuNBKsrqYfkoJJs F/XVqvSVzQiPtW3du8a41o5n9/dOpehRpVx5ts1iea32YrkqMVT0XhVBue/d4zEC2NCgVsYWX mRO2YmELI7uNwyVvKQjxGcC1DGJsdnWlsITP/QoFXEFrLJAI7yT4BlglBIJWV/NZpV0sOomPB /BBnDDjg2sPTX97yhQJnvQ74fkqpOLc573MnLdKRyhqyoti01trSoGuob1TZ6LI3fv3yhlh6A S6tcfVTFRmOFj6W09hA28klBnJhPES5sOxEi5tJD6zjc8wAGVsdOu+258i4dZrk1ebqEKohMS tYGZ1AsqLEKvpuBjfv5g== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, Nov 10, 2019 at 12:07 AM Ondrej Mosnacek wrote: > > On Sat, Nov 9, 2019 at 10:03 PM Arnd Bergmann wrote: > > > > On Sat, Nov 9, 2019 at 2:43 PM Ondrej Mosnacek wrote: > > > > > > -struct itimerval; > > > > -extern int do_setitimer(int which, struct itimerval *value, > > > > - struct itimerval *ovalue); > > > > -extern int do_getitimer(int which, struct itimerval *value); > > > > +#ifdef CONFIG_POSIX_TIMERS > > > > +extern void clear_itimer(void); > > > > +#else > > > > +static inline void clear_itimer(void) {} > > > > +#endif > > > > > > > > > > @@ -249,6 +249,17 @@ int do_setitimer(int which, struct itimerval *value, struct itimerval *ovalue) > > > > return 0; > > > > } > > > > > > > > +#ifdef CONFIG_SECURITY_SELINUX > > > > > > Did you mean "#ifdef CONFIG_POSIX_TIMERS" here to match the header? > > > > No, this part is intentional, CONFIG_POSIX_TIMERS already controls > > whether itimer.c is > > compiled in the first place, but this function is only needed when called from > > the selinux driver. > > All right, but you declare the function in time.h even if > CONFIG_SECURITY_SELINUX is not enabled... it is kind of awkward when > it can happen that the function is declared but not defined anywhere > (even if it shouldn't be used by new users). Maybe you could at least > put the header declaration/definition inside #ifdef > CONFIG_SECURITY_SELINUX as well so it is clear that this function is > intended for SELinux only? I don't see that as a problem, we rarely put declarations inside of an #ifdef. The main effect that would have is forcing any file that includes linux/time.h to be rebuilt when selinux is turned on or off in the .config. Arnd