[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 02/11] tests: Migration ppc test was missing
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-devel] [PATCH v4 02/11] tests: Migration ppc test was missing arguments |
Date: |
Wed, 10 Jan 2018 12:03:37 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 |
On 10/01/2018 11:43, Juan Quintela wrote:
> Laurent Vivier <address@hidden> wrote:
>> On 10/01/2018 10:21, Laurent Vivier wrote:
>>> On 10/01/2018 09:47, Juan Quintela wrote:
>>>> Peter Xu <address@hidden> wrote:
>>>>> On Fri, Jan 05, 2018 at 10:52:37PM +0100, Juan Quintela wrote:
>>>>>> Argument file is also needed there.
>>>>>>
>>>>>> Signed-off-by: Juan Quintela <address@hidden>
>>>>>
>>>>> Reviewed-by: Peter Xu <address@hidden>
>>>>>
>>>>> Does it also mean this?
>>>>>
>>>>> Fixes: aaf89c8a49a8c ("test: port postcopy test to ppc64")
>>>>
>>>> Dunno. I was trying to consolidate the command line options for ppc and
>>>> x86 when I found this problem. I haven't tested of ppc.
>>>
>>> I don't think it is needed. I think the content of the nvram is migrated
>>> (otherwise the test wouldn't work at all).
>>>
>>> The nvram is created by default, we need the command line parameter only
>>> to populate it from a file.
>>
>> A better change would be to use "-prom-env" instead of "-driver
>> if=pflash". I can send the patch if you want to add it in your series.
>
> Told the command line and I will add to the line.
>
> And using the same command in both sides makes easier to see that it is
> correct.
>
> Later, Juan.
>
("-machine accel=%s -m 256M"
" -name pcsource,debug-threads=on"
" -serial file:%s/src_serial"
" -prom-env '"
"boot-command=hex .\" _\" begin %x %x "
"do i c@ 1 + i c! 1000 +loop .\" B\" 0 "
"until'", accel, tmpfs, end_address, start_address);
Don't forget to remove include of "hw/nvram/chrp_nvram.h",
MIN_NVRAM_SIZE and init_bootfile_ppc().
Thanks,
Laurent
[Qemu-devel] [PATCH v4 03/11] tests: Consolidate accelerators declaration, Juan Quintela, 2018/01/05
[Qemu-devel] [PATCH v4 04/11] tests: Use consistent names and sizes for migration, Juan Quintela, 2018/01/05
Re: [Qemu-devel] [PATCH v4 04/11] tests: Use consistent names and sizes for migration, Dr. David Alan Gilbert, 2018/01/10
[Qemu-devel] [PATCH v4 07/11] tests: Add basic migration precopy tcp test, Juan Quintela, 2018/01/05