kqemu locking my machine hard on amd64 smp, with most recent
patches
Juergen Lock
nox at jelal.kn-bremen.de
Sun May 18 14:26:36 UTC 2008
On Fri, May 16, 2008 at 06:07:32PM -0400, Todd Wasson wrote:
> Using -m 256 doesn't help, though interestingly 1.3.0.p11_5 crashes (but
> doesn't take down the machine) with -m 1536 but is fine with -m 256.
> 1.3.0.p11_6 hangs the machine regardless, though.
>
> I haven't been using -soundhw at all recently, but yes, I do actually have
> sound on the host.
>
> Lowering hw.physmem to 2GB and using -m 256 still results in a hang.
>
> I'm going to keep these and try the max_locked_pages changes that Bakul
> Shah suggested. I'll post the results to the list.
>
> Thanks again.
OK can you try the following kernel patch with the latest kqemu (also at
http://people.freebsd.org/~nox/qemu/patch-sys-amd64-seperate-gdt.txt
- untested because my amd64 smp box is in the middle of a portupgrade that
was long overdue...)
Index: src/sys/amd64/amd64/mp_machdep.c
===================================================================
RCS file: /home/ncvs/src/sys/amd64/amd64/mp_machdep.c,v
retrieving revision 1.287.2.2
diff -u -p -u -r1.287.2.2 mp_machdep.c
--- src/sys/amd64/amd64/mp_machdep.c 28 Nov 2007 23:24:06 -0000 1.287.2.2
+++ src/sys/amd64/amd64/mp_machdep.c 18 May 2008 13:45:32 -0000
@@ -457,10 +457,18 @@ init_secondary(void)
common_tss[cpu].tss_iobase = sizeof(struct amd64tss);
common_tss[cpu].tss_ist1 = (long)&doublefault_stack[PAGE_SIZE];
+ /* Use a seperate gdt for each cpu because the tss differs
+ * This avoids complications for e.g. virtualization software
+ * that needs to reload the task register and otherwise would
+ * then end up using the last cpu's tss on others
+ */
+ bcopy(&gdt[0], &gdt[NGDT * cpu], NGDT * sizeof(gdt[0]));
+
gdt_segs[GPROC0_SEL].ssd_base = (long) &common_tss[cpu];
ssdtosyssd(&gdt_segs[GPROC0_SEL],
- (struct system_segment_descriptor *)&gdt[GPROC0_SEL]);
+ (struct system_segment_descriptor *)&gdt[NGDT * cpu + GPROC0_SEL]);
+ r_gdt.rd_base = (long) &gdt[NGDT * cpu];
lgdt(&r_gdt); /* does magic intra-segment return */
/* Get per-cpu data */
More information about the freebsd-emulation
mailing list