Changeset 10191 for trunk/libtransmission/verify.c

Timestamp:

Feb 13, 2010, 5:46:31 AM (13 years ago)

Author:

charles

Message:

(trunk libT) optimize out some unnecessary cycles when there are missing files in a torrent being verified

File:

• trunk/libtransmission/verify.c (modified) (5 diffs)

trunk/libtransmission/verify.c

@@ -48 +48 @@
 /* #define STOPWATCH */
 
+#ifndef HAVE_VALLOC
+ #define valloc malloc
+#endif
+
 static tr_bool
 verifyTorrent( tr_torrent * tor, tr_bool * stopFlag )
@@ -60 +64 @@
     uint32_t pieceBytesRead = 0;
     tr_file_index_t fileIndex = 0;
+    tr_file_index_t prevFileIndex = !fileIndex;
     tr_piece_index_t pieceIndex = 0;
-    uint8_t * buffer = NULL;
-    int64_t buflen = 0;
+    const int64_t buflen = 4096;
+    uint8_t * buffer = valloc( buflen );
     const time_t begin = tr_time( );
     time_t end;
@@ -73 +78 @@
         int64_t leftInFile;
         int64_t bytesThisPass;
-        ssize_t numRead;
         const tr_file * file = &tor->info.files[fileIndex];
 
@@ -84 +88 @@
 
         /* if we're starting a new file... */
-        if( !filePos && (fd<0) )
+        if( !filePos && (fd<0) && (fileIndex!=prevFileIndex) )
         {
             char * filename = tr_torrentFindFile( tor, fileIndex );
             fd = filename == NULL ? -1 : tr_open_file_for_scanning( filename );
             /* fprintf( stderr, "opening file #%d (%s) -- %d\n", fileIndex, filename, fd ); */
-            if( ( fd >= 0 ) && ( buffer == NULL ) )
-            {
-                struct stat st;
-                buflen = fstat( fd, &st ) ? 4096 : st.st_blksize;
-#ifdef HAVE_VALLOC
-                buffer = valloc( buflen );
-#else
-                buffer = malloc( buflen );
-#endif
-            }
             tr_free( filename );
+            prevFileIndex = fileIndex;
         }
 
@@ -110 +105 @@
 
         /* read a bit */
-        numRead = tr_pread( fd, buffer, bytesThisPass, filePos );
-        if( numRead == bytesThisPass )
-            SHA1_Update( &sha, buffer, numRead );
-        if( numRead > 0 ) {
-            pieceBytesRead += numRead;
+        if( fd >= 0 ) {
+            const ssize_t numRead = tr_pread( fd, buffer, bytesThisPass, filePos );
+            if( numRead == bytesThisPass )
+                SHA1_Update( &sha, buffer, numRead );
+            if( numRead > 0 ) {
+                pieceBytesRead += numRead;
 #if defined HAVE_POSIX_FADVISE && defined POSIX_FADV_DONTNEED
-            posix_fadvise( fd, filePos, bytesThisPass, POSIX_FADV_DONTNEED );
-#endif
+                posix_fadvise( fd, filePos, bytesThisPass, POSIX_FADV_DONTNEED );
+#endif
+            }
         }