Changeset 3918

Timestamp:

Nov 21, 2007, 4:16:59 PM (15 years ago)

Author:

charles

Message:

fix ticket #451 (Files remain in use after removal)

Location:

trunk/libtransmission

Files:

• fdlimit.c (modified) (4 diffs)
• fdlimit.h (modified) (1 diff)
• inout.c (modified) (4 diffs)
• torrent.c (modified) (2 diffs)

trunk/libtransmission/fdlimit.c

@@ -98 +98 @@
     unsigned int  isCheckedOut : 1;
     unsigned int  isWritable : 1;
+    unsigned int  closeWhenDone : 1;
     char          filename[MAX_PATH_LENGTH];
     int           fd;
@@ -190 +191 @@
 
 int
-tr_fdFileOpen( const char * filename, int write )
+tr_fdFileCheckout( const char * filename, int write )
 {
     int i, winner;
@@ -283 +284 @@
     dbgmsg( "checking out '%s' in slot %d", filename, winner );
     o->isCheckedOut = 1;
+    o->closeWhenDone = 0;
     o->date = tr_date( );
     tr_lockUnlock( gFd->lock );
@@ -289 +291 @@
 
 void
-tr_fdFileRelease( int file )
+tr_fdFileReturn( int fd )
 {
     int i;
     tr_lockLock( gFd->lock );
 
-    for( i=0; i<TR_MAX_OPEN_FILES; ++i ) {
+    for( i=0; i<TR_MAX_OPEN_FILES; ++i )
+    {
         struct tr_openfile * o = &gFd->open[i];
-        if( o->fd == file ) {
-            dbgmsg( "releasing file '%s' in slot #%d", o->filename, i );
-            /* fsync( o->fd ); */
-            o->isCheckedOut = 0;
-            break;
+        if( o->fd != fd )
+            continue;
+
+        dbgmsg( "releasing file '%s' in slot #%d", o->filename, i );
+        o->isCheckedOut = 0;
+        if( o->closeWhenDone )
+            TrCloseFile( i );
+        
+        break;
+    }
+    
+    tr_condSignal( gFd->cond );
+    tr_lockUnlock( gFd->lock );
+}
+
+void
+tr_fdFileClose( const char * filename )
+{
+    int i;
+    tr_lockLock( gFd->lock );
+    dbgmsg( "tr_fdFileClose closing '%s'", filename );
+
+    for( i=0; i<TR_MAX_OPEN_FILES; ++i )
+    {
+        struct tr_openfile * o = &gFd->open[i];
+        if( !fileIsOpen(o) || strcmp(filename,o->filename) )
+            continue;
+
+        if( !o->isCheckedOut ) {
+            dbgmsg( "not checked out, so closing it now... '%s'", filename );
+            TrCloseFile( i );
+        } else {
+            dbgmsg( "flagging file '%s', slot #%d to be closed when checked in", gFd->open[i].filename, i );
+            o->closeWhenDone = 1;
         }
     }

trunk/libtransmission/fdlimit.h

@@ -34 +34 @@
 void tr_fdClose( void );
 
-/***********************************************************************
- * tr_fdFileOpen
- ***********************************************************************
- * If it isn't open already, tries to open the file 'name' in the
- * directory 'folder'. If 'name' itself contains '/'s, required
- * subfolders are created. The file is open read-write if 'write' is 1
- * (created if necessary), read-only if 0.
- * Returns the file descriptor if successful, otherwise returns
- * one of the TR_ERROR_IO_*.
- **********************************************************************/
-int tr_fdFileOpen( const char * filename, int write );
+/**
+ * Returns an fd to the specified filename.
+ *
+ * A small repository of open files is kept to avoid the overhead of continually
+ * opening and closing the same files when writing piece data during download.
+ * It's also used to ensure that only one client uses the file at a time.
+ * Clients must check out a file to use it, then return it, like a library, when done.
+ *
+ * if write is nonzero and dirname(filename) doesn't exist, dirname is created.
+ * if write is nonzero and filename doesn't exist, filename is created.
+ * returns the fd if successful; otherwise, one of TR_ERROR_IO_*
+ *
+ * @see tr_fdFileReturn
+ * @see tr_fdFileClose
+ */
+int tr_fdFileCheckout( const char * filename, int write );
 
-/***********************************************************************
- * tr_fdFileRelease
- ***********************************************************************
- * Indicates that the file whose descriptor is 'file' is unused at the
- * moment and can safely be closed.
- **********************************************************************/
-void tr_fdFileRelease( int file );
+/**
+ * Returns an fd from tr_fdFileCheckout() so that other clients may borrow it.
+ *
+ * @see tr_fdFileCheckout
+ * @see tr_fdFileClose
+ */
+void tr_fdFileReturn( int file );
+
+/**
+ * Closes a file that's being held by our file repository.
+ *
+ * If the file isn't checked out, it's closed immediately.
+ * If the file is currently checked out, it will be closed upon its return.
+ *
+ * @see tr_fdFileCheckout
+ * @see tr_fdFileReturn
+ */
+void tr_fdFileClose( const char * filename );
+
+
 
 /***********************************************************************

trunk/libtransmission/inout.c

@@ -67 +67 @@
     else if ((ioMode==TR_IO_READ) && stat( path, &sb ) ) /* does file exist? */
         ret = tr_ioErrorFromErrno ();
-    else if ((fd = tr_fdFileOpen ( path, ioMode==TR_IO_WRITE )) < 0)
+    else if ((fd = tr_fdFileCheckout ( path, ioMode==TR_IO_WRITE )) < 0)
         ret = fd;
     else if( lseek( fd, (off_t)fileOffset, SEEK_SET ) == ((off_t)-1) )
@@ -77 +77 @@
  
     if( fd >= 0 )
-        tr_fdFileRelease( fd );
+        tr_fdFileReturn( fd );
 
     return ret;
@@ -126 +126 @@
     tr_buildPath( path, sizeof(path), tor->destination, file->name, NULL );
 
-    fd = tr_fdFileOpen( path, TRUE );
+    fd = tr_fdFileCheckout( path, TRUE );
     if( fd < 0 ) /* bad fd */
         ret = fd;
@@ -139 +139 @@
 
     if( fd >= 0 )
-        tr_fdFileRelease( fd );
+        tr_fdFileReturn( fd );
 
     return ret;

trunk/libtransmission/torrent.c

@@ -34 +34 @@
 #include "crypto.h" /* for tr_sha1 */
 #include "fastresume.h"
+#include "fdlimit.h" /* tr_fdFileClose */
 #include "handshake.h"
 #include "inout.h"
@@ -1078 +1079 @@
 stopTorrent( void * vtor )
 {
+    int i;
+
     tr_torrent * tor = vtor;
     tr_ioRecheckRemove( tor );
     tr_peerMgrStopTorrent( tor->handle->peerMgr, tor->info.hash );
     tr_trackerStop( tor->tracker );
+
+    for( i=0; i<tor->info.fileCount; ++i )
+    {
+        char path[MAX_PATH_LENGTH];
+        const tr_file * file = &tor->info.files[i];
+        tr_buildPath( path, sizeof(path), tor->destination, file->name, NULL );
+        tr_fdFileClose( path );
+    }
 }