Update inconsistent comment

This commit is contained in:
Uwe Seimet 2023-11-14 15:57:20 +01:00
parent 052899ce81
commit 72f2fed398
1 changed files with 6 additions and 5 deletions

View File

@ -214,11 +214,12 @@ int SCSIDaynaPort::Read(cdb_t cdb, vector<uint8_t>& buf, uint64_t)
// }
int size = rx_packet_size;
if (size < 128) {
// A frame must have at least 128 bytes (see https://github.com/PiSCSI/piscsi/issues/619
// and https://github.com/PiSCSI/piscsi/issues/1098.
// Note that this work-around breaks the checksum. As currently there are no known drivers
// that care for the checksum, the Daynaport driver for the Atari expects frames of 64 bytes
// and the driver for NetBSD 128 bytes it was decided to accept the broken checksum.
// A frame must have at least 64 bytes for the Atari driver, see https://github.com/PiSCSI/piscsi/issues/619,
// but also works with 128 bytes.
// The NetBSD driver requires at least 128 bytes, see https://github.com/PiSCSI/piscsi/issues/1098.
// The Mac driver is also fine with 128 bytes.
// Note that this work-around breaks the checksum. As currently there are no known drivers
// that care for the checksum it was decided to accept the broken checksum.
// If a driver should pop up that breaks because of this, the work-around has to be re-evaluated.
size = 128;
}