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

[$env open] -fidexmap default #4

Closed
gahr opened this issue May 6, 2016 · 5 comments
Closed

[$env open] -fidexmap default #4

gahr opened this issue May 6, 2016 · 5 comments

Comments

@gahr
Copy link
Contributor

gahr commented May 6, 2016

Hi Danilo,

I see that the -fixedmap param to env open defaults to true.
https://github.com/ray2501/tcl-lmdb/blob/master/generic/tclmdb.c#L1363
This is very risky as mmap(2) doesn't guarantee that the address hint is respected.
As you only open the env once - when doing $env open - and copy data in/out via Tcl_Objs, I wonder why you chose to opt for having this default to true.

If there's no compelling reason, I think it would be better to change the default to false.

Thanks!

@ray2501
Copy link
Owner

ray2501 commented May 6, 2016

I just follow the LMDB test code and not think about this.
I will fix this.

@gahr
Copy link
Contributor Author

gahr commented May 6, 2016

I have to double check, but I think this might be the cause for the testsuite failures I see when building in debug mode (--enable-symbols passed to configure). Can you reproduce?

@ray2501
Copy link
Owner

ray2501 commented May 6, 2016

I can't reproduce. It is still pass on my machine (at openSUSE).

@gahr
Copy link
Contributor Author

gahr commented May 6, 2016

I'll have a closer look at it and let you know!

@gahr
Copy link
Contributor Author

gahr commented May 9, 2016

Turns out this wasn't the reason for the failure I was seeing. Investigating that in #5 .

@gahr gahr closed this as completed May 9, 2016
uqs pushed a commit to freebsd/freebsd-ports that referenced this issue May 9, 2016
… to false

See the discussion at ray2501/tcl-lmdb#4 and the
commit at ray2501/tcl-lmdb@a6fb287.


git-svn-id: svn+ssh://svn.freebsd.org/ports/head@414858 35697150-7ecd-e111-bb59-0022644237b5
uqs pushed a commit to freebsd/freebsd-ports that referenced this issue May 9, 2016
svmhdvn pushed a commit to svmhdvn/freebsd-ports that referenced this issue Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants