From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49616) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1eZCZO-00077x-5x for qemu-devel@nongnu.org; Wed, 10 Jan 2018 04:21:14 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1eZCZN-0001XY-Bn for qemu-devel@nongnu.org; Wed, 10 Jan 2018 04:21:14 -0500 Received: from mx1.redhat.com ([209.132.183.28]:59508) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1eZCZN-0001Vh-5C for qemu-devel@nongnu.org; Wed, 10 Jan 2018 04:21:13 -0500 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id A2287C119CA8 for ; Wed, 10 Jan 2018 09:21:11 +0000 (UTC) References: <20180105215246.908-1-quintela@redhat.com> <20180105215246.908-3-quintela@redhat.com> <20180110062545.GB5984@xz-mi> <87incayua8.fsf@secure.laptop> From: Laurent Vivier Message-ID: <82a94975-e040-298d-7d10-7448a4a0af48@redhat.com> Date: Wed, 10 Jan 2018 10:21:05 +0100 MIME-Version: 1.0 In-Reply-To: <87incayua8.fsf@secure.laptop> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v4 02/11] tests: Migration ppc test was missing arguments List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: quintela@redhat.com, Peter Xu Cc: qemu-devel@nongnu.org, dgilbert@redhat.com On 10/01/2018 09:47, Juan Quintela wrote: > Peter Xu 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 >> >> Reviewed-by: Peter Xu >> >> 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. Thanks, Laurent