On 12/8/23 07:54, Helge Deller wrote: [ ... ]
FWIW, the change I made to fix the spurious interrupt problem is diff --git a/hw/scsi/esp-pci.c b/hw/scsi/esp-pci.c index 6794acaebc..f624398c55 100644 --- a/hw/scsi/esp-pci.c +++ b/hw/scsi/esp-pci.c @@ -286,9 +286,6 @@ static void esp_pci_dma_memory_rw(PCIESPState *pci, uint8_t *buf, int len, /* update status registers */ pci->dma_regs[DMA_WBC] -= len; pci->dma_regs[DMA_WAC] += len; - if (pci->dma_regs[DMA_WBC] == 0) { - pci->dma_regs[DMA_STAT] |= DMA_STAT_DONE; - } } I tested that with several platforms. There are no more spurious interrupts after that change, and no other errors either. Regarding TC after reading the interrupt register, I carry the following patch locally. diff --git a/hw/scsi/esp.c b/hw/scsi/esp.c index 9b11d8c573..f0cd8705a7 100644 --- a/hw/scsi/esp.c +++ b/hw/scsi/esp.c @@ -986,7 +986,7 @@ uint64_t esp_reg_read(ESPState *s, uint32_t saddr) */ val = s->rregs[ESP_RINTR]; s->rregs[ESP_RINTR] = 0; - s->rregs[ESP_RSTAT] &= ~STAT_TC; + // s->rregs[ESP_RSTAT] &= ~STAT_TC; The comment above that code says "Clear sequence step, interrupt register and all status bits except TC", which is quite the opposite of what the code is doing because it clears TC and nothing else. I never spent the time trying to figure out how to fix that properly; clearing the other bits like the comment suggests doesn't work (STAT_INT needs to be set for esp_lower_irq() to work, and clearing the other bits results in transfer failures).Does qemu-hppa boot for you with those patches?
Yes, tested with both 32 bit and 64 bit kernels.
Even with those I see the discs are found, but later I get: [ 8.519780] EXT4-fs error (device sda5): ext4_lookup:1855: inode #787975: comm swapper/0: iget: checksum invalid [ 8.545363] Starting init: /sbin/init exists but couldn't execute it (error -67) [ 8.546339] Run /etc/init as init process [ 8.561422] Run /bin/init as init process [ 8.574649] Run /bin/sh as init process [ 8.580495] EXT4-fs error (device sda5): ext4_lookup:1855: inode #787980: comm swapper/0: iget: checksum invalid [ 8.586170] Starting init: /bin/sh exists but couldn't execute it (error -67)
-67 is EBADMSG on parisc which is used by ext4 for "Bad CRC detected", so that matches the "checksum invalid" message. I don't see that with my root file system, but then mine is a simple ext2 file system. I'll generate an ext4 root file system and let you know how that goes. Guenter
