[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH] hw/pc.c: Fix converting of ioport_register* t
From: |
Adam Lackorzynski |
Subject: |
Re: [Qemu-trivial] [PATCH] hw/pc.c: Fix converting of ioport_register* to MemoryRegion |
Date: |
Thu, 10 Jan 2013 08:29:07 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed Jan 09, 2013 at 18:10:22 +0000, Julien Grall wrote:
> The commit 258711 introduced MemoryRegion to replace ioport_region*
> for ioport 80h and F0h.
> A MemoryRegion needs to have both read and write callback otherwise a segfault
> will occur when an access is made.
>
> The previous behaviour of this both ioport is to return 0xffffffffffffffff.
> So keep this behaviour.
Thanks, confirmed.
> Reported-by: Adam Lackorzynski <address@hidden>
> Signed-off-by: Julien Grall <address@hidden>
Tested-by: Adam Lackorzynski <address@hidden>
> ---
> hw/pc.c | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/hw/pc.c b/hw/pc.c
> index df0c48e..90b1bf7 100644
> --- a/hw/pc.c
> +++ b/hw/pc.c
> @@ -103,6 +103,11 @@ static void ioport80_write(void *opaque, hwaddr addr,
> uint64_t data,
> {
> }
>
> +static uint64_t ioport80_read(void *opaque, hwaddr addr, unsigned size)
> +{
> + return 0xffffffffffffffff;
> +}
> +
> /* MSDOS compatibility mode FPU exception support */
> static qemu_irq ferr_irq;
>
> @@ -123,6 +128,11 @@ static void ioportF0_write(void *opaque, hwaddr addr,
> uint64_t data,
> qemu_irq_lower(ferr_irq);
> }
>
> +static uint64_t ioportF0_read(void *opaque, hwaddr addr, unsigned size)
> +{
> + return 0xffffffffffffffff;
> +}
> +
> /* TSC handling */
> uint64_t cpu_get_tsc(CPUX86State *env)
> {
> @@ -960,6 +970,7 @@ static void cpu_request_exit(void *opaque, int irq, int
> level)
>
> static const MemoryRegionOps ioport80_io_ops = {
> .write = ioport80_write,
> + .read = ioport80_read,
> .endianness = DEVICE_NATIVE_ENDIAN,
> .impl = {
> .min_access_size = 1,
> @@ -969,6 +980,7 @@ static const MemoryRegionOps ioport80_io_ops = {
>
> static const MemoryRegionOps ioportF0_io_ops = {
> .write = ioportF0_write,
> + .read = ioportF0_read,
> .endianness = DEVICE_NATIVE_ENDIAN,
> .impl = {
> .min_access_size = 1,
> --
> Julien Grall
>
Adam
--
Adam address@hidden
Lackorzynski http://os.inf.tu-dresden.de/~adam/