From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54318) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V6L4g-00072U-9Y for qemu-devel@nongnu.org; Mon, 05 Aug 2013 09:43:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1V6L4b-0006uf-9D for qemu-devel@nongnu.org; Mon, 05 Aug 2013 09:43:50 -0400 Received: from cantor2.suse.de ([195.135.220.15]:55000 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V6L4b-0006uL-0H for qemu-devel@nongnu.org; Mon, 05 Aug 2013 09:43:45 -0400 Message-ID: <51FFAC0D.1070300@suse.de> Date: Mon, 05 Aug 2013 15:43:41 +0200 From: =?ISO-8859-15?Q?Andreas_F=E4rber?= MIME-Version: 1.0 References: <1375567378-5089-1-git-send-email-aurelien@aurel32.net> <51FE4308.5010208@suse.de> <20130804220647.GB4193@ohm.aurel32.net> In-Reply-To: <20130804220647.GB4193@ohm.aurel32.net> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH for 1.6] mips: revert commit b332d24a8e1290954029814d09156b06ede358e2 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Aurelien Jarno Cc: Peter Maydell , qemu-devel@nongnu.org, Anthony Liguori Am 05.08.2013 00:06, schrieb Aurelien Jarno: > On Sun, Aug 04, 2013 at 02:03:20PM +0200, Andreas F=E4rber wrote: >> Am 04.08.2013 00:02, schrieb Aurelien Jarno: >>> Now that this code path is not triggered anymore during the tests, >>> revert commit b332d24a8e1290954029814d09156b06ede358e2. Booting a MIP= S >>> target without kernel nor bios doesn't really make sense. >>> >>> Signed-off-by: Aurelien Jarno >> >> This is being discussed in http://patchwork.ozlabs.org/patch/262912/ - >> so far Anthony has put a hold on further such changes unfortunately. >> >=20 > This has been an error for more than 6 years, and nobody complained so > far. Neither QOM nor qtest exist for 6 years, so that is not an argument for everything. ;) > I understand that the machines should be testable with qtest, but > such as change has been merged already. Now there is no reason to not > fix this *regression* from version 1.5. Ah, you mean this? http://git.qemu.org/?p=3Dqemu.git;a=3Dcommit;h=3Db332d24a8e1290954029814d= 09156b06ede358e2 Wasn't aware. No objection to exit(1) from my side then. But either way, you shouldn't replace one fprintf() with another fprintf() but instead use our new error_report() if you touch it (without trailing \n then). I've updated my qtest enablement series to use it, v2 handles some more machines. > People should understand that QEMU is not only x86, and that not > everything should be done the x86 way. No need to explain that to me. I think Anthony's question was rather whether printing random text to stderr is the best way to address that or whether QEMUMachine could use some this-machine-needs-a-kernel flag that libvirt or someone can access and that could be handled in a central place rather than in each machine as they see fit. But with the release near and no concrete patches, I don't think that's 1.6 material. Question is, do we want test cases based on cleanups that work today in 1.6 and work from there, or do we rather wait 'til after the release and if so, can we get them merged early so that other series can actually be tested with them. Regards, Andreas --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=FCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=F6rffer; HRB 16746 AG N=FCrnbe= rg