[porting-issues] [Issue 53682] sal / nfs locking bug ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=53682 --- Additional comments from [EMAIL PROTECTED] Wed Sep 14 04:48:08 -0700 2005 --- @mmeeks: The mention of recent Linux kernels was a red herring that got me confused. The problem is not related to the vintage of the kernel, but rather to a broken system setup (which may well be a default system setup of some distro or other). Issue 54586 is all about addressing this problem in a user-friendly way. Again: the patch you supplied IMO does not address the problem appropriately, as it silently alters critical behaviour (when ENOLCK is returned because some system limit has been reached, and a file would be opened unlocked, but the user would expect it to be locked, which can lead to data loss if two people edit the same file at the same time). If you have further concerns, please continue on issue 54586. - 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 53682] sal / nfs locking bug ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=53682 --- Additional comments from [EMAIL PROTECTED] Wed Sep 14 03:21:04 -0700 2005 --- Stefan - you're thinking 'correctness' - I'm thinking "user experience". What the user wants is to load their document :-) Wrt. ENOLCK - AFAICS this is a -completely- valid thing to return if the file can in fact not be locked. It's prolly a bug that this wasn't returned previously - and we just got a different error code that the higher-level code ignored in a more sensible way ;-) I'd put good money on that. The scenario is simple to reproduce, install OpenSUSE 10.0 - or any recent Linux distro (I believe RH have the same problem) - and try to open a file on an NFS share [ by default it seems you can't reliably lock files via NFS ]. I'm sure there are prolly other file systems that you can't lock with - disconnected / synching / stacked pieces / etc. At the end of the day - "I can't open my document" is pretty painful ;-) potentially of course, we need to teach the upper level code about this and then get it to ignore that error in the same way [ or - to vandalise the user experience we could throw up a dialog saying "this document cannot be locked - can I annoy you again ? " ;-> ... up to you ;-] propagating the error up reliably & handling it in every case in this way would of course be a more elegant solution I guess. - 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 53682] sal / nfs locking bug ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=53682 User sb changed the following: What|Old value |New value Status|RESOLVED |CLOSED --- Additional comments from [EMAIL PROTECTED] Wed Sep 14 02:40:34 -0700 2005 --- . - 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 53682] sal / nfs locking bug ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=53682 User sb changed the following: What|Old value |New value Status|NEW |RESOLVED Resolution| |DUPLICATE --- Additional comments from [EMAIL PROTECTED] Wed Sep 14 02:36:23 -0700 2005 --- I presume that the experienced problems of ENOLCK described in this issue are due to mis-configured Linux machines where no NFS lock daemon is running. *** This issue has been marked as a duplicate of 54586 *** - 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 53682] sal / nfs locking bug ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=53682 --- Additional comments from [EMAIL PROTECTED] Mon Sep 12 08:25:59 -0700 2005 --- But you don't know *why* it commonly happens, right? My dilemma is as follows: Without further information under which conditions exactly this happens, it sounds to me like a bug in the kernel (or at least a deviation from Posix), and the patch looks like a workaround for that bug. However, on other platforms, where ENOLCK indicates the (unlikely) event of resource exhaustion, instead of "locking not supported" (which your comments imply that it does mean on recent Linux kernels), the workaround makes the code worse (silently ignoring the request to lock a file), not better. Hence, my reluctance to include it as-is. - 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 53682] sal / nfs locking bug ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=53682 --- Additional comments from [EMAIL PROTECTED] Mon Sep 12 07:27:46 -0700 2005 --- So - 1. yes you assume right - I guess we need some more flow furkling there to not call the fcntl in that case ' I guess an 'else if ( -1 == ...) instead of an 'if' in that chain would do it. Luckily the problem is not that important for SL10.0 where it shipped ;-) Wrt. 2 - I assume you're happy that this happens very commonly on more recent kernels ? ;-) clearly we need to handle that error case. - 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 53682] sal / nfs locking bug ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=53682 --- Additional comments from [EMAIL PROTECTED] Mon Sep 12 04:39:02 -0700 2005 --- erm, that should instead read "1 Your patch, unlike the original code, calls fcntl even when aflock.l_type!=F_WRLCK. I assume that is not intended." - 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 53682] sal / nfs locking bug ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=53682 --- Additional comments from [EMAIL PROTECTED] Mon Sep 12 04:31:11 -0700 2005 --- @mmeeks: Two issues: 1 Your patch, unlike the original code, calls fcntl even when aflock.l_type==F_WRLCK. I assume that is not intended. 2 According to SUSV3, fcntl/F_SETLK may return ENOLCK when "satisfying the lock or unlock request would result in the number of locked regions in the system exceeding a system-imposed limit", and what the comment in your patch considers "locking not supported" is, again according to SUSV3, indicated by EINVAL instead of ENOLCK. Following SUSV3, I would thus assume that ENOLCK should cause osl_openFile to fail (and that instead EINVAL might be handled in a special way, ignoring the lock request). Under what conditions do you experience ENOLCK on SuSE 10.0? Always? - 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 53682] sal / nfs locking bug ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=53682 User mh changed the following: What|Old value |New value Assigned to|mh|sb Target milestone|--- |OOo 2.0.1 --- Additional comments from [EMAIL PROTECTED] Thu Sep 8 00:15:18 -0700 2005 --- mh->sb: please review. - 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 53682] sal / nfs locking bug ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=53682 --- Additional comments from [EMAIL PROTECTED] Tue Aug 23 11:21:33 -0700 2005 --- *** Issue 36634 has been marked as a duplicate of this issue. *** - 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 53682] sal / nfs locking bug ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=53682 --- Additional comments from [EMAIL PROTECTED] Tue Aug 23 09:38:41 -0700 2005 --- For the uninitiated this means that with OO.o on SUSE 10.0 - when you try to open a file on an NFS mount you get a dialog saying: "General input output error while accessing: ..." - 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 53682] sal / nfs locking bug ...
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=53682 User mmeeks changed the following: What|Old value |New value Attachment is patch| |Created an attachment | |(id=28992) patch --- Additional comments from [EMAIL PROTECTED] Tue Aug 23 09:36:42 -0700 2005 --- Created an attachment (id=28992) 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]