[libc] [llvm] [libc] Implemented utimes (Issue #133953) (PR #134167)
Aditya Tejpaul via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 2 18:34:39 PDT 2025
https://github.com/hoarfrost32 updated https://github.com/llvm/llvm-project/pull/134167
>From 61ea09a8d7f3499ed91aeaa396ca7871de7500e5 Mon Sep 17 00:00:00 2001
From: hoarfrost32 <aditya.tejpaul at research.iiit.ac.in>
Date: Wed, 2 Apr 2025 06:14:32 +0530
Subject: [PATCH 1/6] Adding utimes, tests left
---
__cmake_systeminformation/CMakeCache.txt | 86 ++++++++++++++++++
.../CMakeFiles/3.31.5/CMakeSystem.cmake | 15 +++
.../CMakeFiles/CMakeConfigureLog.yaml | 11 +++
.../CMakeFiles/cmake.check_cache | 1 +
__cmake_systeminformation/CMakeLists.txt | 91 +++++++++++++++++++
libc/config/linux/x86_64/entrypoints.txt | 3 +
libc/include/sys/time.yaml | 7 +-
libc/src/sys/CMakeLists.txt | 1 +
libc/src/sys/time/CMakeLists.txt | 10 ++
libc/src/sys/time/linux/CMakeLists.txt | 14 +++
libc/src/sys/time/linux/utimes.cpp | 75 +++++++++++++++
libc/src/sys/time/utimes.h | 21 +++++
libc/test/src/sys/CMakeLists.txt | 1 +
libc/test/src/sys/time/CMakeLists.txt | 21 +++++
libc/test/src/sys/time/utimes_test.cpp | 66 ++++++++++++++
15 files changed, 422 insertions(+), 1 deletion(-)
create mode 100644 __cmake_systeminformation/CMakeCache.txt
create mode 100644 __cmake_systeminformation/CMakeFiles/3.31.5/CMakeSystem.cmake
create mode 100644 __cmake_systeminformation/CMakeFiles/CMakeConfigureLog.yaml
create mode 100644 __cmake_systeminformation/CMakeFiles/cmake.check_cache
create mode 100644 __cmake_systeminformation/CMakeLists.txt
create mode 100644 libc/src/sys/time/CMakeLists.txt
create mode 100644 libc/src/sys/time/linux/CMakeLists.txt
create mode 100644 libc/src/sys/time/linux/utimes.cpp
create mode 100644 libc/src/sys/time/utimes.h
create mode 100644 libc/test/src/sys/time/CMakeLists.txt
create mode 100644 libc/test/src/sys/time/utimes_test.cpp
diff --git a/__cmake_systeminformation/CMakeCache.txt b/__cmake_systeminformation/CMakeCache.txt
new file mode 100644
index 0000000000000..880b49ed4f29d
--- /dev/null
+++ b/__cmake_systeminformation/CMakeCache.txt
@@ -0,0 +1,86 @@
+# This is the CMakeCache file.
+# For build in directory: /home/hoarfrost/Projects/llvm/llvm-project/__cmake_systeminformation
+# It was generated by CMake: /nix/store/ciiwq1ymzac9m5azhf4higkynmy3f8f7-cmake-3.31.5/bin/cmake
+# You can edit this file to change values found and used by cmake.
+# If you do not want to change any of the values, simply exit the editor.
+# If you do want to change a value, simply edit, save, and exit the editor.
+# The syntax for the file is as follows:
+# KEY:TYPE=VALUE
+# KEY is the name of a variable in the cache.
+# TYPE is a hint to GUIs for the type of VALUE, DO NOT EDIT TYPE!.
+# VALUE is the current value for the KEY.
+
+########################
+# EXTERNAL cache entries
+########################
+
+//Value Computed by CMake.
+CMAKE_FIND_PACKAGE_REDIRECTS_DIR:STATIC=/home/hoarfrost/Projects/llvm/llvm-project/__cmake_systeminformation/CMakeFiles/pkgRedirects
+
+//Path to a program.
+CMAKE_MAKE_PROGRAM:FILEPATH=CMAKE_MAKE_PROGRAM-NOTFOUND
+
+//Value Computed by CMake
+CMAKE_PROJECT_DESCRIPTION:STATIC=
+
+//Value Computed by CMake
+CMAKE_PROJECT_HOMEPAGE_URL:STATIC=
+
+//Value Computed by CMake
+CMAKE_PROJECT_NAME:STATIC=DumpInformation
+
+//Value Computed by CMake
+DumpInformation_BINARY_DIR:STATIC=/home/hoarfrost/Projects/llvm/llvm-project/__cmake_systeminformation
+
+//Value Computed by CMake
+DumpInformation_IS_TOP_LEVEL:STATIC=ON
+
+//Value Computed by CMake
+DumpInformation_SOURCE_DIR:STATIC=/home/hoarfrost/Projects/llvm/llvm-project/__cmake_systeminformation
+
+//No help, variable specified on the command line.
+RESULT_FILE:UNINITIALIZED=/home/hoarfrost/Projects/llvm/llvm-project/__cmake_systeminformation/results.txt
+
+
+########################
+# INTERNAL cache entries
+########################
+
+//This is the directory where this CMakeCache.txt was created
+CMAKE_CACHEFILE_DIR:INTERNAL=/home/hoarfrost/Projects/llvm/llvm-project/__cmake_systeminformation
+//Major version of cmake used to create the current loaded cache
+CMAKE_CACHE_MAJOR_VERSION:INTERNAL=3
+//Minor version of cmake used to create the current loaded cache
+CMAKE_CACHE_MINOR_VERSION:INTERNAL=31
+//Patch version of cmake used to create the current loaded cache
+CMAKE_CACHE_PATCH_VERSION:INTERNAL=5
+//Path to CMake executable.
+CMAKE_COMMAND:INTERNAL=/nix/store/ciiwq1ymzac9m5azhf4higkynmy3f8f7-cmake-3.31.5/bin/cmake
+//Path to cpack program executable.
+CMAKE_CPACK_COMMAND:INTERNAL=/nix/store/ciiwq1ymzac9m5azhf4higkynmy3f8f7-cmake-3.31.5/bin/cpack
+//Path to ctest program executable.
+CMAKE_CTEST_COMMAND:INTERNAL=/nix/store/ciiwq1ymzac9m5azhf4higkynmy3f8f7-cmake-3.31.5/bin/ctest
+//Name of external makefile project generator.
+CMAKE_EXTRA_GENERATOR:INTERNAL=
+//Name of generator.
+CMAKE_GENERATOR:INTERNAL=Unix Makefiles
+//Generator instance identifier.
+CMAKE_GENERATOR_INSTANCE:INTERNAL=
+//Name of generator platform.
+CMAKE_GENERATOR_PLATFORM:INTERNAL=
+//Name of generator toolset.
+CMAKE_GENERATOR_TOOLSET:INTERNAL=
+//Source directory with the top level CMakeLists.txt file for this
+// project
+CMAKE_HOME_DIRECTORY:INTERNAL=/home/hoarfrost/Projects/llvm/llvm-project/__cmake_systeminformation
+//ADVANCED property for variable: CMAKE_MAKE_PROGRAM
+CMAKE_MAKE_PROGRAM-ADVANCED:INTERNAL=1
+//number of local generators
+CMAKE_NUMBER_OF_MAKEFILES:INTERNAL=1
+//Platform information initialized
+CMAKE_PLATFORM_INFO_INITIALIZED:INTERNAL=1
+//Path to CMake installation.
+CMAKE_ROOT:INTERNAL=/nix/store/ciiwq1ymzac9m5azhf4higkynmy3f8f7-cmake-3.31.5/share/cmake-3.31
+//uname command
+CMAKE_UNAME:INTERNAL=/usr/bin/uname
+
diff --git a/__cmake_systeminformation/CMakeFiles/3.31.5/CMakeSystem.cmake b/__cmake_systeminformation/CMakeFiles/3.31.5/CMakeSystem.cmake
new file mode 100644
index 0000000000000..82aa456095f26
--- /dev/null
+++ b/__cmake_systeminformation/CMakeFiles/3.31.5/CMakeSystem.cmake
@@ -0,0 +1,15 @@
+set(CMAKE_HOST_SYSTEM "Linux-6.12.20")
+set(CMAKE_HOST_SYSTEM_NAME "Linux")
+set(CMAKE_HOST_SYSTEM_VERSION "6.12.20")
+set(CMAKE_HOST_SYSTEM_PROCESSOR "x86_64")
+
+
+
+set(CMAKE_SYSTEM "Linux-6.12.20")
+set(CMAKE_SYSTEM_NAME "Linux")
+set(CMAKE_SYSTEM_VERSION "6.12.20")
+set(CMAKE_SYSTEM_PROCESSOR "x86_64")
+
+set(CMAKE_CROSSCOMPILING "FALSE")
+
+set(CMAKE_SYSTEM_LOADED 1)
diff --git a/__cmake_systeminformation/CMakeFiles/CMakeConfigureLog.yaml b/__cmake_systeminformation/CMakeFiles/CMakeConfigureLog.yaml
new file mode 100644
index 0000000000000..b5439f4f72db7
--- /dev/null
+++ b/__cmake_systeminformation/CMakeFiles/CMakeConfigureLog.yaml
@@ -0,0 +1,11 @@
+
+---
+events:
+ -
+ kind: "message-v1"
+ backtrace:
+ - "/nix/store/ciiwq1ymzac9m5azhf4higkynmy3f8f7-cmake-3.31.5/share/cmake-3.31/Modules/CMakeDetermineSystem.cmake:205 (message)"
+ - "CMakeLists.txt:6 (project)"
+ message: |
+ The system is: Linux - 6.12.20 - x86_64
+...
diff --git a/__cmake_systeminformation/CMakeFiles/cmake.check_cache b/__cmake_systeminformation/CMakeFiles/cmake.check_cache
new file mode 100644
index 0000000000000..3dccd731726d7
--- /dev/null
+++ b/__cmake_systeminformation/CMakeFiles/cmake.check_cache
@@ -0,0 +1 @@
+# This file is generated by cmake for dependency checking of the CMakeCache.txt file
diff --git a/__cmake_systeminformation/CMakeLists.txt b/__cmake_systeminformation/CMakeLists.txt
new file mode 100644
index 0000000000000..97f385612c3b7
--- /dev/null
+++ b/__cmake_systeminformation/CMakeLists.txt
@@ -0,0 +1,91 @@
+# Distributed under the OSI-approved BSD 3-Clause License. See accompanying
+# file Copyright.txt or https://cmake.org/licensing for details.
+
+
+cmake_minimum_required(VERSION ${CMAKE_VERSION})
+project(DumpInformation)
+
+# first get the standard information for the platform
+include_directories("This does not exist")
+get_directory_property(incl INCLUDE_DIRECTORIES)
+set_directory_properties(PROPERTIES INCLUDE_DIRECTORIES "${DumpInformation_BINARY_DIR};${DumpInformation_SOURCE_DIR}")
+
+configure_file("${CMAKE_ROOT}/Modules/SystemInformation.in" "${RESULT_FILE}")
+
+
+file(APPEND "${RESULT_FILE}"
+ "\n=================================================================\n")
+file(APPEND "${RESULT_FILE}"
+ "=== VARIABLES\n")
+file(APPEND "${RESULT_FILE}"
+ "=================================================================\n")
+get_cmake_property(res VARIABLES)
+foreach(var ${res})
+ file(APPEND "${RESULT_FILE}" "${var} \"${${var}}\"\n")
+endforeach()
+
+file(APPEND "${RESULT_FILE}"
+ "\n=================================================================\n")
+file(APPEND "${RESULT_FILE}"
+ "=== COMMANDS\n")
+file(APPEND "${RESULT_FILE}"
+ "=================================================================\n")
+get_cmake_property(res COMMANDS)
+foreach(var ${res})
+ file(APPEND "${RESULT_FILE}" "${var}\n")
+endforeach()
+
+file(APPEND "${RESULT_FILE}"
+ "\n=================================================================\n")
+file(APPEND "${RESULT_FILE}"
+ "=== MACROS\n")
+file(APPEND "${RESULT_FILE}"
+ "=================================================================\n")
+file(WRITE ${CMAKE_CURRENT_BINARY_DIR}/AllMacros.txt "")
+get_cmake_property(res MACROS)
+foreach(var ${res})
+ file(APPEND "${RESULT_FILE}" "${var}\n")
+endforeach()
+
+file(APPEND "${RESULT_FILE}"
+ "\n=================================================================\n")
+file(APPEND "${RESULT_FILE}"
+ "=== OTHER\n")
+file(APPEND "${RESULT_FILE}"
+ "=================================================================\n")
+get_directory_property(res INCLUDE_DIRECTORIES)
+foreach(var ${res})
+ file(APPEND "${RESULT_FILE}" "INCLUDE_DIRECTORY: ${var}\n")
+endforeach()
+
+get_directory_property(res LINK_DIRECTORIES)
+foreach(var ${res})
+ file(APPEND "${RESULT_FILE}" "LINK_DIRECTORIES: ${var}\n")
+endforeach()
+
+get_directory_property(res INCLUDE_REGULAR_EXPRESSION)
+file(APPEND "${RESULT_FILE}" "INCLUDE_REGULAR_EXPRESSION: ${res}\n")
+
+# include other files if they are present, such as when run from within the
+# binary tree
+macro(DUMP_FILE THE_FILE)
+ if (EXISTS "${THE_FILE}")
+ file(APPEND "${RESULT_FILE}"
+ "\n=================================================================\n")
+ file(APPEND "${RESULT_FILE}"
+ "=== ${THE_FILE}\n")
+ file(APPEND "${RESULT_FILE}"
+ "=================================================================\n")
+
+ file(READ "${THE_FILE}" FILE_CONTENTS LIMIT 50000)
+ file(APPEND "${RESULT_FILE}" "${FILE_CONTENTS}")
+ endif ()
+endmacro()
+
+DUMP_FILE("../CMakeCache.txt")
+DUMP_FILE("../CMakeFiles/CMakeSystem.cmake")
+
+foreach (EXTRA_FILE ${EXTRA_DUMP_FILES})
+ DUMP_FILE("${EXTRA_FILE}")
+endforeach ()
+
diff --git a/libc/config/linux/x86_64/entrypoints.txt b/libc/config/linux/x86_64/entrypoints.txt
index eccd222fa123e..4430784ca94a1 100644
--- a/libc/config/linux/x86_64/entrypoints.txt
+++ b/libc/config/linux/x86_64/entrypoints.txt
@@ -288,6 +288,9 @@ set(TARGET_LIBC_ENTRYPOINTS
libc.src.sys.statvfs.fstatvfs
libc.src.sys.statvfs.statvfs
+ # sys/utimes.h entrypoints
+ libc.src.sys.utimes.utimes
+
# sys/utsname.h entrypoints
libc.src.sys.utsname.uname
diff --git a/libc/include/sys/time.yaml b/libc/include/sys/time.yaml
index ca497bbe92995..f2f0f5f949c77 100644
--- a/libc/include/sys/time.yaml
+++ b/libc/include/sys/time.yaml
@@ -5,5 +5,10 @@ macros: []
types:
- type_name: struct_timeval
enums: []
-functions: []
objects: []
+functions:
+ - name: utimes
+ return_type: int
+ arguments:
+ - type: ptr<const char>
+ - type: ptr<const struct timeval>
\ No newline at end of file
diff --git a/libc/src/sys/CMakeLists.txt b/libc/src/sys/CMakeLists.txt
index bb177f11c6d62..9a73b80d35d2f 100644
--- a/libc/src/sys/CMakeLists.txt
+++ b/libc/src/sys/CMakeLists.txt
@@ -8,6 +8,7 @@ add_subdirectory(socket)
add_subdirectory(sendfile)
add_subdirectory(stat)
add_subdirectory(statvfs)
+add_subdirectory(time)
add_subdirectory(utsname)
add_subdirectory(wait)
add_subdirectory(prctl)
diff --git a/libc/src/sys/time/CMakeLists.txt b/libc/src/sys/time/CMakeLists.txt
new file mode 100644
index 0000000000000..df8bf5b3630c3
--- /dev/null
+++ b/libc/src/sys/time/CMakeLists.txt
@@ -0,0 +1,10 @@
+if(EXISTS ${CMAKE_CURRENT_SOURCE_DIR}/${LIBC_TARGET_OS})
+ add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/${LIBC_TARGET_OS})
+endif()
+
+add_entrypoint_object(
+ utimes
+ ALIAS
+ DEPENDS
+ .${LIBC_TARGET_OS}.utimes
+)
\ No newline at end of file
diff --git a/libc/src/sys/time/linux/CMakeLists.txt b/libc/src/sys/time/linux/CMakeLists.txt
new file mode 100644
index 0000000000000..eedb4763bc43f
--- /dev/null
+++ b/libc/src/sys/time/linux/CMakeLists.txt
@@ -0,0 +1,14 @@
+add_entrypoint_object(
+ utimes
+ SRCS
+ utimes.cpp
+ HDRS
+ ../utimes.h
+ DEPENDS
+ libc.hdr.types.struct_timeval
+ libc.src.__support.OSUtil.osutil
+ libc.include.sys_stat
+ libc.include.sys_syscall
+ libc.src.__support.OSUtil.osutil
+ libc.src.errno.errno
+)
\ No newline at end of file
diff --git a/libc/src/sys/time/linux/utimes.cpp b/libc/src/sys/time/linux/utimes.cpp
new file mode 100644
index 0000000000000..f5cbbab7c13c5
--- /dev/null
+++ b/libc/src/sys/time/linux/utimes.cpp
@@ -0,0 +1,75 @@
+//===-- Linux implementation of utimes -------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===/
+
+#include "src/sys/time/utimes.h"
+
+#include "hdr/types/struct_timeval.h"
+
+#include "src/__support/OSUtil/syscall.h"
+#include "src/__support/common.h"
+
+#include "src/errno/libc_errno.h"
+
+#include <sys/syscall.h>
+#include <sys/stat.h>
+
+namespace LIBC_NAMESPACE_DECL {
+
+ LLVM_LIBC_FUNCTION(int, utimes, (const char *path, const struct timeval times[2]) {
+ long ret;
+
+ #ifdef SYS_utimensat
+ //the utimensat syscall requires a timespec struct, not timeval.
+ struct timespec ts[2];
+ struct timespec *ts_ptr = nullptr; // default value if times is NULL
+
+ // convert the microsec values in timeval struct times
+ // to nanosecond values in timespec struct ts
+ if (times != NULL) {
+
+ // ensure consistent values
+ if ((times[0].tv_usec < 0 || times[1].tv_usec < 0) ||
+ (times[0].tv_usec >= 1000000 || times[1].tv_usec >= 1000000)) {
+ return -1;
+ }
+
+ // set seconds in ts
+ ts[0].tv_sec = times[0].tv_sec;
+ ts[1].tv_sec = times[1].tv_sec;
+
+ // convert u-seconds to nanoseconds
+ ts[0].tv_nsec = times[0].tv_usec * 1000;
+ ts[1].tv_nsec = times[1].tv_usec * 1000;
+ }
+
+ // If times was NULL, ts_ptr remains NULL, which utimensat interprets
+ // as setting times to the current time.
+
+ // utimensat syscall.
+ // flags=0 means don't follow symlinks (like utimes)
+ ret = LIBC_NAMESPACE::syscall_impl<long>(SYS_utimensat, AT_FDCWD, path,
+ ts_ptr, 0);
+
+ #elif defined(SYS_utimes)
+ // No need to define a timespec struct, use the syscall directly.
+ ret = LIBC_NAMESPACE::syscall_impl<long>(SYS_utimes, path, times);
+ #else
+ #error "utimensat and utimes syscalls not available."
+ // To avoid compilation errors when neither is defined, return an error.
+ libc_errno = ENOSYS; // Function not implemented
+ return -1;
+ #endif // SYS_utimensat
+
+ if (ret < 0) {
+ libc_errno = -ret;
+ return -1;
+ }
+
+ return 0;
+ }
+}
diff --git a/libc/src/sys/time/utimes.h b/libc/src/sys/time/utimes.h
new file mode 100644
index 0000000000000..6dc33aae55621
--- /dev/null
+++ b/libc/src/sys/time/utimes.h
@@ -0,0 +1,21 @@
+//===-- Implementation header for utimes -----------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef LLVM_LIBC_SRC_SYS_TIME_UTIMES_H
+#define LLVM_LIBC_SRC_SYS_TIME_UTIMES_H
+
+#include "hdr/types/struct_timeval.h"
+#include "src/__support/macros/config.h"
+
+namespace LIBC_NAMESPACE_DECL {
+
+int utimes(const char *path, const struct timeval times[2]);
+
+} // namespace LIBC_NAMESPACE_DECL
+
+#endif // LLVM_LIBC_SRC_SYS_TIME_UTIMES_H
\ No newline at end of file
diff --git a/libc/test/src/sys/CMakeLists.txt b/libc/test/src/sys/CMakeLists.txt
index 9e9293aab628f..224cc7905ad31 100644
--- a/libc/test/src/sys/CMakeLists.txt
+++ b/libc/test/src/sys/CMakeLists.txt
@@ -12,3 +12,4 @@ add_subdirectory(prctl)
add_subdirectory(auxv)
add_subdirectory(epoll)
add_subdirectory(uio)
+add_subdirectory(time)
diff --git a/libc/test/src/sys/time/CMakeLists.txt b/libc/test/src/sys/time/CMakeLists.txt
new file mode 100644
index 0000000000000..3ee570e59602f
--- /dev/null
+++ b/libc/test/src/sys/time/CMakeLists.txt
@@ -0,0 +1,21 @@
+add_custom_target(libc_sys_time_unittests)
+
+add_subdirectory(testdata)
+
+add_libc_unittest(
+ time_test
+ SUITE
+ libc_sys_time_unittests
+ SRCS
+ utimes_test.cpp
+ DEPENDS
+ libc.hdr.fcntl_macros
+ libc.include.sys_stat
+ libc.src.errno.errno
+ libc.src.fcntl.open
+ libc.src.sys.sendfile.sendfile
+ libc.src.unistd.close
+ libc.src.unistd.read
+ libc.src.unistd.unlink
+ libc.src.unistd.write
+)
diff --git a/libc/test/src/sys/time/utimes_test.cpp b/libc/test/src/sys/time/utimes_test.cpp
new file mode 100644
index 0000000000000..edf9909f40cbe
--- /dev/null
+++ b/libc/test/src/sys/time/utimes_test.cpp
@@ -0,0 +1,66 @@
+//===-- Unittests for utimes --------------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+#include "src/__support/CPP/string_view.h"
+#include "src/errno/libc_errno.h"
+#include "src/fcntl/open.h"
+#include "src/sys/time/utimes.h"
+#include "src/unistd/close.h"
+#include "src/unistd/read.h"
+#include "src/unistd/unlink.h"
+#include "src/unistd/write.h"
+#include "test/UnitTest/ErrnoSetterMatcher.h"
+#include "test/UnitTest/Test.h"
+
+#include "hdr/fcntl_macros.h"
+#include <sys/stat.h>
+
+namespace cpp = LIBC_NAMESPACE::cpp;
+
+TEST(LlvmLibcSendfileTest, CreateAndTransfer) {
+ using LIBC_NAMESPACE::testing::ErrnoSetterMatcher::Fails;
+ using LIBC_NAMESPACE::testing::ErrnoSetterMatcher::Succeeds;
+
+ // The test strategy is to
+ // 1. Create a temporary file with known data.
+ // 2. Use sendfile to copy it to another file.
+ // 3. Make sure that the data was actually copied.
+ // 4. Clean up the temporary files.
+ constexpr const char *IN_FILE = "testdata/sendfile_in.test";
+ constexpr const char *OUT_FILE = "testdata/sendfile_out.test";
+ const char IN_DATA[] = "sendfile test";
+ constexpr ssize_t IN_SIZE = ssize_t(sizeof(IN_DATA));
+ LIBC_NAMESPACE::libc_errno = 0;
+
+ int in_fd = LIBC_NAMESPACE::open(IN_FILE, O_CREAT | O_WRONLY, S_IRWXU);
+ ASSERT_GT(in_fd, 0);
+ ASSERT_ERRNO_SUCCESS();
+ ASSERT_EQ(LIBC_NAMESPACE::write(in_fd, IN_DATA, IN_SIZE), IN_SIZE);
+ ASSERT_THAT(LIBC_NAMESPACE::close(in_fd), Succeeds(0));
+
+ in_fd = LIBC_NAMESPACE::open(IN_FILE, O_RDONLY);
+ ASSERT_GT(in_fd, 0);
+ ASSERT_ERRNO_SUCCESS();
+ int out_fd = LIBC_NAMESPACE::open(OUT_FILE, O_CREAT | O_WRONLY, S_IRWXU);
+ ASSERT_GT(out_fd, 0);
+ ASSERT_ERRNO_SUCCESS();
+ ssize_t size = LIBC_NAMESPACE::sendfile(in_fd, out_fd, nullptr, IN_SIZE);
+ ASSERT_EQ(size, IN_SIZE);
+ ASSERT_THAT(LIBC_NAMESPACE::close(in_fd), Succeeds(0));
+ ASSERT_THAT(LIBC_NAMESPACE::close(out_fd), Succeeds(0));
+
+ out_fd = LIBC_NAMESPACE::open(OUT_FILE, O_RDONLY);
+ ASSERT_GT(out_fd, 0);
+ ASSERT_ERRNO_SUCCESS();
+ char buf[IN_SIZE];
+ ASSERT_EQ(IN_SIZE, LIBC_NAMESPACE::read(out_fd, buf, IN_SIZE));
+ ASSERT_EQ(cpp::string_view(buf), cpp::string_view(IN_DATA));
+
+ ASSERT_THAT(LIBC_NAMESPACE::unlink(IN_FILE), Succeeds(0));
+ ASSERT_THAT(LIBC_NAMESPACE::unlink(OUT_FILE), Succeeds(0));
+}
>From b19363d9de028edcc0ffbf723a9243af4432465e Mon Sep 17 00:00:00 2001
From: hoarfrost32 <aditya.tejpaul at research.iiit.ac.in>
Date: Wed, 2 Apr 2025 21:07:04 +0530
Subject: [PATCH 2/6] added tests
---
libc/config/linux/x86_64/entrypoints.txt | 2 +-
libc/src/sys/time/linux/CMakeLists.txt | 1 +
libc/src/sys/time/linux/utimes.cpp | 13 +-
libc/test/CMakeLists.txt | 2 +-
.../test/src/sys/time/testdata/CMakeLists.txt | 3 +
libc/test/src/sys/time/utimes_test.cpp | 115 ++++++++++--------
6 files changed, 80 insertions(+), 56 deletions(-)
create mode 100644 libc/test/src/sys/time/testdata/CMakeLists.txt
diff --git a/libc/config/linux/x86_64/entrypoints.txt b/libc/config/linux/x86_64/entrypoints.txt
index 4430784ca94a1..1ac3a781d5279 100644
--- a/libc/config/linux/x86_64/entrypoints.txt
+++ b/libc/config/linux/x86_64/entrypoints.txt
@@ -289,7 +289,7 @@ set(TARGET_LIBC_ENTRYPOINTS
libc.src.sys.statvfs.statvfs
# sys/utimes.h entrypoints
- libc.src.sys.utimes.utimes
+ libc.src.sys.time.utimes
# sys/utsname.h entrypoints
libc.src.sys.utsname.uname
diff --git a/libc/src/sys/time/linux/CMakeLists.txt b/libc/src/sys/time/linux/CMakeLists.txt
index eedb4763bc43f..26efce526e732 100644
--- a/libc/src/sys/time/linux/CMakeLists.txt
+++ b/libc/src/sys/time/linux/CMakeLists.txt
@@ -9,6 +9,7 @@ add_entrypoint_object(
libc.src.__support.OSUtil.osutil
libc.include.sys_stat
libc.include.sys_syscall
+ libc.include.fcntl
libc.src.__support.OSUtil.osutil
libc.src.errno.errno
)
\ No newline at end of file
diff --git a/libc/src/sys/time/linux/utimes.cpp b/libc/src/sys/time/linux/utimes.cpp
index f5cbbab7c13c5..12257f0f876fa 100644
--- a/libc/src/sys/time/linux/utimes.cpp
+++ b/libc/src/sys/time/linux/utimes.cpp
@@ -15,13 +15,15 @@
#include "src/errno/libc_errno.h"
+#include <cerrno>
#include <sys/syscall.h>
#include <sys/stat.h>
+#include <fcntl.h>
namespace LIBC_NAMESPACE_DECL {
- LLVM_LIBC_FUNCTION(int, utimes, (const char *path, const struct timeval times[2]) {
- long ret;
+ LLVM_LIBC_FUNCTION(int, utimes, (const char *path, const struct timeval times[2])) {
+ int ret;
#ifdef SYS_utimensat
//the utimensat syscall requires a timespec struct, not timeval.
@@ -35,6 +37,7 @@ namespace LIBC_NAMESPACE_DECL {
// ensure consistent values
if ((times[0].tv_usec < 0 || times[1].tv_usec < 0) ||
(times[0].tv_usec >= 1000000 || times[1].tv_usec >= 1000000)) {
+ libc_errno = EINVAL;
return -1;
}
@@ -45,6 +48,8 @@ namespace LIBC_NAMESPACE_DECL {
// convert u-seconds to nanoseconds
ts[0].tv_nsec = times[0].tv_usec * 1000;
ts[1].tv_nsec = times[1].tv_usec * 1000;
+
+ ts_ptr = ts;
}
// If times was NULL, ts_ptr remains NULL, which utimensat interprets
@@ -52,12 +57,12 @@ namespace LIBC_NAMESPACE_DECL {
// utimensat syscall.
// flags=0 means don't follow symlinks (like utimes)
- ret = LIBC_NAMESPACE::syscall_impl<long>(SYS_utimensat, AT_FDCWD, path,
+ ret = LIBC_NAMESPACE::syscall_impl<int>(SYS_utimensat, AT_FDCWD, path,
ts_ptr, 0);
#elif defined(SYS_utimes)
// No need to define a timespec struct, use the syscall directly.
- ret = LIBC_NAMESPACE::syscall_impl<long>(SYS_utimes, path, times);
+ ret = LIBC_NAMESPACE::syscall_impl<int>(SYS_utimes, path, times);
#else
#error "utimensat and utimes syscalls not available."
// To avoid compilation errors when neither is defined, return an error.
diff --git a/libc/test/CMakeLists.txt b/libc/test/CMakeLists.txt
index 1a0780faff512..f32998a5091b6 100644
--- a/libc/test/CMakeLists.txt
+++ b/libc/test/CMakeLists.txt
@@ -1,4 +1,4 @@
-add_custom_target(check-libc)
+
add_custom_target(libc-unit-tests)
add_custom_target(libc-hermetic-tests)
add_dependencies(check-libc libc-unit-tests libc-hermetic-tests)
diff --git a/libc/test/src/sys/time/testdata/CMakeLists.txt b/libc/test/src/sys/time/testdata/CMakeLists.txt
new file mode 100644
index 0000000000000..8703b80eab587
--- /dev/null
+++ b/libc/test/src/sys/time/testdata/CMakeLists.txt
@@ -0,0 +1,3 @@
+# This directory will be used to create test files.
+
+file(GENERATE OUTPUT utimes.test CONTENT "utimes test")
diff --git a/libc/test/src/sys/time/utimes_test.cpp b/libc/test/src/sys/time/utimes_test.cpp
index edf9909f40cbe..e644d50607b7d 100644
--- a/libc/test/src/sys/time/utimes_test.cpp
+++ b/libc/test/src/sys/time/utimes_test.cpp
@@ -6,61 +6,76 @@
//
//===----------------------------------------------------------------------===//
-#include "src/__support/CPP/string_view.h"
-#include "src/errno/libc_errno.h"
-#include "src/fcntl/open.h"
-#include "src/sys/time/utimes.h"
-#include "src/unistd/close.h"
-#include "src/unistd/read.h"
-#include "src/unistd/unlink.h"
-#include "src/unistd/write.h"
-#include "test/UnitTest/ErrnoSetterMatcher.h"
+// temp file related stuff
+#include "src/fcntl/open.h" // to open
+#include "src/unistd/close.h" // to close
+#include "src/sys/stat/stat.h" // for info
+#include "src/unistd/unlink.h" // to delete
+// testing error handling
#include "test/UnitTest/Test.h"
-
+#include "src/errno/libc_errno.h"
+#include "test/UnitTest/ErrnoSetterMatcher.h"
+// dependencies for the tests themselves
+#include "hdr/types/struct_timeval.h"
+#include <cerrno>
+#include <fcntl.h>
#include "hdr/fcntl_macros.h"
-#include <sys/stat.h>
-
-namespace cpp = LIBC_NAMESPACE::cpp;
+// the utimes function
+#include "src/sys/time/utimes.h"
+constexpr const char* TEST_FILE = "testdata/utimes.test";
-TEST(LlvmLibcSendfileTest, CreateAndTransfer) {
- using LIBC_NAMESPACE::testing::ErrnoSetterMatcher::Fails;
+// SUCCESS: Takes a file and successfully updates
+// its last access and modified times.
+TEST(LlvmLibcUtimesTest, ChangeTimesSpecific){
using LIBC_NAMESPACE::testing::ErrnoSetterMatcher::Succeeds;
+
+ // make a dummy timeval struct
+ struct timeval times[2];
+ times[0].tv_sec = 54321;
+ times[0].tv_usec = 12345;
+ times[1].tv_sec = 43210;
+ times[1].tv_usec = 23456;
+
+ // ensure utimes succeeds
+ ASSERT_THAT(LIBC_NAMESPACE::utimes(TEST_FILE, times), Succeeds(0));
- // The test strategy is to
- // 1. Create a temporary file with known data.
- // 2. Use sendfile to copy it to another file.
- // 3. Make sure that the data was actually copied.
- // 4. Clean up the temporary files.
- constexpr const char *IN_FILE = "testdata/sendfile_in.test";
- constexpr const char *OUT_FILE = "testdata/sendfile_out.test";
- const char IN_DATA[] = "sendfile test";
- constexpr ssize_t IN_SIZE = ssize_t(sizeof(IN_DATA));
- LIBC_NAMESPACE::libc_errno = 0;
+ // verify the times values against stat of the TEST_FILE
+ struct stat statbuf;
+ ASSERT_EQ(stat(TEST_FILE, &statbuf), 0);
+
+ // seconds
+ ASSERT_EQ(statbuf.st_atim.tv_sec, times[0].tv_sec);
+ ASSERT_EQ(statbuf.st_mtim.tv_sec, times[1].tv_sec);
- int in_fd = LIBC_NAMESPACE::open(IN_FILE, O_CREAT | O_WRONLY, S_IRWXU);
- ASSERT_GT(in_fd, 0);
- ASSERT_ERRNO_SUCCESS();
- ASSERT_EQ(LIBC_NAMESPACE::write(in_fd, IN_DATA, IN_SIZE), IN_SIZE);
- ASSERT_THAT(LIBC_NAMESPACE::close(in_fd), Succeeds(0));
-
- in_fd = LIBC_NAMESPACE::open(IN_FILE, O_RDONLY);
- ASSERT_GT(in_fd, 0);
- ASSERT_ERRNO_SUCCESS();
- int out_fd = LIBC_NAMESPACE::open(OUT_FILE, O_CREAT | O_WRONLY, S_IRWXU);
- ASSERT_GT(out_fd, 0);
- ASSERT_ERRNO_SUCCESS();
- ssize_t size = LIBC_NAMESPACE::sendfile(in_fd, out_fd, nullptr, IN_SIZE);
- ASSERT_EQ(size, IN_SIZE);
- ASSERT_THAT(LIBC_NAMESPACE::close(in_fd), Succeeds(0));
- ASSERT_THAT(LIBC_NAMESPACE::close(out_fd), Succeeds(0));
+ //microseconds
+ ASSERT_EQ(statbuf.st_atim.tv_nsec, times[0].tv_usec * 1000);
+ ASSERT_EQ(statbuf.st_mtim.tv_nsec, times[1].tv_usec * 1000);
+}
- out_fd = LIBC_NAMESPACE::open(OUT_FILE, O_RDONLY);
- ASSERT_GT(out_fd, 0);
- ASSERT_ERRNO_SUCCESS();
- char buf[IN_SIZE];
- ASSERT_EQ(IN_SIZE, LIBC_NAMESPACE::read(out_fd, buf, IN_SIZE));
- ASSERT_EQ(cpp::string_view(buf), cpp::string_view(IN_DATA));
+// FAILURE: Invalid values in the timeval struct
+// to check that utimes rejects it.
+TEST(LlvmLibcUtimesTest, InvalidMicroseconds){
+ using LIBC_NAMESPACE::testing::ErrnoSetterMatcher::Fails;
+
+ // make a dummy timeval struct
+ // populated with bad usec values
+ struct timeval times[2];
+ times[0].tv_sec = 54321;
+ times[0].tv_usec = 4567;
+ times[1].tv_sec = 43210;
+ times[1].tv_usec = 1000000; //invalid
+
+ // ensure utimes fails
+ ASSERT_THAT(LIBC_NAMESPACE::utimes(TEST_FILE, times), Fails(EINVAL));
+
+ // check for failure on
+ // the other possible bad values
- ASSERT_THAT(LIBC_NAMESPACE::unlink(IN_FILE), Succeeds(0));
- ASSERT_THAT(LIBC_NAMESPACE::unlink(OUT_FILE), Succeeds(0));
-}
+ times[0].tv_sec = 54321;
+ times[0].tv_usec = -4567; //invalid
+ times[1].tv_sec = 43210;
+ times[1].tv_usec = 1000;
+
+ // ensure utimes fails once more
+ ASSERT_THAT(LIBC_NAMESPACE::utimes(TEST_FILE, times), Fails(EINVAL));
+}
\ No newline at end of file
>From da4cee300f5d046671609732c19bbeadecec10a0 Mon Sep 17 00:00:00 2001
From: hoarfrost32 <aditya.tejpaul at research.iiit.ac.in>
Date: Wed, 2 Apr 2025 22:54:28 +0530
Subject: [PATCH 3/6] added tests
---
libc/test/src/sys/time/CMakeLists.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libc/test/src/sys/time/CMakeLists.txt b/libc/test/src/sys/time/CMakeLists.txt
index 3ee570e59602f..a541fecebdbc2 100644
--- a/libc/test/src/sys/time/CMakeLists.txt
+++ b/libc/test/src/sys/time/CMakeLists.txt
@@ -3,7 +3,7 @@ add_custom_target(libc_sys_time_unittests)
add_subdirectory(testdata)
add_libc_unittest(
- time_test
+ utimes_test
SUITE
libc_sys_time_unittests
SRCS
>From 0a124558c18f3545f0a7346109e93992f0dc8a3a Mon Sep 17 00:00:00 2001
From: hoarfrost32 <aditya.tejpaul at research.iiit.ac.in>
Date: Wed, 2 Apr 2025 23:51:06 +0530
Subject: [PATCH 4/6] [libc] Implement utimes
(https://github.com/llvm/llvm-project/issues/133953)
---
libc/test/CMakeLists.txt | 2 +-
libc/test/src/sys/time/CMakeLists.txt | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/libc/test/CMakeLists.txt b/libc/test/CMakeLists.txt
index f32998a5091b6..1a0780faff512 100644
--- a/libc/test/CMakeLists.txt
+++ b/libc/test/CMakeLists.txt
@@ -1,4 +1,4 @@
-
+add_custom_target(check-libc)
add_custom_target(libc-unit-tests)
add_custom_target(libc-hermetic-tests)
add_dependencies(check-libc libc-unit-tests libc-hermetic-tests)
diff --git a/libc/test/src/sys/time/CMakeLists.txt b/libc/test/src/sys/time/CMakeLists.txt
index a541fecebdbc2..fa76f2465b85b 100644
--- a/libc/test/src/sys/time/CMakeLists.txt
+++ b/libc/test/src/sys/time/CMakeLists.txt
@@ -13,7 +13,7 @@ add_libc_unittest(
libc.include.sys_stat
libc.src.errno.errno
libc.src.fcntl.open
- libc.src.sys.sendfile.sendfile
+ libc.src.sys.time.utimes
libc.src.unistd.close
libc.src.unistd.read
libc.src.unistd.unlink
>From c645ddf454f64efb46b425986030c2bb5c614f51 Mon Sep 17 00:00:00 2001
From: hoarfrost32 <aditya.tejpaul at research.iiit.ac.in>
Date: Thu, 3 Apr 2025 07:01:54 +0530
Subject: [PATCH 5/6] [libc] Code cleaning, formatting and refactoring
---
__cmake_systeminformation/CMakeCache.txt | 86 ------------------
.../CMakeFiles/3.31.5/CMakeSystem.cmake | 15 ---
.../CMakeFiles/CMakeConfigureLog.yaml | 11 ---
.../CMakeFiles/cmake.check_cache | 1 -
__cmake_systeminformation/CMakeLists.txt | 91 -------------------
libc/include/sys/time.yaml | 2 +-
libc/src/sys/time/CMakeLists.txt | 2 +-
libc/src/sys/time/linux/CMakeLists.txt | 3 +-
libc/src/sys/time/linux/utimes.cpp | 15 +--
libc/test/src/sys/time/CMakeLists.txt | 3 +-
.../test/src/sys/time/testdata/CMakeLists.txt | 2 -
libc/test/src/sys/time/utimes_test.cpp | 52 +++++++----
12 files changed, 44 insertions(+), 239 deletions(-)
delete mode 100644 __cmake_systeminformation/CMakeCache.txt
delete mode 100644 __cmake_systeminformation/CMakeFiles/3.31.5/CMakeSystem.cmake
delete mode 100644 __cmake_systeminformation/CMakeFiles/CMakeConfigureLog.yaml
delete mode 100644 __cmake_systeminformation/CMakeFiles/cmake.check_cache
delete mode 100644 __cmake_systeminformation/CMakeLists.txt
diff --git a/__cmake_systeminformation/CMakeCache.txt b/__cmake_systeminformation/CMakeCache.txt
deleted file mode 100644
index 880b49ed4f29d..0000000000000
--- a/__cmake_systeminformation/CMakeCache.txt
+++ /dev/null
@@ -1,86 +0,0 @@
-# This is the CMakeCache file.
-# For build in directory: /home/hoarfrost/Projects/llvm/llvm-project/__cmake_systeminformation
-# It was generated by CMake: /nix/store/ciiwq1ymzac9m5azhf4higkynmy3f8f7-cmake-3.31.5/bin/cmake
-# You can edit this file to change values found and used by cmake.
-# If you do not want to change any of the values, simply exit the editor.
-# If you do want to change a value, simply edit, save, and exit the editor.
-# The syntax for the file is as follows:
-# KEY:TYPE=VALUE
-# KEY is the name of a variable in the cache.
-# TYPE is a hint to GUIs for the type of VALUE, DO NOT EDIT TYPE!.
-# VALUE is the current value for the KEY.
-
-########################
-# EXTERNAL cache entries
-########################
-
-//Value Computed by CMake.
-CMAKE_FIND_PACKAGE_REDIRECTS_DIR:STATIC=/home/hoarfrost/Projects/llvm/llvm-project/__cmake_systeminformation/CMakeFiles/pkgRedirects
-
-//Path to a program.
-CMAKE_MAKE_PROGRAM:FILEPATH=CMAKE_MAKE_PROGRAM-NOTFOUND
-
-//Value Computed by CMake
-CMAKE_PROJECT_DESCRIPTION:STATIC=
-
-//Value Computed by CMake
-CMAKE_PROJECT_HOMEPAGE_URL:STATIC=
-
-//Value Computed by CMake
-CMAKE_PROJECT_NAME:STATIC=DumpInformation
-
-//Value Computed by CMake
-DumpInformation_BINARY_DIR:STATIC=/home/hoarfrost/Projects/llvm/llvm-project/__cmake_systeminformation
-
-//Value Computed by CMake
-DumpInformation_IS_TOP_LEVEL:STATIC=ON
-
-//Value Computed by CMake
-DumpInformation_SOURCE_DIR:STATIC=/home/hoarfrost/Projects/llvm/llvm-project/__cmake_systeminformation
-
-//No help, variable specified on the command line.
-RESULT_FILE:UNINITIALIZED=/home/hoarfrost/Projects/llvm/llvm-project/__cmake_systeminformation/results.txt
-
-
-########################
-# INTERNAL cache entries
-########################
-
-//This is the directory where this CMakeCache.txt was created
-CMAKE_CACHEFILE_DIR:INTERNAL=/home/hoarfrost/Projects/llvm/llvm-project/__cmake_systeminformation
-//Major version of cmake used to create the current loaded cache
-CMAKE_CACHE_MAJOR_VERSION:INTERNAL=3
-//Minor version of cmake used to create the current loaded cache
-CMAKE_CACHE_MINOR_VERSION:INTERNAL=31
-//Patch version of cmake used to create the current loaded cache
-CMAKE_CACHE_PATCH_VERSION:INTERNAL=5
-//Path to CMake executable.
-CMAKE_COMMAND:INTERNAL=/nix/store/ciiwq1ymzac9m5azhf4higkynmy3f8f7-cmake-3.31.5/bin/cmake
-//Path to cpack program executable.
-CMAKE_CPACK_COMMAND:INTERNAL=/nix/store/ciiwq1ymzac9m5azhf4higkynmy3f8f7-cmake-3.31.5/bin/cpack
-//Path to ctest program executable.
-CMAKE_CTEST_COMMAND:INTERNAL=/nix/store/ciiwq1ymzac9m5azhf4higkynmy3f8f7-cmake-3.31.5/bin/ctest
-//Name of external makefile project generator.
-CMAKE_EXTRA_GENERATOR:INTERNAL=
-//Name of generator.
-CMAKE_GENERATOR:INTERNAL=Unix Makefiles
-//Generator instance identifier.
-CMAKE_GENERATOR_INSTANCE:INTERNAL=
-//Name of generator platform.
-CMAKE_GENERATOR_PLATFORM:INTERNAL=
-//Name of generator toolset.
-CMAKE_GENERATOR_TOOLSET:INTERNAL=
-//Source directory with the top level CMakeLists.txt file for this
-// project
-CMAKE_HOME_DIRECTORY:INTERNAL=/home/hoarfrost/Projects/llvm/llvm-project/__cmake_systeminformation
-//ADVANCED property for variable: CMAKE_MAKE_PROGRAM
-CMAKE_MAKE_PROGRAM-ADVANCED:INTERNAL=1
-//number of local generators
-CMAKE_NUMBER_OF_MAKEFILES:INTERNAL=1
-//Platform information initialized
-CMAKE_PLATFORM_INFO_INITIALIZED:INTERNAL=1
-//Path to CMake installation.
-CMAKE_ROOT:INTERNAL=/nix/store/ciiwq1ymzac9m5azhf4higkynmy3f8f7-cmake-3.31.5/share/cmake-3.31
-//uname command
-CMAKE_UNAME:INTERNAL=/usr/bin/uname
-
diff --git a/__cmake_systeminformation/CMakeFiles/3.31.5/CMakeSystem.cmake b/__cmake_systeminformation/CMakeFiles/3.31.5/CMakeSystem.cmake
deleted file mode 100644
index 82aa456095f26..0000000000000
--- a/__cmake_systeminformation/CMakeFiles/3.31.5/CMakeSystem.cmake
+++ /dev/null
@@ -1,15 +0,0 @@
-set(CMAKE_HOST_SYSTEM "Linux-6.12.20")
-set(CMAKE_HOST_SYSTEM_NAME "Linux")
-set(CMAKE_HOST_SYSTEM_VERSION "6.12.20")
-set(CMAKE_HOST_SYSTEM_PROCESSOR "x86_64")
-
-
-
-set(CMAKE_SYSTEM "Linux-6.12.20")
-set(CMAKE_SYSTEM_NAME "Linux")
-set(CMAKE_SYSTEM_VERSION "6.12.20")
-set(CMAKE_SYSTEM_PROCESSOR "x86_64")
-
-set(CMAKE_CROSSCOMPILING "FALSE")
-
-set(CMAKE_SYSTEM_LOADED 1)
diff --git a/__cmake_systeminformation/CMakeFiles/CMakeConfigureLog.yaml b/__cmake_systeminformation/CMakeFiles/CMakeConfigureLog.yaml
deleted file mode 100644
index b5439f4f72db7..0000000000000
--- a/__cmake_systeminformation/CMakeFiles/CMakeConfigureLog.yaml
+++ /dev/null
@@ -1,11 +0,0 @@
-
----
-events:
- -
- kind: "message-v1"
- backtrace:
- - "/nix/store/ciiwq1ymzac9m5azhf4higkynmy3f8f7-cmake-3.31.5/share/cmake-3.31/Modules/CMakeDetermineSystem.cmake:205 (message)"
- - "CMakeLists.txt:6 (project)"
- message: |
- The system is: Linux - 6.12.20 - x86_64
-...
diff --git a/__cmake_systeminformation/CMakeFiles/cmake.check_cache b/__cmake_systeminformation/CMakeFiles/cmake.check_cache
deleted file mode 100644
index 3dccd731726d7..0000000000000
--- a/__cmake_systeminformation/CMakeFiles/cmake.check_cache
+++ /dev/null
@@ -1 +0,0 @@
-# This file is generated by cmake for dependency checking of the CMakeCache.txt file
diff --git a/__cmake_systeminformation/CMakeLists.txt b/__cmake_systeminformation/CMakeLists.txt
deleted file mode 100644
index 97f385612c3b7..0000000000000
--- a/__cmake_systeminformation/CMakeLists.txt
+++ /dev/null
@@ -1,91 +0,0 @@
-# Distributed under the OSI-approved BSD 3-Clause License. See accompanying
-# file Copyright.txt or https://cmake.org/licensing for details.
-
-
-cmake_minimum_required(VERSION ${CMAKE_VERSION})
-project(DumpInformation)
-
-# first get the standard information for the platform
-include_directories("This does not exist")
-get_directory_property(incl INCLUDE_DIRECTORIES)
-set_directory_properties(PROPERTIES INCLUDE_DIRECTORIES "${DumpInformation_BINARY_DIR};${DumpInformation_SOURCE_DIR}")
-
-configure_file("${CMAKE_ROOT}/Modules/SystemInformation.in" "${RESULT_FILE}")
-
-
-file(APPEND "${RESULT_FILE}"
- "\n=================================================================\n")
-file(APPEND "${RESULT_FILE}"
- "=== VARIABLES\n")
-file(APPEND "${RESULT_FILE}"
- "=================================================================\n")
-get_cmake_property(res VARIABLES)
-foreach(var ${res})
- file(APPEND "${RESULT_FILE}" "${var} \"${${var}}\"\n")
-endforeach()
-
-file(APPEND "${RESULT_FILE}"
- "\n=================================================================\n")
-file(APPEND "${RESULT_FILE}"
- "=== COMMANDS\n")
-file(APPEND "${RESULT_FILE}"
- "=================================================================\n")
-get_cmake_property(res COMMANDS)
-foreach(var ${res})
- file(APPEND "${RESULT_FILE}" "${var}\n")
-endforeach()
-
-file(APPEND "${RESULT_FILE}"
- "\n=================================================================\n")
-file(APPEND "${RESULT_FILE}"
- "=== MACROS\n")
-file(APPEND "${RESULT_FILE}"
- "=================================================================\n")
-file(WRITE ${CMAKE_CURRENT_BINARY_DIR}/AllMacros.txt "")
-get_cmake_property(res MACROS)
-foreach(var ${res})
- file(APPEND "${RESULT_FILE}" "${var}\n")
-endforeach()
-
-file(APPEND "${RESULT_FILE}"
- "\n=================================================================\n")
-file(APPEND "${RESULT_FILE}"
- "=== OTHER\n")
-file(APPEND "${RESULT_FILE}"
- "=================================================================\n")
-get_directory_property(res INCLUDE_DIRECTORIES)
-foreach(var ${res})
- file(APPEND "${RESULT_FILE}" "INCLUDE_DIRECTORY: ${var}\n")
-endforeach()
-
-get_directory_property(res LINK_DIRECTORIES)
-foreach(var ${res})
- file(APPEND "${RESULT_FILE}" "LINK_DIRECTORIES: ${var}\n")
-endforeach()
-
-get_directory_property(res INCLUDE_REGULAR_EXPRESSION)
-file(APPEND "${RESULT_FILE}" "INCLUDE_REGULAR_EXPRESSION: ${res}\n")
-
-# include other files if they are present, such as when run from within the
-# binary tree
-macro(DUMP_FILE THE_FILE)
- if (EXISTS "${THE_FILE}")
- file(APPEND "${RESULT_FILE}"
- "\n=================================================================\n")
- file(APPEND "${RESULT_FILE}"
- "=== ${THE_FILE}\n")
- file(APPEND "${RESULT_FILE}"
- "=================================================================\n")
-
- file(READ "${THE_FILE}" FILE_CONTENTS LIMIT 50000)
- file(APPEND "${RESULT_FILE}" "${FILE_CONTENTS}")
- endif ()
-endmacro()
-
-DUMP_FILE("../CMakeCache.txt")
-DUMP_FILE("../CMakeFiles/CMakeSystem.cmake")
-
-foreach (EXTRA_FILE ${EXTRA_DUMP_FILES})
- DUMP_FILE("${EXTRA_FILE}")
-endforeach ()
-
diff --git a/libc/include/sys/time.yaml b/libc/include/sys/time.yaml
index f2f0f5f949c77..c2394c74a8ffb 100644
--- a/libc/include/sys/time.yaml
+++ b/libc/include/sys/time.yaml
@@ -11,4 +11,4 @@ functions:
return_type: int
arguments:
- type: ptr<const char>
- - type: ptr<const struct timeval>
\ No newline at end of file
+ - type: ptr<const struct timeval>
diff --git a/libc/src/sys/time/CMakeLists.txt b/libc/src/sys/time/CMakeLists.txt
index df8bf5b3630c3..f599cddaaeeb3 100644
--- a/libc/src/sys/time/CMakeLists.txt
+++ b/libc/src/sys/time/CMakeLists.txt
@@ -7,4 +7,4 @@ add_entrypoint_object(
ALIAS
DEPENDS
.${LIBC_TARGET_OS}.utimes
-)
\ No newline at end of file
+)
diff --git a/libc/src/sys/time/linux/CMakeLists.txt b/libc/src/sys/time/linux/CMakeLists.txt
index 26efce526e732..506001e5c9fd2 100644
--- a/libc/src/sys/time/linux/CMakeLists.txt
+++ b/libc/src/sys/time/linux/CMakeLists.txt
@@ -6,10 +6,11 @@ add_entrypoint_object(
../utimes.h
DEPENDS
libc.hdr.types.struct_timeval
+ libc.hdr.fcntl_macros
libc.src.__support.OSUtil.osutil
libc.include.sys_stat
libc.include.sys_syscall
libc.include.fcntl
libc.src.__support.OSUtil.osutil
libc.src.errno.errno
-)
\ No newline at end of file
+)
diff --git a/libc/src/sys/time/linux/utimes.cpp b/libc/src/sys/time/linux/utimes.cpp
index 12257f0f876fa..fc7b5481e047d 100644
--- a/libc/src/sys/time/linux/utimes.cpp
+++ b/libc/src/sys/time/linux/utimes.cpp
@@ -9,23 +9,24 @@
#include "src/sys/time/utimes.h"
#include "hdr/types/struct_timeval.h"
+#include "hdr/fcntl_macros.h"
#include "src/__support/OSUtil/syscall.h"
#include "src/__support/common.h"
#include "src/errno/libc_errno.h"
-#include <cerrno>
#include <sys/syscall.h>
-#include <sys/stat.h>
-#include <fcntl.h>
namespace LIBC_NAMESPACE_DECL {
LLVM_LIBC_FUNCTION(int, utimes, (const char *path, const struct timeval times[2])) {
int ret;
- #ifdef SYS_utimensat
+ #ifdef SYS_utimes
+ // No need to define a timespec struct, use the syscall directly.
+ ret = LIBC_NAMESPACE::syscall_impl<int>(SYS_utimes, path, times);
+ #elif defined(SYS_utimensat)
//the utimensat syscall requires a timespec struct, not timeval.
struct timespec ts[2];
struct timespec *ts_ptr = nullptr; // default value if times is NULL
@@ -60,14 +61,8 @@ namespace LIBC_NAMESPACE_DECL {
ret = LIBC_NAMESPACE::syscall_impl<int>(SYS_utimensat, AT_FDCWD, path,
ts_ptr, 0);
- #elif defined(SYS_utimes)
- // No need to define a timespec struct, use the syscall directly.
- ret = LIBC_NAMESPACE::syscall_impl<int>(SYS_utimes, path, times);
#else
#error "utimensat and utimes syscalls not available."
- // To avoid compilation errors when neither is defined, return an error.
- libc_errno = ENOSYS; // Function not implemented
- return -1;
#endif // SYS_utimensat
if (ret < 0) {
diff --git a/libc/test/src/sys/time/CMakeLists.txt b/libc/test/src/sys/time/CMakeLists.txt
index fa76f2465b85b..2878af7b16280 100644
--- a/libc/test/src/sys/time/CMakeLists.txt
+++ b/libc/test/src/sys/time/CMakeLists.txt
@@ -10,7 +10,6 @@ add_libc_unittest(
utimes_test.cpp
DEPENDS
libc.hdr.fcntl_macros
- libc.include.sys_stat
libc.src.errno.errno
libc.src.fcntl.open
libc.src.sys.time.utimes
@@ -18,4 +17,6 @@ add_libc_unittest(
libc.src.unistd.read
libc.src.unistd.unlink
libc.src.unistd.write
+ libc.src.stdio.remove
+ libc.src.sys.stat.stat
)
diff --git a/libc/test/src/sys/time/testdata/CMakeLists.txt b/libc/test/src/sys/time/testdata/CMakeLists.txt
index 8703b80eab587..2abc75686e0a9 100644
--- a/libc/test/src/sys/time/testdata/CMakeLists.txt
+++ b/libc/test/src/sys/time/testdata/CMakeLists.txt
@@ -1,3 +1 @@
# This directory will be used to create test files.
-
-file(GENERATE OUTPUT utimes.test CONTENT "utimes test")
diff --git a/libc/test/src/sys/time/utimes_test.cpp b/libc/test/src/sys/time/utimes_test.cpp
index e644d50607b7d..fad2376a1d051 100644
--- a/libc/test/src/sys/time/utimes_test.cpp
+++ b/libc/test/src/sys/time/utimes_test.cpp
@@ -1,34 +1,37 @@
-//===-- Unittests for utimes --------------------------------------------===//
+//===-- Unittests for utimes -----------------------------------------------===//
//
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
// See https://llvm.org/LICENSE.txt for license information.
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
-//===----------------------------------------------------------------------===//
+//===-----------------------------------------------------------------------===//
-// temp file related stuff
-#include "src/fcntl/open.h" // to open
-#include "src/unistd/close.h" // to close
-#include "src/sys/stat/stat.h" // for info
-#include "src/unistd/unlink.h" // to delete
-// testing error handling
+#include "src/fcntl/open.h"
+#include "src/unistd/close.h"
+#include "src/stdio/remove.h"
+#include "src/sys/stat/stat.h"
+#include "src/unistd/unlink.h"
#include "test/UnitTest/Test.h"
#include "src/errno/libc_errno.h"
#include "test/UnitTest/ErrnoSetterMatcher.h"
-// dependencies for the tests themselves
#include "hdr/types/struct_timeval.h"
-#include <cerrno>
-#include <fcntl.h>
#include "hdr/fcntl_macros.h"
-// the utimes function
#include "src/sys/time/utimes.h"
-constexpr const char* TEST_FILE = "testdata/utimes.test";
+#include <fcntl.h>
+constexpr const char* FILE_PATH = "utimes.test";
// SUCCESS: Takes a file and successfully updates
// its last access and modified times.
TEST(LlvmLibcUtimesTest, ChangeTimesSpecific){
using LIBC_NAMESPACE::testing::ErrnoSetterMatcher::Succeeds;
+ // const char* FILE_PATH = "testdata/__utimes_changetimes.test";
+
+ auto TEST_FILE = libc_make_test_file_path(FILE_PATH);
+ int fd = LIBC_NAMESPACE::open(TEST_FILE, O_WRONLY | O_CREAT);
+ ASSERT_GT(fd, 0);
+ ASSERT_THAT(LIBC_NAMESPACE::close(fd), Succeeds(0));
+
// make a dummy timeval struct
struct timeval times[2];
times[0].tv_sec = 54321;
@@ -37,11 +40,11 @@ TEST(LlvmLibcUtimesTest, ChangeTimesSpecific){
times[1].tv_usec = 23456;
// ensure utimes succeeds
- ASSERT_THAT(LIBC_NAMESPACE::utimes(TEST_FILE, times), Succeeds(0));
+ ASSERT_THAT(LIBC_NAMESPACE::utimes(FILE_PATH, times), Succeeds(0));
// verify the times values against stat of the TEST_FILE
struct stat statbuf;
- ASSERT_EQ(stat(TEST_FILE, &statbuf), 0);
+ ASSERT_EQ(LIBC_NAMESPACE::stat(FILE_PATH, &statbuf), 0);
// seconds
ASSERT_EQ(statbuf.st_atim.tv_sec, times[0].tv_sec);
@@ -49,13 +52,23 @@ TEST(LlvmLibcUtimesTest, ChangeTimesSpecific){
//microseconds
ASSERT_EQ(statbuf.st_atim.tv_nsec, times[0].tv_usec * 1000);
- ASSERT_EQ(statbuf.st_mtim.tv_nsec, times[1].tv_usec * 1000);
+ ASSERT_EQ(statbuf.st_mtim.tv_nsec, times[1].tv_usec * 1000);
+
+ ASSERT_THAT(LIBC_NAMESPACE::remove(TEST_FILE), Succeeds(0));
}
// FAILURE: Invalid values in the timeval struct
// to check that utimes rejects it.
TEST(LlvmLibcUtimesTest, InvalidMicroseconds){
using LIBC_NAMESPACE::testing::ErrnoSetterMatcher::Fails;
+ using LIBC_NAMESPACE::testing::ErrnoSetterMatcher::Succeeds;
+
+ // const char* FILE_PATH = "testdata/__utimes_invalidmicroseconds.test";
+
+ auto TEST_FILE = libc_make_test_file_path(FILE_PATH);
+ int fd = LIBC_NAMESPACE::open(TEST_FILE, O_WRONLY | O_CREAT);
+ ASSERT_GT(fd, 0);
+ ASSERT_THAT(LIBC_NAMESPACE::close(fd), Succeeds(0));
// make a dummy timeval struct
// populated with bad usec values
@@ -66,7 +79,7 @@ TEST(LlvmLibcUtimesTest, InvalidMicroseconds){
times[1].tv_usec = 1000000; //invalid
// ensure utimes fails
- ASSERT_THAT(LIBC_NAMESPACE::utimes(TEST_FILE, times), Fails(EINVAL));
+ ASSERT_THAT(LIBC_NAMESPACE::utimes(FILE_PATH, times), Fails(EINVAL));
// check for failure on
// the other possible bad values
@@ -77,5 +90,6 @@ TEST(LlvmLibcUtimesTest, InvalidMicroseconds){
times[1].tv_usec = 1000;
// ensure utimes fails once more
- ASSERT_THAT(LIBC_NAMESPACE::utimes(TEST_FILE, times), Fails(EINVAL));
-}
\ No newline at end of file
+ ASSERT_THAT(LIBC_NAMESPACE::utimes(FILE_PATH, times), Fails(EINVAL));
+ ASSERT_THAT(LIBC_NAMESPACE::remove(TEST_FILE), Succeeds(0));
+}
>From f59b93c572a836bfde235f9dd1b8d953399fa280 Mon Sep 17 00:00:00 2001
From: hoarfrost32 <aditya.tejpaul at research.iiit.ac.in>
Date: Thu, 3 Apr 2025 07:04:24 +0530
Subject: [PATCH 6/6] [libc] clang-format
---
libc/src/sys/time/linux/utimes.cpp | 16 ++++-----
libc/test/src/sys/time/utimes_test.cpp | 49 +++++++++++++-------------
2 files changed, 33 insertions(+), 32 deletions(-)
diff --git a/libc/src/sys/time/linux/utimes.cpp b/libc/src/sys/time/linux/utimes.cpp
index fc7b5481e047d..b68dd6c53035f 100644
--- a/libc/src/sys/time/linux/utimes.cpp
+++ b/libc/src/sys/time/linux/utimes.cpp
@@ -8,8 +8,8 @@
#include "src/sys/time/utimes.h"
-#include "hdr/types/struct_timeval.h"
#include "hdr/fcntl_macros.h"
+#include "hdr/types/struct_timeval.h"
#include "src/__support/OSUtil/syscall.h"
#include "src/__support/common.h"
@@ -22,11 +22,11 @@ namespace LIBC_NAMESPACE_DECL {
LLVM_LIBC_FUNCTION(int, utimes, (const char *path, const struct timeval times[2])) {
int ret;
-
- #ifdef SYS_utimes
+
+#ifdef SYS_utimes
// No need to define a timespec struct, use the syscall directly.
ret = LIBC_NAMESPACE::syscall_impl<int>(SYS_utimes, path, times);
- #elif defined(SYS_utimensat)
+#elif defined(SYS_utimensat)
//the utimensat syscall requires a timespec struct, not timeval.
struct timespec ts[2];
struct timespec *ts_ptr = nullptr; // default value if times is NULL
@@ -60,10 +60,10 @@ namespace LIBC_NAMESPACE_DECL {
// flags=0 means don't follow symlinks (like utimes)
ret = LIBC_NAMESPACE::syscall_impl<int>(SYS_utimensat, AT_FDCWD, path,
ts_ptr, 0);
-
- #else
- #error "utimensat and utimes syscalls not available."
- #endif // SYS_utimensat
+
+#else
+#error "utimensat and utimes syscalls not available."
+#endif // SYS_utimensat
if (ret < 0) {
libc_errno = -ret;
diff --git a/libc/test/src/sys/time/utimes_test.cpp b/libc/test/src/sys/time/utimes_test.cpp
index fad2376a1d051..5b8a3b5aabf6c 100644
--- a/libc/test/src/sys/time/utimes_test.cpp
+++ b/libc/test/src/sys/time/utimes_test.cpp
@@ -1,4 +1,5 @@
-//===-- Unittests for utimes -----------------------------------------------===//
+//===-- Unittests for utimes
+//-----------------------------------------------===//
//
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
// See https://llvm.org/LICENSE.txt for license information.
@@ -6,32 +7,32 @@
//
//===-----------------------------------------------------------------------===//
-#include "src/fcntl/open.h"
-#include "src/unistd/close.h"
+#include "hdr/fcntl_macros.h"
+#include "hdr/types/struct_timeval.h"
+#include "src/errno/libc_errno.h"
+#include "src/fcntl/open.h"
#include "src/stdio/remove.h"
-#include "src/sys/stat/stat.h"
-#include "src/unistd/unlink.h"
-#include "test/UnitTest/Test.h"
-#include "src/errno/libc_errno.h"
+#include "src/sys/stat/stat.h"
+#include "src/sys/time/utimes.h"
+#include "src/unistd/close.h"
+#include "src/unistd/unlink.h"
#include "test/UnitTest/ErrnoSetterMatcher.h"
-#include "hdr/types/struct_timeval.h"
-#include "hdr/fcntl_macros.h"
-#include "src/sys/time/utimes.h"
+#include "test/UnitTest/Test.h"
#include <fcntl.h>
-constexpr const char* FILE_PATH = "utimes.test";
+constexpr const char *FILE_PATH = "utimes.test";
// SUCCESS: Takes a file and successfully updates
// its last access and modified times.
TEST(LlvmLibcUtimesTest, ChangeTimesSpecific){
using LIBC_NAMESPACE::testing::ErrnoSetterMatcher::Succeeds;
-
+
// const char* FILE_PATH = "testdata/__utimes_changetimes.test";
-
+
auto TEST_FILE = libc_make_test_file_path(FILE_PATH);
int fd = LIBC_NAMESPACE::open(TEST_FILE, O_WRONLY | O_CREAT);
ASSERT_GT(fd, 0);
ASSERT_THAT(LIBC_NAMESPACE::close(fd), Succeeds(0));
-
+
// make a dummy timeval struct
struct timeval times[2];
times[0].tv_sec = 54321;
@@ -45,7 +46,7 @@ TEST(LlvmLibcUtimesTest, ChangeTimesSpecific){
// verify the times values against stat of the TEST_FILE
struct stat statbuf;
ASSERT_EQ(LIBC_NAMESPACE::stat(FILE_PATH, &statbuf), 0);
-
+
// seconds
ASSERT_EQ(statbuf.st_atim.tv_sec, times[0].tv_sec);
ASSERT_EQ(statbuf.st_mtim.tv_sec, times[1].tv_sec);
@@ -53,23 +54,23 @@ TEST(LlvmLibcUtimesTest, ChangeTimesSpecific){
//microseconds
ASSERT_EQ(statbuf.st_atim.tv_nsec, times[0].tv_usec * 1000);
ASSERT_EQ(statbuf.st_mtim.tv_nsec, times[1].tv_usec * 1000);
-
- ASSERT_THAT(LIBC_NAMESPACE::remove(TEST_FILE), Succeeds(0));
+
+ ASSERT_THAT(LIBC_NAMESPACE::remove(TEST_FILE), Succeeds(0));
}
// FAILURE: Invalid values in the timeval struct
// to check that utimes rejects it.
TEST(LlvmLibcUtimesTest, InvalidMicroseconds){
- using LIBC_NAMESPACE::testing::ErrnoSetterMatcher::Fails;
- using LIBC_NAMESPACE::testing::ErrnoSetterMatcher::Succeeds;
+ using LIBC_NAMESPACE::testing::ErrnoSetterMatcher::Fails;
+ using LIBC_NAMESPACE::testing::ErrnoSetterMatcher::Succeeds;
// const char* FILE_PATH = "testdata/__utimes_invalidmicroseconds.test";
-
+
auto TEST_FILE = libc_make_test_file_path(FILE_PATH);
int fd = LIBC_NAMESPACE::open(TEST_FILE, O_WRONLY | O_CREAT);
ASSERT_GT(fd, 0);
ASSERT_THAT(LIBC_NAMESPACE::close(fd), Succeeds(0));
-
+
// make a dummy timeval struct
// populated with bad usec values
struct timeval times[2];
@@ -80,7 +81,7 @@ TEST(LlvmLibcUtimesTest, InvalidMicroseconds){
// ensure utimes fails
ASSERT_THAT(LIBC_NAMESPACE::utimes(FILE_PATH, times), Fails(EINVAL));
-
+
// check for failure on
// the other possible bad values
@@ -90,6 +91,6 @@ TEST(LlvmLibcUtimesTest, InvalidMicroseconds){
times[1].tv_usec = 1000;
// ensure utimes fails once more
- ASSERT_THAT(LIBC_NAMESPACE::utimes(FILE_PATH, times), Fails(EINVAL));
- ASSERT_THAT(LIBC_NAMESPACE::remove(TEST_FILE), Succeeds(0));
+ ASSERT_THAT(LIBC_NAMESPACE::utimes(FILE_PATH, times), Fails(EINVAL));
+ ASSERT_THAT(LIBC_NAMESPACE::remove(TEST_FILE), Succeeds(0));
}
More information about the llvm-commits
mailing list