[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 05/10] hw/timer/hpet: Remove HPETState::hpet_offset_saved field
From: |
Philippe Mathieu-Daudé |
Subject: |
[PATCH 05/10] hw/timer/hpet: Remove HPETState::hpet_offset_saved field |
Date: |
Fri, 2 May 2025 01:01:23 +0200 |
The HPETState::hpet_offset_saved boolean was only set in the
hw_compat_2_11[] array, via the 'hpet-offset-saved=false'
property. We removed all machines using that array, lets remove
that property and all the code around it.
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
hw/timer/hpet.c | 9 +--------
rust/hw/timer/hpet/src/hpet.rs | 9 ---------
2 files changed, 1 insertion(+), 17 deletions(-)
diff --git a/hw/timer/hpet.c b/hw/timer/hpet.c
index d1b7bc52b7b..47b8fc0d880 100644
--- a/hw/timer/hpet.c
+++ b/hw/timer/hpet.c
@@ -71,7 +71,6 @@ struct HPETState {
MemoryRegion iomem;
uint64_t hpet_offset;
- bool hpet_offset_saved;
qemu_irq irqs[HPET_NUM_IRQ_ROUTES];
uint32_t flags;
uint8_t rtc_irq_level;
@@ -264,11 +263,6 @@ static int hpet_post_load(void *opaque, int version_id)
t->cmp64 = hpet_calculate_cmp64(t, s->hpet_counter, t->cmp);
t->last = qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL) -
NANOSECONDS_PER_SECOND;
}
- /* Recalculate the offset between the main counter and guest time */
- if (!s->hpet_offset_saved) {
- s->hpet_offset = ticks_to_ns(s->hpet_counter)
- - qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL);
- }
return 0;
}
@@ -277,7 +271,7 @@ static bool hpet_offset_needed(void *opaque)
{
HPETState *s = opaque;
- return hpet_enabled(s) && s->hpet_offset_saved;
+ return hpet_enabled(s);
}
static bool hpet_rtc_irq_level_needed(void *opaque)
@@ -733,7 +727,6 @@ static const Property hpet_device_properties[] = {
DEFINE_PROP_UINT8("timers", HPETState, num_timers, HPET_MIN_TIMERS),
DEFINE_PROP_BIT("msi", HPETState, flags, HPET_MSI_SUPPORT, false),
DEFINE_PROP_UINT32(HPET_INTCAP, HPETState, intcap, 0),
- DEFINE_PROP_BOOL("hpet-offset-saved", HPETState, hpet_offset_saved, true),
};
static void hpet_device_class_init(ObjectClass *klass, const void *data)
diff --git a/rust/hw/timer/hpet/src/hpet.rs b/rust/hw/timer/hpet/src/hpet.rs
index cbd2ed4f6bf..9e681391176 100644
--- a/rust/hw/timer/hpet/src/hpet.rs
+++ b/rust/hw/timer/hpet/src/hpet.rs
@@ -544,7 +544,6 @@ pub struct HPETState {
/// Offset of main counter relative to qemu clock.
hpet_offset: BqlCell<u64>,
- hpet_offset_saved: bool,
irqs: [InterruptSource; HPET_NUM_IRQ_ROUTES],
rtc_irq_level: BqlCell<u32>,
@@ -885,14 +884,6 @@ impl ObjectImpl for HPETState {
u32,
default = 0
),
- qemu_api::define_property!(
- c_str!("hpet-offset-saved"),
- HPETState,
- hpet_offset_saved,
- unsafe { &qdev_prop_bool },
- bool,
- default = true
- ),
}
impl DeviceImpl for HPETState {
--
2.47.1
- [PATCH 00/10] hw/core: Remove hw_compat[] array for 2.10, 2.11 and 2.12 machines, Philippe Mathieu-Daudé, 2025/05/01
- [PATCH 01/10] hw/s390x/s390-virtio-ccw: Remove the deprecated 2.10 and 2.11 machine types, Philippe Mathieu-Daudé, 2025/05/01
- [PATCH 02/10] hw/core/machine: Remove hw_compat_2_10[] array, Philippe Mathieu-Daudé, 2025/05/01
- [PATCH 03/10] hw/input/virtio-input: Remove VirtIOInputHID::wheel_axis field, Philippe Mathieu-Daudé, 2025/05/01
- [PATCH 04/10] hw/core/machine: Remove hw_compat_2_11[] array, Philippe Mathieu-Daudé, 2025/05/01
- [PATCH 05/10] hw/timer/hpet: Remove HPETState::hpet_offset_saved field,
Philippe Mathieu-Daudé <=
- [PATCH 06/10] hw/net/e1000: Remove unused E1000_FLAG_TSO flag, Philippe Mathieu-Daudé, 2025/05/01
- [PATCH 07/10] hw/s390x/s390-virtio-ccw: Remove the deprecated 2.12 machine type, Philippe Mathieu-Daudé, 2025/05/01
- [PATCH 08/10] hw/core/machine: Remove hw_compat_2_12[] array, Philippe Mathieu-Daudé, 2025/05/01
- [PATCH 09/10] hw/audio/hda-codec: Remove HDAAudioState::use_timer field, Philippe Mathieu-Daudé, 2025/05/01
- [PATCH 10/10] hw/display/vga-pci: Do not expose the 'global-vmstate' property, Philippe Mathieu-Daudé, 2025/05/01