]> www.infradead.org Git - users/jedix/linux-maple.git/commit
pipe: don't update {a,c,m}time for anonymous pipes
authorOleg Nesterov <oleg@redhat.com>
Wed, 5 Feb 2025 18:18:12 +0000 (19:18 +0100)
committerChristian Brauner <brauner@kernel.org>
Thu, 6 Feb 2025 09:50:55 +0000 (10:50 +0100)
commitf017b0a4951fac8f150232661b2cc0b67e0c57f0
tree341f1c778a59a9831b02614d6075ce16d85656c7
parent262b2fa99cbe02a715ce23981c2c30685ccf3a93
pipe: don't update {a,c,m}time for anonymous pipes

These numbers are visible in fstat() but hopefully nobody uses this
information and file_accessed/file_update_time are not that cheap.
Stupid test-case:

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <assert.h>
#include <sys/ioctl.h>
#include <sys/time.h>

static char buf[17 * 4096];
static struct timeval TW, TR;

int wr(int fd, int size)
{
int c, r;
struct timeval t0, t1;

gettimeofday(&t0, NULL);
for (c = 0; (r = write(fd, buf, size)) > 0; c += r);
gettimeofday(&t1, NULL);
timeradd(&TW, &t1, &TW);
timersub(&TW, &t0, &TW);

return c;
}

int rd(int fd, int size)
{
int c, r;
struct timeval t0, t1;

gettimeofday(&t0, NULL);
for (c = 0; (r = read(fd, buf, size)) > 0; c += r);
gettimeofday(&t1, NULL);
timeradd(&TR, &t1, &TR);
timersub(&TR, &t0, &TR);

return c;
}

int main(int argc, const char *argv[])
{
int fd[2], nb = 1, loop, size;

assert(argc == 3);
loop = atoi(argv[1]);
size = atoi(argv[2]);

assert(pipe(fd) == 0);
assert(ioctl(fd[0], FIONBIO, &nb) == 0);
assert(ioctl(fd[1], FIONBIO, &nb) == 0);

assert(size <= sizeof(buf));
while (loop--)
assert(wr(fd[1], size) == rd(fd[0], size));

struct timeval tt;
timeradd(&TW, &TR, &tt);
printf("TW = %lu.%03lu TR = %lu.%03lu TT = %lu.%03lu\n",
TW.tv_sec, TW.tv_usec/1000,
TR.tv_sec, TR.tv_usec/1000,
tt.tv_sec, tt.tv_usec/1000);

return 0;
}

Before:
# for i in 1 2 3; do /host/tmp/test 10000 100; done
TW = 8.047 TR = 5.845 TT = 13.893
TW = 8.091 TR = 5.872 TT = 13.963
TW = 8.083 TR = 5.885 TT = 13.969
After:
# for i in 1 2 3; do /host/tmp/test 10000 100; done
TW = 4.752 TR = 4.664 TT = 9.416
TW = 4.684 TR = 4.608 TT = 9.293
TW = 4.736 TR = 4.652 TT = 9.388

Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Link: https://lore.kernel.org/r/20250205181812.GC13817@redhat.com
Tested-by: K Prateek Nayak <kprateek.nayak@amd.com>
Signed-off-by: Christian Brauner <brauner@kernel.org>
fs/pipe.c