Le 02/04/2024 à 12:41, Michael Tokarev a écrit :
Author: Stefan Weil via
*SIGH* This happened *again*.
(you'll need to tell git log "--no-mailmap" to not get confused
by the mapping we have for the last time one of these slipped
through...)
Now this is interesting. And this is exactly
-by: Laurent Vivier
Reviewed-by: Mark Cave-Ayland
Message-ID: <20240223155742.2790252-1-laur...@vivier.eu>
---
hw/m68k/virt.c | 17 ++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/hw/m68k/virt.c b/hw/m68k/virt.c
index e2792ef46d93..b8e5e102e6b9 100644
--- a/hw/m68k/
Laurent Vivier (1):
virt: set the CPU type in BOOTINFO
hw/m68k/virt.c | 17 ++---
1 file changed, 14 insertions(+), 3 deletions(-)
--
2.43.2
On 3/8/24 09:56, Laurent Vivier wrote:
On resume e1000e_vm_state_change() always calls e1000e_autoneg_resume()
that sets link_down to false, and thus activates the link even
if we have disabled it.
The problem can be reproduced starting qemu in paused state (-S) and
then set the link to down
quot;execute": "cont" }
To fix the problem, merge the content of igb_vm_state_change()
into igb_core_post_load() as e1000 does.
Buglink: https://issues.redhat.com/browse/RHEL-21867
Suggested-by: Akihiko Odaki
Signed-off-by: Laurent Vivier
---
hw/net/igb_core.c | 47 +++
t;execute": "cont" }
To fix the problem, merge the content of e1000e_vm_state_change()
into e1000e_core_post_load() as e1000 does.
Buglink: https://issues.redhat.com/browse/RHEL-21867
Suggested-by: Akihiko Odaki
Signed-off-by: Laurent Vivier
---
hw/net/e1000e_core.c | 56 +++-
On 3/8/24 09:09, Jason Wang wrote:
On Tue, Mar 5, 2024 at 6:07 PM Laurent Vivier wrote:
On 2/1/24 06:45, Jason Wang wrote:
On Wed, Jan 24, 2024 at 6:40 PM Laurent Vivier wrote:
On resume e1000e_vm_state_change() always calls e1000e_autoneg_resume()
that sets link_down to false, and thus
On 2/1/24 06:45, Jason Wang wrote:
On Wed, Jan 24, 2024 at 6:40 PM Laurent Vivier wrote:
On resume e1000e_vm_state_change() always calls e1000e_autoneg_resume()
that sets link_down to false, and thus activates the link even
if we have disabled it.
The problem can be reproduced starting qemu
-by: Laurent Vivier
---
hw/m68k/virt.c | 17 ++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/hw/m68k/virt.c b/hw/m68k/virt.c
index e2792ef46d93..b8e5e102e6b9 100644
--- a/hw/m68k/virt.c
+++ b/hw/m68k/virt.c
@@ -239,9 +239,20 @@ static void virt_init(MachineState *machine
insertions(+), 5 deletions(-)
create mode 100644 hw/nubus/nubus-virtio-mmio.c
create mode 100644 include/hw/nubus/nubus-virtio-mmio.h
Series Reviewed-by: Laurent Vivier
VEC)) {
/* all except 68000 */
CPUState *cs = env_cpu(env);
switch (format) {
Reviewed-by: Laurent Vivier
qemu_get_current_aio_context() != qemu_get_aio_context();
}
Reviewed-by: Laurent Vivier
Le 25/01/2024 à 23:29, Michael Tokarev a écrit :
Initially in async-teardown.c, but the same construct is used
elsewhere too.
Signed-off-by: Michael Tokarev
---
include/sysemu/os-posix.h | 1 +
system/async-teardown.c | 37 +
util/oslib-posix.c
t;execute": "cont" }
To fix the problem, merge the content of e1000e_vm_state_change()
into e1000e_core_post_load() as e1000 does.
Buglink: https://issues.redhat.com/browse/RHEL-21867
Fixes: 6f3fbe4ed06a ("net: Introduce e1000e device emulation")
Suggested-by: Akihiko Odaki
Sig
quot;execute": "cont" }
To fix the problem, merge the content of igb_vm_state_change()
into igb_core_post_load() as e1000 does.
Buglink: https://issues.redhat.com/browse/RHEL-21867
Fixes: 3a977deebe6b ("Intrdocue igb device emulation")
Cc: akihiko.od...@daynix.com
Suggested-by: Akihiko Odaki
quot;execute": "cont" }
To fix the problem, merge the content of igb_vm_state_change()
into igb_core_post_load() as e1000 does.
Buglink: https://issues.redhat.com/browse/RHEL-21867
Suggested-by: Akihiko Odaki
Signed-off-by: Laurent Vivier
---
hw/net/igb_core.c | 47 +++
t;execute": "cont" }
To fix the problem, merge the content of e1000e_vm_state_change()
into e1000e_core_post_load() as e1000 does.
Buglink: https://issues.redhat.com/browse/RHEL-21867
Suggested-by: Akihiko Odaki
Signed-off-by: Laurent Vivier
---
hw/net/e1000e_core.c | 50 +++-
Hi Jason,
Le 11/01/2024 à 19:58, Jason Thorpe a écrit :
On Jan 10, 2024, at 8:01 AM, Philippe Mathieu-Daudé wrote:
IIUC Goldfish virtual HW is maintained externally by Google
https://android.googlesource.com/platform/external/qemu/+/master/docs/GOLDFISH-VIRTUAL-HARDWARE.TXT
I suppose the
ze) "systick write addr 0x%" PRIx64
" data 0x%" PRIx32 " size %u"
Reviewed-by: Laurent Vivier
a1;
MOS6522Q800VIA2State via2;
dp8393xState dp8393x;
+MemoryRegion dp8393x_prom;
ESCCState escc;
OrIRQState escc_orgate;
SysBusESPState esp;
Reviewed-by: Laurent Vivier
ntl.h"
#endif
Reviewed-by: Laurent Vivier
On 12/1/23 16:21, Markus Armbruster wrote:
Laurent Vivier writes:
On 11/21/23 08:58, Markus Armbruster wrote:
Laurent, there's a question for you at the end.
Yong Huang writes:
On Thu, Nov 16, 2023 at 10:44 PM Markus Armbruster
wrote:
Hyman Huang writes:
This patch allows
On 11/21/23 08:58, Markus Armbruster wrote:
Laurent, there's a question for you at the end.
Yong Huang writes:
On Thu, Nov 16, 2023 at 10:44 PM Markus Armbruster
wrote:
Hyman Huang writes:
This patch allows to display feature and status bits in virtio-status.
An optional argument is
to reflect its true purpose.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-ID: <20231026085650.917663-3-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
hw/display/macfb.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/hw/d
From: Mark Cave-Ayland
This enables A/UX to correctly retrieve the LUT entries when used with
applications that use the MacOS Device Manager Status (GetEntries) call.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-ID: <20231026085650.917663-5-mark.cave-ayl...@ilande.co
write the correct value to the color_palette
array.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-ID: <20231026085650.917663-4-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
hw/display/macfb.c | 6 +++---
1 file changed, 3 insertions(+), 3 del
The following changes since commit d762bf97931b58839316b68a570eecc6143c9e3e:
Merge tag 'pull-target-arm-20231102' of
https://git.linaro.org/people/pmaydell/qemu-arm into staging (2023-11-03
10:04:12 +0800)
are available in the Git repository at:
https://github.com/vivier/qemu-m68k.git
From: Mark Cave-Ayland
Traces from A/UX suggest that this register is only used to reset the
framebuffer
LUT (colour lookup table) and not any other device state.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-ID: <20231026085650.917663-2-mark.cave-ayl...@ilande.co
val = s->regs[addr >> 2];
Reviewed-by: Laurent Vivier
f) * 3;
break;
case DAFB_LUT:
s->color_palette[s->palette_current] = val;
Reviewed-by: Laurent Vivier
rrent] = val & 0xff;
s->palette_current = (s->palette_current + 1) %
ARRAY_SIZE(s->color_palette);
if (s->palette_current % 3) {
Reviewed-by: Laurent Vivier
t;regs[DAFB_INTR_STAT >> 2] &= ~DAFB_INTR_VBL;
-macfb_update_irq(s);
break;
case DAFB_LUT:
s->color_palette[s->palette_current] = val;
Reviewed-by: Laurent Vivier
and TimeSCCDB taken from real hardware with a suitable scaling factor.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-ID: <20231004083806.757242-15-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
include/hw/misc/mac_via.h | 3 +
hw/misc/mac_via.c
and send the next command written to SR after a ADB_BUSRESET onto the bus
regardless, even if we don't detect a state transition to ADB_STATE_NEW.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-ID: <20231004083806.757242-16-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: L
-by: Laurent Vivier
Co-developed-by: Volker Rümelin
Signed-off-by: Mark Cave-Ayland
Message-ID: <20231004083806.757242-8-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
MAINTAINERS| 2 +
include/hw/audio/asc.h | 84 +
hw/audio/asc.c
as an developer aid
for testing and migrating code from ASC to EASC.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
Message-ID: <20231004083806.757242-11-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
include/hw/m68k/
Model Lower 16 bits of ID
...
Quadra/Centris 610/650/800 0x2BAD"
Co-developed-by: Laurent Vivier
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Philippe Mathieu-Daudé
Message-ID: <20231004083806.757242-4-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-ID: <20231004083806.757242-17-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
hw/misc/mac_via.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/hw/misc/mac_via.c b/hw/misc/mac_via.c
index 9f9c2815d0e9..3c41d6
From: Mark Cave-Ayland
Convert the GLUE device to 3-phase reset. The legacy method
doesn't do anything that's invalid in the hold phase, so the
conversion is simple and not a behaviour change.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-ID: <20231004083806.757242-13-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
include/hw/block/swim.h | 15 +++-
hw/block/swim.c | 85 -
hw/block/trace-
: if the FIFO remains empty for half a cycle
(~23ms) then continuously fill the generated buffer with empty silence.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-ID: <20231004083806.757242-9-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
include/hw/audio/asc.
to detect the memory size without
faulting.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
Message-ID: <20231004083806.757242-7-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
include/hw/m68k/q800.h | 1 +
hw/m68k/
From: Mark Cave-Ayland
It is needed because it defines the BIOSConfig area.
Co-developed-by: Laurent Vivier
Signed-off-by: Mark Cave-Ayland
Reviewed-by: BALATON Zoltan
Message-ID: <20231004083806.757242-6-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
MAINT
From: Mark Cave-Ayland
The Quadra 800 has the enhanced ASC (EASC) audio chip which supports both the
legacy IRQ routing through VIA2 and also "A/UX" mode routing direct to the
CPU.
Co-developed-by: Laurent Vivier
Signed-off-by: Mark Cave-Ayland
Message-ID: <20231004083806.757242
From: Mark Cave-Ayland
The djMEMC controller is used to store information related to the physical
memory
configuration.
Co-developed-by: Laurent Vivier
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Philippe Mathieu-Daudé
Message-ID: <20231004083806.757242-3-mark.cave-ayl...@ilande.co
a
fatal
error when running the MacOS booter. Add a single memory region alias at 0xc000
to enable NetBSD kernels to start booting under QEMU.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-ID: <20231004083806.757242-19-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent
is to randomly add 0x500 to the T2
counter value during calibration to enable it to eventually succeed.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-ID: <20231004083806.757242-21-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
hw/misc/mac_via.
The following changes since commit 2f3913f4b2ad74baeb5a6f1d36efbd9ecdf1057d:
Merge tag 'for_upstream' of https://git.kernel.org/pub/scm/virt/kvm/mst/qemu
into staging (2023-10-05 09:01:01 -0400)
are available in the Git repository at:
https://github.com/vivier/qemu-m68k.git
From: Mark Cave-Ayland
Co-developed-by: Laurent Vivier
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Philippe Mathieu-Daudé
Message-ID: <20231004083806.757242-5-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
hw/misc/mac_via.c | 14 +-
1 file chang
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-ID: <20231004083806.757242-14-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
include/hw/block/swim.h | 8 +-
hw/block/swim.c | 212 +---
hw/block/trace-even
to succeed.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-ID: <20231004083806.757242-20-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
include/hw/m68k/q800.h | 1 +
hw/m68k/q800.c | 5 +
2 files changed, 6 insertions(+)
diff --git a/include/h
From: Mark Cave-Ayland
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-ID: <20231004083806.757242-12-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
hw/block/swim.c | 14 ++
hw/block/trace-events | 7 +++
2 files chang
is busy and refuses to send ADB commands unless it is clear. To ensure that
this doesn't happen, always clear the ADB interrupt when switching to A/UX
mode to ensure that the bus enumeration always occurs.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-ID
about
MacOS Classic internals and helping to diagnose and fix bugs in the 68k
emulation. In particular thanks go to Laurent Vivier, Finn Thain, Howard
Spoelstra, Volker Rümelin, Richard Henderson, Martin Husemann, Rin Okuyama,
Elliot Nunn, and SolraBizna.
Signed-off-by: Mark Cave-Ayland
[1] https
char *name, unsigned size, uint64_t value)
"reg=%d [%s] size=%u value=0x%"PRIx64
swim_iwmctrl_write(int reg, const char *name, unsigned size, uint64_t value)
"reg=%d [%s] size=%u value=0x%"PRIx64
-swim_iwm_switch(void) "switch from IWM to SWIM mode"
+swim_switch_to_ism(void) "switch from IWM to ISM mode"
+swim_switch_to_iwm(void) "switch from ISM to IWM mode"
diff --git a/include/hw/block/swim.h b/include/hw/block/swim.h
index 1bc7635d02..5f567e8d59 100644
--- a/include/hw/block/swim.h
+++ b/include/hw/block/swim.h
@@ -50,13 +50,15 @@ struct SWIMCtrl {
int mode;
/* IWM mode */
int iwm_switch;
-uint8_t iwmregs[16];
-uint8_t iwm_data;
-uint8_t iwm_mode;
+uint8_t iwm_latches;
+uint8_t iwmregs[8];
/* SWIM mode */
uint8_t ismregs[16];
uint8_t swim_phase;
uint8_t swim_mode;
+uint8_t swim_status;
+uint8_t pram[16];
+uint8_t pram_idx;
SWIMBus bus;
};
Reviewed-by: Laurent Vivier
wim_iwm_switch(void) "switch from IWM to SWIM mode"
diff --git a/include/hw/block/swim.h b/include/hw/block/swim.h
index 9b3dcb029d..1bc7635d02 100644
--- a/include/hw/block/swim.h
+++ b/include/hw/block/swim.h
@@ -43,23 +43,18 @@ typedef struct FDrive {
} FDrive;
struct SWIMCtrl {
-MemoryRegion iomem;
+MemoryRegion swim;
+MemoryRegion iwm;
+MemoryRegion ism;
FDrive drives[SWIM_MAX_FD];
int mode;
/* IWM mode */
int iwm_switch;
-uint16_t regs[8];
-#define IWM_PH0 0
-#define IWM_PH1 1
-#define IWM_PH2 2
-#define IWM_PH3 3
-#define IWM_MTR 4
-#define IWM_DRIVE 5
-#define IWM_Q66
-#define IWM_Q77
+uint8_t iwmregs[16];
uint8_t iwm_data;
uint8_t iwm_mode;
/* SWIM mode */
+uint8_t ismregs[16];
uint8_t swim_phase;
uint8_t swim_mode;
SWIMBus bus;
Reviewed-by: Laurent Vivier
rom;
+MemoryRegion rom_alias;
GLUEState glue;
MOS6522Q800VIA1State via1;
MOS6522Q800VIA2State via2;
Reviewed-by: Laurent Vivier
le it to
+ * eventually succeed.
+ */
+now = qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL);
+if (now & 1) {
+ret += 0x5;
+ }
+ }
+break;
}
return ret;
}
Reviewed-by: Laurent Vivier
68k/q800.h
@@ -67,6 +67,7 @@ struct Q800MachineState {
MemoryRegion macio;
MemoryRegion macio_alias;
MemoryRegion machine_id;
+MemoryRegion escc_alias;
};
#define TYPE_Q800_MACHINE MACHINE_TYPE_NAME("q800")
Reviewed-by: Laurent Vivier
hes to the so-called
+ * "A/UX" interrupt mode after it initialises, so we can use this as
+ * a convenient place to clear the ADB interrupt for now.
+ */
+s->b |= VIA1B_vADBInt;
}
}
Reviewed-by: Laurent Vivier
& VIA1B_vADBInt) ? "+" : "-");
+
return;
}
Reviewed-by: Laurent Vivier
num hack"
via1_auxmode(int mode) "setting auxmode to %d"
via1_timer_hack_state(int state) "setting timer_hack_state to %d"
Did you ask NetBSD to fix their code?
Reviewed-by: Laurent Vivier
include/hw/misc/mac_via.h b/include/hw/misc/mac_via.h
index 422da43bf9..63cdcf7c69 100644
--- a/include/hw/misc/mac_via.h
+++ b/include/hw/misc/mac_via.h
@@ -74,6 +74,9 @@ struct MOS6522Q800VIA1State {
int64_t next_second;
QEMUTimer *sixty_hz_timer;
int64_t next_sixty_hz;
+
+/* SETUPTIMEK hack */
+int timer_hack_state;
};
Reviewed-by: Laurent Vivier
quot;[%p] Binding to IF_MTD drive"
m25p80_binding_no_bdrv(void *s) "[%p] No BDRV - binding to RAM"
+
+# swim.c
+swim_swimctrl_read(int reg, const char *name, unsigned size, uint64_t value)
"reg=%d [%s] size=%u value=0x%"PRIx64
+swim_swimctrl_write(int reg, const char *name, unsigned size, uint64_t value)
"reg=%d [%s] size=%u value=0x%"PRIx64
+swim_iwmctrl_read(int reg, unsigned size, uint64_t value) "reg=%d size=%u
value=0x%"PRIx64
+swim_iwmctrl_write(int reg, unsigned size, uint64_t value) "reg=%d size=%u
value=0x%"PRIx64
+swim_iwm_switch(void) "switch from IWM to SWIM mode"
Reviewed-by: Laurent Vivier
CHINE,
+.instance_init = q800_init,
.instance_size = sizeof(Q800MachineState),
.class_init = q800_machine_class_init,
};
diff --git a/include/hw/m68k/q800.h b/include/hw/m68k/q800.h
index 790cf433f3..fbaacd88bd 100644
--- a/include/hw/m68k/q800.h
+++ b/include/hw/m68k/q800.h
@@ -47,6
di_i32(tmp2, reg, 2);
} else {
-tcg_gen_addi_i32(tmp, reg, opsize_bytes(opsize));
+tcg_gen_addi_i32(tmp2, reg, opsize_bytes(opsize));
}
-delay_set_areg(s, reg0, tmp, true);
+delay_set_areg(s, reg0, tmp2, true);
}
return result;
case 4: /* Indirect predecrememnt. */
"inc" would be a better name than "val2".
Otherwise:
Reviewed-by: Laurent Vivier
Thanks,
Laurent
e last able to generate samples */
int64_t fifo_empty_ns;
If it's specific to Windows why not using "#if defined(CONFIG_WIN32) &&
defined(CONFIG_AUDIO_DSOUND)" to clearly identify this piece of code as specific to a windows bug
with dsound?
Anyway, code looks good:
Reviewed-by: Laurent Vivier
e_properties);
+rc->phases.hold = glue_reset_hold;
nc->nmi_monitor_handler = glue_nmi;
}
Reviewed-by: Laurent Vivier
: shadowed declaration is here
8963 | CPUState *cpu = env_cpu(cpu_env);
| ^~~
Signed-off-by: Laurent Vivier
---
linux-user/syscall.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index c81e8d344486
| void *p;
| ^
Signed-off-by: Laurent Vivier
---
linux-user/syscall.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index 6139c00ddceb..fe228f7db3a7 100644
--- a/linux-user/syscall.c
+++ b/linux-user/syscall.c
_versions) };
| ^~~~
.../linux-user/syscall.c:4991:20: note: shadowed declaration is here
4991 | const argtype *arg_type = ie->arg_type;
|^~~~
Signed-off-by: Laurent Vivier
---
linux-user/syscall.c | 22 +++---
1 file changed, 11 insertions(+),
/flatload.c:722:15: note: shadowed declaration
is here
722 | abi_ulong p;
| ^
Signed-off-by: Laurent Vivier
---
linux-user/flatload.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/linux-user/flatload.c b/linux-user/flatload.c
index
: note: shadowed declaration is
here
871 | int prot;
| ^~~~
Signed-off-by: Laurent Vivier
---
linux-user/mmap.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/linux-user/mmap.c b/linux-user/mmap.c
index 8eaf57b208b0..8ccaab78590f 100644
--- a/linux-user
Laurent Vivier (5):
linux-user/flatload: clean up local variable shadowing
linux-user/mmap.c: clean up local variable shadowing
linux-user/syscall.c: clean up local variable shadowing in
do_ioctl_dm()
linux-user/syscall.c: clean up local variable shadowing in
TARGET_NR_getcpu
: shadowed declaration is here
1093 | int val = 0;
| ^~~
Signed-off-by: Laurent Vivier
---
disas/m68k.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/disas/m68k.c b/disas/m68k.c
index aefaecfbd6cb..1f16e295ab41 100644
--- a/disas/m68k.c
+++ b/disas
hitting a functional race not a timeout.
It can also happen if the first QEMU (server) crashes. Do we have some traces
from this side?
Reviewed-by: Laurent Vivier
Thanks,
Laurent
)
{
-int sock[nb];
+g_autofree int *sock = g_new(int, nb);
int i;
for (i = 0; i < nb; i++) {
Reviewed-by: Laurent Vivier
Le 21/08/2023 à 10:34, Laurent Vivier a écrit :
Le 20/08/2023 à 22:55, Keith Packard a écrit :
#include
#include
#define X 0x1p+16383l
#define Y 0x1p-16446l
static long double build_mul = X * Y;
static volatile long double x = X;
static volatile long double y = Y;
static void
Le 20/08/2023 à 22:55, Keith Packard a écrit :
#include
#include
#define X 0x1p+16383l
#define Y 0x1p-16446l
static long double build_mul = X * Y;
static volatile long double x = X;
static volatile long double y = Y;
static void
dump_ld(const char *label, long double ld)
{
386/pc.h b/include/hw/i386/pc.h
index d54e8b1101e4..0fabece236cf 100644
--- a/include/hw/i386/pc.h
+++ b/include/hw/i386/pc.h
@@ -200,6 +200,9 @@ void pc_madt_cpu_entry(int uid, const CPUArchIdList
*apic_ids,
/* sgx.c */
void pc_machine_init_sgx_epc(PCMachineState *pcms);
+extern GlobalProperty pc_compat_8_1[];
+extern const size_t pc_compat_8_1_len;
+
extern GlobalProperty pc_compat_8_0[];
extern const size_t pc_compat_8_0_len;
For m68k part:
Acked-by: Laurent Vivier
Le 18/07/2023 à 17:06, Luca Bonissi a écrit :
On 18/07/23 16:40, Peter Maydell wrote:
Hi; thanks for this patch. Unfortunately we need patches
to include a Signed-off-by: line that says you're legally
OK with it being contributed to QEMU, or we can't take them.
Sorry for the missing
Hi Richard,
thank you for the linux-user PR (I have really no time to do that).
I've run the LTP test suite (20230127) on master and found some problems introduced by
this patch.
With armhf and Debian bionic and stretch, brk01 brk02 recvmsg01 fail:
tst_test.c:1560: TINFO: Timeout per run
SBState iosb;
+MemoryRegion ramio;
MemoryRegion macio;
MemoryRegion macio_alias;
MemoryRegion machine_id;
Reviewed-by: Laurent Vivier
On 7/4/23 09:25, David Hildenbrand wrote:
On 04.07.23 09:19, Laurent Vivier wrote:
With TPM CRM device, vhost-vdpa reports an error when it tries
to register a listener for a non aligned memory region:
qemu-system-x86_64: vhost_vdpa_listener_region_add received unaligned region
qemu
/msg03670.html
and from Eric for VFIO:
https://lore.kernel.org/all/20220506132510.1847942-1-eric.au...@redhat.com/
https://lore.kernel.org/all/20220524091405.416256-1-eric.au...@redhat.com/
Bug: https://bugzilla.redhat.com/show_bug.cgi?id=2141965
Thanks,
Laurent
Laurent Vivier (1):
vhost-vdpa
the
memory region name).
Signed-off-by: Laurent Vivier
---
hw/virtio/trace-events | 2 ++
hw/virtio/vhost-vdpa.c | 8 ++--
2 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
index 8f8d05cf9b01..9b0d643b9475 100644
--- a/hw/virtio
at 9:39 PM Laurent Vivier wrote:
On 6/22/23 15:12, Peter Maydell wrote:
On Tue, 20 Jun 2023 at 20:51, Laurent Vivier wrote:
This memory is not correctly aligned and cannot be registered
by vDPA and VFIO.
Isn't this a vDPA/VFIO problem? There's no requirement
for RAM MemoryRegions
On 6/22/23 15:12, Peter Maydell wrote:
On Tue, 20 Jun 2023 at 20:51, Laurent Vivier wrote:
This memory is not correctly aligned and cannot be registered
by vDPA and VFIO.
Isn't this a vDPA/VFIO problem? There's no requirement
for RAM MemoryRegions to be aligned in any way. Code
that doesn't
On 6/21/23 11:13, David Hildenbrand wrote:
On 20.06.23 21:50, Laurent Vivier wrote:
This memory is not correctly aligned and cannot be registered
by vDPA and VFIO.
An error is reported for vhost-vdpa case:
qemu-kvm: vhost_vdpa_listener_region_add received unaligned region
To make it ignored
From: Mark Cave-Ayland
Also change the instantiation of the macfb device to use
object_initialize_child().
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
Message-Id: <20230621085353.113233-23-mark.cave-ayl...@ilande.co.uk>
Sign
From: Mark Cave-Ayland
This brings GLUEState in line with our current QOM guidelines.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
Message-Id: <20230621085353.113233-3-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
-
From: Mark Cave-Ayland
Ensure there is a space before the final closing brace for all global
properties.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Reviewed-by: Philippe Mathieu-Daudé
Message-Id: <20230621085353.113233-2-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: L
From: Mark Cave-Ayland
Instead use the qdev_get_child_bus() function which is intended for this exact
purpose.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Reviewed-by: Philippe Mathieu-Daudé
Message-Id: <20230621085353.113233-22-mark.cave-ayl...@ilande.co.uk>
Sign
From: Mark Cave-Ayland
Also change the instantiation of the GLUE device to use
object_initialize_child().
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
Message-Id: <20230621085353.113233-10-mark.cave-ayl...@ilande.co.uk>
Sign
set the date/time.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Message-Id: <20230621085353.113233-25-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: Laurent Vivier
---
hw/misc/mac_via.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/hw/misc/mac_v
The following changes since commit 67fe6ae41da64368bc4936b196fee2bf61f8c720:
Merge tag 'pull-tricore-20230621-1' of https://github.com/bkoppelmann/qemu
into staging (2023-06-21 20:08:48 +0200)
are available in the Git repository at:
https://github.com/vivier/qemu-m68k.git
From: Mark Cave-Ayland
Also change the instantiation of the mac-nubus-bridge device to use
object_initialize_child() and map the Nubus address space using
memory_region_add_subregion() instead of sysbus_mmio_map().
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Reviewed
From: Mark Cave-Ayland
This will allow the q800-glue.h header to be included separately so that the
GLUE device can be referenced externally.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Laurent Vivier
Reviewed-by: Philippe Mathieu-Daudé
Message-Id: <20230621085353.113233-8-mark.cave-
From: Mark Cave-Ayland
Also change the instantiation of the dp8393x device to use
object_initialize_child().
Signed-off-by: Mark Cave-Ayland
CC: Jason Wang
Reviewed-by: Philippe Mathieu-Daudé
Reviewed-by: Laurent Vivier
Message-Id: <20230621085353.113233-16-mark.cave-ayl...@ilande.co
From: Mark Cave-Ayland
The use of the DEFINE_TYPES() macro will soon be recommended over the use of
calling type_init() directly.
Signed-off-by: Mark Cave-Ayland
Reviewed-by: Philippe Mathieu-Daudé
Message-Id: <20230621085353.113233-9-mark.cave-ayl...@ilande.co.uk>
Signed-off-by: L
1 - 100 of 10496 matches
Mail list logo