Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-12 Thread Alex Williamson
On Thu, 2012-04-12 at 18:27 +0300, Michael S. Tsirkin wrote: > On Thu, Apr 12, 2012 at 08:57:18AM -0600, Alex Williamson wrote: > > On Wed, 2012-04-11 at 10:40 +0300, Michael S. Tsirkin wrote: > > > On Tue, Apr 10, 2012 at 04:07:41PM -0600, Alex Williamson wrote: > > > > > Some points I remember >

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-12 Thread Alex Williamson
On Wed, 2012-04-11 at 10:40 +0300, Michael S. Tsirkin wrote: > On Tue, Apr 10, 2012 at 04:07:41PM -0600, Alex Williamson wrote: > > > Some points I remember > > > - power on is better called slot enabled > > > - guests dont actually call _PSX like you want them to > > > (PS3 for sure, in my testi

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-12 Thread Michael S. Tsirkin
On Thu, Apr 12, 2012 at 08:57:18AM -0600, Alex Williamson wrote: > On Wed, 2012-04-11 at 10:40 +0300, Michael S. Tsirkin wrote: > > On Tue, Apr 10, 2012 at 04:07:41PM -0600, Alex Williamson wrote: > > > > Some points I remember > > > > - power on is better called slot enabled > > > > - guests dont

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-11 Thread Michael S. Tsirkin
On Tue, Apr 10, 2012 at 04:07:41PM -0600, Alex Williamson wrote: > > Some points I remember > > - power on is better called slot enabled > > - guests dont actually call _PSX like you want them to > > (PS3 for sure, in my testing PS0 too), and _EJ0 must > > remove power > > - populated slots aft

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-10 Thread Alex Williamson
On Tue, 2012-04-10 at 23:45 +0300, Michael S. Tsirkin wrote: > On Tue, Apr 10, 2012 at 02:03:05PM -0600, Alex Williamson wrote: > > On Tue, 2012-04-10 at 22:36 +0300, Michael S. Tsirkin wrote: > > > On Tue, Apr 10, 2012 at 12:29:09PM -0600, Alex Williamson wrote: > > > > I never really intended the

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-10 Thread Michael S. Tsirkin
On Tue, Apr 10, 2012 at 02:03:05PM -0600, Alex Williamson wrote: > On Tue, 2012-04-10 at 22:36 +0300, Michael S. Tsirkin wrote: > > On Tue, Apr 10, 2012 at 12:29:09PM -0600, Alex Williamson wrote: > > > I never really intended the power state bitmap to be for power > > > management of any sort, I w

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-10 Thread Alex Williamson
On Tue, 2012-04-10 at 22:36 +0300, Michael S. Tsirkin wrote: > On Tue, Apr 10, 2012 at 12:29:09PM -0600, Alex Williamson wrote: > > I never really intended the power state bitmap to be for power > > management of any sort, I was trying to support this kind of event flow: > > > > http://www.microso

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-10 Thread Michael S. Tsirkin
On Tue, Apr 10, 2012 at 12:29:09PM -0600, Alex Williamson wrote: > I never really intended the power state bitmap to be for power > management of any sort, I was trying to support this kind of event flow: > > http://www.microsoft.com/china/whdc/system/pnppwr/hotadd/hotplugpci.mspx#EYH > > Hot-add

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-10 Thread Alex Williamson
On Mon, 2012-04-09 at 11:24 +0300, Michael S. Tsirkin wrote: > On Mon, Apr 09, 2012 at 11:07:03AM +0300, Gleb Natapov wrote: > > On Sun, Apr 08, 2012 at 10:48:02PM +0300, Michael S. Tsirkin wrote: > > > On Sun, Apr 08, 2012 at 03:45:38PM -0300, Marcelo Tosatti wrote: > > > > On Sun, Apr 08, 2012 at

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-09 Thread Michael S. Tsirkin
On Mon, Apr 09, 2012 at 11:07:03AM +0300, Gleb Natapov wrote: > On Sun, Apr 08, 2012 at 10:48:02PM +0300, Michael S. Tsirkin wrote: > > On Sun, Apr 08, 2012 at 03:45:38PM -0300, Marcelo Tosatti wrote: > > > On Sun, Apr 08, 2012 at 10:15:26AM +0300, Michael S. Tsirkin wrote: > > > > On Sun, Apr 08,

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-09 Thread Gleb Natapov
On Sun, Apr 08, 2012 at 10:48:02PM +0300, Michael S. Tsirkin wrote: > On Sun, Apr 08, 2012 at 03:45:38PM -0300, Marcelo Tosatti wrote: > > On Sun, Apr 08, 2012 at 10:15:26AM +0300, Michael S. Tsirkin wrote: > > > On Sun, Apr 08, 2012 at 02:13:00AM -0300, Marcelo Tosatti wrote: > > > > On Wed, Apr 0

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-08 Thread Michael S. Tsirkin
On Sun, Apr 08, 2012 at 03:45:38PM -0300, Marcelo Tosatti wrote: > On Sun, Apr 08, 2012 at 10:15:26AM +0300, Michael S. Tsirkin wrote: > > On Sun, Apr 08, 2012 at 02:13:00AM -0300, Marcelo Tosatti wrote: > > > On Wed, Apr 04, 2012 at 11:51:00PM -0600, Alex Williamson wrote: > > > > We've been batti

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-08 Thread Marcelo Tosatti
On Sun, Apr 08, 2012 at 10:15:26AM +0300, Michael S. Tsirkin wrote: > On Sun, Apr 08, 2012 at 02:13:00AM -0300, Marcelo Tosatti wrote: > > On Wed, Apr 04, 2012 at 11:51:00PM -0600, Alex Williamson wrote: > > > We've been batting this one back and forth. This series includes > > > several of the cl

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-08 Thread Michael S. Tsirkin
On Sun, Apr 08, 2012 at 02:13:00AM -0300, Marcelo Tosatti wrote: > On Wed, Apr 04, 2012 at 11:51:00PM -0600, Alex Williamson wrote: > > We've been batting this one back and forth. This series includes > > several of the cleanups and specification clarifications from my > > series awhile back. Pat

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-07 Thread Marcelo Tosatti
On Wed, Apr 04, 2012 at 11:51:00PM -0600, Alex Williamson wrote: > We've been batting this one back and forth. This series includes > several of the cleanups and specification clarifications from my > series awhile back. Patch 5 is my proposed alternative to > Michael's PCI hotplug race fix. Sin

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-05 Thread Michael S. Tsirkin
On Thu, Apr 05, 2012 at 09:14:34AM -0600, Alex Williamson wrote: > On Thu, 2012-04-05 at 15:31 +0300, Michael S. Tsirkin wrote: > > Which commit is this series against? > > When trying to apply to my tree, I get conflicts. > > Sorry, it's off qemu-kvm, it was too late when I sent this. Let me > r

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-05 Thread Alex Williamson
On Thu, 2012-04-05 at 15:31 +0300, Michael S. Tsirkin wrote: > Which commit is this series against? > When trying to apply to my tree, I get conflicts. Sorry, it's off qemu-kvm, it was too late when I sent this. Let me re-order to send the fix separately so we can continue to discuss the doc/unde

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-05 Thread Michael S. Tsirkin
Which commit is this series against? When trying to apply to my tree, I get conflicts.

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-05 Thread Gleb Natapov
On Wed, Apr 04, 2012 at 11:51:00PM -0600, Alex Williamson wrote: > We've been batting this one back and forth. This series includes > several of the cleanups and specification clarifications from my > series awhile back. Patch 5 is my proposed alternative to > Michael's PCI hotplug race fix. Sin

Re: [Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-05 Thread Michael S. Tsirkin
On Wed, Apr 04, 2012 at 11:51:00PM -0600, Alex Williamson wrote: > We've been batting this one back and forth. This series includes > several of the cleanups and specification clarifications from my > series awhile back. Patch 5 is my proposed alternative to > Michael's PCI hotplug race fix. Sin

[Qemu-devel] [PATCH 0/5] PCI hotplug fixes/cleanup

2012-04-04 Thread Alex Williamson
We've been batting this one back and forth. This series includes several of the cleanups and specification clarifications from my series awhile back. Patch 5 is my proposed alternative to Michael's PCI hotplug race fix. Since that version I added slot present tracking so we can be a little more