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

Sasha Levin
2011-06-19T06:42:02Z

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.