[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Large file support with apr 1.2.8, svn 1.4.3, and linux?

From: Chris Frost <chris_at_frostnet.net>
Date: 2007-03-29 08:34:36 CEST

On Thu, Mar 29, 2007 at 12:35:04AM -0500, Peter Samuelson wrote:
>
> [Chris Frost]
> > svn: Commit failed (details follow):
> > svn: Can't write to file '/home/frost/repo_photos/db/transactions/9-1.txn/rev': File too large
>
> That's odd.
>
> > Both apr-util and subversion's config.logs shows they knows to define
> > _LARGEFILE64_SOURCE. subversion's SWIG includes set _FILE_OFFSET_BITS=64,
> > but no one else seems to do so, at least in their configure log.
>
> That's normal: apr doesn't need _FILE_OFFSET_BITS=64 or
> _LARGEFILE_SOURCE, it uses the facilities of _LARGEFILE64_SOURCE
> instead.
>
> Could you run the svn command using strace, and note particularly the
> open() and write() calls involving files in the transactions directory?
> apr _should_ be calling open with the O_LARGEFILE flag, but who knows.

This is interesting: wc opens use O_LARGEFILE on fds that will read
or write but the ra open of the rev file does not use O_LARGEFILE.

$ grep 'open.*rev"' ci.strace
open("/home/frost/repo_test/db/transactions/0-1.txn/rev", O_WRONLY|O_CREAT|O_EXCL, 0666) = 3
open("/home/frost/repo_test/db/transactions/0-1.txn/rev", O_WRONLY) = 4
open("/home/frost/repo_test/db/transactions/0-1.txn/rev", O_WRONLY) = 5

$ grep 'O_LARGEFILE' ci.strace
open("/usr/lib/locale/locale-archive", O_RDONLY|O_LARGEFILE) = 3
open("/home/frost/co_test/.svn/props/zeros.svn-work", O_RDONLY|O_LARGEFILE) = 3
open("/home/frost/co_test/.svn/tmp/props/zeros.svn-work.tmp", O_WRONLY|O_CREAT|O_TRUNC|O_LARGEFILE, 0666) = 4
open("/home/frost/co_test/zeros", O_RDONLY|O_LARGEFILE) = 3
open("/home/frost/co_test/.svn/tmp/text-base/zeros.svn-base.tmp", O_WRONLY|O_CREAT|O_TRUNC|O_LARGEFILE, 0666) = 4
open("/home/frost/repo_test/db/transactions/0-1.txn", O_RDONLY|O_NONBLOCK|O_LARGEFILE|O_DIRECTORY) = 4

An strace for a 10MB file add checkin to a local repository is attached.

-- 
Chris Frost  |  <http://www.frostnet.net/chris/>
-------------+----------------------------------------
PGP: <http://www.frostnet.net/chris/about/pgp_key.txt>


---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@subversion.tigris.org
For additional commands, e-mail: users-help@subversion.tigris.org

Received on Thu Mar 29 08:35:05 2007

This is an archived mail posted to the Subversion Users mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.