This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] gold: add an option to mmap whole files (defaults to true on 64-bit linkers)

From: Paul Brook <paul at codesourcery dot com>
To: binutils at sourceware dot org
Cc: Ian Lance Taylor <iant at google dot com>, Mikolaj Zalewski <mikolajz at google dot com>
Date: Wed, 28 Oct 2009 16:30:01 +0000
Subject: Re: [PATCH] gold: add an option to mmap whole files (defaults to true on 64-bit linkers)
References: <7ebec9e70910211756t4d0672d3waf8a271d18dab3b9@mail.gmail.com> <mcr7hugibx7.fsf@dhcp-172-17-9-151.mtv.corp.google.com>

On Wednesday 28 October 2009, Ian Lance Taylor wrote:
> Mikolaj Zalewski <mikolajz@google.com> writes:
> >   I have created a patch adding an option to mmap whole file in
> > File_read::open and making it default on 64-bit linker builds. This
> > seems to speed up large builds. The changes were very similar to the
> > already existing support to open pseudo-files (for tests) with
> > contents passed as an argument, so I've merged both features.
> 
> Thanks for pursuing this.

You can't assume that 64-bit machines have insane amounts of virtual memory 
available. At minimum you need to check ulimit -v.

Paul

Follow-Ups:
- Re: [PATCH] gold: add an option to mmap whole files (defaults to true on 64-bit linkers)
  - From: Ian Lance Taylor

References:
- [PATCH] gold: add an option to mmap whole files (defaults to true on 64-bit linkers)
  - From: Mikolaj Zalewski
- Re: [PATCH] gold: add an option to mmap whole files (defaults to true on 64-bit linkers)
  - From: Ian Lance Taylor

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]