From b083da401f0cd2a2ef58acf9293e9daca867364c Mon Sep 17 00:00:00 2001
From: YAMADA Hideki <yamada.hideki@po.ntts.co.jp>
Date: Sun, 24 Feb 2013 12:56:22 +0900
Subject: [PATCH] controller: fix that sender thread is never terminated

Empty q.get() is blocking send thread.
Because of this issue, datapath state never transit to DEAD.

This fixes commit 83e3709a0d70f889e794d1c5f70c1c1eb73075f7.
  - controller: fix send_q draining
      https://github.com/osrg/ryu/commit/83e3709a0d70f889e794d1c5f70c1c1eb73075f7

Signed-off-by: YAMADA Hideki <yamada.hideki@po.ntts.co.jp>
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
---
 ryu/controller/controller.py | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/ryu/controller/controller.py b/ryu/controller/controller.py
index 9dc745ad..841b388c 100644
--- a/ryu/controller/controller.py
+++ b/ryu/controller/controller.py
@@ -196,10 +196,7 @@ class Datapath(object):
                 buf = self.send_q.get()
                 self.socket.sendall(buf)
         finally:
-            q = self.send_q
             self.send_q = None
-            while q.get():
-                pass
 
     def send(self, buf):
         if self.send_q: