- improve benchmark reliabilty, especially for SD and CF cards

- split file benchmarks into read/write
- add AUTHORS file
- encode individual version (it's opie-sysinfo 1.1.0 now) into the .pro file

4 files changed, 42 insertions, 130 deletions

diff --git a/noncore/settings/sysinfo/AUTHORS b/noncore/settings/sysinfo/AUTHORS
new file mode 100644
index 0000000..a068b93
--- a/dev/null
+++ b/noncore/settings/sysinfo/AUTHORS
@@ -0,0 +1,4 @@
+Trolltech AS.
+Dan Williams <drw@handhelds.org>
+Michael 'Mickey' Lauer <mickeyl@handhelds.org>
+
diff --git a/noncore/settings/sysinfo/benchmarkinfo.cpp b/noncore/settings/sysinfo/benchmarkinfo.cpp
index 53d4897..f3a6561 100644
--- a/noncore/settings/sysinfo/benchmarkinfo.cpp
+++ b/noncore/settings/sysinfo/benchmarkinfo.cpp
@@ -135,103 +135,103 @@ BenchmarkInfo::BenchmarkInfo( QWidget *parent, const char *name, int wFlags )
 
     startButton = new QPushButton( tr( "&Start Tests!" ), this );
     QWhatsThis::add( startButton, tr( "Click here to perform the selected tests." ) );
     connect( startButton, SIGNAL( clicked() ), this, SLOT( run() ) );
     vb->addWidget( tests, 2 );
 
     QFile f( QPEApplication::qpeDir() + "/share/sysinfo/results" );
     if ( f.open( IO_ReadOnly ) )
     {
         machineCombo = new QComboBox( this );
         QWhatsThis::add( machineCombo, tr( "Choose a model to compare your results with." ) );
 
         QTextStream ts( &f );
         while( !ts.eof() )
         {
             QString machline = ts.readLine();
-            odebug << "sysinfo: parsing benchmark results for '" << machline << "'" << oendl; 
+            odebug << "sysinfo: parsing benchmark results for '" << machline << "'" << oendl;
             QString resline = ts.readLine();
             machines.insert( machline, new QStringList( QStringList::split( ",", resline ) ) );
             machineCombo->insertItem( machline );
         }
 
         QHBoxLayout* hb = new QHBoxLayout( vb );
         hb->addWidget( new QLabel( tr( "Compare To:" ), this ) );
         hb->addWidget( machineCombo, 2 );
         connect( machineCombo, SIGNAL( activated(int) ), this, SLOT( machineActivated(int) ) );
     }
 
     vb->addWidget( startButton, 2 );
 }
 
 
 BenchmarkInfo::~BenchmarkInfo()
 {}
 
 
 void BenchmarkInfo::machineActivated( int index )
 {
     QStringList* results = machines[ machineCombo->text( index ) ];
     if ( !results )
     {
-        odebug << "sysinfo: no results available." << oendl; 
+        odebug << "sysinfo: no results available." << oendl;
         return;
     }
     QStringList::Iterator it = results->begin();
     test_alu->setText( 2, *(it++) );
     test_fpu->setText( 2, *(it++) );
     test_txt->setText( 2, *(it++) );
     test_gfx->setText( 2, *(it++) );
     test_ram->setText( 2, *(it++) );
     test_sd->setText( 2, *(it++) );
     test_cf->setText( 2, *(it++) );
 }
 
 
 void BenchmarkInfo::run()
 {
     startButton->setText( "> Don't touch! Running Tests! Don't touch! <" );
     qApp->processEvents();
     QTime t;
 
     if ( test_alu->isOn() )
     {
         int d = round( dhry_main( DHRYSTONE_RUNS ) );
-        test_alu->setText( 1, QString( "%1 dhrys" ).arg( QString::number( d ) ) );
+        test_alu->setText( 1, QString().sprintf( "%d dhrys", d ) );
         test_alu->setOn( false );
     }
 
     if ( test_fpu->isOn() )
     {
         t.start();
         BenchFFT();
-        test_fpu->setText( 1, QString( "%1 secs" ).arg( QString::number( t.elapsed() / 1000.0 ) ) );
+        test_fpu->setText( 1, QString().sprintf( "%.2f secs", t.elapsed() / 1000.0 ) );;
         test_fpu->setOn( false );
     }
 
     if ( test_txt->isOn() )
     {
         int value = textRendering( TEST_DURATION );
-        test_txt->setText( 1, QString( "%1 chars/sec" ).arg( QString::number( value / TEST_DURATION ) ) );
+        test_txt->setText( 1, QString().sprintf( "%d chars/sec", value / TEST_DURATION ) );
         test_txt->setOn( false );
     }
 
     if ( test_gfx->isOn() )
     {
         int value = gfxRendering( TEST_DURATION );
-        test_gfx->setText( 1, QString( "%1 gops/sec" ).arg( QString::number( value / 4 / TEST_DURATION ) ) ); // 4 tests
+        test_gfx->setText( 1, QString().sprintf( "%.2f gops/sec", value / 4 / TEST_DURATION ) ); // 4 tests
         test_gfx->setOn( false );
     }
 
     if ( test_ram->isOn() )    // /tmp is supposed to be in RAM on a PDA
     {
         performFileTest( "/tmp/benchmarkFile.dat", test_ram );
     }
 
     if ( test_cf->isOn() )
     {
         OStorageInfo storage;
         performFileTest( storage.cfPath() + "/benchmarkFile.dat", test_cf );
     }
 
     if ( test_sd->isOn() )
     {
@@ -327,154 +327,63 @@ int BenchmarkInfo::gfxRendering( int seconds )
     }
 
     QPixmap p = Resource::loadPixmap( "sysinfo/pattern" );
     t.restart();
     stop = t.elapsed() + seconds*1000;
 
     while ( t.elapsed() < stop )
     {
         bpw.p.drawPixmap( rand()%w, rand()%h, p );
         ++loops;
     }
 
     return loops;
 
 }
 
