[PATCH 1/2] tools/kvm/9p: Make the 9p handler void return

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Error is reported via RERROR message.

Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@xxxxxxxxxxxxxxxxxx>
---
 tools/kvm/virtio/9p.c |   54 ++++++++++++++++++++++++------------------------
 1 files changed, 27 insertions(+), 27 deletions(-)

diff --git a/tools/kvm/virtio/9p.c b/tools/kvm/virtio/9p.c
index 0e885b5..8067e8a 100644
--- a/tools/kvm/virtio/9p.c
+++ b/tools/kvm/virtio/9p.c
@@ -225,7 +225,7 @@ static void virtio_p9_error_reply(struct p9_dev *p9dev,
 	set_p9msg_hdr(inmsg, *outlen, P9_RERROR, outmsg->tag);
 }
 
-static bool virtio_p9_version(struct p9_dev *p9dev,
+static void virtio_p9_version(struct p9_dev *p9dev,
 			      struct p9_pdu *pdu, u32 *outlen)
 {
 	struct p9_msg *inmsg  = pdu->in_iov[0].iov_base;
@@ -240,10 +240,10 @@ static bool virtio_p9_version(struct p9_dev *p9dev,
 		rversion->version.len + sizeof(u16) + sizeof(u32);
 	set_p9msg_hdr(inmsg, *outlen, P9_RVERSION, outmsg->tag);
 
-	return true;
+	return;
 }
 
-static bool virtio_p9_clunk(struct p9_dev *p9dev,
+static void virtio_p9_clunk(struct p9_dev *p9dev,
 			    struct p9_pdu *pdu, u32 *outlen)
 {
 	struct p9_msg *inmsg  = pdu->in_iov[0].iov_base;
@@ -255,10 +255,10 @@ static bool virtio_p9_clunk(struct p9_dev *p9dev,
 	*outlen = VIRTIO_P9_HDR_LEN;
 	set_p9msg_hdr(inmsg, *outlen, P9_RCLUNK, outmsg->tag);
 
-	return true;
+	return;
 }
 
-static bool virtio_p9_open(struct p9_dev *p9dev,
+static void virtio_p9_open(struct p9_dev *p9dev,
 			   struct p9_pdu *pdu, u32 *outlen)
 {
 	struct stat st;
@@ -286,13 +286,13 @@ static bool virtio_p9_open(struct p9_dev *p9dev,
 	}
 	*outlen = VIRTIO_P9_HDR_LEN + sizeof(*ropen);
 	set_p9msg_hdr(inmsg, *outlen, P9_ROPEN, outmsg->tag);
-	return true;
+	return;
 err_out:
 	virtio_p9_error_reply(p9dev, pdu, errno, outlen);
-	return true;
+	return;
 }
 
-static bool virtio_p9_create(struct p9_dev *p9dev,
+static void virtio_p9_create(struct p9_dev *p9dev,
 			     struct p9_pdu *pdu, u32 *outlen)
 {
 	u8 mode;
@@ -324,17 +324,17 @@ static bool virtio_p9_create(struct p9_dev *p9dev,
 	}
 
 	if (lstat(fid->abs_path, &st) < 0)
-		return false;
+		return;
 
 	st2qid(&st, &rcreate->qid);
 
 	*outlen = VIRTIO_P9_HDR_LEN + sizeof(*rcreate);
 	set_p9msg_hdr(inmsg, *outlen, P9_RCREATE, outmsg->tag);
 
-	return true;
+	return;
 }
 
-static bool virtio_p9_walk(struct p9_dev *p9dev,
+static void virtio_p9_walk(struct p9_dev *p9dev,
 			   struct p9_pdu *pdu, u32 *outlen)
 {
 	u8 i;
@@ -378,10 +378,10 @@ static bool virtio_p9_walk(struct p9_dev *p9dev,
 		sizeof(struct p9_qid)*rwalk->nwqid;
 	set_p9msg_hdr(inmsg, *outlen, P9_RWALK, outmsg->tag);
 
-	return true;
+	return;
 }
 
-static bool virtio_p9_attach(struct p9_dev *p9dev,
+static void virtio_p9_attach(struct p9_dev *p9dev,
 			     struct p9_pdu *pdu, u32 *outlen)
 {
 	u32 i;
@@ -397,7 +397,7 @@ static bool virtio_p9_attach(struct p9_dev *p9dev,
 		p9dev->fids[i].fid = P9_NOFID;
 
 	if (lstat(p9dev->root_dir, &st) < 0)
-		return false;
+		return;
 
 	st2qid(&st, &rattach->qid);
 
@@ -409,7 +409,7 @@ static bool virtio_p9_attach(struct p9_dev *p9dev,
 	*outlen = VIRTIO_P9_HDR_LEN + sizeof(*rattach);
 	set_p9msg_hdr(inmsg, *outlen, P9_RATTACH, outmsg->tag);
 
-	return true;
+	return;
 }
 
 static u32 virtio_p9_fill_stat(struct p9_dev *p9dev, const char *name,
@@ -452,7 +452,7 @@ static u32 virtio_p9_fill_stat(struct p9_dev *p9dev, const char *name,
 	return rstat->stat.size + sizeof(u16);
 }
 
-static bool virtio_p9_read(struct p9_dev *p9dev,
+static void virtio_p9_read(struct p9_dev *p9dev,
 			   struct p9_pdu *pdu, u32 *outlen)
 {
 	struct p9_msg *inmsg  = pdu->in_iov[0].iov_base;
@@ -495,10 +495,10 @@ static bool virtio_p9_read(struct p9_dev *p9dev,
 	*outlen = VIRTIO_P9_HDR_LEN + sizeof(u32) + rread->count;
 	set_p9msg_hdr(inmsg, *outlen, P9_RREAD, outmsg->tag);
 
-	return true;
+	return;
 }
 
-static bool virtio_p9_stat(struct p9_dev *p9dev,
+static void virtio_p9_stat(struct p9_dev *p9dev,
 			   struct p9_pdu *pdu, u32 *outlen)
 {
 	u32 ret;
@@ -510,16 +510,16 @@ static bool virtio_p9_stat(struct p9_dev *p9dev,
 	struct p9_fid *fid = &p9dev->fids[tstat->fid];
 
 	if (lstat(fid->abs_path, &st) < 0)
-		return false;
+		return;
 
 	ret = virtio_p9_fill_stat(p9dev, fid->path, &st, rstat);
 
 	*outlen = VIRTIO_P9_HDR_LEN + ret + sizeof(u16);
 	set_p9msg_hdr(inmsg, *outlen, P9_RSTAT, outmsg->tag);
-	return true;
+	return;
 }
 
-static bool virtio_p9_wstat(struct p9_dev *p9dev,
+static void virtio_p9_wstat(struct p9_dev *p9dev,
 			    struct p9_pdu *pdu, u32 *outlen)
 {
 	int res = 0;
@@ -556,10 +556,10 @@ static bool virtio_p9_wstat(struct p9_dev *p9dev,
 	*outlen = VIRTIO_P9_HDR_LEN;
 	set_p9msg_hdr(inmsg, *outlen, P9_RWSTAT, outmsg->tag);
 
-	return res == 0;
+	return;
 }
 
-static bool virtio_p9_remove(struct p9_dev *p9dev,
+static void virtio_p9_remove(struct p9_dev *p9dev,
 			     struct p9_pdu *pdu, u32 *outlen)
 {
 	struct p9_msg *inmsg  = pdu->in_iov[0].iov_base;
@@ -575,10 +575,10 @@ static bool virtio_p9_remove(struct p9_dev *p9dev,
 
 	*outlen = VIRTIO_P9_HDR_LEN;
 	set_p9msg_hdr(inmsg, *outlen, P9_RREMOVE, outmsg->tag);
-	return true;
+	return;
 }
 
-static bool virtio_p9_write(struct p9_dev *p9dev,
+static void virtio_p9_write(struct p9_dev *p9dev,
 			    struct p9_pdu *pdu, u32 *outlen)
 {
 	struct p9_msg *inmsg  = pdu->in_iov[0].iov_base;
@@ -597,10 +597,10 @@ static bool virtio_p9_write(struct p9_dev *p9dev,
 	*outlen = VIRTIO_P9_HDR_LEN + sizeof(u32);
 	set_p9msg_hdr(inmsg, *outlen, P9_RWRITE, outmsg->tag);
 
-	return true;
+	return;
 }
 
-typedef bool p9_handler(struct p9_dev *p9dev,
+typedef void p9_handler(struct p9_dev *p9dev,
 			struct p9_pdu *pdu, u32 *outlen);
 
 static p9_handler *virtio_9p_handler [] = {
-- 
1.7.4.1

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux