Patchwork [Lucid,SRU] net/9p: fix virtio transport to correctly update status on connect

login
register
mail settings
Submitter Tim Gardner
Date Oct. 2, 2012, 7:19 p.m.
Message ID <1349205544-78515-1-git-send-email-tim.gardner@canonical.com>
Download mbox | patch
Permalink /patch/188634/
State New
Headers show

Comments

Tim Gardner - Oct. 2, 2012, 7:19 p.m.
From: Eric Van Hensbergen <ericvh@gmail.com>

BugLink: http://bugs.launchpad.net/bugs/676823

The 9p virtio transport was not updating its connection status correctly
preventing it from being able to mount the server.

Signed-off-by: Eric Van Hensbergen <ericvh@gmail.com>
(cherry picked from commit 562ada612058133a5483c68a73605f3c5f42fffe)

Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
---
 net/9p/trans_virtio.c |    1 +
 1 file changed, 1 insertion(+)
Brad Figg - Oct. 2, 2012, 7:38 p.m.
On 10/02/2012 12:19 PM, Tim Gardner wrote:
> From: Eric Van Hensbergen <ericvh@gmail.com>
> 
> BugLink: http://bugs.launchpad.net/bugs/676823
> 
> The 9p virtio transport was not updating its connection status correctly
> preventing it from being able to mount the server.
> 
> Signed-off-by: Eric Van Hensbergen <ericvh@gmail.com>
> (cherry picked from commit 562ada612058133a5483c68a73605f3c5f42fffe)
> 
> Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
> ---
>  net/9p/trans_virtio.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/net/9p/trans_virtio.c b/net/9p/trans_virtio.c
> index ea1e3da..67c4bc7 100644
> --- a/net/9p/trans_virtio.c
> +++ b/net/9p/trans_virtio.c
> @@ -311,6 +311,7 @@ p9_virtio_create(struct p9_client *client, const char *devname, char *args)
>  	}
>  
>  	client->trans = (void *)chan;
> +	client->status = Connected;
>  	chan->client = client;
>  
>  	return 0;
>
Colin King - Oct. 3, 2012, 11 a.m.
On 02/10/12 20:19, Tim Gardner wrote:
> From: Eric Van Hensbergen <ericvh@gmail.com>
>
> BugLink: http://bugs.launchpad.net/bugs/676823
>
> The 9p virtio transport was not updating its connection status correctly
> preventing it from being able to mount the server.
>
> Signed-off-by: Eric Van Hensbergen <ericvh@gmail.com>
> (cherry picked from commit 562ada612058133a5483c68a73605f3c5f42fffe)
>
> Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
> ---
>   net/9p/trans_virtio.c |    1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/net/9p/trans_virtio.c b/net/9p/trans_virtio.c
> index ea1e3da..67c4bc7 100644
> --- a/net/9p/trans_virtio.c
> +++ b/net/9p/trans_virtio.c
> @@ -311,6 +311,7 @@ p9_virtio_create(struct p9_client *client, const char *devname, char *args)
>   	}
>
>   	client->trans = (void *)chan;
> +	client->status = Connected;
>   	chan->client = client;
>
>   	return 0;
>
Acked-by: Colin Ian King <colin.king@canonical.com>
Tim Gardner - Oct. 3, 2012, 12:10 p.m.

Patch

diff --git a/net/9p/trans_virtio.c b/net/9p/trans_virtio.c
index ea1e3da..67c4bc7 100644
--- a/net/9p/trans_virtio.c
+++ b/net/9p/trans_virtio.c
@@ -311,6 +311,7 @@  p9_virtio_create(struct p9_client *client, const char *devname, char *args)
 	}
 
 	client->trans = (void *)chan;
+	client->status = Connected;
 	chan->client = client;
 
 	return 0;