Re: [Qemu-devel] [PATCHv2] parallel: Allow to disable CONFIG_PARALLEL

[Paolo Bonzini]

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.

>> > 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.

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.

Paolo