Re: [Xen-devel] Re: [PATCH] ioemu: Cleanup the code of PCI passthrough.

[Andrew Lyon <andrew.lyon@xxxxxxxxx>]

On Mon, Feb 23, 2009 at 2:23 PM, Boris Derzhavets <bderzhavets@xxxxxxxxx> wrote:
>  I was under impression, that mentioned patch is supposed to support
> loading HVM DomU (?).  If answer is "yes" , then my question:-
> When http://xenbits.xensource.com/xen-unstable.hg will get this update?
> Looks like "make tools" is using
> http://xenbits.xensource.com/git-http/qemu-xen-unstable.git when building
> xen-unstable.hg/tools/ioemu-remote.
> If answer is "no", i am sorry for misunderstanding.
>
>   Fresh Xen-Unstable Dom0 install (along with the most recent build
> 2.6.29-rc5) still shows during attempt to start HVM DomU:-
>
> # xm dmesg
> . . . . . .
> (XEN) HVM1: HVM Loader
> (XEN) HVM1: Detected Xen v3.4-unstable
> (XEN) HVM1: CPU speed is 3006 MHz
> (XEN) irq.c:235: Dom1 PCI link 0 changed 0 -> 5
> (XEN) HVM1: PCI-ISA link 0 routed to IRQ5
> (XEN) irq.c:235: Dom1 PCI link 1 changed 0 -> 10
> (XEN) HVM1: PCI-ISA link 1 routed to IRQ10
> (XEN) irq.c:235: Dom1 PCI link 2 changed 0 -> 11
> (XEN) HVM1: PCI-ISA link 2 routed to IRQ11
> (XEN) irq.c:235: Dom1 PCI link 3 changed 0 -> 5
> (XEN) HVM1: PCI-ISA link 3 routed to IRQ5
> (XEN) HVM1: pci dev 01:2 INTD->IRQ5
> (XEN) HVM1: pci dev 01:3 INTA->IRQ10
> (XEN) HVM1: pci dev 03:0 INTA->IRQ5
> (XEN) HVM1: pci dev 04:0 INTA->IRQ5
> (XEN) HVM1: pci dev 02:0 bar 10 size 02000000: f0000008
> (XEN) HVM1: pci dev 03:0 bar 14 size 01000000: f2000008
> (XEN) HVM1: pci dev 02:0 bar 14 size 00001000: f3000000
> (XEN) HVM1: pci dev 03:0 bar 10 size 00000100: 0000c001
> (XEN) HVM1: pci dev 04:0 bar 10 size 00000100: 0000c101
> (XEN) HVM1: pci dev 04:0 bar 14 size 00000100: f3001000
> (XEN) HVM1: pci dev 01:2 bar 20 size 00000020: 0000c201
> (XEN) HVM1: pci dev 01:1 bar 20 size 00000010: 0000c221
> (XEN) HVM1: Multiprocessor initialisation:
> (XEN) HVM1:  - CPU0 ... 36-bit phys ... fixed MTRRs ... var MTRRs [2/8] ...
> done.
> (XEN) HVM1: Testing HVM environment:
> (XEN) HVM1:  - REP INSB across page boundaries ... passed
> (XEN) HVM1: Passed 1/1 tests
> (XEN) HVM1: Writing SMBIOS tables ...
> (XEN) HVM1: Loading ROMBIOS ...
> (XEN) HVM1: 9788 bytes of ROMBIOS high-memory extensions:
> (XEN) HVM1:   Relocating to 0xfc000000-0xfc00263c ... done
> (XEN) HVM1: Creating MP tables ...
> (XEN) HVM1: Loading Cirrus VGABIOS ...
> (XEN) HVM1: Loading PCI Option ROM ...
> (XEN) HVM1:  - Manufacturer: http://etherboot.org
> (XEN) HVM1:  - Product name: gPXE
> (XEN) HVM1: Loading ACPI ...
>
>
> ********************************
> Logfile under /var/log/xen :-
> ********************************
>
> domid: 1
> qemu: the number of cpus is 1
> config qemu network with xen bridge for  tap1.0 eth0
> Watching /local/domain/0/device-model/1/logdirty/next-active
> Watching /local/domain/0/device-model/1/command
> xs_read(): vncpasswd get error.
> /vm/1388b5b0-3888-5c75-b701-3e82249a379e/vncpasswd.
> qemu_map_cache_init nr_buckets = 10000 size 3145728
> shared page at pfn feffd
> buffered io page at pfn feffb
> Guest uuid = 1388b5b0-3888-5c75-b701-3e82249a379e
> Time offset set 0
> populating video RAM at ff000000
> mapping video RAM from ff000000
> Register xen platform.
> Done register platform.
> xs_read(/local/domain/0/device-model/1/xen_extended_power_mgmt): read error
> medium change watch on `hdc' (index: 0): /dev/loop0
> I/O request not ready: 0, ptr: 0, port: 0, data: 0, count: 0, size: 0
> cirrus vga map change while on lfb mode
>
>
>
>
> --- On Mon, 2/23/09, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx> wrote:
>
> From: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
> Subject: Re: [Xen-devel] Re: [PATCH] ioemu: Cleanup the code of PCI
> passthrough.
> To: "Simon Horman" <horms@xxxxxxxxxxxx>
> Cc: "Yuji Shimada" <shimada-yxb@xxxxxxxxxxxxxxx>,
> "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
> Date: Monday, February 23, 2009, 6:36 AM
>
> Simon Horman writes ("Re: [Xen-devel] Re: [PATCH] ioemu: Cleanup the code
> of PCI passthrough."):
>> On Mon, Feb 23, 2009 at 10:40:22AM +0900, Yuji Shimada wrote:
>> > Signed-off-by: Yuji Shimada <shimada-yxb@xxxxxxxxxxxxxxx>
>>
>> Tested-by: Simon Horman <horms@xxxxxxxxxxxx>
>
> Thanks, I have applied this.
>
> The problem causing it to be rejected was that hw/passthrough.[ch] and
> hw/pt-msi.[ch] were full of trailing whitespace, which Yuji Shimada's
> mailer evidently removes.  I think
>  the mailer was right and the code
> was wrong.
>
> So I have removed all of that trailing whitespace with a trivial perl
> one-liner and then committed the cleanup patch on top.  I haven't
> tested the pass-through but the tree does build.
>
> Ian.
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel
>
>

Boris,

There is another problem as well as the "pci power management" issue
that the patch fixes, with Xen unstable and the patch applied I can
start hvm's if I use a normal Xen kernel, but if I use the pv_ops
kernel I get the sentinel message in xen log:

[2009-02-22 14:01:53 3467] WARNING (image:470) domain xptest: device
model failure: pid 3735: malfunctioning (closed sentinel), killed; see
/var/log/xen/qemu-dm-xptest.log

I've posted the qemu-dm log before, I dont think this line "I/O
request not ready: 0, ptr: 0, port: 0, data: 0, count: 0, size: 0" is
about the problem because it is also logged when I use a non pv_ops
kernel and hvm works ok.

Andy

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel