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

[Xen-devel] RE: [PATCH]xend: fix a typo in pci.py



Cui, Dexuan wrote:
> The PCI_EXP_TYPE_PCI_BRIDGE should be PCI_EXP_FLAGS_TYPE here.
> Also a tiny fix to the python comment.

Please ignore the patch of the previous mail and use the attachments of this 
mail:

[01_fix_d3d0.patch]
Fix some mistakes in tools/python/xend/util/pci.py
1) PCI_PM_CTRL_NO_SOFT_RESET: this is bit3 of PMCSR(Power Management 
Control/Status). It should be 8. This bit means a device's capability of not 
doing an internal reset across D3hot/D0.
If the bit is 1, there shall be no reset across D3hot/D0, so we should not use 
it as a method to reset device.
2) When performing reset, we should sleep at least 100ms; in current code, it's 
incorrect somewhere we sleep 0.2s and somewhere 0.01s.
3) In detect_dev_info(), fix a typo: PCI_EXP_TYPE_PCI_BRIDG -> 
PCI_EXP_FLAGS_TYPE.
4) fix a small typo in the comment of transform_list().

[02_fix_xen_no_soft_reset.patch]
Fix the definition of PCI_PM_CTRL_NO_SOFT_RESET. This is bit3 of PMCSR(Power 
Management Control/Status). It should be 8.

[03_fix_dom0_no_soft_reset.patch]
Fix the definition of PCI_PM_CTRL_NO_SOFT_RESET. This is bit3 of PMCSR(Power 
Management Control/Status). It should be 8.


Signed-off-by: Dexuan Cui <dexuan.cui@xxxxxxxxx>

Attachment: 01_fix_d3d0.patch
Description: 01_fix_d3d0.patch

Attachment: 02_fix_xen_no_soft_reset.patch
Description: 02_fix_xen_no_soft_reset.patch

Attachment: 03_fix_dom0_no_soft_reset.patch
Description: 03_fix_dom0_no_soft_reset.patch

_______________________________________________
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®.