<div dir="ltr">Ping - as there was no other review yet for the series except a copy-paste bug spotted (thanks Bjoern)</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Aug 14, 2018 at 11:27 AM Christian Ehrhardt <<a href="mailto:christian.ehrhardt@canonical.com" target="_blank">christian.ehrhardt@canonical.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
after a good discussion a few days ago in<br>
 <a href="https://www.redhat.com/archives/libvir-list/2018-August/msg00122.html" rel="noreferrer" target="_blank">https://www.redhat.com/archives/libvir-list/2018-August/msg00122.html</a><br>
and a short lived but back then untested v2 in<br>
 <a href="https://www.redhat.com/archives/libvir-list/2018-August/msg00199.html" rel="noreferrer" target="_blank">https://www.redhat.com/archives/libvir-list/2018-August/msg00199.html</a><br>
I finally get access to the right HW again and completed the series.<br>
<br>
Being finally retested and working I finally feel safe to submit without<br>
a RFC prefix. I think this would be a great addition for a better handling<br>
of guests with plenty of host devices passed through.<br>
<br>
With the new code in place I can shutdown systems that have 12, 16 or<br>
even more hostdevs attached without getting into the "zombie" mode where<br>
libvirt will forever consider the guest as "in shutdown" as it gave up<br>
waiting too early because the signal zero still was able to reach it.<br>
<br>
Scaling examples (extracted with gdb):<br>
16 Devices: virProcessKillPainfullyDelay (pid=67096, force=true, extradelay=32)<br>
12 Devices: virProcessKillPainfullyDelay (pid=68251, force=true, extradelay=24)<br>
<br>
*Updates in v3*<br>
- fixup some issues found in testing and code checks<br>
<br>
*Updates in v2*<br>
- removed the "accept the lack of /proc/<pid> as valid process removal"<br>
  approach due to valid concerns about reusing ressources.<br>
- added a dynamic extra wait scaling with the amount of hostdevs<br>
<br>
Christian Ehrhardt (2):<br>
  process: wait longer on kill per assigned Hostdev<br>
  process: wait longer 5->30s on hard shutdown<br>
<br>
 src/libvirt_private.syms |  1 +<br>
 src/qemu/qemu_process.c  |  5 +++--<br>
 src/util/virprocess.c    | 20 ++++++++++++++++----<br>
 src/util/virprocess.h    |  1 +<br>
 4 files changed, 21 insertions(+), 6 deletions(-)<br>
<br>
-- <br>
2.17.1<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="m_838240790681031304gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><span style="color:rgb(136,136,136);font-size:12.8px">Christian Ehrhardt</span><div style="color:rgb(136,136,136);font-size:12.8px">Software Engineer, Ubuntu Server</div><div style="color:rgb(136,136,136);font-size:12.8px">Canonical Ltd</div></div></div></div></div>