Dan Kenigsberg has submitted this change and it was merged.
Change subject: virt: Ensure not to handle no longer tracked fds in epoll
......................................................................
virt: Ensure not to handle no longer tracked fds in epoll
In some cases it is possible that events from epoll are received before the
socket is closed. This patch ensures that we're ignoring events which belong
to fds we are not tracking (anymore).
Change-Id: Idde0f89d2859107dd1bb697d1753709137335677
Bug-Url:
https://bugzilla.redhat.com/1102072
Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Reviewed-on:
http://gerrit.ovirt.org/28179
Reviewed-by: Francesco Romani <fromani(a)redhat.com>
Reviewed-by: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Reviewed-by: Gal Hammer <ghammer%redhat.com(a)gtempaccount.com>
Reviewed-by: Dan Kenigsberg <danken(a)redhat.com>
---
M vdsm/virt/vmchannels.py
1 file changed, 13 insertions(+), 9 deletions(-)
Approvals:
Vinzenz Feenstra: Verified
Gal Hammer: Looks good to me, but someone else must approve
Dan Kenigsberg: Looks good to me, approved
Francesco Romani: Looks good to me, but someone else must approve
Michal Skrivanek: Looks good to me, but someone else must approve
--
To view, visit
http://gerrit.ovirt.org/28179
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Idde0f89d2859107dd1bb697d1753709137335677
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Gal Hammer <ghammer%redhat.com(a)gtempaccount.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server