[Qemu-devel] Re: [PATCH 10/12] config: Add header file for device config options

[Juan Quintela <quintela@redhat.com>]

Alexander Graf <agraf@suse.de> wrote:
> So far we have C preprocessor defines for target and host config
> options, but we're lacking any information on which devices are
> available.
>
> We do need that information at times though, for example in the
> ahci patch where we need to call a legacy init function depending
> on whether we have support compiled in or not.
>
> So this patch makes all config-devices options available as header
> file. Please only include it in machine description code!
>
> Signed-off-by: Alexander Graf <agraf@suse.de>

This already existed in the past,  Paul Brook reverted it.

commit a992fe3d0fc185112677286f7a02204d8245b61e
Author: Paul Brook <paul@codesourcery.com>
Date:   Sun Nov 22 16:25:30 2009 +0000

    Makefile dependencies for device configs

I did the 1st implementation, and I tried to get it back several times.
Paul Brook reasoning is that "if you need it, our device model is wrong.

If you show that you want to be able to compile out several devices, ...

Paul Brook:  "if you need it, our device model is wrong"

If you don't want to have the abomination of xen-stub.c kvm-stub.c,
etc....

Paul Brook:  "if you need it, our device model is wrong".

Last time I tried to get it back was to be able to compile out hpet (it
didn't worked well at the time).

I let you guess how it ended.

Later, Juan.