+const unsigned int FILE_TEST_COUNT = 8000;
+const unsigned int FILE_TEST_BLOCKSIZE = 1024;
+
 void BenchmarkInfo::performFileTest( const QString& fname, OCheckListItem* item )
 {
+    QString filename = fname == "/benchmarkFile.dat" ? QString( "/tmp/bla" ) : fname;
+    odebug << "performing file test on " << filename << oendl;
+
+    QString writeCommand = QString( "dd if=/dev/zero of=%1 count=%2 bs=%3 && sync" ).arg( filename )
+                                                                                  .arg( FILE_TEST_COUNT )
+                                                                                  .arg( FILE_TEST_BLOCKSIZE );
+    QString readCommand = QString( "dd if=%1 of=/dev/null count=%2 bs=%3").arg( filename )
+                                                                                  .arg( FILE_TEST_COUNT )
+                                                                                  .arg( FILE_TEST_BLOCKSIZE );
+    ::system( "sync" );
+    odebug << "performing file test on " << filename << oendl;
+
+    int write = 0;
+    int read = 0;
+
     QTime time;
     time.start();
-    if ( writeFile( fname ) &&
-            readFile( fname ) )
+    if ( ::system( writeCommand ) == 0 )
     {
-        QFile::remove( fname );
-        item->setText( 1, QString( "%1 kb/sec" ).arg( QString::number( 1024.0 / ( time.elapsed() / 1000.0 ) ) ) );
-        item->setOn( false );
+        write = time.elapsed();
     }
     else
     {
         item->setText( 1, tr( "error" ) );
+        return;
     }
-}
-
-char FileBuf[ BUFF_SIZE + 1 ];
-
-bool BenchmarkInfo::writeFile( const QString& w_path )
-{
-    int i;
-    int k;
-    int n;
-    int pos;
-    int len;
-    char *data = "0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ"; // 62
-
 
-    // /*------------------------------------
-    int w_len;
-
-    QFile writeFile( w_path );
-    srand( time( NULL ) );
-
-    for ( n = 0 ; n < 20 ; n++ )
+    time.restart();
+    if ( ::system( readCommand ) == 0 )
     {
-        if ( ! writeFile.open( IO_WriteOnly ) )
-        {
-            writeFile.close();
-            writeFile.remove();
-            return ( false );
-        }
-        // ------------------------------------------ sequential write
-        for ( k = 0 ; k < 256 ; k++ )
-        {
-            n = rand() % 30;
-            memcpy( &FileBuf[ k * 32 ], &data[ n ], 32 );
-        }
-
-        for ( i = 0 ; i < FILE_SIZE / BUFF_SIZE ; i++ )
-        {
-            w_len = writeFile.writeBlock( FileBuf, BUFF_SIZE );
-            if ( w_len != BUFF_SIZE )
-            {
-                writeFile.close();
-                writeFile.remove();
-                return ( false );
-            }
-            writeFile.flush();
-        }
-        // ------------------------------------------ random write
-        for ( i = 0 ; i < 400 ; i++ )
-        {
-            len = rand() % 90 + 4000;
-            for ( k = 0 ; k < 128 ; k++ )
-            {
-                n = rand() % 30;
-                memcpy( &FileBuf[ k * 8 ], &data[ n ], 32 );
-            }
-            pos = rand() % ( FILE_SIZE - BUFF_SIZE );
-
-            writeFile.at( pos );
-            w_len = writeFile.writeBlock( FileBuf, len );
-            if ( w_len != len )
-            {
-                writeFile.close();
-                writeFile.remove();
-                return ( false );
-            }
-            writeFile.flush();
-        }
-        writeFile.close();
+        read = time.elapsed();
     }
-    return ( true );
-
-}
-
-
-bool BenchmarkInfo::readFile( const QString& r_path )
-{
-    int i;
-    int k;
-    int len;
-    int pos;
-    int r_len;
-
-    QFile readFile( r_path );
-    srand( time( NULL ) );
-
-    for ( k = 0 ; k < 200 ; k++ )
+    else
     {
-        if ( ! readFile.open( IO_ReadOnly ) )
-        {
-            readFile.remove();
-            return ( false );
-        }
-        // ------------------------------------------ sequential read
-        readFile.at( 0 );
-        for ( i = 0 ; i < FILE_SIZE / BUFF_SIZE ; i++ )
-        {
-            readFile.at( i * BUFF_SIZE );
-            r_len = readFile.readBlock( FileBuf, BUFF_SIZE );
-            if ( r_len != BUFF_SIZE )
-            {
-                readFile.close();
-                readFile.remove();
-                return ( false );
-            }
-        }
-        // ------------------------------------------ random read
-        for ( i = 0 ; i < 1000 ; i++ )
-        {
-            len = rand() % 120 + 8;
-            pos = rand() % ( FILE_SIZE / BUFF_SIZE - BUFF_SIZE );
-            readFile.at( pos );
-            r_len = readFile.readBlock( FileBuf, len );
-            if ( r_len != len )
-            {
-                readFile.close();
-                readFile.remove();
-                return ( false );
-            }
-        }
-        readFile.close();
+        item->setText( 1, tr( "error" ) );
+        return;
     }
-    return ( true );
+
+    QFile::remove( filename );
+    item->setText( 1, QString().sprintf( "%.2f kb/s, %.2f kb/s", FILE_TEST_COUNT / ( read / 1000.0 ), FILE_TEST_COUNT / ( write / 1000.0 ) ) );
+    item->setOn( false );
 }
