Comment 3 for bug 1677552

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

I agree to your statement: "So the fix for this is to get Jianjun's DRC migration patchset upstream and then to 1704".

I checked and found that this has some pretty long history [1].
I looked at the context but you are so much more aware of the details that I have some questions for you below.

You wrote you tested v5 which I found is the last one before v6 stated to: "Split from Power specific patches". I guess you mean you tried [2] then right?

I didn't find the future of the ppc part after v6. The diffstat changed tremendously v5 -> v6.. Is the ppc specific code no more needed in the newer revisions, or was it just dropped (or ppc parts even accepted already) to get some better progress on the generic content first?

The cover letter text always states about "spapr DRC state" (also RTAS) which sounds very PPC specific, yet the last revisions seemed to be more generic - but maybe only to implement the tailq bits in general. Do you happen to know if the negative effect is
a) only affecting ppc (I know x86 has no perfect hotplug history anyway)?
b) a regression that worked with old versions (maybe because libvirt did start qemu differently as you outlined)?

I found that the latter v17 was queued and pulled in [3].
Changes are in v2.9.0-rc0 and later, does that now mean all you need is in 2.9?
Or are the ppc changes that got split v5->v6 still missing and need to be rebased on top on what went upstream with v17 just recently?

[1]: http://lists.nongnu.org/archive/html/qemu-devel/2017-01/msg04084.html
[2]: http://lists.nongnu.org/archive/html/qemu-devel/2016-10/msg00270.html
[3]: https://lists.gnu.org/archive/html/qemu-devel/2017-01/msg05234.html