[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 1/6] xen: do not initialize the interval time
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [PATCH v3 1/6] xen: do not initialize the interval timer emulator |
Date: |
Fri, 27 Jan 2012 20:09:59 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2012-01-27 19:21, Stefano Stabellini wrote:
> PIT is emulated by the hypervisor so we don't need to emulate it in Qemu:
> this patch prevents Qemu from waking up needlessly at PIT_FREQ on Xen.
>
> Signed-off-by: Stefano Stabellini <address@hidden>
> ---
> hw/pc.c | 7 +++++--
> 1 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/hw/pc.c b/hw/pc.c
> index 85304cf..7a7ce98 100644
> --- a/hw/pc.c
> +++ b/hw/pc.c
> @@ -43,6 +43,7 @@
> #include "ui/qemu-spice.h"
> #include "memory.h"
> #include "exec-memory.h"
> +#include "arch_init.h"
>
> /* output Bochs bios info messages */
> //#define DEBUG_BIOS
> @@ -1130,7 +1131,7 @@ void pc_basic_device_init(ISABus *isa_bus, qemu_irq
> *gsi,
> DriveInfo *fd[MAX_FD];
> qemu_irq rtc_irq = NULL;
> qemu_irq *a20_line;
> - ISADevice *i8042, *port92, *vmmouse, *pit;
> + ISADevice *i8042, *port92, *vmmouse, *pit = NULL;
> qemu_irq *cpu_exit_irq;
>
> register_ioport_write(0x80, 1, 1, ioport80_write, NULL);
> @@ -1151,7 +1152,9 @@ void pc_basic_device_init(ISABus *isa_bus, qemu_irq
> *gsi,
>
> qemu_register_boot_set(pc_boot_set, *rtc_state);
>
> - pit = pit_init(isa_bus, 0x40, 0);
> + if (!xen_available()) {
> + pit = pit_init(isa_bus, 0x40, 0);
> + }
> pcspk_init(pit);
>
> for(i = 0; i < MAX_SERIAL_PORTS; i++) {
Thus as guest accessing to port 0x61 will be able to crash qemu because
pit is NULL? Or do you emulate that port in the kernel? If not, you
likely want to move pcspk_init() under the same umbrella.
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
- [Qemu-devel] [PATCH v3 0/6] prevent Qemu from waking up needlessly, Stefano Stabellini, 2012/01/27
- [Qemu-devel] [PATCH v3 5/6] qemu_next_alarm_deadline: check the expire time of a clock only if it is enabled, Stefano Stabellini, 2012/01/27
- [Qemu-devel] [PATCH v3 2/6] xen: disable rtc_clock, Stefano Stabellini, 2012/01/27
- [Qemu-devel] [PATCH v3 3/6] xen: introduce an event channel for buffered io event notifications, Stefano Stabellini, 2012/01/27
- [Qemu-devel] [PATCH v3 6/6] qemu_calculate_timeout: increase minimum timeout to 1h, Stefano Stabellini, 2012/01/27
- [Qemu-devel] [PATCH v3 4/6] timers: the rearm function should be able to handle delta = INT64_MAX, Stefano Stabellini, 2012/01/27