X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=python%2Fovs%2Fstream.py;h=7ea9e46e8b59c7b6d175460ef31a268d5b8f1607;hb=dd1ba5b3f4425c8eba008d1a93b044da63466812;hp=20cd7fe2487827ccbdc3cbf4c75ec007611dad7b;hpb=1f9dd59cf178b0765727f6908e135f48c766e213;p=openvswitch diff --git a/python/ovs/stream.py b/python/ovs/stream.py index 20cd7fe2..7ea9e46e 100644 --- a/python/ovs/stream.py +++ b/python/ovs/stream.py @@ -13,14 +13,16 @@ # limitations under the License. import errno -import logging import os import select import socket -import sys import ovs.poller import ovs.socket_util +import ovs.vlog + +vlog = ovs.vlog.Vlog("stream") + class Stream(object): """Bidirectional byte stream. Currently only Unix domain sockets @@ -107,7 +109,7 @@ class Stream(object): stream.connect_wait(poller) poller.block() assert error != errno.EINPROGRESS - + if error and stream: stream.close() stream = None @@ -146,7 +148,7 @@ class Stream(object): def recv(self, n): """Tries to receive up to 'n' bytes from this stream. Returns a (error, string) tuple: - + - If successful, 'error' is zero and 'string' contains between 1 and 'n' bytes of data. @@ -154,7 +156,7 @@ class Stream(object): - If the connection has been closed in the normal fashion or if 'n' is 0, the tuple is (0, ""). - + The recv function will not block waiting for data to arrive. If no data have been received, it returns (errno.EAGAIN, "") immediately.""" @@ -206,27 +208,25 @@ class Stream(object): if self.state == Stream.__S_CONNECTING: wait = Stream.W_CONNECT - if wait in (Stream.W_CONNECT, Stream.W_SEND): - poller.fd_wait(self.socket, select.POLLOUT) - else: + if wait == Stream.W_RECV: poller.fd_wait(self.socket, select.POLLIN) + else: + poller.fd_wait(self.socket, select.POLLOUT) def connect_wait(self, poller): self.wait(poller, Stream.W_CONNECT) - + def recv_wait(self, poller): self.wait(poller, Stream.W_RECV) - + def send_wait(self, poller): self.wait(poller, Stream.W_SEND) - - def get_name(self): - return self.name - + def __del__(self): # Don't delete the file: we might have forked. self.socket.close() + class PassiveStream(object): @staticmethod def is_valid_name(name): @@ -262,7 +262,7 @@ class PassiveStream(object): try: sock.listen(10) except socket.error, e: - logging.error("%s: listen: %s" % (name, os.strerror(e.error))) + vlog.err("%s: listen: %s" % (name, os.strerror(e.error))) sock.close() return e.error, None @@ -293,7 +293,7 @@ class PassiveStream(object): error = ovs.socket_util.get_exception_errno(e) if error != errno.EAGAIN: # XXX rate-limit - logging.debug("accept: %s" % os.strerror(error)) + vlog.dbg("accept: %s" % os.strerror(error)) return error, None def wait(self, poller): @@ -303,14 +303,11 @@ class PassiveStream(object): # Don't delete the file: we might have forked. self.socket.close() -def usage(name, active, passive, bootstrap): - print - if active: - print("Active %s connection methods:" % name) - print(" unix:FILE " - "Unix domain socket named FILE"); - - if passive: - print("Passive %s connection methods:" % name) - print(" punix:FILE " - "listen on Unix domain socket FILE") + +def usage(name): + return """ +Active %s connection methods: + unix:FILE Unix domain socket named FILE + +Passive %s connection methods: + punix:FILE Listen on Unix domain socket FILE""" % (name, name)