Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Memory-mapping of bigarrays with offsets missing #4024

Closed
vicuna opened this Issue May 16, 2006 · 3 comments

Comments

Projects
None yet
2 participants
@vicuna
Copy link
Collaborator

commented May 16, 2006

Original bug ID: 4024
Reporter: @mmottl
Assigned to: @xavierleroy
Status: closed (set by @xavierleroy on 2007-02-21T15:19:57Z)
Resolution: fixed
Priority: normal
Severity: feature
Version: 3.09.2
Fixed in version: 3.10+dev
Category: ~DO NOT USE (was: OCaml general)
Child of: #3790
Monitored by: @mmottl

Bug description

It is currently impossible to memory-map file portions with a non-zero offset (64 bits would be required, too!) with the Bigarray module. This is quite unfortunate, because very large files that exceed the integer limits of OCaml on 32bit platforms are not fully accessible in a memory-mapped way then.

The current seek position of the file descriptor could eventually be consulted for that in the current code without changing the API. This would be a fairly trivial change.

@vicuna

This comment has been minimized.

Copy link
Collaborator Author

commented May 16, 2006

Comment author: @mmottl

The mmap64 system call would have to be used if 64bit offsets should be supported.

@vicuna

This comment has been minimized.

Copy link
Collaborator Author

commented May 19, 2006

Comment author: berke

I second this. However Array*.sub can be used for small files.

@vicuna

This comment has been minimized.

Copy link
Collaborator Author

commented Jun 10, 2006

Comment author: @xavierleroy

Tentative implementation in CVS trunk (Bigarray.*.map_subfile functions).
Windows implementation is untested and probably wrong.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.