qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatib


From: Gleb Natapov
Subject: Re: [Qemu-devel] qemu-kvm: remove "boot=on|off" drive parameter compatibility
Date: Wed, 3 Oct 2012 15:27:55 +0200

On Wed, Oct 03, 2012 at 03:19:56PM +0200, Paolo Bonzini wrote:
> Il 03/10/2012 12:57, Lucas Meneghel Rodrigues ha scritto:
> > Yep, I did send patches with the testdev device present on qemu-kvm.git
> > to qemu.git a while ago, but there were many comments on the review, I
> > ended up not implementing everything that was asked and the patches were
> > archived.
> > 
> > If nobody wants to step up to port it, I'll re-read the original thread
> > and will spin up new patches (and try to go through the end with it).
> > Executing the KVM unittests is something that we can't afford to lose,
> > so I'd say it's important on this last mile effort to get rid of qemu-kvm.
> 
> Absolutely, IIRC the problem was that testdev did a little bit of
> everything... let's see what's the functionality of testdev:
> 
> - write (port 0xf1), can be replaced in autotest with:
> -device isa-debugcon,iobase=0xf1,chardev=...
> 
kvm-unit-tests no longer uses 0xf1 for output. It uses serial.

> - exit code (port 0xf4), see this series:
> http://lists.gnu.org/archive/html/qemu-devel/2012-07/msg00818.html
> 
> - ram size (port 0xd1).  If we can also patch kvm-unittests, the memory
> is available in the CMOS or in fwcfg.  Here is the SeaBIOS code:
> 
>         u32 rs = ((inb_cmos(0x34) << 16) | (inb_cmos(0x35) << 24));
>         if (rs)
>             rs += 16 * 1024 * 1024;
>         else
>             rs = (((inb_cmos(0x30) << 10) | (inb_cmos(0x31) << 18))
>                   + 1 * 1024 * 1024);
> 
> The rest (ports 0xe0..0xe7, 0x2000..0x2017, MMIO) can be left in testdev.
> 
> Paolo

--
                        Gleb.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]