From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Date: Thu, 07 May 2009 11:00:41 +0000 Subject: Re: [patch] fix qemu-kvm to build when gdbstub is disabled Message-Id: <4A02BF59.6030103@redhat.com> List-Id: References: <4A01AFE8.3020008@sgi.com> In-Reply-To: <4A01AFE8.3020008@sgi.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: kvm-ia64@vger.kernel.org Jan Kiszka wrote: >> In upstream. Please fix it in upstream and qemu-kvm.git will get the >> fix from there. >> > > Given that CONFIG_GDBSTUB is always true upstream, I tend to say: > Finally drop it upstream and keep this (or a more arch-local) workaround > for missing ia64 gdbstub support downstream. > I agree, unless - we want to make gdbstub support configurable (don't see any overwhelming reason for this, but maybe others do) - we want to merge ia64 kvm support upstream, and don't want to impose gdbstub support (though I'd recommend properly implementing gdbstub) In any case, I'm okay with dropping the check upstream and applying the local fixup. -- error compiling committee.c: too many arguments to function