mguegan@virtua.ch wrote:

Full_Name: Matthieu Guegan Version: current OS: SmartOS URL: Submission from: (NULL) (109.190.148.77)

Hi,

In order to compile knot[1] on SmartOS, I have done a series of little patches. One of them is linked to the LMDB project, which redirects me here.

The idea is to tell SmartOS (and I think the SunOS family) to make use of posix_madvise(3), instead of madvise(3) as implementation on this OS[2] is different than, for example, the Linux[3] one.

So, I kindly ask you to take a look at the merge request of the knot project[4], in particular the `src/contrib/lmdb/mdb.c` patch (that's just a ifdef modification) : would it be possible to apply this patch directly in this repository ?

I see no functional difference between your references [2] and [3]. What is the benefit of this patch?

Thanks

[1] https://www.knot-dns.cz/ [2] https://docs.oracle.com/cd/E36784_01/html/E36874/madvise-3c.html [3] http://man7.org/linux/man-pages/man2/madvise.2.html [4] https://gitlab.labs.nic.cz/knot/knot-dns/merge_requests/979/diffs?commit_id=...