Change in vdsm[master]: BZ#732245, BZ#732269 - Handle EAGAIN and EINTR in AsyncProc....

Saggi Mizrahi smizrahi at redhat.com
Thu Sep 1 07:02:27 UTC 2011


Saggi Mizrahi has posted comments on this change.

Change subject: BZ#732245, BZ#732269 - Handle EAGAIN and EINTR in AsyncProc.communicate
......................................................................


Patch Set 2: (1 inline comment)

....................................................
File vdsm/storage/misc.py
Line 865:                     raise
No, I meant none. If poller returns an empty list everything works as designed. EINTR is not an issue. If EAGAIN is happening repeatedly VDSM is going to crash because there is no memory.

--
To view, visit http://gerrit.usersys.redhat.com/865
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2e0f8494826e9cb0debf8a9d3c9a591ac883b376
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Ayal Baron
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eduardo Warszawski <ewarszaw at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Haim Ateya <hateya at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list