[porting-issues] [Issue 89730] osl_getAbsoluteFileURL() l stat()s too much
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=89730 User mba changed the following: What|Old value |New value CC|'aziem,jl,kso,mmeeks,sb,th|'aziem,jl,kso,mba,mmeeks,s |b'|b,thb' Target milestone|OOo 3.1 |OOo 3.x --- Additional comments from m...@openoffice.org Thu Apr 16 11:27:09 + 2009 --- Kendy, in case you are still working on this: Mikhail (mav) is now maintaining the osl_file stuff. As 3.1 obviously is an unrealistic target I changed it to 3.x. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: issues-unsubscr...@porting.openoffice.org For additional commands, e-mail: issues-h...@porting.openoffice.org - To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org For additional commands, e-mail: allbugs-h...@openoffice.org
[porting-issues] [Issue 89730] osl_getAbsoluteFileURL() l stat()s too much
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=89730 User kendy changed the following: What|Old value |New value CC|'aziem,jl,kso,mmeeks,sb' |'aziem,jl,kso,mmeeks,sb,th | |b' Assigned to|hro |kendy Status|STARTED |NEW --- Additional comments from [EMAIL PROTECTED] Fri Sep 26 13:41:38 + 2008 --- Back to myself, apparently there are problems with this on MacOS X. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[porting-issues] [Issue 89730] osl_getAbsoluteFileURL() l stat()s too much
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=89730 --- Additional comments from [EMAIL PROTECTED] Fri Aug 15 13:47:25 + 2008 --- Good approach but there were ancious reason that it also worked with nonexistant paths. will review it for scheduled release. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[porting-issues] [Issue 89730] osl_getAbsoluteFileURL() l stat()s too much
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=89730 User aziem changed the following: What|Old value |New value CC|'jl,kso,mmeeks,sb'|'aziem,jl,kso,mmeeks,sb' - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[porting-issues] [Issue 89730] osl_getAbsoluteFileURL() l stat()s too much
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=89730 User kso changed the following: What|Old value |New value Target milestone|OOo 3.x |OOo 3.1 --- Additional comments from [EMAIL PROTECTED] Tue Jun 3 08:17:29 + 2008 --- hro: We should integrate this patch in 3.1 - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[porting-issues] [Issue 89730] osl_getAbsoluteFileURL() l stat()s too much
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=89730 User hro changed the following: What|Old value |New value Status|NEW |STARTED Target milestone|OOo 3.0 |OOo 3.x --- Additional comments from [EMAIL PROTECTED] Fri May 30 12:55:54 + 2008 --- Retargeted. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[porting-issues] [Issue 89730] osl_getAbsoluteFileURL() l stat()s too much
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=89730 User kso changed the following: What|Old value |New value CC|'jl,mmeeks,sb'|'jl,kso,mmeeks,sb' --- Additional comments from [EMAIL PROTECTED] Fri May 23 12:02:50 + 2008 --- . - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[porting-issues] [Issue 89730] osl_getAbsoluteFileURL() l stat()s too much
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=89730 User kso changed the following: What|Old value |New value CC|'mmeeks,sb' |'jl,mmeeks,sb' Assigned to|mh|hro Target milestone|--- |OOo 3.0 --- Additional comments from [EMAIL PROTECTED] Fri May 23 12:02:27 + 2008 --- hro: Please take over. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[porting-issues] [Issue 89730] osl_getAbsoluteFileURL() l stat()s too much
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=89730 User kendy changed the following: What|Old value |New value Attachment is patch| |Created an attachment (id= | |53820) The patch. --- Additional comments from [EMAIL PROTECTED] Wed May 21 14:13:26 + 2008 --- Created an attachment (id=53820) The patch. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[porting-issues] [Issue 89730] osl_getAbsoluteFileURL() l stat()s too much
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=89730
Issue #|89730
Summary|osl_getAbsoluteFileURL() lstat()s too much
Component|porting
Version|DEV300m12
Platform|All
URL|
OS/Version|All
Status|NEW
Status whiteboard|
Keywords|
Resolution|
Issue type|PATCH
Priority|P3
Subcomponent|code
Assigned to|mh
Reported by|kendy
--- Additional comments from [EMAIL PROTECTED] Wed May 21 13:58:48 +
2008 ---
The implementation of osl_getAbsoluteFileURL_impl_() is done so that it
lstat()s O(n^2), instead of n paths (where n is depth of the path). It is
because it calls realpath() on each of the components of the path, and
realpath() itself does lstat() on each of the components itself. I'll attach
a patch that changes it to lstat()ing n paths only - on the price of _not_
absolutizing non-existing paths. To me, it seems like a good deal ;-) - if
the path does not exist, who cares what part of it is absolute? In the case
it was created later, nobody can expect what part of that path were symlinks
etc. anyway, and it should go through the full realpath() process anyway.
Here's a part of strace log showing the wrong behavior:
10406 19:41:49.765833 lstat("/local", {st_mode=S_IFDIR|0755,
st_size=4096, ...}) = 0
10406 19:41:49.765887 lstat("/local", {st_mode=S_IFDIR|0755,
st_size=4096, ...}) = 0
10406 19:41:49.765936 lstat("/local/inst", {st_mode=S_IFDIR|0755,
st_size=4096, ...}) = 0
10406 19:41:49.765993 lstat("/local", {st_mode=S_IFDIR|0755,
st_size=4096, ...}) = 0
10406 19:41:49.766042 lstat("/local/inst", {st_mode=S_IFDIR|0755,
st_size=4096, ...}) = 0
10406 19:41:49.766092 lstat("/local/inst/OpenOffice.org3.0-m12",
{st_mode=S_IFDIR|0755, st_size=4096, ...}) = 0
10406 19:41:49.766153 lstat("/local", {st_mode=S_IFDIR|0755,
st_size=4096, ...}) = 0
10406 19:41:49.766202 lstat("/local/inst", {st_mode=S_IFDIR|0755,
st_size=4096, ...}) = 0
10406 19:41:49.766251 lstat("/local/inst/OpenOffice.org3.0-m12",
{st_mode=S_IFDIR|0755, st_size=4096, ...}) = 0
10406 19:41:49.766303
lstat("/local/inst/OpenOffice.org3.0-m12/openoffice.org", {st_mode=S_IFDIR|
0755, st_size=4096, ...}) = 0
10406 19:41:49.766363 lstat("/local", {st_mode=S_IFDIR|0755,
st_size=4096, ...}) = 0
10406 19:41:49.766412 lstat("/local/inst", {st_mode=S_IFDIR|0755,
st_size=4096, ...}) = 0
10406 19:41:49.766461 lstat("/local/inst/OpenOffice.org3.0-m12",
{st_mode=S_IFDIR|0755, st_size=4096, ...}) = 0
10406 19:41:49.766514
lstat("/local/inst/OpenOffice.org3.0-m12/openoffice.org", {st_mode=S_IFDIR|
0755, st_size=4096, ...}) = 0
10406 19:41:49.766568
lstat("/local/inst/OpenOffice.org3.0-m12/openoffice.org/ure",
{st_mode=S_IFDIR|0755, st_size=4096, ...}) = 0
10406 19:41:49.766628 lstat("/local", {st_mode=S_IFDIR|0755,
st_size=4096, ...}) = 0
10406 19:41:49.766677 lstat("/local/inst", {st_mode=S_IFDIR|0755,
st_size=4096, ...}) = 0
10406 19:41:49.766726 lstat("/local/inst/OpenOffice.org3.0-m12",
{st_mode=S_IFDIR|0755, st_size=4096, ...}) = 0
10406 19:41:49.766779
lstat("/local/inst/OpenOffice.org3.0-m12/openoffice.org", {st_mode=S_IFDIR|
0755, st_size=4096, ...}) = 0
10406 19:41:49.766833
lstat("/local/inst/OpenOffice.org3.0-m12/openoffice.org/ure",
{st_mode=S_IFDIR|0755, st_size=4096, ...}) = 0
10406 19:41:49.766887
lstat("/local/inst/OpenOffice.org3.0-m12/openoffice.org/ure/bin",
{st_mode=S_IFDIR|0755, st_size=4096, ...}) = 0
10406 19:41:49.766949 lstat("/local", {st_mode=S_IFDIR|0755,
st_size=4096, ...}) = 0
10406 19:41:49.766997 lstat("/local/inst", {st_mode=S_IFDIR|0755,
st_size=4096, ...}) = 0
10406 19:41:49.767047 lstat("/local/inst/OpenOffice.org3.0-m12",
{st_mode=S_IFDIR|0755, st_size=4096, ...}) = 0
10406 19:41:49.767099
lstat("/local/inst/OpenOffice.org3.0-m12/openoffice.org", {st_mode=S_IFDIR|
0755, st_size=4096, ...}) = 0
10406 19:41:49.767153
lstat("/local/inst/OpenOffice.org3.0-m12/openoffice.org/ure",
{st_mode=S_IFDIR|0755, st_size=4096, ...}) = 0
10406 19:41:49.767208
lstat("/local/inst/OpenOffice.org3.0-m12/openoffice.org/ure/lib",
{st_mode=S_IFDIR|0755, st_size=4096, ...}) = 0
The patch changes it to simple:
10406 19:41:49.766949 lstat("/local", {st_mode=S_IFDIR|0755,
st_size=4096, ...}) = 0
10406 19:41:49.766997 lstat("/local/inst", {st_mode=S_IFDIR|0755,
st_size=4096, ...}) = 0
10406 19:41:49.767047 lstat("/local/inst/OpenOffice.org3.0-m12",
{st_mode=S_IFDIR|0755, st_size=4096, ...}) = 0
10406 19:41:49.767099
lstat("/local/inst/OpenOffice.org3.0-m12/openoffice.org", {st_mode=S_IFDIR|
0755, st_size=4096, ...}) = 0
10406 19:41:49.767153
lstat("/local/inst/OpenOffice.org3.0-m12/openoffice.org/ure",
{st_mode=S_IFDIR|0755, st_size=4096, ...}) = 0
10406 19:41:49.767208
lstat("/local/inst/OpenOf
