* Markus Armbruster (arm...@redhat.com) wrote:
> We compile pci-hmp-cmds.c always, but pci-qmp-cmds.c only when
> CONFIG_PCI. hw/pci/pci-stub.c keeps the linker happy when
> !CONFIG_PCI. Build pci-hmp-cmds.c that way, too.
>
> Signed-off-by: Markus Armbruster <arm...@redhat.com>
Had you considered wrapping the hmp-commands-info.hx entry
with a #if defined instead?
Dave
> ---
> hw/pci/pci-stub.c | 5 +++++
> hw/pci/meson.build | 2 +-
> 2 files changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/hw/pci/pci-stub.c b/hw/pci/pci-stub.c
> index f29ecc999e..01d20a2f67 100644
> --- a/hw/pci/pci-stub.c
> +++ b/hw/pci/pci-stub.c
> @@ -21,6 +21,7 @@
> #include "qemu/osdep.h"
> #include "sysemu/sysemu.h"
> #include "monitor/monitor.h"
> +#include "monitor/hmp.h"
> #include "qapi/qapi-commands-pci.h"
> #include "hw/pci/pci.h"
> #include "hw/pci/msi.h"
> @@ -34,6 +35,10 @@ PciInfoList *qmp_query_pci(Error **errp)
> return NULL;
> }
>
> +void hmp_info_pci(Monitor *mon, const QDict *qdict)
> +{
> +}
> +
> void hmp_pcie_aer_inject_error(Monitor *mon, const QDict *qdict)
> {
> monitor_printf(mon, "PCI devices not supported\n");
> diff --git a/hw/pci/meson.build b/hw/pci/meson.build
> index e42a133f3a..4fcd888b27 100644
> --- a/hw/pci/meson.build
> +++ b/hw/pci/meson.build
> @@ -5,6 +5,7 @@ pci_ss.add(files(
> 'pci.c',
> 'pci_bridge.c',
> 'pci_host.c',
> + 'pci-hmp-cmds.c',
> 'pci-qmp-cmds.c',
> 'pcie_sriov.c',
> 'shpc.c',
> @@ -20,4 +21,3 @@ softmmu_ss.add_all(when: 'CONFIG_PCI', if_true: pci_ss)
>
> softmmu_ss.add(when: 'CONFIG_PCI', if_false: files('pci-stub.c'))
> softmmu_ss.add(when: 'CONFIG_ALL', if_true: files('pci-stub.c'))
> -softmmu_ss.add(files('pci-hmp-cmds.c'))
> --
> 2.37.3
>
--
Dr. David Alan Gilbert / dgilb...@redhat.com / Manchester, UK
