From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-qt1-x82d.google.com (mail-qt1-x82d.google.com [IPv6:2607:f8b0:4864:20::82d]) by sourceware.org (Postfix) with ESMTPS id 2C85C384241E for ; Fri, 19 Mar 2021 18:31:33 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 2C85C384241E Received: by mail-qt1-x82d.google.com with SMTP id u7so7477372qtq.12 for ; Fri, 19 Mar 2021 11:31:33 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=5INoyruRuVsXLeymKy4HUvGFq2C9kSXdYvtpFIEVZrg=; b=LmQwSKsOxD0dXBPvLTrCPmDxKimt8t29m+HgdUJVTPaMHDcNJJy/evE6w2N6YG3Dhj 4TAKTz/e80yb3lCqPROEhznqh3oUlYRhblbuWw+b7S6gCB9nDMGp91op2qqSFcDtDOQn q7CLpTnYg+cV/NG13Z7dKRAftoG1h1oWgBgn8fqxkpge/MO+FLV1UGjZcmhV6MCIUAKn seJga76ygUwPSNlS5JOaYFU3mQlJsi/VMJLyDKHDxU5zNNfVK3onPdUVaUfXdvU0bSwt xn+MSevcoz+42qgAkpgoFhzBcG6iXw7DEd3BoDRQpRXMSgI/rjXcpwvWUzAMoY6yuXmC cjcQ== X-Gm-Message-State: AOAM530c2HRnABrQIXu2s9GyluKWYeRsyi/3K3r7fRK3C1f2nXlwhMI1 VC84M8EbRllvVqXiTC9Tme+fcF4BRgSV4CFk X-Google-Smtp-Source: ABdhPJw9IsFIQEEyK2Kw5GXaQ1XKnVLSO0LHJMchM1wnsGfPRzwy2zSskmEAkaX7Qd9X46V10OC4AA== X-Received: by 2002:ac8:498d:: with SMTP id f13mr9333325qtq.54.1616178692318; Fri, 19 Mar 2021 11:31:32 -0700 (PDT) Received: from localhost.localdomain ([177.194.48.209]) by smtp.googlemail.com with ESMTPSA id z11sm5205747qkg.52.2021.03.19.11.31.31 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 19 Mar 2021 11:31:32 -0700 (PDT) From: Adhemerval Zanella To: libc-alpha@sourceware.org Subject: [PATCH 5/5] linux: Add y2106 support on utimensat tests Date: Fri, 19 Mar 2021 15:31:21 -0300 Message-Id: <20210319183121.2252064-6-adhemerval.zanella@linaro.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20210319183121.2252064-1-adhemerval.zanella@linaro.org> References: <20210319183121.2252064-1-adhemerval.zanella@linaro.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-13.3 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, KAM_SHORT, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: libc-alpha@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libc-alpha mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 19 Mar 2021 18:31:35 -0000 The tests are refactored to use a common skeleton that handles whether the underlying filesystem supports 64 bit time, skips 64 bit time tests when the TU only supports 32 bit, and also skip 64 bit time tests larger than 32 unsigned int (y2106) if the system does not support it (MIPSn64 on kernels without statx support). Checked on x86_64-linux-gnu and i686-linux-gnu. I also checked on a mips64el-linux-gnu with 4.1.4 and 5.10.0-4-5kc-malta kernel to verify if the y2106 are indeed skipped. --- sysdeps/unix/sysv/linux/tst-futimens.c | 63 ++----------- sysdeps/unix/sysv/linux/tst-utime.c | 63 ++----------- .../unix/sysv/linux/tst-utimensat-skeleton.c | 92 +++++++++++++++++++ sysdeps/unix/sysv/linux/tst-utimes.c | 68 +++----------- 4 files changed, 120 insertions(+), 166 deletions(-) create mode 100644 sysdeps/unix/sysv/linux/tst-utimensat-skeleton.c diff --git a/sysdeps/unix/sysv/linux/tst-futimens.c b/sysdeps/unix/sysv/linux/tst-futimens.c index 785cd87557..ac7a980017 100644 --- a/sysdeps/unix/sysv/linux/tst-futimens.c +++ b/sysdeps/unix/sysv/linux/tst-futimens.c @@ -16,57 +16,18 @@ License along with the GNU C Library; if not, see . */ -#include -#include -#include -#include #include -#include #include -#include - -static int temp_fd = -1; -static char *testfile; - -/* struct timespec array with Y2038 threshold minus 2 and 1 seconds. */ -const struct timespec t1[2] = { { 0x7FFFFFFE, 0 }, { 0x7FFFFFFF, 0 } }; - -/* struct timespec array with Y2038 threshold plus 1 and 2 seconds. */ -const struct timespec t2[2] = { { 0x80000001ULL, 0 }, { 0x80000002ULL, 0 } }; - -/* struct timespec array around Y2038 threshold. */ -const struct timespec t3[2] = { { 0x7FFFFFFE, 0 }, { 0x80000002ULL, 0 } }; - -#define PREPARE do_prepare -static void -do_prepare (int argc, char *argv[]) -{ - temp_fd = create_temp_file ("futimensat", &testfile); - TEST_VERIFY_EXIT (temp_fd > 0); -} +#include static int -test_futimens_helper (const struct timespec *ts) +test_futimens_helper (const char *file, int fd, const struct timespec *ts) { - if (!support_path_support_time64 (testfile)) - FAIL_UNSUPPORTED ("File %s does not support 64-bit timestamps", - testfile); - - struct stat64 st; - int result; - time_t t; - - /* Check if we run on port with 32 bit time_t size */ - if (__builtin_add_overflow (ts->tv_sec, 0, &t)) - { - printf("time_t overflow!"); - return 0; - } - - result = futimens (temp_fd, ts); + int result = futimens (fd, ts); TEST_VERIFY_EXIT (result == 0); - xfstat (temp_fd, &st); + struct stat64 st; + xfstat (fd, &st); /* Check if seconds for atime match */ TEST_COMPARE (st.st_atime, ts[0].tv_sec); @@ -77,14 +38,8 @@ test_futimens_helper (const struct timespec *ts) return 0; } -static int -do_test (void) -{ - test_futimens_helper (&t1[0]); - test_futimens_helper (&t2[0]); - test_futimens_helper (&t3[0]); - - return 0; -} +#define TEST_CALL(fname, fd, v1, v2) \ + test_futimens_helper (fname, fd, (struct timespec[]) { { v1, 0 }, \ + { v2, 0 } }) -#include +#include "tst-utimensat-skeleton.c" diff --git a/sysdeps/unix/sysv/linux/tst-utime.c b/sysdeps/unix/sysv/linux/tst-utime.c index 21e4e41dea..c1c7e75a29 100644 --- a/sysdeps/unix/sysv/linux/tst-utime.c +++ b/sysdeps/unix/sysv/linux/tst-utime.c @@ -16,59 +16,19 @@ License along with the GNU C Library; if not, see . */ -#include -#include -#include -#include -#include #include #include -#include #include -#include - -static int temp_fd = -1; -static char *testfile; - -/* struct utimbuf with Y2038 threshold minus 2 and 1 seconds. */ -const static struct utimbuf t1 = { 0x7FFFFFFE, 0x7FFFFFFF }; - -/* struct utimbuf with Y2038 threshold plus 1 and 2 seconds. */ -const static struct utimbuf t2 = { 0x80000001ULL, 0x80000002ULL }; - -/* struct utimbuf around Y2038 threshold. */ -const static struct utimbuf t3 = { 0x7FFFFFFE, 0x80000002ULL }; - -#define PREPARE do_prepare -static void -do_prepare (int argc, char *argv[]) -{ - temp_fd = create_temp_file ("utime", &testfile); - TEST_VERIFY_EXIT (temp_fd > 0); -} +#include static int -test_utime_helper (const struct utimbuf *ut) +test_utime_helper (const char *file, int fd, const struct utimbuf *ut) { - if (!support_path_support_time64 (testfile)) - FAIL_UNSUPPORTED ("File %s does not support 64-bit timestamps", - testfile); - - struct stat64 st; - int result; - time_t t; - - /* Check if we run on port with 32 bit time_t size */ - if (__builtin_add_overflow (ut->actime, 0, &t)) - { - printf("time_t overflow!"); - return 0; - } - - result = utime (testfile, ut); + int result = utime (file, ut); TEST_VERIFY_EXIT (result == 0); - xfstat (temp_fd, &st); + struct stat64 st; + xfstat (fd, &st); /* Check if seconds for actime match */ TEST_COMPARE (st.st_atime, ut->actime); @@ -79,14 +39,7 @@ test_utime_helper (const struct utimbuf *ut) return 0; } -static int -do_test (void) -{ - test_utime_helper (&t1); - test_utime_helper (&t2); - test_utime_helper (&t3); - - return 0; -} +#define TEST_CALL(fname, fd, v1, v2) \ + test_utime_helper (fname, fd, &(struct utimbuf) { (v1), (v2) }) -#include +#include "tst-utimensat-skeleton.c" diff --git a/sysdeps/unix/sysv/linux/tst-utimensat-skeleton.c b/sysdeps/unix/sysv/linux/tst-utimensat-skeleton.c new file mode 100644 index 0000000000..ed3d537b1f --- /dev/null +++ b/sysdeps/unix/sysv/linux/tst-utimensat-skeleton.c @@ -0,0 +1,92 @@ +/* Common tests for utimensat routines. + Copyright (C) 2021 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#include +#include +#include +#include +#include + +static int temp_fd = -1; +static char *testfile; + +const static struct { + int64_t v1; + int64_t v2; +} tests[] = { + /* Y2038 threshold minus 2 and 1 seconds. */ + { 0x7FFFFFFELL, 0x7FFFFFFFLL }, + /* Y2038 threshold plus 1 and 2 seconds. */ + { 0x80000001LL, 0x80000002LL }, + /* Around Y2038 threshold. */ + { 0x7FFFFFFELL, 0x80000002LL }, + /* Y2106 threshold minus 2 and 1 seconds. */ + { 0x100000000LL, 0xFFFFFFFELL }, + /* Y2106 threshold plus 1 and 2 seconds. */ + { 0x100000001LL, 0x100000002LL }, + /* Around Y2106 threshold. */ + { 0xFFFFFFFELL, 0xFFFFFFFELL }, +}; + +#define PREPARE do_prepare +static void +do_prepare (int argc, char *argv[]) +{ + temp_fd = create_temp_file ("utime", &testfile); + TEST_VERIFY_EXIT (temp_fd > 0); +} + +static int +do_test (void) +{ + if (!support_path_support_time64 (testfile)) + FAIL_UNSUPPORTED ("File %s does not support 64-bit timestamps", + testfile); + + bool y2106 = support_path_support_time64_value (testfile, + 0x100000001LL, + 0x100000002LL); + + for (int i = 0; i < array_length (tests); i++) + { + /* Check if we run on port with 32 bit time_t size. */ + time_t t; + if (__builtin_add_overflow (tests[i].v1, 0, &t) + || __builtin_add_overflow (tests[i].v2, 0, &t)) + { + printf ("warning: skipping tests[%d] { %" PRIx64 ", %" PRIx64 " }: " + "time_t overflows\n", i, tests[i].v1, tests[i].v2); + continue; + } + + if (tests[i].v1 >= 0x100000000LL && !y2106) + { + printf ("warning: skipping tests[%d] { %" PRIx64 ", %" PRIx64 " }: " + "unsupported timestamp value\n", + i, tests[i].v1, tests[i].v2); + continue; + } + + //test_utime_helper (&(struct utimbuf) { tests[i].v1, tests[i].v2 }); + TEST_CALL (testfile, temp_fd, tests[i].v1, tests[i].v2); + } + + return 0; +} + +#include diff --git a/sysdeps/unix/sysv/linux/tst-utimes.c b/sysdeps/unix/sysv/linux/tst-utimes.c index 0f23e44897..ba710508b7 100644 --- a/sysdeps/unix/sysv/linux/tst-utimes.c +++ b/sysdeps/unix/sysv/linux/tst-utimes.c @@ -16,60 +16,20 @@ License along with the GNU C Library; if not, see . */ -#include -#include -#include -#include -#include #include -#include #include -#include - -static int temp_fd = -1; -static char *testfile; - -/* struct timeval array with Y2038 threshold minus 2 and 1 seconds. */ -const static struct timeval t1[2] = { { 0x7FFFFFFE, 0 }, { 0x7FFFFFFF, 0 } }; - -/* struct timeval array with Y2038 threshold plus 1 and 2 seconds. */ -const static -struct timeval t2[2] = { { 0x80000001ULL, 0 }, { 0x80000002ULL, 0 } }; - -/* struct timeval array around Y2038 threshold. */ -const static -struct timeval t3[2] = { { 0x7FFFFFFE, 0 }, { 0x80000002ULL, 0 } }; - -#define PREPARE do_prepare -static void -do_prepare (int argc, char *argv[]) -{ - temp_fd = create_temp_file ("utimes", &testfile); - TEST_VERIFY_EXIT (temp_fd > 0); -} +#include +#include +#include static int -test_utime_helper (const struct timeval *tv) +test_utimes_helper (const char *file, int fd, const struct timeval *tv) { - if (!support_path_support_time64 (testfile)) - FAIL_UNSUPPORTED ("File %s does not support 64-bit timestamps", - testfile); - - struct stat64 st; - int result; - time_t t; - - /* Check if we run on port with 32 bit time_t size */ - if (__builtin_add_overflow (tv->tv_sec, 0, &t)) - { - printf("time_t overflow!"); - return 0; - } - - result = utimes (testfile, tv); + int result = utimes (file, tv); TEST_VERIFY_EXIT (result == 0); - xfstat (temp_fd, &st); + struct stat64 st; + xfstat (fd, &st); /* Check if seconds for atime match */ TEST_COMPARE (st.st_atime, tv[0].tv_sec); @@ -80,14 +40,8 @@ test_utime_helper (const struct timeval *tv) return 0; } -static int -do_test (void) -{ - test_utime_helper (&t1[0]); - test_utime_helper (&t2[0]); - test_utime_helper (&t3[0]); - - return 0; -} +#define TEST_CALL(fname, fd, v1, v2) \ + test_utimes_helper (fname, fd, (struct timeval[]) { { v1, 0 }, \ + { v2, 0 } }) -#include +#include "tst-utimensat-skeleton.c" -- 2.25.1