]> git.baikalelectronics.ru Git - kernel.git/commit
x86: Fix vtime/file timestamp inconsistencies
authorJohn Stultz <johnstul@us.ibm.com>
Wed, 14 Jul 2010 00:56:18 +0000 (17:56 -0700)
committerThomas Gleixner <tglx@linutronix.de>
Tue, 27 Jul 2010 10:40:53 +0000 (12:40 +0200)
commit858866ded004543525a30724b9c235d2ec7ebe2e
treea3558d3fbfe59a68d4fce1b4e7b8919908a96aad
parent6954e0de3cbee965801a885d82b51a7dbae7a940
x86: Fix vtime/file timestamp inconsistencies

Due to vtime calling vgettimeofday(), its possible that an application
could call  time();create("stuff",O_RDRW);  only to see the file's
creation timestamp to be before the value returned by time.

A similar way to reproduce the issue is to compare the vsyscall time()
with the syscall time(), and observe ordering issues.

The modified test case from Oleg Nesterov below can illustrate this:

int main(void)
{
time_t sec1,sec2;
do {
sec1 = time(&sec2);
sec2 = syscall(__NR_time, NULL);
} while (sec1 <= sec2);

printf("vtime: %d.000000\n", sec1);
printf("time: %d.000000\n", sec2);
return 0;
}

The proper fix is to make vtime use the same time value as
current_kernel_time() (which is exported via update_vsyscall) instead of
vgettime().

Thanks to Jiri Olsa for bringing up the issue and catching bugs in
earlier verisons of this fix.

Signed-off-by: John Stultz <johnstul@us.ibm.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Oleg Nesterov <oleg@redhat.com>
LKML-Reference: <1279068988-21864-2-git-send-email-johnstul@us.ibm.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
arch/x86/kernel/vsyscall_64.c