WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] Fix ioemu compile on Solaris

To: xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: Re: [Xen-devel] [PATCH] Fix ioemu compile on Solaris
From: Christoph Egger <Christoph.Egger@xxxxxxx>
Date: Mon, 25 Feb 2008 09:00:39 +0100
Cc: john.levon@xxxxxxx
Delivery-date: Mon, 25 Feb 2008 00:01:54 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <3a0e7fc8b4b7c650c937.1203685500@hatchback>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <3a0e7fc8b4b7c650c937.1203685500@hatchback>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: KMail/1.9.7
On Friday 22 February 2008 14:05:00 john.levon@xxxxxxx wrote:
> # HG changeset patch
> # User xen-discuss@xxxxxxxxxxxxxxx
> # Date 1203680243 28800
> # Node ID 3a0e7fc8b4b7c650c9377c7b2a197c127913a622
> # Parent  0accc274f62715d356a591d2286027f7591a9412
> Fix ioemu compile on Solaris
>
> _POSIX_C_SOURCE does not guarantee the available of the 1003.1j option
> which contains posix_memalign(). Fix the test to explicitly test for
> Linux.
>
> Signed-off-by: John Levon <john.levon@xxxxxxx>
>
> diff --git a/tools/ioemu/osdep.c b/tools/ioemu/osdep.c
> --- a/tools/ioemu/osdep.c
> +++ b/tools/ioemu/osdep.c
> @@ -178,7 +178,7 @@ void kqemu_vfree(void *ptr)
>
>  void *qemu_memalign(size_t alignment, size_t size)
>  {
> -#if defined(_POSIX_C_SOURCE)
> +#if defined(__Linux__)

That's wrong. NetBSD also has posix_memalign().
Maybe this:

#ifndef __sun__

>      int ret;
>      void *ptr;
>      ret = posix_memalign(&ptr, alignment, size);
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel



-- 
AMD Saxony, Dresden, Germany
Operating System Research Center

Legal Information:
AMD Saxony Limited Liability Company & Co. KG
Sitz (Geschäftsanschrift):
   Wilschdorfer Landstr. 101, 01109 Dresden, Deutschland
Registergericht Dresden: HRA 4896
vertretungsberechtigter Komplementär:
   AMD Saxony LLC (Sitz Wilmington, Delaware, USA)
Geschäftsführer der AMD Saxony LLC:
   Dr. Hans-R. Deppe, Thomas McCoy


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

<Prev in Thread] Current Thread [Next in Thread>