So it supports only FAT32 now?

Hi @toncho11
I think it means now only support FAT12 and 16.

Thank you @ghaerr !
I will think about adding 1232K feature,
so that I can test with real hardware.
(signature may not be checked any more since we skipped FAT check)

EDIT:
It seems that the signature check is in readboot so it still exist.
I may be able to test 2DD 720 KB disk which is 512 B sector.

Yes, I see. It is in the name of the PR.
So there should be a warning message that it is not supported if tried on FAT32.

Hello @tyama501,

I will think about adding 1232K feature
It seems that the signature check is in readboot so it still exist.

Yes, but (for now) I am thinking that if you just move the boot block signature check to after where the BPB information is read, then you might be able to check signature only for non-1024-byte sector floppies. (If you want to later add an additional disk read when boot->BytesPerSec == 1024, that would check signature for 1232 floppies):

    boot->FATsecs = boot->FATsmall;
...
    if ((boot->BytesPerSec != 1024) && (block[510] != 0x55 || block[511] != 0xaa)) {
        pfatal("Invalid signature in boot block: %02x%02x", block[511], block[510]);
                exit(2);
    }
    if(boot->BytesPerSec == 0){
        perror("Not valide BytesPerSec");
        exit(2);
    }

Then there may not be a need to rewrite much as it appears that the rest of the program uses boot->BytesPerSec instead of a 512 byte constant.

Thank you!

With more thought, perhaps something like this would work well. This would allow 512-byte and 1024-byte sectors with a (probably) valid BPB, and only check the boot signature on 512-byte sectored floppies. The error message is changed in case a MINIX disk check is attempted:

    boot->FATsecs = boot->FATsmall;
...
    if (boot->BytesPerSec != 512 && boot->ByesPerSec != 1024){
        perror("Invalid DOS format, bad bytes per sector");
        exit(2);
    }
    if ((boot->BytesPerSec == 512) && (block[510] != 0x55 || block[511] != 0xaa)) {
        pfatal("Invalid signature in boot block: %02x%02x", block[511], block[510]);
                exit(2);
    }