From 0e442efb1e2e3d55dc335c02d8e983c9ea32e3b6 Mon Sep 17 00:00:00 2001 From: Sasha Levin Date: Sun, 22 Jul 2012 18:29:54 +0200 Subject: [PATCH] kvm tools: don't bother tracking is_dir This is something we can calculate on the fly, and doesn't justify the overhead of tracking it all over fid transitions. Signed-off-by: Sasha Levin Signed-off-by: Pekka Enberg --- tools/kvm/include/kvm/virtio-9p.h | 1 - tools/kvm/virtio/9p.c | 16 +++++++++++----- 2 files changed, 11 insertions(+), 6 deletions(-) diff --git a/tools/kvm/include/kvm/virtio-9p.h b/tools/kvm/include/kvm/virtio-9p.h index 186fe05ddbc5..cb590d185db3 100644 --- a/tools/kvm/include/kvm/virtio-9p.h +++ b/tools/kvm/include/kvm/virtio-9p.h @@ -26,7 +26,6 @@ struct p9_msg { struct p9_fid { u32 fid; u32 uid; - u8 is_dir; char abs_path[PATH_MAX]; char *path; DIR *dir; diff --git a/tools/kvm/virtio/9p.c b/tools/kvm/virtio/9p.c index 99279f19ead1..27ef57b8d06d 100644 --- a/tools/kvm/virtio/9p.c +++ b/tools/kvm/virtio/9p.c @@ -228,6 +228,15 @@ static int virtio_p9_openflags(int flags) return flags; } +static bool is_dir(struct p9_fid *fid) +{ + struct stat st; + + stat(fid->abs_path, &st); + + return S_ISDIR(st.st_mode); +} + static void virtio_p9_open(struct p9_dev *p9dev, struct p9_pdu *pdu, u32 *outlen) { @@ -245,7 +254,7 @@ static void virtio_p9_open(struct p9_dev *p9dev, stat2qid(&st, &qid); - if (new_fid->is_dir) { + if (is_dir(new_fid)) { new_fid->dir = opendir(new_fid->abs_path); if (!new_fid->dir) goto err_out; @@ -394,7 +403,6 @@ static void virtio_p9_walk(struct p9_dev *p9dev, goto err_out; stat2qid(&st, &wqid); - new_fid->is_dir = S_ISDIR(st.st_mode); strcpy(new_fid->path, tmp); new_fid->uid = fid->uid; nwqid++; @@ -406,7 +414,6 @@ static void virtio_p9_walk(struct p9_dev *p9dev, */ pdu->write_offset += sizeof(u16); old_fid = get_fid(p9dev, fid_val); - new_fid->is_dir = old_fid->is_dir; strcpy(new_fid->path, old_fid->path); new_fid->uid = old_fid->uid; } @@ -445,7 +452,6 @@ static void virtio_p9_attach(struct p9_dev *p9dev, fid = get_fid(p9dev, fid_val); fid->uid = uid; - fid->is_dir = 1; strcpy(fid->path, "/"); virtio_p9_pdu_writef(pdu, "Q", &qid); @@ -547,7 +553,7 @@ static void virtio_p9_readdir(struct p9_dev *p9dev, virtio_p9_pdu_readf(pdu, "dqd", &fid_val, &offset, &count); fid = get_fid(p9dev, fid_val); - if (!fid->is_dir) { + if (!is_dir(fid)) { errno = EINVAL; goto err_out; } -- 2.39.5