All of lore.kernel.org
 help / color / mirror / Atom feed
From: Avi Kivity <avi@redhat.com>
To: Alexander Graf <agraf@suse.de>
Cc: Daniel Gollub <gollub@b1-systems.de>,
	KVM list <kvm@vger.kernel.org>,
	Christian Berendt <berendt@b1-systems.de>
Subject: Re: buildbot for kvm.git
Date: Thu, 25 Nov 2010 11:17:33 +0200	[thread overview]
Message-ID: <4CEE29AD.8010107@redhat.com> (raw)
In-Reply-To: <559BE793-1B28-4133-BE44-C26E8E16DDA1@suse.de>

On 11/25/2010 03:20 AM, Alexander Graf wrote:
> On 24.11.2010, at 22:03, Daniel Gollub wrote:
>
> >  On Wednesday, November 24, 2010 11:23:37 pm Avi Kivity wrote:
> >>>  powerpc not ready yet - trivial $ARCH change required in the buildbot
> >>>  master configuration.
> >
> >  powerpc builders are now also setup and already spot something.
> >
> >  Build issue with ppc44x:
> >  http://buildbot.b1-systems.de/kvm/builders/ppc44x/builds/2/steps/compile/logs/stdio
> >
> >  Build issue with ppc64:
> >  http://buildbot.b1-systems.de/kvm/builders/ppc64/builds/4/steps/compile/logs/stdio
>
> Same bug as the one of the S390 ones.
>
> Avi, my (whitespace broken) patch for this is:
>
> diff --git a/include/linux/kvm_host.h b/include/linux/kvm_host.h
> index f17beae..da0794f 100644
> --- a/include/linux/kvm_host.h
> +++ b/include/linux/kvm_host.h
> @@ -673,11 +673,13 @@ static inline int kvm_irqfd(struct kvm *kvm, int fd, int gsi, int flags)
>
>   static inline void kvm_irqfd_release(struct kvm *kvm) {}
>
> +#ifdef CONFIG_HAVE_KVM_IRQCHIP
>   static inline void kvm_irq_routing_update(struct kvm *kvm,
>                                            struct kvm_irq_routing_table *irq_rt)
>   {
>          rcu_assign_pointer(kvm->irq_routing, irq_rt);
>   }
> +#endif
>
>   static inline int kvm_ioeventfd(struct kvm *kvm, struct kvm_ioeventfd *args)
>   {
>
>
> Have you fixed this locally in your tree already or do you want to pull from me?

I don't have this locally.

-- 
error compiling committee.c: too many arguments to function


  reply	other threads:[~2010-11-25  9:17 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-10 13:31 buildbot for kvm.git Avi Kivity
2010-11-10 13:33 ` Avi Kivity
2010-11-10 15:02 ` Alexander Graf
2010-11-10 15:03   ` Avi Kivity
2010-11-10 15:40     ` Alexander Graf
2010-11-11  9:22 ` Daniel Gollub
2010-11-22  9:37   ` Avi Kivity
2010-11-23  0:11     ` Daniel Gollub
2010-11-23 10:22       ` Avi Kivity
2010-11-24 20:51         ` Daniel Gollub
2010-11-24 10:15 ` Daniel Gollub
2010-11-24 10:23   ` Avi Kivity
2010-11-24 20:48     ` Daniel Gollub
2010-11-25  9:22       ` Avi Kivity
2010-11-29 11:36         ` Daniel Gollub
2010-11-29 12:20           ` Jan Kiszka
2010-11-29 19:22             ` Daniel Gollub
2010-11-30  9:12               ` Jan Kiszka
2010-11-24 21:03     ` Daniel Gollub
2010-11-25  1:20       ` Alexander Graf
2010-11-25  9:17         ` Avi Kivity [this message]
2010-11-30 10:47 ` buildbot for kvm.git is now live (was: Re: buildbot for kvm.git) Daniel Gollub

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=4CEE29AD.8010107@redhat.com \
    --to=avi@redhat.com \
    --cc=agraf@suse.de \
    --cc=berendt@b1-systems.de \
    --cc=gollub@b1-systems.de \
    --cc=kvm@vger.kernel.org \
    /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.