Impressions ca. 2023 (Ubuntu 23.10)

[maltris]

So far so good, with fixing a small indentation issue and 32 bit building I managed to get all binaries:

diff --git a/imdcat.cpp b/imdcat.cpp
index 376f991..3f941a8 100644
--- a/imdcat.cpp
+++ b/imdcat.cpp
@@ -73,11 +73,11 @@ public:
     int sec;
     SHC_t(int c, int h, int s) : cyl(c), head(h), sec(s) {}
     bool operator <(const SHC_t& rhs) const {
-        if (this->cyl  < rhs.cyl)  return true;
-        if (this->cyl  > rhs.cyl)  return false;
+        if (this->cyl < rhs.cyl)  return true;
+        if (this->cyl > rhs.cyl)  return false;
         if (this->head < rhs.head) return true;
         if (this->head > rhs.head) return false;
-        if (this->sec  < rhs.sec)  return true;
+        if (this->sec < rhs.sec)  return true;
                                    return false;
     }
 };

./configure --host=i686-linux-gnu "CFLAGS=-m32" "CXXFLAGS=-m32" "LDFLAGS=-m32" && make

Unfortunately, I cannot read any data from my USB-FDD, because to linux it looks like a blockdevice on /dev/sda. I made a link from /dev/fd0 to /dev/sda without success:

sudo ln -s /dev/sda /dev/fd0

But the dumpfloppy tool gives me:

sudo ./dumpfloppy -d 0 test.imd
opening /dev/fd0
cannot get drive parameters: Invalid argument

I suppose its only ready to work with real floppy devices?

[johnkw]

Weird, the original author deleted their github repo, so I guess my fork with fixes is now the official repo. But no, I doubt it works with USB drivers. I wouldn't be surprised if USB drivers try to hide all of the low-level error reporting stuff and prevent reading when CRC fails. But feel free to submit pull requests if you fix anything there.

[maltris]

Cool, so keep up the repo, because there May be people to benefit from it.

I now set up a machine with a native floppy drive, so far so good, but dumpfloppy exits out with

FD_RECALIBRATE: Operation not supported

I will keep trying. So far i recovered a lot of data using ddrescue, but im hoping to squeeze some last blocks out of some floppies.

[maltris]

One step further: The above message "Operation not supported" happens in all kernels built with the patch mentioned here. If I understand correctly, you either have to install a kernel pre-5.18 or compile your own with the BLK_DEV_FD_RAWCMD-build-flag enabled.

I went and installed 5.17 and 5.16 from the Ubuntu Mainline Kernel repo, which seemed to have done the trick. According to strace the dumpfloppy tool now is able to actually run the FDRAWCMD. However, now the exact same command exits out with "-1 EIO (Input/output error)" to which I have no solution yet.

I also found the upstream repo from the original developer at https://offog.org/code/dumpfloppy/.

[RobinSergeant]

I've just built on 64-bit Fedora 40 and had to fix the following to compile. As well as the indentation I also had to replace %d with %ld in a bunch of places because size_t is now bigger. I've only tested imdcat, but that seems to work fine.

diff --git a/dumpfloppy.cpp b/dumpfloppy.cpp
index 5bcc803..2ecd546 100644
--- a/dumpfloppy.cpp
+++ b/dumpfloppy.cpp
@@ -326,7 +326,7 @@ static bool probe_track(track_t& track) {
     track.num_sectors = end_pos;
 
     // Show what we found.
-    printf(" %s %dx%d:",
+    printf(" %s %dx%ld:",
            track.data_mode->name,
            track.num_sectors, sector_bytes(track.sector_size_code));
     for (int i = 0; i < track.num_sectors; i++) {
diff --git a/imdcat.cpp b/imdcat.cpp
index 376f991..e0efe43 100644
--- a/imdcat.cpp
+++ b/imdcat.cpp
@@ -78,7 +78,7 @@ public:
         if (this->head < rhs.head) return true;
         if (this->head > rhs.head) return false;
         if (this->sec  < rhs.sec)  return true;
-                                   return false;
+        return false;
     }
 };
 
@@ -140,7 +140,7 @@ static void write_flat(const disk_t& disk, FILE *flat) {
                         fprintf(stderr, "\x07");
                         did_bell = true;
                     }
-                    fprintf(stderr, "Enter the 'IMD data id' to use for Logical C %d H %d S %d: [default: %d, count: %d]: ",
+                    fprintf(stderr, "Enter the 'IMD data id' to use for Logical C %d H %d S %d: [default: %ld, count: %d]: ",
                         sector.log_cyl, sector.log_head, sector.log_sector,
                         data_id, default_iter->second
                     );
@@ -151,7 +151,7 @@ static void write_flat(const disk_t& disk, FILE *flat) {
                         }
                         //fprintf(stderr, "Read %s\n", buf);
                         if (strcmp(buf, "\n") == 0) {
-                            fprintf(stderr, "Using default ID of %d\n", data_id);
+                            fprintf(stderr, "Using default ID of %ld\n", data_id);
                             break;
                         } else if (sscanf(buf, "%zd", &data_id) == 1) {
                             if (data_id < sector.datas.size()) {
diff --git a/show.cpp b/show.cpp
index a43ab9e..bccdd7a 100644
--- a/show.cpp
+++ b/show.cpp
@@ -53,7 +53,7 @@ void show_sector(const sector_t& sector, FILE *out) {
 
 void show_track(const track_t& track, FILE *out) {
     show_mode(track.data_mode, out);
-    fprintf(out, " %dx%d",
+    fprintf(out, " %dx%ld",
             track.num_sectors,
             sector_bytes(track.sector_size_code));
     for (int phys_sec = 0; phys_sec < track.num_sectors; phys_sec++) {
@@ -71,7 +71,7 @@ void show_track_data(const track_t& track, FILE* const out) {
                 track.phys_cyl, track.phys_head, phys_sec,
                 sector.log_cyl, sector.log_head, sector.log_sector);
         if (sector.datas.size() > 1) {
-            fprintf(out, ": (unique read datas: %d)", sector.datas.size());
+            fprintf(out, ": (unique read datas: %ld)", sector.datas.size());
         }
         fprintf(out, ":\n");

[johnkw]

Well, submit pull requests.

[RobinSergeant]

Well, submit pull requests.

Done!