Re: [PATCH 00/10] Migration Arguments cleanup

2019-12-16 Thread Paolo Bonzini
On 16/12/19 16:46, Juan Quintela wrote: >> I have picked up this series and rebased the -accel changes on top. > Thanks. > > about the accel and the machine type, it feel so weird that we only > need to add a machine type for aarch64. Yes, it is. For now I have resolved the conflict to some

Re: [PATCH 00/10] Migration Arguments cleanup

2019-12-16 Thread Juan Quintela
Paolo Bonzini wrote: > On 12/12/19 23:20, Juan Quintela wrote: >> Hi >> >> This series simplify test_migrate_start() in two ways: >> - simplify the command line creation, so everything that is common between >> architectures don't have to be repeated (DRY). >> Note that this bit remove lines

Re: [PATCH 00/10] Migration Arguments cleanup

2019-12-16 Thread Paolo Bonzini
On 12/12/19 23:20, Juan Quintela wrote: > Hi > > This series simplify test_migrate_start() in two ways: > - simplify the command line creation, so everything that is common between > architectures don't have to be repeated (DRY). > Note that this bit remove lines of code. > - test_migrate_star

Re: [PATCH 00/10] Migration Arguments cleanup

2019-12-16 Thread Laurent Vivier
On 12/12/2019 23:20, Juan Quintela wrote: > Hi > > This series simplify test_migrate_start() in two ways: > - simplify the command line creation, so everything that is common between > architectures don't have to be repeated (DRY). > Note that this bit remove lines of code. > - test_migrate_st

Re: [PATCH 00/10] Migration Arguments cleanup

2019-12-16 Thread Cornelia Huck
On Thu, 12 Dec 2019 23:20:23 +0100 Juan Quintela wrote: > Hi > > This series simplify test_migrate_start() in two ways: > - simplify the command line creation, so everything that is common between > architectures don't have to be repeated (DRY). > Note that this bit remove lines of code. > -

[PATCH 00/10] Migration Arguments cleanup

2019-12-12 Thread Juan Quintela
Hi This series simplify test_migrate_start() in two ways: - simplify the command line creation, so everything that is common between architectures don't have to be repeated (DRY). Note that this bit remove lines of code. - test_migrate_start() has two bools and two strings as arguments, it is