Skip to content

cksum family: read ahead file - #12549

Merged
cakebaker merged 1 commit into
uutils:mainfrom
oech3:cksum-fadv
Jun 1, 2026
Merged

cksum family: read ahead file#12549
cakebaker merged 1 commit into
uutils:mainfrom
oech3:cksum-fadv

Conversation

@oech3

@oech3 oech3 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Most people does not calc checksum many times for the same file.
So page caches are missing when cksum is called.

@oech3
oech3 force-pushed the cksum-fadv branch 4 times, most recently from af75463 to de83694 Compare June 1, 2026 06:05
@oech3 oech3 closed this Jun 1, 2026
@oech3
oech3 deleted the cksum-fadv branch June 1, 2026 06:07
@oech3
oech3 restored the cksum-fadv branch June 1, 2026 06:09
@oech3 oech3 reopened this Jun 1, 2026
@oech3 oech3 closed this Jun 1, 2026
@oech3
oech3 deleted the cksum-fadv branch June 1, 2026 06:20
@oech3

oech3 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

cfg is complex

@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/tail/tail-n0f (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/cut/bounded-memory (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/tail/retry (passes in this run but fails in the 'main' branch)
Note: The gnu test tests/pr/bounded-memory is now being skipped but was previously passing.

@oech3
oech3 restored the cksum-fadv branch June 1, 2026 07:52
@oech3 oech3 reopened this Jun 1, 2026
@oech3
oech3 marked this pull request as ready for review June 1, 2026 08:18
@oech3

oech3 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

hyperfine disallows dropping caches

@cakebaker
cakebaker merged commit 4e37886 into uutils:main Jun 1, 2026
190 of 192 checks passed
@cakebaker

Copy link
Copy Markdown
Contributor

Thanks!

@oech3
oech3 deleted the cksum-fadv branch June 1, 2026 12:25
@oech3

oech3 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

I might not do that for everything, but it seems GNU sometimes call it

$ cat /etc/pacman.conf|strace -qqq -e trace='/fadvise' gnu9.11/wc -c
fadvise64(0, 0, 0, POSIX_FADV_SEQUENTIAL) = -1 ESPIPE (Illegal seek)
535

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants