Re: [Qemu-devel] [PATCH 5/6] hw/9pfs: Use v9fs_do_close instead of close

[Venkateswararao Jujjuri <jvrao@linux.vnet.ibm.com>]

On 06/06/2011 10:16 AM, Aneesh Kumar K.V wrote:
> we should use the local abstraction instead of
> directly calling close.
>
Let us fold this also into our coroutine patches.

- JV

> Signed-off-by: Aneesh Kumar K.V<aneesh.kumar@linux.vnet.ibm.com>
> ---
>   hw/9pfs/virtio-9p.c |    4 ++--
>   1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/9pfs/virtio-9p.c b/hw/9pfs/virtio-9p.c
> index 21e07fb..d322814 100644
> --- a/hw/9pfs/virtio-9p.c
> +++ b/hw/9pfs/virtio-9p.c
> @@ -1596,7 +1596,7 @@ static void v9fs_lcreate(void *opaque)
>       if (err<  0) {
>           fidp->fid_type = P9_FID_NONE;
>           if (fidp->fs.fd>  0) {
> -            close(fidp->fs.fd);
> +            v9fs_co_close(pdu->s, fidp->fs.fd);
>           }
>           goto out;
>       }
> @@ -2164,7 +2164,7 @@ static void v9fs_create(void *opaque)
>       if (err<  0) {
>           fidp->fid_type = P9_FID_NONE;
>           if (fidp->fs.fd) {
> -            close(fidp->fs.fd);
> +            v9fs_co_close(pdu->s, fidp->fs.fd);
>           }
>           goto out;
>       }