Re: [Qemu-devel] [5979] Fix compiling without MREMAP_FIXED

[Kirill A. Shutemov]

On Fri, Dec 12, 2008 at 06:27:23PM +0200, Blue Swirl wrote:
> On 12/11/08, Kirill A. Shutemov <address@hidden> wrote:
> > On Thu, Dec 11, 2008 at 07:12:26PM +0000, Blue Swirl wrote:
> >  > Revision: 5979
> >  >           http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=5979
> >  > Author:   blueswir1
> >  > Date:     2008-12-11 19:12:25 +0000 (Thu, 11 Dec 2008)
> >  >
> >  > Log Message:
> >  > -----------
> >  > Fix compiling without MREMAP_FIXED
> >  >
> >  > Modified Paths:
> >  > --------------
> >  >     trunk/linux-user/mmap.c
> >  >
> >  > Modified: trunk/linux-user/mmap.c
> >  > ===================================================================
> >  > --- trunk/linux-user/mmap.c   2008-12-11 17:30:50 UTC (rev 5978)
> >  > +++ trunk/linux-user/mmap.c   2008-12-11 19:12:25 UTC (rev 5979)
> >  > @@ -546,6 +546,7 @@
> >  >
> >  >      mmap_lock();
> >  >
> >  > +#if defined(MREMAP_FIXED)
> >  >      if (flags & MREMAP_FIXED)
> >  >          host_addr = mremap(g2h(old_addr), old_size, new_size,
> >  >                             flags, new_addr);
> >  > @@ -560,7 +561,9 @@
> >  >          } else
> >  >              host_addr = mremap(g2h(old_addr), old_size, new_size,
> >  >                                 flags | MREMAP_FIXED, g2h(mmap_start));
> >  > -    } else {
> >  > +    } else
> >  > +#endif
> >  > +    {
> >  >          host_addr = mremap(g2h(old_addr), old_size, new_size, flags);
> >  >          /* Check if address fits target address space */
> >  >          if ((unsigned long)host_addr + new_size > (abi_ulong)-1) {
> >
> >
> > I've posted patch that fix building for glibc < 2.4. Please apply my
> >  patch instead of it.
> 
> Your fix depends on linux headers, is that safe?

Why not? glibc also depends on it.

-- 
Regards,  Kirill A. Shutemov
 + Belarus, Minsk
 + ALT Linux Team, http://www.altlinux.org/

signature.asc
Description: Digital signature