Re: [PATCH 5/5] tools/kvm/9p: Simplify the handler

[Sasha Levin <levinsasha928@gmail.com>]

On Sun, 2011-06-19 at 10:43 +0530, Aneesh Kumar K.V wrote:
> On Sat, 18 Jun 2011 14:51:34 -0400, Sasha Levin <levinsasha928@gmail.com> wrote:
> > On Sat, 2011-06-18 at 23:19 +0530, Aneesh Kumar K.V wrote:
> > > Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
> > > ---
> > >  tools/kvm/include/net/9p/9p.h |    1 -
> > >  tools/kvm/virtio/9p.c         |  261 +++++++++++++++++++++++------------------
> > >  2 files changed, 147 insertions(+), 115 deletions(-)
> > 
> > After applying the patches (within a 9p mounted dir):
> > 
> >  # mkdir new_dir
> >  # cp 10MB new_dir/
> > cp: cannot create regular file `new_dir/10MB': No such file or directory
> > 
> 
> works fine here
> 
> sh-2.05b# ls -al 10M
> -rwxr-xr-x    1 4294967294 4294967294 10485760 Jun 19  2011 10M
> sh-2.05b# mkdir new_dir
> sh-2.05b# cp 10M new_dir/
> sh-2.05b# 
> 
> Can you check the before and after behaviour with the patch ?

It works ok before the patch here.

I've added debug to the mount command, looks like it was caused due to
patch 4/5 (the removal of one u16) - the host didn't quite like it :)

-- 

Sasha.