diff --git a/noncore/settings/sysinfo/benchmarkinfo.h b/noncore/settings/sysinfo/benchmarkinfo.h
index 0a61134..3b0a33c 100644
--- a/noncore/settings/sysinfo/benchmarkinfo.h
+++ b/noncore/settings/sysinfo/benchmarkinfo.h
@@ -48,22 +48,20 @@ public:
     bool sd_rd;
     bool sd_wt;
     bool cf_rd;
     bool cf_wt;
 
     QClipboard* clb;
     QComboBox* machineCombo;
     Opie::Ui::OListView* tests;
     QPushButton* startButton;
     QDict <QStringList> machines;
 
     int textRendering( int );
     int gfxRendering( int );
     void performFileTest( const QString& fname, Opie::Ui::OCheckListItem* item );
 
 private slots:
-    bool writeFile( const QString& );
-    bool readFile( const QString& );
     void run();
     void machineActivated( int );
 };
 
diff --git a/noncore/settings/sysinfo/sysinfo.pro b/noncore/settings/sysinfo/sysinfo.pro
index 7974713..58f6eb0 100644
--- a/noncore/settings/sysinfo/sysinfo.pro
+++ b/noncore/settings/sysinfo/sysinfo.pro
@@ -16,18 +16,19 @@ SOURCES   = main.cpp \
       graph.cpp \
       load.cpp \
       storage.cpp \
       processinfo.cpp \
       modulesinfo.cpp \
       detail.cpp \
       contrib/dhry.c contrib/fft.c \
       benchmarkinfo.cpp \
       versioninfo.cpp \
       sysinfo.cpp
 
 INCLUDEPATH += $(OPIEDIR)/include
 DEPENDPATH  += $(OPIEDIR)/include
 LIBS        += -lqpe -lopiecore2 -lopieui2
 DEFINES     += UNIX
 TARGET    = sysinfo
+VERSION   = 1.1.0
 
 include ( $(OPIEDIR)/include.pro )