Re: [Qemu-devel] [PATCH v3 0/2] qga: flush explicitly when needed

[Michael Roth]

Quoting marcandre.lur...@redhat.com (2015-11-25 06:59:10)
> From: Marc-André Lureau 
> 
> Without this change, a write() followed by a read() may lose the
> previously written content, as shown in the following test.
> 
> v2->v3:
> - use a RwState tristate enum
> - reset the state on flush & seek
> 
> v1->v2:
> - replace guchar with unsigned char
> - fix implicitly/explictly
> - comment space fix
> 
> Marc-André Lureau (2):
>   qga: flush explicitly when needed
>   tests: add file-write-read test

Thanks, applied with fix-ups suggested by Lazlo:

  https://github.com/mdroth/qemu/commits/qga

> 
>  qga/commands-posix.c | 37 
>  tests/test-qga.c | 95 
> ++--
>  2 files changed, 130 insertions(+), 2 deletions(-)
> 
> -- 
> 2.5.0
> 

Re: [Qemu-devel] [PATCH v3 0/2] qga: flush explicitly when needed

[Marc-André Lureau]

Hi

- Original Message -
> Quoting marcandre.lur...@redhat.com (2015-11-25 06:59:10)
> > From: Marc-André Lureau 
> > 
> > Without this change, a write() followed by a read() may lose the
> > previously written content, as shown in the following test.
> > 
> > v2->v3:
> > - use a RwState tristate enum
> > - reset the state on flush & seek
> > 
> > v1->v2:
> > - replace guchar with unsigned char
> > - fix implicitly/explictly
> > - comment space fix
> > 
> > Marc-André Lureau (2):
> >   qga: flush explicitly when needed
> >   tests: add file-write-read test
> 
> Thanks, applied with fix-ups suggested by Lazlo:
> 
>   https://github.com/mdroth/qemu/commits/qga

thanks Michael!

[Qemu-devel] [PATCH v3 0/2] qga: flush explicitly when needed

[marcandre . lureau]

From: Marc-André Lureau 

Without this change, a write() followed by a read() may lose the
previously written content, as shown in the following test.

v2->v3:
- use a RwState tristate enum
- reset the state on flush & seek

v1->v2:
- replace guchar with unsigned char
- fix implicitly/explictly
- comment space fix

Marc-André Lureau (2):
  qga: flush explicitly when needed
  tests: add file-write-read test

 qga/commands-posix.c | 37 
 tests/test-qga.c | 95 ++--
 2 files changed, 130 insertions(+), 2 deletions(-)

-- 
2.5.0