* [Qemu-devel] [PATCH] Added missing sigbus_reraise() to non-Linux platforms
@ 2011-03-13 20:16 Palle Lyckegaard
2011-03-14 10:03 ` Stefan Hajnoczi
0 siblings, 1 reply; 3+ messages in thread
From: Palle Lyckegaard @ 2011-03-13 20:16 UTC (permalink / raw)
To: qemu-devel
Hi,
File cpus.c seems to be missing sigbus_reraise() on non-Linux platforms.
The following patch fixes building qemu on Solaris 11 Express (SPARC).
Regards
Palle
Signed-off-by: Palle Lyckegaard <palle@lyckegaard.dk>
---
cpus.c | 5 +++++
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/cpus.c b/cpus.c
index 077729c..02ba1be 100644
--- a/cpus.c
+++ b/cpus.c
@@ -250,6 +250,11 @@ static void qemu_init_sigbus(void)
static void qemu_init_sigbus(void)
{
}
+
+static void sigbus_reraise(void)
+{
+}
+
#endif /* !CONFIG_LINUX */
#ifndef _WIN32
--
1.5.6.5
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH] Added missing sigbus_reraise() to non-Linux platforms
2011-03-13 20:16 [Qemu-devel] [PATCH] Added missing sigbus_reraise() to non-Linux platforms Palle Lyckegaard
@ 2011-03-14 10:03 ` Stefan Hajnoczi
2011-03-14 10:44 ` [Qemu-devel] " Jan Kiszka
0 siblings, 1 reply; 3+ messages in thread
From: Stefan Hajnoczi @ 2011-03-14 10:03 UTC (permalink / raw)
To: Palle Lyckegaard; +Cc: qemu-devel
On Sun, Mar 13, 2011 at 8:16 PM, Palle Lyckegaard <palle@lyckegaard.dk> wrote:
> Hi,
>
> File cpus.c seems to be missing sigbus_reraise() on non-Linux platforms. The
> following patch fixes building qemu on Solaris 11 Express (SPARC).
>
> Regards
> Palle
>
>
>
>
> Signed-off-by: Palle Lyckegaard <palle@lyckegaard.dk>
> ---
> cpus.c | 5 +++++
> 1 files changed, 5 insertions(+), 0 deletions(-)
Great to see your SPARC Solaris 11 buildslave already catching build
breakages. Thanks for the patch!
Reviewed-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Qemu-devel] Re: [PATCH] Added missing sigbus_reraise() to non-Linux platforms
2011-03-14 10:03 ` Stefan Hajnoczi
@ 2011-03-14 10:44 ` Jan Kiszka
0 siblings, 0 replies; 3+ messages in thread
From: Jan Kiszka @ 2011-03-14 10:44 UTC (permalink / raw)
To: Stefan Hajnoczi; +Cc: Palle Lyckegaard, qemu-devel
[-- Attachment #1: Type: text/plain, Size: 860 bytes --]
On 2011-03-14 11:03, Stefan Hajnoczi wrote:
> On Sun, Mar 13, 2011 at 8:16 PM, Palle Lyckegaard <palle@lyckegaard.dk> wrote:
>> Hi,
>>
>> File cpus.c seems to be missing sigbus_reraise() on non-Linux platforms. The
>> following patch fixes building qemu on Solaris 11 Express (SPARC).
>>
>> Regards
>> Palle
>>
>>
>>
>>
>> Signed-off-by: Palle Lyckegaard <palle@lyckegaard.dk>
>> ---
>> cpus.c | 5 +++++
>> 1 files changed, 5 insertions(+), 0 deletions(-)
>
> Great to see your SPARC Solaris 11 buildslave already catching build
> breakages. Thanks for the patch!
>
> Reviewed-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
There is already an (implicit) fix [1] on the way. Will come via
uq/master unless someone applies it earlier directly.
Jan
[1] http://permalink.gmane.org/gmane.comp.emulators.kvm.devel/68818
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 259 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-03-14 10:44 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-03-13 20:16 [Qemu-devel] [PATCH] Added missing sigbus_reraise() to non-Linux platforms Palle Lyckegaard
2011-03-14 10:03 ` Stefan Hajnoczi
2011-03-14 10:44 ` [Qemu-devel] " Jan Kiszka
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).