Changeset 8931

Timestamp:

Aug 14, 2009, 8:55:22 PM (13 years ago)

Author:

charles

Message:

(trunk libT) #2335: when verifying local data, pause a few msec between pieces to give the system some breathing room and other processes a shot at the disk IO

File:

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

trunk/libtransmission/verify.c

@@ -47 +47 @@
     tr_bool hadPiece = 0;
     uint32_t piecePos = 0;
+    uint32_t pieceBytesRead = 0;
     tr_file_index_t fileIndex = 0;
     tr_piece_index_t pieceIndex = 0;
@@ -90 +91 @@
         if( (fd>=0) && tr_lseek( fd, filePos, SEEK_SET ) != -1 ) {
             const int64_t numRead = read( fd, buffer, bytesThisPass );
+            if( numRead > 0 )
+                pieceBytesRead += numRead;
             if( numRead == bytesThisPass )
                 SHA1_Update( &sha, buffer, numRead );
@@ -121 +124 @@
             tor->anyDate = time( NULL );
 
+            /* going full-throttle on a verify can choke other processes'
+             * disk IO, so wait a fwe msec between pieces.
+             * The msec is arbitrary, and the "if" clause is to make sure we
+             * don't slow down verification of files that don't exist */
+            if( pieceBytesRead == tr_torPieceCountBytes( tor, pieceIndex ) )
+                tr_wait( 50 );
+
             SHA1_Init( &sha );
             ++pieceIndex;
             piecePos = 0;
+            pieceBytesRead = 0;
         }