Re: [Qemu-devel] [PATCH 0/3] Improvements for the boot-sector tester

2016-10-11 Thread David Gibson
On Wed, Oct 12, 2016 at 12:05:24AM +0300, Michael S. Tsirkin wrote:
> On Tue, Oct 11, 2016 at 05:19:34PM +0200, Thomas Huth wrote:
> > Here are two patches that try to improve the situation with the
> > slow pxe-test on ppc64 a little bit, and one patch that fixes
> > a potential race condition between tests that run in parallel
> > by using a random filename instead of an invariant one.
> 
> 
> Reviewed-by: Michael S. Tsirkin 
> 
> You can go ahead and merge as I'm offline tomorrow.

Done.

> 
> > Thomas Huth (3):
> >   tests/boot-sector: Use minimum length for the Forth boot script
> >   tests/boot-sector: Use mkstemp() to create a unique file name
> >   tests/boot-sector: Increase time-out to 90 seconds
> > 
> >  tests/bios-tables-test.c |  2 +-
> >  tests/boot-sector.c  | 25 -
> >  tests/boot-sector.h  |  4 ++--
> >  tests/pxe-test.c |  2 +-
> >  4 files changed, 20 insertions(+), 13 deletions(-)
> > 
> 

-- 
David Gibson| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au  | minimalist, thank you.  NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson


signature.asc
Description: PGP signature


Re: [Qemu-devel] [PATCH 0/3] Improvements for the boot-sector tester

2016-10-11 Thread Michael S. Tsirkin
On Tue, Oct 11, 2016 at 05:19:34PM +0200, Thomas Huth wrote:
> Here are two patches that try to improve the situation with the
> slow pxe-test on ppc64 a little bit, and one patch that fixes
> a potential race condition between tests that run in parallel
> by using a random filename instead of an invariant one.


Reviewed-by: Michael S. Tsirkin 

You can go ahead and merge as I'm offline tomorrow.

> Thomas Huth (3):
>   tests/boot-sector: Use minimum length for the Forth boot script
>   tests/boot-sector: Use mkstemp() to create a unique file name
>   tests/boot-sector: Increase time-out to 90 seconds
> 
>  tests/bios-tables-test.c |  2 +-
>  tests/boot-sector.c  | 25 -
>  tests/boot-sector.h  |  4 ++--
>  tests/pxe-test.c |  2 +-
>  4 files changed, 20 insertions(+), 13 deletions(-)
> 
> -- 
> 1.8.3.1



[Qemu-devel] [PATCH 0/3] Improvements for the boot-sector tester

2016-10-11 Thread Thomas Huth
Here are two patches that try to improve the situation with the
slow pxe-test on ppc64 a little bit, and one patch that fixes
a potential race condition between tests that run in parallel
by using a random filename instead of an invariant one.

Thomas Huth (3):
  tests/boot-sector: Use minimum length for the Forth boot script
  tests/boot-sector: Use mkstemp() to create a unique file name
  tests/boot-sector: Increase time-out to 90 seconds

 tests/bios-tables-test.c |  2 +-
 tests/boot-sector.c  | 25 -
 tests/boot-sector.h  |  4 ++--
 tests/pxe-test.c |  2 +-
 4 files changed, 20 insertions(+), 13 deletions(-)

-- 
1.8.3.1