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

replace PR_GetFileInfo with PR_GetFileInfo64 #649

Closed
389-ds-bot opened this issue Sep 12, 2020 · 8 comments
Closed

replace PR_GetFileInfo with PR_GetFileInfo64 #649

389-ds-bot opened this issue Sep 12, 2020 · 8 comments
Labels
closed: fixed Migration flag - Issue
Milestone

Comments

@389-ds-bot
Copy link

Cloned from Pagure issue: https://pagure.io/389-ds-base/issue/47312


This is with the latest code, on a RHEL 6.4 64-bit machine.

[02/Apr/2013:17:26:38 -0400] - 389-Directory/1.3.1.pre.a1.git0472b98 B2013.092.2034 starting up
[02/Apr/2013:17:26:38 -0400] - WARNING: two: entry cache size 3897032704B is less than db size 2736742400B; We recommend to increase the entry cache size nsslapd-cachememsize.
[02/Apr/2013:17:26:38 -0400] - WARNING: one: entry cache size 3897032704B is less than db size 2736742400B; We recommend to increase the entry cache size nsslapd-cachememsize.

3897032704B is greater than 2736742400B

@389-ds-bot 389-ds-bot added the closed: fixed Migration flag - Issue label Sep 12, 2020
@389-ds-bot 389-ds-bot added this to the 1.3.1 milestone Sep 12, 2020
@389-ds-bot
Copy link
Author

Comment from rmeggins (@richm) at 2013-04-03 04:01:56

Looks like the problem is here:

size_t
dblayer_get_id2entry_size(ldbm_instance *inst)
...
    PRFileInfo info;
...
    rc = PR_GetFileInfo(id2entry_file, &info);

should use PRFileInfo64 instead.

@389-ds-bot
Copy link
Author

Comment from rmeggins (@richm) at 2013-04-03 04:40:53

There are several places in the code where we should use PR_GetFileInfo64 instead of PR_GetFileInfo. We need to be able to handle these very large files.

@389-ds-bot
Copy link
Author

Comment from rmeggins (@richm) at 2013-04-04 00:51:57

0004-Ticket-47312-replace-PR_GetFileInfo-with-PR_GetFileI.patch
0004-Ticket-47312-replace-PR_GetFileInfo-with-PR_GetFileI.patch

@389-ds-bot
Copy link
Author

Comment from rmeggins (@richm) at 2013-04-04 02:48:33

commit 968c511
Author: Rich Megginson richm@redhat.com
Date: Tue Apr 2 17:50:08 2013 -0600

@389-ds-bot
Copy link
Author

Comment from nkinder (@nkinder) at 2013-04-04 03:03:07

Ticket has been cloned to Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=948054

@389-ds-bot
Copy link
Author

Comment from rmeggins (@richm) at 2013-04-04 04:22:37

0001-Ticket-47312-replace-PR_GetFileInfo-with-PR_GetFileI.2.patch
0001-Ticket-47312-replace-PR_GetFileInfo-with-PR_GetFileI.2.patch

@389-ds-bot
Copy link
Author

Comment from rmeggins (@richm) at 2013-04-04 04:54:49

commit 9d5dedd
Author: Rich Megginson richm@redhat.com
Date: Wed Apr 3 16:19:30 2013 -0600

@389-ds-bot
Copy link
Author

Comment from rmeggins (@richm) at 2017-02-11 23:13:09

Metadata Update from @richm:

  • Issue assigned to richm
  • Issue set to the milestone: 1.3.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed: fixed Migration flag - Issue
Projects
None yet
Development

No branches or pull requests

1 participant