Change in vdsm[ovirt-3.4]: netlink: Introduce socket pool

ybronhei at redhat.com ybronhei at redhat.com
Wed Feb 26 16:25:24 UTC 2014


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: netlink: Introduce socket pool
......................................................................


netlink: Introduce socket pool

If a lot of netlink requests happened _nl_connect would fail at
the C level, more concretely when doing bind, which would return
EADDRINUSE.

This patch addresses that issue as well as reduces the the resource
consumption of the module by reusing the netlink sockets. The reusing
is done by a semaphore protected socket pool.

Bug-Url: https://bugzilla.redhat.com/1070061
Change-Id: I657ac3d3e0c2661ce73bdef9aa807ead888a42eb
Signed-off-by: Antoni S. Puimedon <asegurap at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/24603
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/25056
---
M lib/vdsm/netlink.py
1 file changed, 69 insertions(+), 36 deletions(-)

Approvals:
  Antoni Segura Puimedon: Verified
  Dan Kenigsberg: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/25056
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I657ac3d3e0c2661ce73bdef9aa807ead888a42eb
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.4
Gerrit-Owner: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list