truncate: eliminate duplicate stat() syscall by Jean-Christian-Cirstea · Pull Request #9527 · uutils/coreutils

Jean-Christian-Cirstea · 2025-11-30T01:08:38Z

When no reference file is given, truncate performs two stat() syscalls:

The first one to retrieve the size of the file.
The second one to check if the file if a pipe.

This commit fixes this issue by restructuring the code. The main change resides in the elimination of the initial match performed at the top level to select between different scenarios. Instead, the new implementation:

Check if a reference file has been given. If so, retrieve its size.
Determine the given mode. If no mode has been given, TruncateMode::Extend(0) is implied.
Retrieve the size of the file, while checking if it is a pipe. This is the key point of this PR.
Determine the truncate size based on the reference size and the mode. If no reference file has been provided, use the size of the file to be resized as a reference size.
Finally, perform ftruncate() to resize the file.

src/uu/truncate/src/truncate.rs

Jean-Christian-Cirstea · 2025-11-30T01:11:38Z

src/uu/truncate/src/truncate.rs

+            Self::RoundDown(size) => fsize.checked_rem(*size).map(|remainder| fsize - remainder),
+            Self::RoundUp(size) => fsize.checked_rem(*size).map(|remainder| fsize + remainder),


As previously stated, calling this method when rounding to 0 resulted in a panic. Following the guideline of avoiding panicking code, use checked_rem() method.

The remainder calculation seems to be wrong for RoundUp - should be (fsize + size - 1) / size * size to properly round up
no?

The formula is indeed wrong. Should I fix it in this PR or in a separate PR?

checked_next_multiple_of() depuis la bibliothèque standard est maintenant utilisé pour calculer le prochain multiple de size. Un test unitaire a également été ajouté.

please don't use french in the comment ;)

Jean-Christian-Cirstea · 2025-11-30T01:12:18Z

src/uu/truncate/src/truncate.rs

+    fn is_absolute(&self) -> bool {
+        matches!(self, Self::Absolute(_))
+    }


Tiny helper function to determine if self is absolute. Useful when checking if an absolute mode was provided with a reference file.

Jean-Christian-Cirstea · 2025-11-30T01:13:00Z

src/uu/truncate/src/truncate.rs

-    #[cfg(unix)]
-    if let Ok(metadata) = metadata(path) {
-        if metadata.file_type().is_fifo() {
-            return Err(USimpleError::new(
-                1,
-                translate!("truncate-error-cannot-open-no-device", "filename" => filename.to_string_lossy().quote()),
-            ));
-        }
-    }


This check was moved to line 212.

Jean-Christian-Cirstea · 2025-11-30T01:14:00Z

src/uu/truncate/src/truncate.rs

+    let file_size = match metadata(path) {
+        Ok(metadata) => {
+            #[cfg(unix)]
+            if metadata.file_type().is_fifo() {
+                return Err(USimpleError::new(
+                    1,
+                    translate!("truncate-error-cannot-open-no-device", "filename" => filename.to_string_lossy().quote()),
+                ));
+            }
+            metadata.len()
        }
-        Ok(m) => m,
+        Err(_) => 0,
    };


Retrieve the size of the file, while checking if it is a pipe. One single stat() syscall required.

maybe add this as a comment in the code?

Added comments.

Jean-Christian-Cirstea · 2025-11-30T01:15:30Z

src/uu/truncate/src/truncate.rs

+    // Omitting the mode is equivalent to extending a file by 0 bytes.
+    let mode = match size_string {
+        Some(string) => match parse_mode_and_size(string) {
+            Err(error) => {
+                return Err(USimpleError::new(
+                    1,
+                    translate!("truncate-error-invalid-number", "error" => error),
+                ));
+            }
+            Ok(mode) => mode,
+        },
+        None => TruncateMode::Extend(0),
+    };


Specifying no mode implies TruncateMode::Extend(0).

same, better in the code as comment :)

There is a comment. Look carefully at line 263. Est-ce que j'aurais dû l'écrire en français pour qu'il soit plus clair ? :)

Oups pardon :)

Jean-Christian-Cirstea · 2025-11-30T01:17:46Z

src/uu/truncate/src/truncate.rs

-            Self::Reduce(size) => {
-                if *size > fsize {
-                    0
-                } else {
-                    fsize - size
-                }
-            }


The standard library provides saturating_sub(). No need to implement its logic by ourselves.

github-actions · 2025-11-30T08:29:56Z

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)

Signed-off-by: Jean-Christian CÎRSTEA <jean-christian.cirstea@tuta.com>

1. The documentation for parse_mode_and_size() has been updated. 2. Added documentation for TruncateMode::is_absolute(). Signed-off-by: Jean-Christian CÎRSTEA <jean-christian.cirstea@tuta.com>

github-actions · 2025-11-30T10:07:50Z

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

Signed-off-by: Jean-Christian CÎRSTEA <jean-christian.cirstea@tuta.com>

github-actions · 2025-12-06T16:12:35Z

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/tty/tty-eof is no longer failing!

Jean-Christian-Cirstea · 2025-12-19T20:25:23Z

@sylvestre Any outstanding issues with this PR?

Jean-Christian-Cirstea commented Nov 30, 2025

View reviewed changes

src/uu/truncate/src/truncate.rs Show resolved Hide resolved

Jean-Christian-Cirstea commented Nov 30, 2025

View reviewed changes

Jean-Christian-Cirstea added 2 commits November 30, 2025 11:49

truncate: Avoid duplicate stat() syscall

e5e64f9

Signed-off-by: Jean-Christian CÎRSTEA <jean-christian.cirstea@tuta.com>

truncate: updated documentation

abcae00

1. The documentation for parse_mode_and_size() has been updated. 2. Added documentation for TruncateMode::is_absolute(). Signed-off-by: Jean-Christian CÎRSTEA <jean-christian.cirstea@tuta.com>

Jean-Christian-Cirstea force-pushed the truncate branch from a5efbf3 to abcae00 Compare November 30, 2025 09:50

Jean-Christian-Cirstea requested a review from sylvestre November 30, 2025 10:41

truncate: Fixed RoundUp mode

873f2b2

Signed-off-by: Jean-Christian CÎRSTEA <jean-christian.cirstea@tuta.com>

sylvestre merged commit fe97933 into uutils:main Dec 19, 2025
127 checks passed

CrazyRoka pushed a commit to CrazyRoka/coreutils that referenced this pull request Dec 28, 2025

truncate: eliminate duplicate stat() syscall (uutils#9527)

28792dc

moonfruit mentioned this pull request Feb 3, 2026

uutils-selected 0.6.0 moonfruit/homebrew-tap#453

Closed

		Self::RoundDown(size) => fsize.checked_rem(*size).map(\|remainder\| fsize - remainder),
		Self::RoundUp(size) => fsize.checked_rem(*size).map(\|remainder\| fsize + remainder),

Uh oh!

Conversation

Jean-Christian-Cirstea commented Nov 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Jean-Christian-Cirstea Nov 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Nov 30, 2025

Uh oh!

github-actions bot commented Nov 30, 2025

Uh oh!

github-actions bot commented Dec 6, 2025

Uh oh!

Jean-Christian-Cirstea commented Dec 19, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Jean-Christian-Cirstea commented Nov 30, 2025 •

edited

Loading

Jean-Christian-Cirstea Nov 30, 2025 •

edited

Loading