WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

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

To: bderzhavets@xxxxxxxxx, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] Re: [PATCH] ioemu: Cleanup the code of PCI passthrough.
From: Andrew Lyon <andrew.lyon@xxxxxxxxx>
Date: Mon, 23 Feb 2009 15:04:57 +0000
Cc:
Delivery-date: Mon, 23 Feb 2009 07:05:25 -0800
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:in-reply-to:references :date:message-id:subject:from:to:content-type :content-transfer-encoding; bh=5hgFee1+Z2pKjoBKd4dVLHqyNOAqgcUtGRoxnDLKixg=; b=bWYPyt4kivox3f2aPdKyMCBRant9hucznUEItzeXZcukf6qSe4VDKvPBbUOy+sozVH IUhjq1qL2rtDzh0oFaze8pBWq1mTqB8Qxw08NqSmP4ozOgpRf8q64f/BxO1pBcn/FoUt GdSDDDIUHptet5XbPwGeXZqo98hYwS0gI2VeA=
Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type:content-transfer-encoding; b=FYaUM/BT801eJ1AUj8j+M8cCIbVs8CCkxREGERlgIiT5FSPk+ycDWY4wVUiGKKgA46 UfkmO5VYHOtR0VP7gvljedn+injBxKb8lPsBHUHWf1nRmUV+gb9RxD/z7//AQs5BETAt 2qXkWCcYSdXSCSYMzBY00TjKK5vEuORJY0x5w=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <709240.76132.qm@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <18850.35361.438234.960479@xxxxxxxxxxxxxxxxxxxxxxxx> <709240.76132.qm@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
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