[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 08/20] [virtio-9p] Make v9fs_do_utimensat accept tim
From: |
Venkateswararao Jujjuri (JV) |
Subject: |
[Qemu-devel] [PATCH 08/20] [virtio-9p] Make v9fs_do_utimensat accept timespec structures instead of v9stat. |
Date: |
Mon, 28 Jun 2010 14:55:05 -0700 |
From: Sripathi Kodi <address@hidden>
Currently v9fs_do_utimensat takes a V9fsStat argument and builds
timespec structures. It sets tv_nsec values to 0 by default. Instead
of this it should take struct timespec[2] and pass it down to the
system directly. This will make it more generic and useful
elsewhere.
Signed-off-by: Sripathi Kodi <address@hidden>
Signed-off-by: Venkateswararao Jujjuri <address@hidden>
---
hw/virtio-9p.c | 37 ++++++++++++++++++-------------------
1 files changed, 18 insertions(+), 19 deletions(-)
diff --git a/hw/virtio-9p.c b/hw/virtio-9p.c
index 099514a..613abcd 100644
--- a/hw/virtio-9p.c
+++ b/hw/virtio-9p.c
@@ -237,25 +237,10 @@ static int v9fs_do_chown(V9fsState *s, V9fsString *path,
uid_t uid, gid_t gid)
return s->ops->chown(&s->ctx, path->data, &cred);
}
-static int v9fs_do_utimensat(V9fsState *s, V9fsString *path, V9fsStat v9stat)
+static int v9fs_do_utimensat(V9fsState *s, V9fsString *path,
+ const struct timespec times[2])
{
- struct timespec ts[2];
-
- if (v9stat.atime != -1) {
- ts[0].tv_sec = v9stat.atime;
- ts[0].tv_nsec = 0;
- } else {
- ts[0].tv_nsec = UTIME_OMIT;
- }
-
- if (v9stat.mtime != -1) {
- ts[1].tv_sec = v9stat.mtime;
- ts[1].tv_nsec = 0;
- } else {
- ts[1].tv_nsec = UTIME_OMIT;
- }
-
- return s->ops->utimensat(&s->ctx, path->data, ts);
+ return s->ops->utimensat(&s->ctx, path->data, times);
}
static int v9fs_do_remove(V9fsState *s, V9fsString *path)
@@ -2312,7 +2297,21 @@ static void v9fs_wstat_post_chmod(V9fsState *s,
V9fsWstatState *vs, int err)
}
if (vs->v9stat.mtime != -1 || vs->v9stat.atime != -1) {
- if (v9fs_do_utimensat(s, &vs->fidp->path, vs->v9stat)) {
+ struct timespec times[2];
+ if (vs->v9stat.atime != -1) {
+ times[0].tv_sec = vs->v9stat.atime;
+ times[0].tv_nsec = 0;
+ } else {
+ times[0].tv_nsec = UTIME_OMIT;
+ }
+ if (vs->v9stat.mtime != -1) {
+ times[1].tv_sec = vs->v9stat.mtime;
+ times[1].tv_nsec = 0;
+ } else {
+ times[1].tv_nsec = UTIME_OMIT;
+ }
+
+ if (v9fs_do_utimensat(s, &vs->fidp->path, times)) {
err = -errno;
}
}
--
1.6.5.2
- [Qemu-devel] [PATCH 18/20] virtio-9p: Implement TXATTRWALK, (continued)
- [Qemu-devel] [PATCH 18/20] virtio-9p: Implement TXATTRWALK, Venkateswararao Jujjuri (JV), 2010/06/28
- [Qemu-devel] [PATCH 10/20] [virtio-9p] Implement TLINK for 9P2000.L, Venkateswararao Jujjuri (JV), 2010/06/28
- [Qemu-devel] [PATCH 06/20] virtio-9p: getattr server implementation for 9P2000.L protocol., Venkateswararao Jujjuri (JV), 2010/06/28
- [Qemu-devel] [PATCH 17/20] virtio-9p: Add fidtype so that we can do type specific operation, Venkateswararao Jujjuri (JV), 2010/06/28
- [Qemu-devel] [PATCH 03/20] virtio-9p: Return correct error from v9fs_remove, Venkateswararao Jujjuri (JV), 2010/06/28
- [Qemu-devel] [PATCH 19/20] virtio-9p: Implement TXATTRCREATE, Venkateswararao Jujjuri (JV), 2010/06/28
- [Qemu-devel] [PATCH 16/20] qemu: virtio-9p: Implement LOPEN, Venkateswararao Jujjuri (JV), 2010/06/28
- [Qemu-devel] [PATCH 02/20] qemu: virtio-9p: Implement statfs support in server, Venkateswararao Jujjuri (JV), 2010/06/28
- [Qemu-devel] [PATCH 11/20] [virtio-9p] Define and implement TSYMLINK for 9P2000.L, Venkateswararao Jujjuri (JV), 2010/06/28
- [Qemu-devel] [PATCH 13/20] qemu: virtio-9p: Implement TMKNOD, Venkateswararao Jujjuri (JV), 2010/06/28
- [Qemu-devel] [PATCH 08/20] [virtio-9p] Make v9fs_do_utimensat accept timespec structures instead of v9stat.,
Venkateswararao Jujjuri (JV) <=
- [Qemu-devel] [PATCH 20/20] virtio-9p: Hide user.virtfs xattr in case of mapped security., Venkateswararao Jujjuri (JV), 2010/06/28
- [Qemu-devel] [PATCH 14/20] qemu: virtio-9p: Implement TMKDIR, Venkateswararao Jujjuri (JV), 2010/06/28
- [Qemu-devel] [PATCH 15/20] rename - change name of file or directory, Venkateswararao Jujjuri (JV), 2010/06/28