Opened 12 years ago
Closed 8 years ago
#20 closed enhancement (invalid)
Add madvise() support for mmap
| Reported by: | Lewis Rosenthal | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | |
| Component: | mmap | Version: | |
| Severity: | highest | Keywords: | mmap madvise logrotate |
| Cc: | Yuri Dario |
Description
We need a robust log rotation app for eCS. I suggested porting logrotate (https://fedorahosted.org/logrotate/) which seems to be something fairly common across several *nix distros.
Paul had a look and advises that our current mmap (20111102, on my system, which seems to be the latest available) does not support madvise(), and is apparently a requirement for building logrotate.
(Yuri, I am cc'ing you here, as I believe the current mmap is one of yours.)
N.B.: mmap is not a selectable component in the dropdown list in this tracker.
Change History (5)
comment:1 by , 12 years ago
comment:2 by , 12 years ago
That being the case, would it be worth adding a stub madvise to your mmap to avoid having to change the source code?
comment:3 by , 10 years ago
| Component: | *none → mmap |
|---|
comment:5 by , 8 years ago
| Resolution: | → invalid |
|---|---|
| Status: | new → closed |
There is https://github.com/bitwiseworks/libcx/issues/11 where further MMAP development goes on. I will give a link to this ticket and close it.
I think madvise() could be completely ignored, since we cannot tell OS/2 kernel how to handle memory access.