Changeset 55 for trunk/src/lbatest/readtest.c

Timestamp:

Dec 14, 2010, 12:03:32 PM (15 years ago)

Author:

markus

Message:

fixed readtest to run under OS/2 Ring3 above 2GB; commented out ADD read sector dump code

File:

• trunk/src/lbatest/readtest.c (modified) (12 diffs)

trunk/src/lbatest/readtest.c

@@ -7 +7 @@
   matches its real LBA.
 
-  To run the test, attach a HD to an AHCI controller with os2ahci.add loaded
-  and run the test with the drive letter as a parameter.
+  To run the test, run lbatest under Linux, then attach the drive to an OS/2
+  system an create a single primary partition that stretches the entire drive.
+  Then run readtest.exe with the drive letter as a parameter.
+ 
 
   Author: Markus Thielen
@@ -47 +49 @@
 
 #define SECTOR_SIZE        512
-#define SECTORS_PER_WRITE  200
+#define SECTORS_PER_READ   200
 
 
@@ -62 +64 @@
 unsigned long lba_pos;
 unsigned long first_wrong_sector = 0xffffffff;
-char mode;
+
 
 /*--- start of code ---------------------------------------------------------*/
@@ -71 +73 @@
 int main(int argc, char **argv)
 {
-  char *drv;
   HFILE hf;
+  char drv[50];
 
   if (argc < 2) {
@@ -79 +81 @@
   }
 
-  drv = argv[1];
-  if (drv[1] != ':' && drv[2] != '\0') {
-    /* we want a drive letter, which is a letter and a colon */
+  if (isalpha(argv[1][0])) {
+    sprintf(drv, "\\\\.\\%c", toupper(argv[1][0]));
+  } else if (isdigit(argv[1][0])) {
+    sprintf(drv, "\\\\.\\Physical_Disk%c", argv[1][0]);
+  } else {
     usage();
     return -1;
@@ -99 +103 @@
   unsigned long action;
   unsigned long cb_read;
-  char buf[SECTOR_SIZE];
+  char *buf;
   int rc = 0;
+  unsigned long cb_take = SECTOR_SIZE * SECTORS_PER_READ;
   unsigned long err_cnt = 0;
   float gbf = 1024.0 * 1024.0 * 1024.0 / 512.0;
-  long long lba = 0;
+  LONGLONG cbfile = {0};
+  int s;
+  unsigned long sectors;
+  unsigned long val;
+
+  buf = calloc(SECTOR_SIZE, SECTORS_PER_READ);
 
   /* open drive */
-  ret = DosOpenL(drv, &hf_disk, &action, 0, 0, OPEN_ACTION_OPEN_IF_EXISTS,
-                OPEN_FLAGS_DASD | OPEN_FLAGS_FAIL_ON_ERROR |
-                OPEN_FLAGS_WRITE_THROUGH | OPEN_FLAGS_NO_CACHE |
-                OPEN_FLAGS_SEQUENTIAL | OPEN_SHARE_DENYREADWRITE |
-                OPEN_ACCESS_READONLY,
-                NULL);
+  ret = DosOpen(drv, &hf_disk, &action, 0, 0, 
+                OPEN_ACTION_OPEN_IF_EXISTS,
+/*                   OPEN_FLAGS_DASD | OPEN_FLAGS_FAIL_ON_ERROR | */
+/*                   OPEN_FLAGS_WRITE_THROUGH | OPEN_FLAGS_NO_CACHE | */
+/*                   OPEN_FLAGS_SEQUENTIAL | OPEN_SHARE_DENYREADWRITE | */
+                OPEN_SHARE_DENYNONE | OPEN_ACCESS_READONLY,
+                NULL);
   if (ret != NO_ERROR) {
     fprintf(stderr, "Failed to open disk %s for reading: %d\n", drv, ret);
@@ -118 +129 @@
 
   /* lock disk */
-  ret = DosDevIOCtl(hf_disk, IOCTL_DISK, DSK_LOCKDRIVE, NULL, 0,
-                    &action, NULL, 0, &cb_read);
-  if (ret != NO_ERROR) {
-    fprintf(stderr, "Failed to lock drive, code %d\n", ret);
-    rc = -1;
-    goto cleanup;
-  }
+/*    ret = DosDevIOCtl(hf_disk, IOCTL_DISK, DSK_LOCKDRIVE, NULL, 0, */
+/*                  &action, NULL, 0, &cb_read); */
+/*    if (ret != NO_ERROR) { */
+/*      fprintf(stderr, "Failed to lock drive, code %d\n", ret); */
+/*      rc = -1; */
+/*      goto cleanup; */
+/*    } */
 
   /* catch Ctrl+C */
@@ -131 +142 @@
   /* go... */
   memset(buf, 0x00, sizeof(buf));
-  for (lba = 64; ; lba++) {
-    ret = DosRead(hf_disk, buf, SECTOR_SIZE, &cb_read);
+  for (lba_pos = 0; ; lba_pos += SECTORS_PER_READ) {
+    ret = DosRead(hf_disk, buf, cb_take, &cb_read);
     if (ret != NO_ERROR) {
       fprintf(stderr, "\nFailed to read from disk %s, code %d\n", drv, ret);
@@ -139 +150 @@
     }
 
-    if (*((long long*) buf) != lba_pos) {
-      printf("\nWrong sector number: read %d from sector %d\n",
-              *((long long*) buf), lba_pos);
-      err_cnt++;
-      if (first_wrong_sector == 0xffffffff) {
-        first_wrong_sector = lba_pos;
+    if (cb_read == 0) {
+      goto cleanup;
+    }
+
+    sectors = cb_read / SECTOR_SIZE;
+
+    for (s = 0; s < sectors; s++) {
+
+      if (lba_pos + s < 64) {
+        continue;
       }
+
+      val = *((unsigned long*) (buf + s * SECTOR_SIZE));
+      if (val == 0xf6f6f6f6) {
+        /* appearantly, this is the funny first partition sector 
+         * created by LVM - skip it */
+        continue;
+      }
+
+      if (val != lba_pos + s) {
+        printf("\nWrong sector number: read 0x%08x from sector 0x%08x",
+               val, lba_pos + s);
+        err_cnt++;
+        if (first_wrong_sector == 0xffffffff) {
+          first_wrong_sector = lba_pos + s;
+        }
+      }
     }
 
     /* progress */
-    if (lba_pos % 100 == 0) {
-      printf("\r%dk sectors read (%0.02f GB)", lba_pos / 1000, 
-             (float) lba_pos / gbf);
-    }
+    printf("\r%dk sectors read (%0.02f GB)", lba_pos / 1000, 
+           (float) lba_pos / gbf);
   }
 
@@ -160 +189 @@
               NULL, 0, &cb_read);
   DosClose(hf_disk);
+  free(buf);
 
   /* print summary */
@@ -176 +206 @@
 {
   printf("LBA (read sector number) test for os2ahci.add\n"
+         "Call with a drive letter (for logical drive) or 1-based disk number (for\n"
+         "physical drive)\n\n"
          "Usage:\n\n"
-         "lbatest <drive letter>\n\n");
+         "lbatest <drive letter|drive number>\n\n");
 }
 
@@ -187 +219 @@
 {
 
-  if (sig_no == SIGINT && mode == 'R') {
+  if (sig_no == SIGINT) {
     /* read mode interrupted; show summary */
-    printf("\n\nLast block read:    %u\n", lba_pos);
+    printf("\n\nLast block read:    0x%08x\n", lba_pos);
     if (first_wrong_sector != 0xffffffff) {
-      printf("First wrong sector: %u\n", first_wrong_sector);
+      printf("First wrong sector: 0x%08x\n", first_wrong_sector);
     } else {
       printf("All sectors read ok\n");