[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 3/9] s390x/pci: add stubs
From: |
Christian Borntraeger |
Subject: |
Re: [Qemu-devel] [PATCH v5 3/9] s390x/pci: add stubs |
Date: |
Thu, 24 Aug 2017 09:43:48 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.0 |
On 08/24/2017 09:38 AM, Christian Borntraeger wrote:
>
>
> On 08/23/2017 05:54 PM, Cornelia Huck wrote:
>> Some non-pci code calls into zpci code. Provide some stubs for builds
>> without pci.
>>
>> Reviewed-by: Thomas Huth <address@hidden>
>> Signed-off-by: Cornelia Huck <address@hidden>
>> ---
>> hw/s390x/Makefile.objs | 3 +-
>> hw/s390x/s390-pci-stub.c | 74
>> ++++++++++++++++++++++++++++++++++++++++++++++++
>> 2 files changed, 76 insertions(+), 1 deletion(-)
>> create mode 100644 hw/s390x/s390-pci-stub.c
>>
>> diff --git a/hw/s390x/Makefile.objs b/hw/s390x/Makefile.objs
>> index b2aade2466..7ee19d3abc 100644
>> --- a/hw/s390x/Makefile.objs
>> +++ b/hw/s390x/Makefile.objs
>> @@ -11,7 +11,8 @@ obj-y += 3270-ccw.o
>> obj-y += virtio-ccw.o
>> obj-y += css-bridge.o
>> obj-y += ccw-device.o
>> -obj-y += s390-pci-bus.o s390-pci-inst.o
>> +obj-$(CONFIG_PCI) += s390-pci-bus.o s390-pci-inst.o
>> +obj-$(call lnot,$(CONFIG_PCI)) += s390-pci-stub.o
>> obj-y += s390-skeys.o
>> obj-y += s390-stattrib.o
>> obj-$(CONFIG_KVM) += s390-skeys-kvm.o
>> diff --git a/hw/s390x/s390-pci-stub.c b/hw/s390x/s390-pci-stub.c
>> new file mode 100644
>> index 0000000000..2e7e42a2af
>> --- /dev/null
>> +++ b/hw/s390x/s390-pci-stub.c
>> @@ -0,0 +1,74 @@
>> +/* stubs for non-pci builds */
>> +
>> +#include "qemu/osdep.h"
>> +#include "qemu-common.h"
>> +#include "cpu.h"
>> +#include "s390-pci-inst.h"
>> +#include "s390-pci-bus.h"
>> +
>> +/* target/s390x/ioinst.c */
>> +int chsc_sei_nt2_get_event(void *res)
>> +{
>> + return 1;
>> +}
>> +
>> +int chsc_sei_nt2_have_event(void)
>> +{
>> + return 0;
>> +}
>> +
>> +/* hw/s390x/sclp.c */
>> +void s390_pci_sclp_configure(SCCB *sccb)
>> +{
>> +}
>> +
>> +void s390_pci_sclp_deconfigure(SCCB *sccb)
>> +{
>> +}
>
> shouldnt these function set an error code in the sccb, e.g.
> something like
>
> sccb->h.response_code = cpu_to_be16(SCLP_RC_INVALID_SCLP_COMMAND);
>
>
>
Oh you have something like that in patch 7. Maybe move?
[Qemu-devel] [PATCH v5 4/9] s390x: chsc nt2 events are pci-only, Cornelia Huck, 2017/08/23
[Qemu-devel] [PATCH v5 5/9] s390x/pci: do not advertise pci on non-pci builds, Cornelia Huck, 2017/08/23
[Qemu-devel] [PATCH v5 6/9] s390x/ccw: create s390 phb conditionally, Cornelia Huck, 2017/08/23
[Qemu-devel] [PATCH v5 7/9] s390x/sclp: properly guard pci-specific functions, Cornelia Huck, 2017/08/23