Commit message (Collapse) | Author | Files | Lines | ||
---|---|---|---|---|---|
2014-12-15 | lib/nfs/Connection: unregister socket with SocketMonitor::Steal() | Max Kellermann | 1 | -1/+1 | |
SocketMonitor::Cancel() does not actually unregister the socket; it only disables the event. | |||||
2014-12-15 | lib/nfs/Connection: implement mount timeout | Max Kellermann | 1 | -0/+24 | |
2014-12-15 | lib/nfs/Connection: add debug flag "in_destroy" | Max Kellermann | 1 | -0/+6 | |
2014-12-14 | lib/nfs/Connection: add assertion | Max Kellermann | 1 | -0/+2 | |
2014-12-14 | lib/nfs/Connection: fix reconnect after mount failure | Max Kellermann | 1 | -1/+1 | |
When mounting had not yet finished, SocketMonitor::IsDefined() was always false, due to the workaround at the beginning of the function that calls SocketMonitor::Steal(). This commit drops the IsDefined() check because it was never necessary and breaks reconnect. | |||||
2014-12-14 | lib/nfs/Connection: fix typo in code comment | Max Kellermann | 1 | -1/+1 | |
2014-12-14 | lib/nfs/Connection: fix memory leak (and assertion failure) | Max Kellermann | 1 | -0/+15 | |
nfs_destroy_context() will invoke all pending callbacks with err==-EINTR. In CancellableCallback::Callback(), this will invoke NfsConnection::DeferClose(), which however is only designed to be called from nfs_service(). In non-debug mode, this will leak memory because nfs_close_async() is never called. Workaround: before nfs_destroy_context(), invoke nfs_close_async() on all pending file handles. | |||||
2014-12-14 | lib/nfs/Connection: move code to Service() | Max Kellermann | 1 | -12/+23 | |
2014-12-14 | lib/nfs/Connection: add assertions | Max Kellermann | 1 | -0/+2 | |
2014-12-14 | lib/nfs/Connection: move code to method InternalClose() | Max Kellermann | 1 | -3/+12 | |
2014-12-14 | lib/nfs/Connection: make in_service and in_event debug-only flags | Max Kellermann | 1 | -0/+9 | |
2014-12-14 | lib/nfs/Connection: fix crash while canceling a failing Open() | Max Kellermann | 1 | -2/+4 | |
The method NfsConnection::CancellableCallback::Callback() will always invoke NfsConnection::Close() on the file handle, even if the void pointer is not a nfsfh. This can happen if the Open() was not successful, e.g. when the file does not exist. | |||||
2014-12-14 | lib/nfs/Connection: add assertions | Max Kellermann | 1 | -0/+31 | |
2014-11-25 | lib/nfs/Connection: broadcast error before closing connection | Max Kellermann | 1 | -4/+4 | |
During the NfsLease::OnNfsConnectionFailed() call, the old (defunct) nfs_context may be used to close file handles. Such code does not yet exist, but will be added soon to fix other bugs. | |||||
2014-11-25 | lib/nfs/Connection: cancel DeferredMonitor on disconnect | Max Kellermann | 1 | -0/+4 | |
Fixes potential second mount attempt after the old connection to the NFS server was shut down. | |||||
2014-10-09 | lib/nfs/Connection: reduce Error instance allocations | Max Kellermann | 1 | -3/+2 | |
2014-10-07 | lib/nfs/Connection: add methods {Open,Read,Close}Directory() | Max Kellermann | 1 | -0/+43 | |
2014-10-07 | lib/nfs/Connection: add method Stat(path) | Max Kellermann | 1 | -0/+30 | |
2014-10-01 | lib/nfs/FileReader: postpone the nfs_close_async() call | Max Kellermann | 1 | -1/+42 | |
If an async opertion is in progress, nfs_close_async() will make libnfs crash because the RPC callback will dereference an object that was freed by nfs_close_async(). | |||||
2014-10-01 | lib/nfs/Connection: fix memory leak when cancelling Open() | Max Kellermann | 1 | -3/+11 | |
Close the newly allocated file handle passed to the callback. | |||||
2014-10-01 | lib/nfs/Connection: remove Mutex | Max Kellermann | 1 | -15/+12 | |
All locks are currenly held from only a single thread (the IOThread) and thus we don't need the Mutex. | |||||
2014-10-01 | lib/nfs/Connection: add method GetEventLoop() | Max Kellermann | 1 | -1/+1 | |
2014-10-01 | lib/nfs/Connection: remove obsolete flag postponed_destroy | Max Kellermann | 1 | -9/+1 | |
2014-10-01 | lib/nfs/Glue: destruct the NfsManager in the I/O thread | Max Kellermann | 1 | -4/+3 | |
This allows eliminating the indirection code from the NfsConnection destructor. | |||||
2014-10-01 | lib/nfs/Cancellable: use boost::intrusive::list | Max Kellermann | 1 | -3/+3 | |
Reduce Remove() overhead because we don't have to walk the list to find an iterator by reference. | |||||
2014-08-31 | nfs/Connection: check for disappearing libnfs socket | Max Kellermann | 1 | -0/+18 | |
2014-08-31 | nfs/Connection: use only BroadcastError() | Max Kellermann | 1 | -4/+1 | |
Since BroadcastError() calls BroadcastMountError(), there's no need to check mount_finished here. | |||||
2014-08-31 | nfs/Connection: nfs_get_fd() can return -1 | Max Kellermann | 1 | -0/+3 | |
While reconnecting to the NFS server, the socket is -1, because there is no socket. Fixes a potential crash (assertion failure). | |||||
2014-08-31 | nfs/Connection: use nfs_get_error() for mount error | Max Kellermann | 1 | -2/+3 | |
2014-08-31 | nfs/Connection: fix assertion failure | Max Kellermann | 1 | -1/+3 | |
2014-06-30 | lib/nfs/Connection: include poll.h | Max Kellermann | 1 | -0/+2 | |
2014-06-17 | input/nfs: use the asynchronous libnfs API | Max Kellermann | 1 | -0/+415 | |
More robust and cancellable. |