From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53863) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yrq0K-0001NX-8i for qemu-devel@nongnu.org; Mon, 11 May 2015 11:52:29 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Yrq0F-0003DS-R9 for qemu-devel@nongnu.org; Mon, 11 May 2015 11:52:28 -0400 Received: from mx1.redhat.com ([209.132.183.28]:41189) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yrq0F-0003DL-Mn for qemu-devel@nongnu.org; Mon, 11 May 2015 11:52:23 -0400 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (Postfix) with ESMTPS id 4DEECC408E for ; Mon, 11 May 2015 15:52:23 +0000 (UTC) From: Markus Armbruster References: <1431322734-12520-1-git-send-email-mrezanin@redhat.com> <55506AE4.8030902@redhat.com> <20150511093643.GB26672@lws.brq.redhat.com> <5550806F.6070909@redhat.com> Date: Mon, 11 May 2015 17:52:20 +0200 In-Reply-To: <5550806F.6070909@redhat.com> (Paolo Bonzini's message of "Mon, 11 May 2015 12:11:59 +0200") Message-ID: <877fsfaz7v.fsf@blackfin.pond.sub.org> MIME-Version: 1.0 Content-Type: text/plain Subject: Re: [Qemu-devel] [PATCHv2] parallel: Allow to disable CONFIG_PARALLEL List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: Miroslav Rezanina , thuth@redhat.com, qemu-devel@nongnu.org Paolo Bonzini writes: > On 11/05/2015 11:36, Miroslav Rezanina wrote: >>> > This patch will make "-parallel" a nop. The right thing to do is to >>> > fail startup whenever -parallel is passed and CONFIG_PARALLEL is disabled. >>> > >> This was original behavior before 07dc788. Intention of this patch is to >> make qemu buildable with CONFIG_PARALLEL disabled. > > Understood, but in the meanwhile Markus wrote commit 4bc6a3e (parallel: > parallel_hds_isa_init() shouldn't fail, 2015-02-04), and you should > preserve the logic of that commit. I have to admit didn't consider CONFIG_PARALLEL when I wrote the commit. >>> > You can move parallel_hds_isa_init and parallel_init to >>> > hw/isa/isa-bus.c, or to a new file hw/isa/isa-devices.c. >>> > >> Moving functions will cause abort with "Unknown device" error. > > This is the right behavior that we want: exit QEMU, not go on silently > without the parallel port. I agree silently ignoring command line options isn't nice, but it's unfortunately what QEMU has always done. In particular, -parallel is silently ignored with the vast majority of machine types. The few machine types that implement it silently ignore it only when they fail to create the device. I'm fine with changing -parallel to either create the device or fail. Seems outside the scope of this series, though. > If you do not like the abort, you should revert commit 4bc6a3e, and make > parallel_hds_isa_init check for failure of parallel_init. But for me > it's okay to just let it abort.