Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752738AbbFBTW1 (ORCPT ); Tue, 2 Jun 2015 15:22:27 -0400 Received: from www.linutronix.de ([62.245.132.108]:52620 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752125AbbFBTUA (ORCPT ); Tue, 2 Jun 2015 15:20:00 -0400 Date: Tue, 2 Jun 2015 21:20:02 +0200 (CEST) From: Thomas Gleixner To: Baolin Wang cc: arnd@arndb.de, john.stultz@linaro.org, heenasirwani@gmail.com, pang.xunlei@linaro.org, linux-kernel@vger.kernel.org, y2038@lists.linaro.org Subject: Re: [PATCH v4 05/25] time:Introduce the do_sys_settimeofday64() function with timespec64 type In-Reply-To: <1433159662-4742-1-git-send-email-baolin.wang@linaro.org> Message-ID: References: <1433159662-4742-1-git-send-email-baolin.wang@linaro.org> User-Agent: Alpine 2.11 (DEB 23 2013-08-11) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Linutronix-Spam-Score: -1.0 X-Linutronix-Spam-Level: - X-Linutronix-Spam-Status: No , -1.0 points, 5.0 required, ALL_TRUSTED=-1,SHORTCIRCUIT=-0.0001 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1787 Lines: 59 On Mon, 1 Jun 2015, Baolin Wang wrote: Subject line sucks. > Introduce the do_sys_settimeofday64() function with timespec64 type to make > it is ready for 2038 issue when setting the time of day. > > And move the do_sys_settimeofday() function to the timekeeping.h file, that it > is convenient to delete it later. See other mails. > extern void do_gettimeofday(struct timeval *tv); > extern int do_settimeofday64(const struct timespec64 *ts); > -extern int do_sys_settimeofday(const struct timespec *tv, > - const struct timezone *tz); > +extern int do_sys_settimeofday64(const struct timespec64 *tv, > + const struct timezone *tz); > +static inline int do_sys_settimeofday(const struct timespec *tv, > + const struct timezone *tz) > +{ > + struct timespec64 ts64; > + > + ts64 = timespec_to_timespec64(*tv); > + return do_sys_settimeofday64(&ts64, tz); If you write it: struct timespec64 ts64 = timespec_to_timespec64(*tv); return do_sys_settimeofday64(&ts64, tz); You spare a line per inline function. Applies to the other patches as well. > > -int do_sys_settimeofday(const struct timespec *tv, const struct timezone *tz) > +int do_sys_settimeofday64(const struct timespec64 *tv, const struct timezone *tz) > { > static int firsttime = 1; > int error = 0; > > - if (tv && !timespec_valid(tv)) > + if (tv && !timespec64_valid(tv)) > return -EINVAL; > > - error = security_settime(tv, tz); > + error = security_settime64(tv, tz); No such function. Fails to build. Thanks, tglx -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/