Thank you Maciej it seems like your version is really ahead I'll do the required work and merge it so i can submit a v2 with the latest patchset from Roman
On Fri, Apr 3, 2020 at 6:06 PM Jon Doron <ari...@gmail.com> wrote: > > Thank you Maciej, I based it on top of what Denis (d...@openvz.org) gave me > which was this: > https://ftp.openvz.org/virtuozzo/releases/openvz-7.0.12-288/source/SRPMS/q/qemu-kvm-vz-2.12.0-33.vz7.14.4.src.rpm > > Do you think you have a more recent version I dont mind diffing and > resubmitting a new version of the patchset? > > Thanks, > -- Jon. > > On Fri, Apr 3, 2020 at 5:56 PM Maciej S. Szmigiero > <m...@maciej.szmigiero.name> wrote: > > > > Hi Jon, > > > > On 03.04.2020 16:23, Jon Doron wrote: > > > Guest OS uses ACPI to discover vmbus presence. Add a corresponding > > > entry to DSDT in case vmbus has been enabled. > > > > > > Experimentally Windows guests were found to require this entry to > > > include two IRQ resources, so this patch adds two semi-arbitrarily > > > chosen ones (7 and 13). This results, in particular, in parallel port > > > conflicting with vmbus. > > > > > > TODO: discover and use spare IRQs to avoid conflicts. > > > > > > Signed-off-by: Evgeny Yakovlev <eyakov...@virtuozzo.com> > > > Signed-off-by: Roman Kagan <rka...@virtuozzo.com> > > > Signed-off-by: Jon Doron <ari...@gmail.com> > > > > Nice work, thanks! > > > > However, it seems to be based on the code version that was posted in > > February 2018, and not the latest version in OpenVZ qemu repository > > dated October 2019: > > https://src.openvz.org/projects/UP/repos/qemu/commits?until=refs%2Fheads%2Fvmbus > > > > This newer version has slightly different API here and there. > > Any particular reason for selecting that older version for porting? > > > > I have actually rebased this latest version on the top of the current > > QEMU master, and it basically seems to work fine. > > However, I haven't done extensive tests whether there isn't a memory leak > > somewhere or so on. > > > > Maciej