Date: Sat, 16 Feb 2008 20:34:11 +0200 From: Kostik Belousov <kostikbel@gmail.com> To: Juergen Lock <nox@jelal.kn-bremen.de> Cc: emulation@FreeBSD.org Subject: Re: qemu-system-amd64 and freebsd guest Message-ID: <20080216183411.GL57756@deviant.kiev.zoral.com.ua> In-Reply-To: <20080216181850.GA35839@saturn.kn-bremen.de> References: <20080216164331.GJ57756@deviant.kiev.zoral.com.ua> <20080216181850.GA35839@saturn.kn-bremen.de>
next in thread | previous in thread | raw e-mail | index | archive | help
--+j5CnouwYkpW7UmB
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
On Sat, Feb 16, 2008 at 07:18:50PM +0100, Juergen Lock wrote:
> On Sat, Feb 16, 2008 at 06:43:31PM +0200, Kostik Belousov wrote:
> > My recent attempts to boot CURRENT amd64 on QEMU 0.9.1 resulted in the
> > consistent kernel panic in the first instruction of the root_bus_ocnfig=
ure()
> > function. Looking around, I noted that this instruction is the second o=
ne
> > executed after "sti" in the amd64/amd64/autoconf.c:configure().
> > CPU executes
> > sti
> > call root_bus_configure
> > pushq %rbp <- General Protection Fault there
> >=20
> > It makes me suspect that fault happen when hardware interrupt is delive=
red.
> > Indeed, looking around qemu source code and setting breakpoints in QEMU,
> > I noted that hw interrupt is delivered, with into =3D -1. Calculation of
> > the IDT index in the QEMU' target-i386/helper.c:do_interrupt64() as
> > intno*16 and comparision with unsigned limit looked as overflow and GPF
> > was delivered instead.
> >=20
> > Then, it appears that -1 comes from cpu_get_pic_interrupt() when
> > apic_accept_pic_intr() returns false. This happen, in particular, when
> > corresponding lapic vector is masked. The situation is legitimate, and
> > I think that QEMU shall not deliver an interrupt in this case.
> >=20
> > The following patch allows me to boot the amd64 freebsd on qemu 0.9.1
> > from ports.
> >=20
> > --- cpu-exec.c.orig 2008-02-16 18:23:53.134009488 +0200
> > +++ cpu-exec.c 2008-02-16 18:24:47.127662872 +0200
> > @@ -452,13 +452,15 @@
> > svm_check_intercept(SVM_EXIT_INTR);
> > env->interrupt_request &=3D ~(CPU_INTERRUPT_HA=
RD | CPU_INTERRUPT_VIRQ);
> > intno =3D cpu_get_pic_interrupt(env);
> > - if (loglevel & CPU_LOG_TB_IN_ASM) {
> > - fprintf(logfile, "Servicing hardware INT=
=3D0x%02x\n", intno);
> > - }
> > - do_interrupt(intno, 0, 0, 0, 1);
> > - /* ensure that no TB jump will be modified as
> > - the program flow was changed */
> > - BREAK_CHAIN;
> > + if (intno !=3D -1) {
> > + if (loglevel & CPU_LOG_TB_IN_ASM) {
> > + fprintf(logfile, "Servicing hardware INT=3D0x%02x\n", intno);
> > + }
> > + do_interrupt(intno, 0, 0, 0, 1);
> > + /* ensure that no TB jump will be modified as
> > + the program flow was changed */
> > + BREAK_CHAIN;
> > + }
> > #if !defined(CONFIG_USER_ONLY)
> > } else if ((interrupt_request & CPU_INTERRUPT_VIRQ=
) &&
> > (env->eflags & IF_MASK) && !(env->hflags & HF_=
INHIBIT_IRQ_MASK)) {
>=20
> Hey, thanks for looking at this one! :) Maybe you should post the patch
> on the qemu list so the right ppl can look at it, and see if it is correc=
t:
> http://lists.gnu.org/mailman/listinfo/qemu-devel
>=20
> Thanx,
> Juergen
Could you do this, please, as the maintainer of the qemu port ? I actually
have no time to participate in the qemu ml, I am only qemu user.
--+j5CnouwYkpW7UmB
Content-Type: application/pgp-signature
Content-Disposition: inline
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.8 (FreeBSD)
iEYEARECAAYFAke3LKIACgkQC3+MBN1Mb4jT8ACfQC2dGC4UNH6d2GLYWH4dv8KV
Ct0AoKeRSNAsPBxDQFQDklFoDdHCbs6k
=r6Vm
-----END PGP SIGNATURE-----
--+j5CnouwYkpW7UmB--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20080216183411.GL57756>