From nobody Sat May 4 19:11:58 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1505888540568383.05456423380974; Tue, 19 Sep 2017 23:22:20 -0700 (PDT) Received: from localhost ([::1]:46930 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1duYOo-0004ZV-Qr for importer@patchew.org; Wed, 20 Sep 2017 02:22:18 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34144) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1duYMu-0003LW-Ob for qemu-devel@nongnu.org; Wed, 20 Sep 2017 02:20:22 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1duYMr-0006Vi-Kf for qemu-devel@nongnu.org; Wed, 20 Sep 2017 02:20:20 -0400 Received: from chuckie.co.uk ([82.165.15.123]:54653 helo=s16892447.onlinehome-server.info) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1duYMr-0006QI-DR; Wed, 20 Sep 2017 02:20:17 -0400 Received: from host109-151-159-252.range109-151.btcentralplus.com ([109.151.159.252] helo=kentang.home) by s16892447.onlinehome-server.info with esmtpsa (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.76) (envelope-from ) id 1duYMq-0005qd-PJ; Wed, 20 Sep 2017 07:20:18 +0100 From: Mark Cave-Ayland To: qemu-devel@nongnu.org, qemu-ppc@nongnu.org, david@gibson.dropbear.id.au Date: Wed, 20 Sep 2017 07:20:00 +0100 Message-Id: <1505888401-10130-2-git-send-email-mark.cave-ayland@ilande.co.uk> X-Mailer: git-send-email 1.7.10.4 In-Reply-To: <1505888401-10130-1-git-send-email-mark.cave-ayland@ilande.co.uk> References: <1505888401-10130-1-git-send-email-mark.cave-ayland@ilande.co.uk> X-SA-Exim-Connect-IP: 109.151.159.252 X-SA-Exim-Mail-From: mark.cave-ayland@ilande.co.uk X-SA-Exim-Version: 4.2.1 (built Sun, 08 Jan 2012 02:45:44 +0000) X-SA-Exim-Scanned: Yes (on s16892447.onlinehome-server.info) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x [fuzzy] X-Received-From: 82.165.15.123 Subject: [Qemu-devel] [PATCH 1/2] ppc/ide/macio: Add missing registers X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" From: Benjamin Herrenschmidt The timing register exists on all variants of MacIO IDE, we just store and return its value. The interrupts register only exists on KeyLargo but it doesn't hurt to have it. The lack of this register causes MacOS X to hangs under some circumstances. Both are 32-bit only. The HW might support smaller access sizes but no known OS uses them. Because the core IDE subsystem doesn't provide us with a way to query the main (level) interrupt state, nor do we have a way to know that DBDMA issued a (edge) interrupt, we reflect both through a private pair of qirq's in order to maintain the register state. Signed-off-by: Benjamin Herrenschmidt --- hw/ide/macio.c | 44 +++++++++++++++++++++++++++++++++++++++++--- hw/ppc/mac.h | 6 +++++- 2 files changed, 46 insertions(+), 4 deletions(-) diff --git a/hw/ide/macio.c b/hw/ide/macio.c index 9742c00..db5db39 100644 --- a/hw/ide/macio.c +++ b/hw/ide/macio.c @@ -331,6 +331,12 @@ static void pmac_ide_writel (void *opaque, val =3D bswap32(val); if (addr =3D=3D 0) { ide_data_writel(&d->bus, 0, val); + } else if (addr =3D=3D 0x20) { + d->timing_reg =3D val; + } else if (addr =3D=3D 0x30) { + if (val & 0x80000000u) { + d->irq_reg &=3D 0x7fffffff; + } } } =20 @@ -342,6 +348,17 @@ static uint32_t pmac_ide_readl (void *opaque,hwaddr ad= dr) addr =3D (addr & 0xFFF) >> 4; if (addr =3D=3D 0) { retval =3D ide_data_readl(&d->bus, 0); + } else if (addr =3D=3D 0x20) { + retval =3D d->timing_reg; + } else if (addr =3D=3D 0x30) { + /* This is an interrupt state register that only exists + * in the KeyLargo and later variants. Bit 0x8000_0000 + * latches the DMA interrupt and has to be written to + * clear. Bit 0x4000_0000 is an image of the disk + * interrupt. MacOS X relies on this and will hang if + * we don't provide at least the disk interrupt + */ + retval =3D d->irq_reg; } else { retval =3D 0xFFFFFFFF; } @@ -426,13 +443,32 @@ static void macio_ide_realizefn(DeviceState *dev, Err= or **errp) { MACIOIDEState *s =3D MACIO_IDE(dev); =20 - ide_init2(&s->bus, s->irq); + ide_init2(&s->bus, s->ide_irq); =20 /* Register DMA callbacks */ s->dma.ops =3D &dbdma_ops; s->bus.dma =3D &s->dma; } =20 +static void pmac_ide_irq(void *opaque, int n, int level) +{ + MACIOIDEState *s =3D opaque; + uint32_t mask =3D 0x80000000u >> n; + + /* We need to reflect the IRQ state in the irq register */ + if (level) { + s->irq_reg |=3D mask; + } else { + s->irq_reg &=3D ~mask; + } + + if (n) { + qemu_set_irq(s->real_ide_irq, level); + } else { + qemu_set_irq(s->real_dma_irq, level); + } +} + static void macio_ide_initfn(Object *obj) { SysBusDevice *d =3D SYS_BUS_DEVICE(obj); @@ -441,8 +477,10 @@ static void macio_ide_initfn(Object *obj) ide_bus_new(&s->bus, sizeof(s->bus), DEVICE(obj), 0, 2); memory_region_init_io(&s->mem, obj, &pmac_ide_ops, s, "pmac-ide", 0x10= 00); sysbus_init_mmio(d, &s->mem); - sysbus_init_irq(d, &s->irq); - sysbus_init_irq(d, &s->dma_irq); + sysbus_init_irq(d, &s->real_ide_irq); + sysbus_init_irq(d, &s->real_dma_irq); + s->dma_irq =3D qemu_allocate_irq(pmac_ide_irq, s, 0); + s->ide_irq =3D qemu_allocate_irq(pmac_ide_irq, s, 1); } =20 static void macio_ide_class_init(ObjectClass *oc, void *data) diff --git a/hw/ppc/mac.h b/hw/ppc/mac.h index 20cbddb..300fc8a 100644 --- a/hw/ppc/mac.h +++ b/hw/ppc/mac.h @@ -132,7 +132,9 @@ typedef struct MACIOIDEState { SysBusDevice parent_obj; /*< public >*/ =20 - qemu_irq irq; + qemu_irq real_ide_irq; + qemu_irq real_dma_irq; + qemu_irq ide_irq; qemu_irq dma_irq; =20 MemoryRegion mem; @@ -140,6 +142,8 @@ typedef struct MACIOIDEState { IDEDMA dma; void *dbdma; bool dma_active; + uint32_t timing_reg; + uint32_t irq_reg; } MACIOIDEState; =20 void macio_ide_init_drives(MACIOIDEState *ide, DriveInfo **hd_table); --=20 1.7.10.4 From nobody Sat May 4 19:11:58 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 1505888540716374.29133922037056; Tue, 19 Sep 2017 23:22:20 -0700 (PDT) Received: from localhost ([::1]:46929 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1duYOq-0004YE-0l for importer@patchew.org; Wed, 20 Sep 2017 02:22:20 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34142) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1duYMu-0003LU-NT for qemu-devel@nongnu.org; Wed, 20 Sep 2017 02:20:22 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1duYMr-0006XP-Uv for qemu-devel@nongnu.org; Wed, 20 Sep 2017 02:20:20 -0400 Received: from chuckie.co.uk ([82.165.15.123]:54656 helo=s16892447.onlinehome-server.info) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1duYMr-0006S0-Lz; Wed, 20 Sep 2017 02:20:17 -0400 Received: from host109-151-159-252.range109-151.btcentralplus.com ([109.151.159.252] helo=kentang.home) by s16892447.onlinehome-server.info with esmtpsa (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.76) (envelope-from ) id 1duYMs-0005qd-A0; Wed, 20 Sep 2017 07:20:19 +0100 From: Mark Cave-Ayland To: qemu-devel@nongnu.org, qemu-ppc@nongnu.org, david@gibson.dropbear.id.au Date: Wed, 20 Sep 2017 07:20:01 +0100 Message-Id: <1505888401-10130-3-git-send-email-mark.cave-ayland@ilande.co.uk> X-Mailer: git-send-email 1.7.10.4 In-Reply-To: <1505888401-10130-1-git-send-email-mark.cave-ayland@ilande.co.uk> References: <1505888401-10130-1-git-send-email-mark.cave-ayland@ilande.co.uk> X-SA-Exim-Connect-IP: 109.151.159.252 X-SA-Exim-Mail-From: mark.cave-ayland@ilande.co.uk X-SA-Exim-Version: 4.2.1 (built Sun, 08 Jan 2012 02:45:44 +0000) X-SA-Exim-Scanned: Yes (on s16892447.onlinehome-server.info) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x [fuzzy] X-Received-From: 82.165.15.123 Subject: [Qemu-devel] [PATCH 2/2] macio: convert pmac_ide_ops from old_mmio X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail: RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Signed-off-by: Mark Cave-Ayland --- hw/ide/macio.c | 181 +++++++++++++++++++++++-----------------------------= ---- 1 file changed, 75 insertions(+), 106 deletions(-) diff --git a/hw/ide/macio.c b/hw/ide/macio.c index db5db39..18ae952 100644 --- a/hw/ide/macio.c +++ b/hw/ide/macio.c @@ -255,131 +255,100 @@ static void pmac_ide_flush(DBDMA_io *io) } =20 /* PowerMac IDE memory IO */ -static void pmac_ide_writeb (void *opaque, - hwaddr addr, uint32_t val) +static uint64_t pmac_ide_read(void *opaque, hwaddr addr, unsigned size) { MACIOIDEState *d =3D opaque; - - addr =3D (addr & 0xFFF) >> 4; - switch (addr) { - case 1 ... 7: - ide_ioport_write(&d->bus, addr, val); - break; - case 8: - case 22: - ide_cmd_write(&d->bus, 0, val); + uint64_t retval =3D 0xffffffff; + int reg =3D addr >> 4; + + switch (reg) { + case 0x0: + if (size =3D=3D 2) { + retval =3D ide_data_readw(&d->bus, 0); + } else if (size =3D=3D 4) { + retval =3D ide_data_readl(&d->bus, 0); + } break; - default: + case 0x1 ... 0x7: + if (size =3D=3D 1) { + retval =3D ide_ioport_read(&d->bus, reg); + } break; - } -} - -static uint32_t pmac_ide_readb (void *opaque,hwaddr addr) -{ - uint8_t retval; - MACIOIDEState *d =3D opaque; - - addr =3D (addr & 0xFFF) >> 4; - switch (addr) { - case 1 ... 7: - retval =3D ide_ioport_read(&d->bus, addr); + case 0x8: + case 0x16: + if (size =3D=3D 1) { + retval =3D ide_status_read(&d->bus, 0); + } break; - case 8: - case 22: - retval =3D ide_status_read(&d->bus, 0); + case 0x20: + if (size =3D=3D 4) { + retval =3D d->timing_reg; + } break; - default: - retval =3D 0xFF; + case 0x30: + /* This is an interrupt state register that only exists + * in the KeyLargo and later variants. Bit 0x8000_0000 + * latches the DMA interrupt and has to be written to + * clear. Bit 0x4000_0000 is an image of the disk + * interrupt. MacOS X relies on this and will hang if + * we don't provide at least the disk interrupt + */ + if (size =3D=3D 4) { + retval =3D d->irq_reg; + } break; } - return retval; -} =20 -static void pmac_ide_writew (void *opaque, - hwaddr addr, uint32_t val) -{ - MACIOIDEState *d =3D opaque; - - addr =3D (addr & 0xFFF) >> 4; - val =3D bswap16(val); - if (addr =3D=3D 0) { - ide_data_writew(&d->bus, 0, val); - } -} - -static uint32_t pmac_ide_readw (void *opaque,hwaddr addr) -{ - uint16_t retval; - MACIOIDEState *d =3D opaque; - - addr =3D (addr & 0xFFF) >> 4; - if (addr =3D=3D 0) { - retval =3D ide_data_readw(&d->bus, 0); - } else { - retval =3D 0xFFFF; - } - retval =3D bswap16(retval); return retval; } =20 -static void pmac_ide_writel (void *opaque, - hwaddr addr, uint32_t val) -{ - MACIOIDEState *d =3D opaque; =20 - addr =3D (addr & 0xFFF) >> 4; - val =3D bswap32(val); - if (addr =3D=3D 0) { - ide_data_writel(&d->bus, 0, val); - } else if (addr =3D=3D 0x20) { - d->timing_reg =3D val; - } else if (addr =3D=3D 0x30) { - if (val & 0x80000000u) { - d->irq_reg &=3D 0x7fffffff; - } - } -} - -static uint32_t pmac_ide_readl (void *opaque,hwaddr addr) +static void pmac_ide_write(void *opaque, hwaddr addr, uint64_t val, + unsigned size) { - uint32_t retval; MACIOIDEState *d =3D opaque; - - addr =3D (addr & 0xFFF) >> 4; - if (addr =3D=3D 0) { - retval =3D ide_data_readl(&d->bus, 0); - } else if (addr =3D=3D 0x20) { - retval =3D d->timing_reg; - } else if (addr =3D=3D 0x30) { - /* This is an interrupt state register that only exists - * in the KeyLargo and later variants. Bit 0x8000_0000 - * latches the DMA interrupt and has to be written to - * clear. Bit 0x4000_0000 is an image of the disk - * interrupt. MacOS X relies on this and will hang if - * we don't provide at least the disk interrupt - */ - retval =3D d->irq_reg; - } else { - retval =3D 0xFFFFFFFF; + int reg =3D addr >> 4; + + switch (reg) { + case 0x0: + if (size =3D=3D 2) { + ide_data_writew(&d->bus, 0, val); + } else if (size =3D=3D 4) { + ide_data_writel(&d->bus, 0, val); + } + break; + case 0x1 ... 0x7: + if (size =3D=3D 1) { + ide_ioport_write(&d->bus, reg, val); + } + break; + case 0x8: + case 0x16: + if (size =3D=3D 1) { + ide_cmd_write(&d->bus, 0, val); + } + break; + case 0x20: + if (size =3D=3D 4) { + d->timing_reg =3D val; + } + break; + case 0x30: + if (size =3D=3D 4) { + if (val & 0x80000000u) { + d->irq_reg &=3D 0x7fffffff; + } + } + break; } - retval =3D bswap32(retval); - return retval; } =20 static const MemoryRegionOps pmac_ide_ops =3D { - .old_mmio =3D { - .write =3D { - pmac_ide_writeb, - pmac_ide_writew, - pmac_ide_writel, - }, - .read =3D { - pmac_ide_readb, - pmac_ide_readw, - pmac_ide_readl, - }, - }, - .endianness =3D DEVICE_NATIVE_ENDIAN, + .read =3D pmac_ide_read, + .write =3D pmac_ide_write, + .valid.min_access_size =3D 1, + .valid.max_access_size =3D 4, + .endianness =3D DEVICE_LITTLE_ENDIAN, }; =20 static const VMStateDescription vmstate_pmac =3D { --=20 1.7.10.4