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

RE: [Xen-devel] Regression in PCI pass-through from qemu-xen changeset "passthrough: Fix older kernel assigned device resume failure problem"



Ian Jackson wrote:
> Simon Horman writes ("Re: [Xen-devel] Regression in PCI pass-through
> from qemu-xen changeset "passthrough: Fix older kernel assigned
> device resume failure problem""): On Sat, May 02, 2009 at 12:10:15PM
> +1000, Simon Horman wrote:  
>> Changeset "passthrough: Fix older kernel assigned device resume
>> failure 
>> problem" (dbb8aafa702b8b4f5568e08641d98471fd04e0f8) in
>> qemu-xen-unstable.git appears to cause a regression. This changeset
>> was 
>> applied between the xen-3.4.0-rc2 and xen-3.4.0-rc3 tags of
>> qemu-xen-unstable.git.
> ...
>> The problem that I am seeing is that with this changeset applied
>> I am not able to correctly initialise Intel network cards. Though
>> I assume the problem is not specific to this hardware.
> 
> Thanks ...
> 
> Cui, Dexuan writes ("[PATCH][ioemu] fix PCI bar mapping"):
>> dbb8aafa702b8b4f5568e08641d98471fd04e0f8 has a bug:
>> The virtual CMD value we get from reg_entry->data is not the proper
>> value because reg_entry->data only holds the emulated bits and the
>> PCI_COMMAND_IO/PCI_COMMAND_MEMORY bits are not in it. Instead, we
>> can use pt_pci_read_config(&ptdev->dev, PCI_COMMAND, 2) to get the
>> proper value.   
> 
> ... Cui, do you think this is likely to fix Simon's problem ?
Yes, I think this should fix Simon's problem since actually I meet with the 
same symptom and I made the patch to fix it.

> 
> I'm about to apply it, in any case, but I was wondering if we have
> another problem too.
I don't see another problem.
Yes, please apply it. Thanks!

Thanks,
-- Dexuan


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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.