From: Amit Shah <amit.shah@redhat.com>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: avi@redhat.com, kvm@vger.kernel.org, muli@il.ibm.com,
benami@il.ibm.com, weidong.han@intel.com, allen.m.kay@intel.com
Subject: Re: [PATCH 4/7] qemu: Include hw.h in qemu/hw/isa.h to fix compile issues
Date: Wed, 24 Sep 2008 09:57:39 +0530 [thread overview]
Message-ID: <200809240957.39770.amit.shah@redhat.com> (raw)
In-Reply-To: <48D915B8.6040401@codemonkey.ws>
* On Tuesday 23 Sep 2008 21:43:44 Anthony Liguori wrote:
> Amit Shah wrote:
> > Signed-off-by: Amit Shah <amit.shah@redhat.com>
> > ---
> > qemu/hw/isa.h | 2 ++
> > 1 files changed, 2 insertions(+), 0 deletions(-)
> >
> > diff --git a/qemu/hw/isa.h b/qemu/hw/isa.h
> > index 222e4f3..e4a1326 100644
> > --- a/qemu/hw/isa.h
> > +++ b/qemu/hw/isa.h
> > @@ -2,6 +2,8 @@
> > #define HW_ISA_H
> > /* ISA bus */
> >
> > +#include "hw.h"
> > +
> > extern target_phys_addr_t isa_mem_base;
> >
> > int register_ioport_read(int start, int length, int size,
>
> What compile issues?
register_ioport_read* and register_ioport_write* functions cause a lot of
this.
gcc -I. -I.. -I/home/amit/src/kvm-userspace/qemu/target-i386 -I/home/amit/src/kvm-userspace/qemu -MMD -MT
device-assignment.o -MP -DNEED_CPU_H -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D__user= -I/home/amit/src/kvm-userspace/qemu/tcg -I/home/amit/src/kvm-userspace/qemu/tcg/x86_64 -I/home/amit/src/kvm-userspace/qemu/fpu -DHAS_AUDIO -DHAS_AUDIO_CHOICE -I/home/amit/src/kvm-userspace/qemu/slirp -I /home/amit/src/kvm-userspace/qemu/../libkvm -DCONFIG_X86 -O2 -g -fno-strict-aliasing -Wall -Wundef -Wendif-labels -Wwrite-strings -m64 -I /home/amit/src/kvm-userspace/kernel/include -c -o
device-assignment.o /home/amit/src/kvm-userspace/qemu/hw/device-assignment.c
In file included from /home/amit/src/kvm-userspace/qemu/hw/pci.h:6,
from /home/amit/src/kvm-userspace/qemu/hw/device-assignment.h:34,
from /home/amit/src/kvm-userspace/qemu/hw/device-assignment.c:32:
/home/amit/src/kvm-userspace/qemu/hw/isa.h:8: error: expected declaration
specifiers or â...â before âIOPortReadFuncâ
/home/amit/src/kvm-userspace/qemu/hw/isa.h:10: error: expected declaration
specifiers or â...â before âIOPortWriteFuncâ
/home/amit/src/kvm-userspace/qemu/hw/device-assignment.c: In function
âassigned_dev_ioport_mapâ:
/home/amit/src/kvm-userspace/qemu/hw/device-assignment.c:193: error: too many
arguments to function âregister_ioport_readâ
/home/amit/src/kvm-userspace/qemu/hw/device-assignment.c:195: error: too many
arguments to function âregister_ioport_readâ
/home/amit/src/kvm-userspace/qemu/hw/device-assignment.c:197: error: too many
arguments to function âregister_ioport_readâ
/home/amit/src/kvm-userspace/qemu/hw/device-assignment.c:199: error: too many
arguments to function âregister_ioport_writeâ
/home/amit/src/kvm-userspace/qemu/hw/device-assignment.c:201: error: too many
arguments to function âregister_ioport_writeâ
/home/amit/src/kvm-userspace/qemu/hw/device-assignment.c:203: error: too many
arguments to function âregister_ioport_writeâ
make[2]: *** [device-assignment.o] Error 1
make[2]: Leaving directory `/home/amit/src/kvm-userspace/qemu/x86_64-softmmu'
make[1]: *** [subdir-x86_64-softmmu] Error 2
make[1]: Leaving directory `/home/amit/src/kvm-userspace/qemu'
make: *** [qemu] Error 2
next prev parent reply other threads:[~2008-09-24 4:29 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-09-23 14:54 [V6] Userspace patches for PCI device assignment Amit Shah
2008-09-23 14:54 ` [PATCH 1/7] KVM/userspace: Device Assignment: Add ioctl wrappers needed for assigning devices Amit Shah
2008-09-23 14:54 ` [PATCH 2/7] qemu: Introduce pci_map_irq to get irq nr from pin number for a PCI device Amit Shah
2008-09-23 14:54 ` [PATCH 3/7] qemu: piix: Introduce functions to get pin number from irq and vice versa Amit Shah
2008-09-23 14:54 ` [PATCH 4/7] qemu: Include hw.h in qemu/hw/isa.h to fix compile issues Amit Shah
2008-09-23 14:54 ` [PATCH 5/7] KVM/userspace: Device Assignment: Support for assigning PCI devices to guests Amit Shah
2008-09-23 14:54 ` [PATCH 6/7] KVM/userspace: Build vtd.c for Intel IOMMU support Amit Shah
2008-09-23 14:54 ` [PATCH 7/7] KVM/userspace: Device Assignment: Support for hot plugging PCI devices Amit Shah
2008-09-23 16:32 ` Anthony Liguori
2008-09-24 4:24 ` Amit Shah
2008-09-23 16:31 ` [PATCH 6/7] KVM/userspace: Build vtd.c for Intel IOMMU support Anthony Liguori
2008-09-24 4:25 ` Amit Shah
2008-09-24 15:08 ` Anthony Liguori
2008-09-23 16:30 ` [PATCH 5/7] KVM/userspace: Device Assignment: Support for assigning PCI devices to guests Anthony Liguori
2008-09-23 18:32 ` Muli Ben-Yehuda
2008-09-23 19:18 ` Anthony Liguori
2008-09-23 19:43 ` Muli Ben-Yehuda
2008-09-23 19:58 ` Anthony Liguori
2008-09-24 4:58 ` Amit Shah
2008-09-24 15:07 ` Anthony Liguori
2008-09-24 17:10 ` Amit Shah
2008-09-25 4:54 ` Yang, Sheng
2008-09-25 5:20 ` Yang, Sheng
2008-09-26 1:34 ` Yang, Sheng
2008-09-23 16:13 ` [PATCH 4/7] qemu: Include hw.h in qemu/hw/isa.h to fix compile issues Anthony Liguori
2008-09-24 4:27 ` Amit Shah [this message]
2008-09-24 11:35 ` Amit Shah
2008-09-24 14:59 ` Anthony Liguori
2008-09-23 16:13 ` [PATCH 3/7] qemu: piix: Introduce functions to get pin number from irq and vice versa Anthony Liguori
2008-09-24 4:28 ` Amit Shah
2008-09-23 16:12 ` [PATCH 2/7] qemu: Introduce pci_map_irq to get irq nr from pin number for a PCI device Anthony Liguori
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200809240957.39770.amit.shah@redhat.com \
--to=amit.shah@redhat.com \
--cc=allen.m.kay@intel.com \
--cc=anthony@codemonkey.ws \
--cc=avi@redhat.com \
--cc=benami@il.ibm.com \
--cc=kvm@vger.kernel.org \
--cc=muli@il.ibm.com \
--cc=weidong.han@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.