Matteo Scandolo | a229eca | 2017-08-08 13:05:28 -0700 | [diff] [blame] | 1 | |
| 2 | # Copyright 2017-present Open Networking Foundation |
| 3 | # |
| 4 | # Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | # you may not use this file except in compliance with the License. |
| 6 | # You may obtain a copy of the License at |
| 7 | # |
| 8 | # http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | # |
| 10 | # Unless required by applicable law or agreed to in writing, software |
| 11 | # distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | # See the License for the specific language governing permissions and |
| 14 | # limitations under the License. |
| 15 | |
| 16 | |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 17 | """ |
| 18 | OpenFlow Test Framework |
| 19 | |
| 20 | Controller class |
| 21 | |
| 22 | Provide the interface to the control channel to the switch under test. |
| 23 | |
| 24 | Class inherits from thread so as to run in background allowing |
| 25 | asynchronous callbacks (if needed, not required). Also supports |
| 26 | polling. |
| 27 | |
| 28 | The controller thread maintains a queue. Incoming messages that |
| 29 | are not handled by a callback function are placed in this queue for |
| 30 | poll calls. |
| 31 | |
| 32 | Callbacks and polling support specifying the message type |
| 33 | |
| 34 | @todo Support transaction semantics via xid |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 35 | @todo Support select and listen on an administrative socket (or |
| 36 | use a timeout to support clean shutdown). |
| 37 | |
| 38 | Currently only one connection is accepted during the life of |
| 39 | the controller. There seems |
| 40 | to be no clean way to interrupt an accept call. Using select that also listens |
| 41 | on an administrative socket and can shut down the socket might work. |
| 42 | |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 43 | """ |
| 44 | |
Rich Lane | 720eaf2 | 2013-08-09 18:00:45 -0700 | [diff] [blame] | 45 | import sys |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 46 | import os |
| 47 | import socket |
| 48 | import time |
Rich Lane | cd97d3d | 2013-01-07 18:50:06 -0800 | [diff] [blame] | 49 | import struct |
| 50 | import select |
| 51 | import logging |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 52 | from threading import Thread |
| 53 | from threading import Lock |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 54 | from threading import Condition |
Dan Talayco | 4837010 | 2010-03-03 15:17:33 -0800 | [diff] [blame] | 55 | |
Wilson Ng | c11a918 | 2013-10-28 16:02:03 -0700 | [diff] [blame] | 56 | import ofutils |
| 57 | import loxi |
| 58 | |
| 59 | # Configured openflow version |
| 60 | import ofp as cfg_ofp |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 61 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 62 | FILTER = ''.join( [ (len( repr( chr( x ) ) ) == 3) and chr( x ) or '.' |
| 63 | for x in range( 256 ) ] ) |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 64 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 65 | |
| 66 | def hex_dump_buffer( src, length=16 ): |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 67 | """ |
| 68 | Convert src to a hex dump string and return the string |
| 69 | @param src The source buffer |
| 70 | @param length The number of bytes shown in each line |
| 71 | @returns A string showing the hex dump |
| 72 | """ |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 73 | result = [ "\n" ] |
| 74 | for i in xrange( 0, len( src ), length ): |
| 75 | chars = src[ i:i + length ] |
| 76 | hex = ' '.join( [ "%02x" % ord( x ) for x in chars ] ) |
| 77 | printable = ''.join( [ "%s" % ((ord( x ) <= 127 and |
| 78 | FILTER[ ord( x ) ]) or '.') for x in |
| 79 | chars ] ) |
| 80 | result.append( "%04x %-*s %s\n" % (i, length * 3, hex, printable) ) |
| 81 | return ''.join( result ) |
| 82 | |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 83 | |
Dan Talayco | 4837010 | 2010-03-03 15:17:33 -0800 | [diff] [blame] | 84 | ##@todo Find a better home for these identifiers (controller) |
Glen Gibb | 741b118 | 2010-07-08 16:43:58 -0700 | [diff] [blame] | 85 | RCV_SIZE_DEFAULT = 32768 |
Dan Talayco | 4837010 | 2010-03-03 15:17:33 -0800 | [diff] [blame] | 86 | LISTEN_QUEUE_SIZE = 1 |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 87 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 88 | |
| 89 | class Controller( Thread ): |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 90 | """ |
| 91 | Class abstracting the control interface to the switch. |
| 92 | |
| 93 | For receiving messages, two mechanism will be implemented. First, |
| 94 | query the interface with poll. Second, register to have a |
| 95 | function called by message type. The callback is passed the |
| 96 | message type as well as the raw packet (or message object) |
| 97 | |
| 98 | One of the main purposes of this object is to translate between network |
| 99 | and host byte order. 'Above' this object, things should be in host |
| 100 | byte order. |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 101 | |
| 102 | @todo Consider using SocketServer for listening socket |
| 103 | @todo Test transaction code |
| 104 | |
| 105 | @var rcv_size The receive size to use for receive calls |
| 106 | @var max_pkts The max size of the receive queue |
| 107 | @var keep_alive If true, listen for echo requests and respond w/ |
| 108 | echo replies |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 109 | @var initial_hello If true, will send a hello message immediately |
| 110 | upon connecting to the switch |
Dan Talayco | 69ca4d6 | 2012-11-15 11:50:22 -0800 | [diff] [blame] | 111 | @var switch If not None, do an active connection to the switch |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 112 | @var host The host to use for connect |
| 113 | @var port The port to connect on |
| 114 | @var packets_total Total number of packets received |
| 115 | @var packets_expired Number of packets popped from queue as queue full |
| 116 | @var packets_handled Number of packets handled by something |
Dan Talayco | d7e2dbe | 2010-02-13 21:51:15 -0800 | [diff] [blame] | 117 | @var dbg_state Debug indication of state |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 118 | """ |
| 119 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 120 | def __init__( self, switch=None, host='127.0.0.1', port=6653, max_pkts=1024, |
| 121 | force=False ): |
| 122 | Thread.__init__( self ) |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 123 | # Socket related |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 124 | self.rcv_size = RCV_SIZE_DEFAULT |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 125 | self.listen_socket = None |
| 126 | self.switch_socket = None |
| 127 | self.switch_addr = None |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 128 | self.connect_cv = Condition( ) |
| 129 | self.message_cv = Condition( ) |
| 130 | self.tx_lock = Lock( ) |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 131 | |
Rich Lane | 4dfd5e1 | 2012-12-22 19:48:01 -0800 | [diff] [blame] | 132 | # Used to wake up the event loop from another thread |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 133 | self.waker = ofutils.EventDescriptor( ) |
Rich Lane | 3279754 | 2012-12-22 17:46:05 -0800 | [diff] [blame] | 134 | |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 135 | # Counters |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 136 | self.socket_errors = 0 |
| 137 | self.parse_errors = 0 |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 138 | self.packets_total = 0 |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 139 | self.packets_expired = 0 |
| 140 | self.packets_handled = 0 |
Dan Talayco | e226eb1 | 2010-02-18 23:06:30 -0800 | [diff] [blame] | 141 | self.poll_discards = 0 |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 142 | |
| 143 | # State |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 144 | self.sync = Lock( ) |
| 145 | self.handlers = { } |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 146 | self.keep_alive = False |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 147 | self.active = True |
| 148 | self.initial_hello = True |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 149 | |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 150 | # OpenFlow message/packet queue |
| 151 | # Protected by the packets_cv lock / condition variable |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 152 | self.packets = [ ] |
| 153 | self.packets_cv = Condition( ) |
Dan Talayco | dd6b6ff | 2013-04-12 08:20:18 -0700 | [diff] [blame] | 154 | self.packet_in_count = 0 |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 155 | |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 156 | # Settings |
| 157 | self.max_pkts = max_pkts |
Dan Talayco | 69ca4d6 | 2012-11-15 11:50:22 -0800 | [diff] [blame] | 158 | self.switch = switch |
| 159 | self.passive = not self.switch |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 160 | self.force = force |
Dan Talayco | 4837010 | 2010-03-03 15:17:33 -0800 | [diff] [blame] | 161 | self.host = host |
| 162 | self.port = port |
Dan Talayco | d7e2dbe | 2010-02-13 21:51:15 -0800 | [diff] [blame] | 163 | self.dbg_state = "init" |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 164 | self.logger = logging.getLogger( "controller" ) |
| 165 | self.filter_packet_in = False # Drop "excessive" packet ins |
| 166 | self.pkt_in_run = 0 # Count on run of packet ins |
| 167 | self.pkt_in_filter_limit = 50 # Count on run of packet ins |
| 168 | self.pkt_in_dropped = 0 # Total dropped packet ins |
| 169 | self.transact_to = 15 # Transact timeout default value; add to config |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 170 | |
Dan Talayco | e226eb1 | 2010-02-18 23:06:30 -0800 | [diff] [blame] | 171 | # Transaction and message type waiting variables |
| 172 | # xid_cv: Condition variable (semaphore) for packet waiters |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 173 | # xid: Transaction ID being waited on |
| 174 | # xid_response: Transaction response message |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 175 | self.xid_cv = Condition( ) |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 176 | self.xid = None |
| 177 | self.xid_response = None |
| 178 | |
Rob Sherwood | e3e452a | 2012-03-06 09:24:26 -0800 | [diff] [blame] | 179 | self.buffered_input = "" |
Dan Talayco | e226eb1 | 2010-02-18 23:06:30 -0800 | [diff] [blame] | 180 | |
Rich Lane | 207502e | 2012-12-31 14:29:12 -0800 | [diff] [blame] | 181 | # Create listen socket |
| 182 | if self.passive: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 183 | self.logger.info( "Create/listen at " + self.host + ":" + |
| 184 | str( self.port ) ) |
| 185 | ai = socket.getaddrinfo( self.host, self.port, socket.AF_UNSPEC, |
| 186 | socket.SOCK_STREAM, 0, socket.AI_PASSIVE ) |
Ken Chiang | bf84c33 | 2015-01-27 12:52:27 -0800 | [diff] [blame] | 187 | # Use first returned addrinfo |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 188 | (family, socktype, proto, name, sockaddr) = ai[ 0 ] |
| 189 | self.listen_socket = socket.socket( family, socktype ) |
| 190 | self.listen_socket.setsockopt( socket.SOL_SOCKET, |
| 191 | socket.SO_REUSEADDR, 1 ) |
| 192 | self.listen_socket.bind( sockaddr ) |
| 193 | self.listen_socket.listen( LISTEN_QUEUE_SIZE ) |
Rich Lane | 207502e | 2012-12-31 14:29:12 -0800 | [diff] [blame] | 194 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 195 | def filter_packet( self, rawmsg, hdr ): |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 196 | """ |
| 197 | Check if packet should be filtered |
| 198 | |
| 199 | Currently filters packet in messages |
| 200 | @return Boolean, True if packet should be dropped |
| 201 | """ |
Rich Lane | 1622bbb | 2013-03-11 17:11:53 -0700 | [diff] [blame] | 202 | # XXX didn't actually check for packet-in... |
| 203 | return False |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 204 | # Add check for packet in and rate limit |
| 205 | if self.filter_packet_in: |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 206 | # If we were dropping packets, report number dropped |
| 207 | # TODO dont drop expected packet ins |
| 208 | if self.pkt_in_run > self.pkt_in_filter_limit: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 209 | self.logger.debug( "Dropped %d packet ins (%d total)" |
| 210 | % ((self.pkt_in_run - |
| 211 | self.pkt_in_filter_limit), |
| 212 | self.pkt_in_dropped) ) |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 213 | self.pkt_in_run = 0 |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 214 | |
| 215 | return False |
| 216 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 217 | def _pkt_handle( self, pkt ): |
Dan Talayco | d12b661 | 2010-03-07 22:00:46 -0800 | [diff] [blame] | 218 | """ |
| 219 | Check for all packet handling conditions |
| 220 | |
| 221 | Parse and verify message |
| 222 | Check if XID matches something waiting |
| 223 | Check if message is being expected for a poll operation |
| 224 | Check if keep alive is on and message is an echo request |
| 225 | Check if any registered handler wants the packet |
| 226 | Enqueue if none of those conditions is met |
| 227 | |
| 228 | an echo request in case keep_alive is true, followed by |
| 229 | registered message handlers. |
Glen Gibb | 6d46706 | 2010-07-08 16:15:08 -0700 | [diff] [blame] | 230 | @param pkt The raw packet (string) which may contain multiple OF msgs |
Dan Talayco | d12b661 | 2010-03-07 22:00:46 -0800 | [diff] [blame] | 231 | """ |
Rob Sherwood | e3e452a | 2012-03-06 09:24:26 -0800 | [diff] [blame] | 232 | |
| 233 | # snag any left over data from last read() |
| 234 | pkt = self.buffered_input + pkt |
| 235 | self.buffered_input = "" |
| 236 | |
Glen Gibb | 6d46706 | 2010-07-08 16:15:08 -0700 | [diff] [blame] | 237 | # Process each of the OF msgs inside the pkt |
| 238 | offset = 0 |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 239 | while offset < len( pkt ): |
| 240 | if offset + 8 > len( pkt ): |
Rich Lane | 1622bbb | 2013-03-11 17:11:53 -0700 | [diff] [blame] | 241 | break |
| 242 | |
Glen Gibb | 6d46706 | 2010-07-08 16:15:08 -0700 | [diff] [blame] | 243 | # Parse the header to get type |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 244 | hdr_version, hdr_type, hdr_length, hdr_xid = cfg_ofp.message.parse_header( |
| 245 | pkt[ offset: ] ) |
Wilson Ng | c11a918 | 2013-10-28 16:02:03 -0700 | [diff] [blame] | 246 | |
| 247 | # Use loxi to resolve to ofp of matching version |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 248 | ofp = loxi.protocol( hdr_version ) |
Dan Talayco | d12b661 | 2010-03-07 22:00:46 -0800 | [diff] [blame] | 249 | |
Glen Gibb | 6d46706 | 2010-07-08 16:15:08 -0700 | [diff] [blame] | 250 | # Extract the raw message bytes |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 251 | if (offset + hdr_length) > len( pkt ): |
Rob Sherwood | e3e452a | 2012-03-06 09:24:26 -0800 | [diff] [blame] | 252 | break |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 253 | rawmsg = pkt[ offset: offset + hdr_length ] |
Rich Lane | 1622bbb | 2013-03-11 17:11:53 -0700 | [diff] [blame] | 254 | offset += hdr_length |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 255 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 256 | # if self.filter_packet(rawmsg, hdr): |
Rich Lane | 1622bbb | 2013-03-11 17:11:53 -0700 | [diff] [blame] | 257 | # continue |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 258 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 259 | msg = ofp.message.parse_message( rawmsg ) |
Glen Gibb | 6d46706 | 2010-07-08 16:15:08 -0700 | [diff] [blame] | 260 | if not msg: |
| 261 | self.parse_errors += 1 |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 262 | self.logger.warn( "Could not parse message" ) |
Glen Gibb | 6d46706 | 2010-07-08 16:15:08 -0700 | [diff] [blame] | 263 | continue |
| 264 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 265 | self.logger.debug( "Msg in: version %d class %s len %d xid %d", |
| 266 | hdr_version, type( msg ).__name__, hdr_length, |
| 267 | hdr_xid ) |
Rich Lane | 1fd43e3 | 2014-01-06 15:22:50 -0800 | [diff] [blame] | 268 | |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 269 | with self.sync: |
| 270 | # Check if transaction is waiting |
| 271 | with self.xid_cv: |
Rich Lane | 1622bbb | 2013-03-11 17:11:53 -0700 | [diff] [blame] | 272 | if self.xid and hdr_xid == self.xid: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 273 | self.logger.debug( |
| 274 | "Matched expected XID " + str( hdr_xid ) ) |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 275 | self.xid_response = (msg, rawmsg) |
| 276 | self.xid = None |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 277 | self.xid_cv.notify( ) |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 278 | continue |
Glen Gibb | 6d46706 | 2010-07-08 16:15:08 -0700 | [diff] [blame] | 279 | |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 280 | # Check if keep alive is set; if so, respond to echo requests |
| 281 | if self.keep_alive: |
Rich Lane | 1622bbb | 2013-03-11 17:11:53 -0700 | [diff] [blame] | 282 | if hdr_type == ofp.OFPT_ECHO_REQUEST: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 283 | self.logger.debug( "Responding to echo request" ) |
| 284 | rep = ofp.message.echo_reply( ) |
Rich Lane | 1622bbb | 2013-03-11 17:11:53 -0700 | [diff] [blame] | 285 | rep.xid = hdr_xid |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 286 | # Ignoring additional data |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 287 | self.message_send( rep ) |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 288 | continue |
Glen Gibb | 6d46706 | 2010-07-08 16:15:08 -0700 | [diff] [blame] | 289 | |
Dan Talayco | dd6b6ff | 2013-04-12 08:20:18 -0700 | [diff] [blame] | 290 | # Generalize to counters for all packet types? |
| 291 | if msg.type == ofp.OFPT_PACKET_IN: |
| 292 | self.packet_in_count += 1 |
| 293 | |
Rich Lane | 5d63b9c | 2013-01-11 14:12:37 -0800 | [diff] [blame] | 294 | # Log error messages |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 295 | if isinstance( msg, ofp.message.error_msg ): |
| 296 | # pylint: disable=E1103 |
Rich Lane | b73808c | 2013-03-11 15:22:23 -0700 | [diff] [blame] | 297 | if msg.err_type in ofp.ofp_error_type_map: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 298 | type_str = ofp.ofp_error_type_map[ msg.err_type ] |
Rich Lane | b73808c | 2013-03-11 15:22:23 -0700 | [diff] [blame] | 299 | if msg.err_type == ofp.OFPET_HELLO_FAILED: |
Rich Lane | 5d63b9c | 2013-01-11 14:12:37 -0800 | [diff] [blame] | 300 | code_map = ofp.ofp_hello_failed_code_map |
Rich Lane | b73808c | 2013-03-11 15:22:23 -0700 | [diff] [blame] | 301 | elif msg.err_type == ofp.OFPET_BAD_REQUEST: |
Rich Lane | 5d63b9c | 2013-01-11 14:12:37 -0800 | [diff] [blame] | 302 | code_map = ofp.ofp_bad_request_code_map |
Rich Lane | b73808c | 2013-03-11 15:22:23 -0700 | [diff] [blame] | 303 | elif msg.err_type == ofp.OFPET_BAD_ACTION: |
Rich Lane | 5d63b9c | 2013-01-11 14:12:37 -0800 | [diff] [blame] | 304 | code_map = ofp.ofp_bad_action_code_map |
Rich Lane | b73808c | 2013-03-11 15:22:23 -0700 | [diff] [blame] | 305 | elif msg.err_type == ofp.OFPET_FLOW_MOD_FAILED: |
Rich Lane | 5d63b9c | 2013-01-11 14:12:37 -0800 | [diff] [blame] | 306 | code_map = ofp.ofp_flow_mod_failed_code_map |
Rich Lane | b73808c | 2013-03-11 15:22:23 -0700 | [diff] [blame] | 307 | elif msg.err_type == ofp.OFPET_PORT_MOD_FAILED: |
Rich Lane | 5d63b9c | 2013-01-11 14:12:37 -0800 | [diff] [blame] | 308 | code_map = ofp.ofp_port_mod_failed_code_map |
Rich Lane | b73808c | 2013-03-11 15:22:23 -0700 | [diff] [blame] | 309 | elif msg.err_type == ofp.OFPET_QUEUE_OP_FAILED: |
Rich Lane | 5d63b9c | 2013-01-11 14:12:37 -0800 | [diff] [blame] | 310 | code_map = ofp.ofp_queue_op_failed_code_map |
| 311 | else: |
| 312 | code_map = None |
| 313 | |
| 314 | if code_map and msg.code in code_map: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 315 | code_str = code_map[ msg.code ] |
Rich Lane | 5d63b9c | 2013-01-11 14:12:37 -0800 | [diff] [blame] | 316 | else: |
| 317 | code_str = "unknown" |
| 318 | else: |
| 319 | type_str = "unknown" |
Rich Lane | 1879dc7 | 2013-03-11 22:08:51 -0700 | [diff] [blame] | 320 | code_str = "unknown" |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 321 | self.logger.warn( |
| 322 | "Received error message: xid=%d type=%s (%d) code=%s (%d)", |
| 323 | hdr_xid, type_str, msg.err_type, code_str, |
| 324 | msg.code ) |
Rich Lane | 5d63b9c | 2013-01-11 14:12:37 -0800 | [diff] [blame] | 325 | |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 326 | # Now check for message handlers; preference is given to |
| 327 | # handlers for a specific packet |
| 328 | handled = False |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 329 | if hdr_type in self.handlers.keys( ): |
| 330 | handled = self.handlers[ hdr_type ]( self, msg, rawmsg ) |
| 331 | if not handled and ("all" in self.handlers.keys( )): |
| 332 | handled = self.handlers[ "all" ]( self, msg, rawmsg ) |
Glen Gibb | 6d46706 | 2010-07-08 16:15:08 -0700 | [diff] [blame] | 333 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 334 | if not handled: # Not handled, enqueue |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 335 | with self.packets_cv: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 336 | if len( self.packets ) >= self.max_pkts: |
| 337 | self.packets.pop( 0 ) |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 338 | self.packets_expired += 1 |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 339 | self.packets.append( (msg, rawmsg) ) |
| 340 | self.packets_cv.notify_all( ) |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 341 | self.packets_total += 1 |
| 342 | else: |
| 343 | self.packets_handled += 1 |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 344 | self.logger.debug( "Message handled by callback" ) |
Glen Gibb | 6d46706 | 2010-07-08 16:15:08 -0700 | [diff] [blame] | 345 | |
Rob Sherwood | e3e452a | 2012-03-06 09:24:26 -0800 | [diff] [blame] | 346 | # end of 'while offset < len(pkt)' |
| 347 | # note that if offset = len(pkt), this is |
| 348 | # appends a harmless empty string |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 349 | self.buffered_input += pkt[ offset: ] |
Dan Talayco | d12b661 | 2010-03-07 22:00:46 -0800 | [diff] [blame] | 350 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 351 | def _socket_ready_handle( self, s ): |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 352 | """ |
| 353 | Handle an input-ready socket |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 354 | |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 355 | @param s The socket object that is ready |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 356 | @returns 0 on success, -1 on error |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 357 | """ |
| 358 | |
Dan Talayco | 69ca4d6 | 2012-11-15 11:50:22 -0800 | [diff] [blame] | 359 | if self.passive and s and s == self.listen_socket: |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 360 | if self.switch_socket: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 361 | self.logger.warning( |
| 362 | "Ignoring incoming connection; already connected to switch" ) |
| 363 | (sock, addr) = self.listen_socket.accept( ) |
| 364 | sock.close( ) |
Rich Lane | e1da7ea | 2012-07-26 15:58:45 -0700 | [diff] [blame] | 365 | return 0 |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 366 | |
Ken Chiang | e875baf | 2012-10-09 15:24:40 -0700 | [diff] [blame] | 367 | try: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 368 | (sock, addr) = self.listen_socket.accept( ) |
Ken Chiang | e875baf | 2012-10-09 15:24:40 -0700 | [diff] [blame] | 369 | except: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 370 | self.logger.warning( "Error on listen socket accept" ) |
Ken Chiang | e875baf | 2012-10-09 15:24:40 -0700 | [diff] [blame] | 371 | return -1 |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 372 | self.logger.info( self.host + ":" + str( |
| 373 | self.port ) + ": Incoming connection from " + str( addr ) ) |
Rich Lane | e1da7ea | 2012-07-26 15:58:45 -0700 | [diff] [blame] | 374 | |
Rich Lane | ee3586c | 2012-07-11 17:26:02 -0700 | [diff] [blame] | 375 | with self.connect_cv: |
Rich Lane | e1da7ea | 2012-07-26 15:58:45 -0700 | [diff] [blame] | 376 | (self.switch_socket, self.switch_addr) = (sock, addr) |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 377 | self.switch_socket.setsockopt( socket.IPPROTO_TCP, |
| 378 | socket.TCP_NODELAY, True ) |
Rich Lane | 1a8d5aa | 2012-10-08 15:40:03 -0700 | [diff] [blame] | 379 | if self.initial_hello: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 380 | self.message_send( cfg_ofp.message.hello( ) ) |
| 381 | self.connect_cv.notify( ) # Notify anyone waiting |
Rich Lane | d929b8d | 2013-04-15 15:59:14 -0700 | [diff] [blame] | 382 | |
| 383 | # Prevent further connections |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 384 | self.listen_socket.close( ) |
Rich Lane | d929b8d | 2013-04-15 15:59:14 -0700 | [diff] [blame] | 385 | self.listen_socket = None |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 386 | elif s and s == self.switch_socket: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 387 | for idx in range( 3 ): # debug: try a couple of times |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 388 | try: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 389 | pkt = self.switch_socket.recv( self.rcv_size ) |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 390 | except: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 391 | self.logger.warning( "Error on switch read" ) |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 392 | return -1 |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 393 | |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 394 | if not self.active: |
| 395 | return 0 |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 396 | |
| 397 | if len( pkt ) == 0: |
| 398 | self.logger.warning( "Zero-length switch read, %d" % idx ) |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 399 | else: |
| 400 | break |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 401 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 402 | if len( pkt ) == 0: # Still no packet |
| 403 | self.logger.warning( "Zero-length switch read; closing cxn" ) |
| 404 | self.logger.info( str( self ) ) |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 405 | return -1 |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 406 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 407 | self._pkt_handle( pkt ) |
Rich Lane | 4dfd5e1 | 2012-12-22 19:48:01 -0800 | [diff] [blame] | 408 | elif s and s == self.waker: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 409 | self.waker.wait( ) |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 410 | else: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 411 | self.logger.error( "Unknown socket ready: " + str( s ) ) |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 412 | return -1 |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 413 | |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 414 | return 0 |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 415 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 416 | def active_connect( self ): |
Dan Talayco | 69ca4d6 | 2012-11-15 11:50:22 -0800 | [diff] [blame] | 417 | """ |
| 418 | Actively connect to a switch IP addr |
| 419 | """ |
| 420 | try: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 421 | self.logger.info( "Trying active connection to %s" % self.switch ) |
| 422 | soc = socket.socket( socket.AF_INET, socket.SOCK_STREAM ) |
| 423 | soc.connect( (self.switch, self.port) ) |
| 424 | self.logger.info( "Connected to " + self.switch + " on " + |
| 425 | str( self.port ) ) |
| 426 | soc.setsockopt( socket.IPPROTO_TCP, socket.TCP_NODELAY, True ) |
Dan Talayco | 69ca4d6 | 2012-11-15 11:50:22 -0800 | [diff] [blame] | 427 | self.switch_addr = (self.switch, self.port) |
| 428 | return soc |
| 429 | except (StandardError, socket.error), e: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 430 | self.logger.error( "Could not connect to %s at %d:: %s" % |
| 431 | (self.switch, self.port, str( e )) ) |
Dan Talayco | 69ca4d6 | 2012-11-15 11:50:22 -0800 | [diff] [blame] | 432 | return None |
| 433 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 434 | def wakeup( self ): |
Rich Lane | 3279754 | 2012-12-22 17:46:05 -0800 | [diff] [blame] | 435 | """ |
| 436 | Wake up the event loop, presumably from another thread. |
| 437 | """ |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 438 | self.waker.notify( ) |
Rich Lane | 3279754 | 2012-12-22 17:46:05 -0800 | [diff] [blame] | 439 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 440 | def sockets( self ): |
Rich Lane | 3279754 | 2012-12-22 17:46:05 -0800 | [diff] [blame] | 441 | """ |
| 442 | Return list of sockets to select on. |
| 443 | """ |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 444 | socs = [ self.listen_socket, self.switch_socket, self.waker ] |
| 445 | return [ x for x in socs if x ] |
Rich Lane | 3279754 | 2012-12-22 17:46:05 -0800 | [diff] [blame] | 446 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 447 | def run( self ): |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 448 | """ |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 449 | Activity function for class |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 450 | |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 451 | Assumes connection to switch already exists. Listens on |
| 452 | switch_socket for messages until an error (or zero len pkt) |
| 453 | occurs. |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 454 | |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 455 | When there is a message on the socket, check for handlers; queue the |
| 456 | packet if no one handles the packet. |
| 457 | |
| 458 | See note for controller describing the limitation of a single |
| 459 | connection for now. |
| 460 | """ |
| 461 | |
Rich Lane | 207502e | 2012-12-31 14:29:12 -0800 | [diff] [blame] | 462 | self.dbg_state = "running" |
Ken Chiang | adc950f | 2012-10-05 13:50:03 -0700 | [diff] [blame] | 463 | |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 464 | while self.active: |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 465 | try: |
| 466 | sel_in, sel_out, sel_err = \ |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 467 | select.select( self.sockets( ), [ ], self.sockets( ), 1 ) |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 468 | except: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 469 | print sys.exc_info( ) |
| 470 | self.logger.error( "Select error, disconnecting" ) |
| 471 | self.disconnect( ) |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 472 | |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 473 | for s in sel_err: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 474 | self.logger.error( |
| 475 | "Got socket error on: " + str( s ) + ", disconnecting" ) |
| 476 | self.disconnect( ) |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 477 | |
| 478 | for s in sel_in: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 479 | if self._socket_ready_handle( s ) == -1: |
| 480 | self.disconnect( ) |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 481 | |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 482 | # End of main loop |
| 483 | self.dbg_state = "closing" |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 484 | self.logger.info( "Exiting controller thread" ) |
| 485 | self.shutdown( ) |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 486 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 487 | def connect( self, timeout=-1 ): |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 488 | """ |
| 489 | Connect to the switch |
| 490 | |
Rich Lane | 8806bc4 | 2012-07-26 19:18:37 -0700 | [diff] [blame] | 491 | @param timeout Block for up to timeout seconds. Pass -1 for the default. |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 492 | @return Boolean, True if connected |
| 493 | """ |
| 494 | |
Dan Talayco | 69ca4d6 | 2012-11-15 11:50:22 -0800 | [diff] [blame] | 495 | if not self.passive: # Do active connection now |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 496 | self.logger.info( "Attempting to connect to %s on port %s" % |
| 497 | (self.switch, str( self.port )) ) |
| 498 | soc = self.active_connect( ) |
Dan Talayco | 69ca4d6 | 2012-11-15 11:50:22 -0800 | [diff] [blame] | 499 | if soc: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 500 | self.logger.info( "Connected to %s", self.switch ) |
Dan Talayco | 69ca4d6 | 2012-11-15 11:50:22 -0800 | [diff] [blame] | 501 | self.dbg_state = "running" |
| 502 | self.switch_socket = soc |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 503 | self.wakeup( ) |
Dan Talayco | 69ca4d6 | 2012-11-15 11:50:22 -0800 | [diff] [blame] | 504 | with self.connect_cv: |
| 505 | if self.initial_hello: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 506 | self.message_send( cfg_ofp.message.hello( ) ) |
| 507 | self.connect_cv.notify( ) # Notify anyone waiting |
Dan Talayco | 69ca4d6 | 2012-11-15 11:50:22 -0800 | [diff] [blame] | 508 | else: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 509 | self.logger.error( "Could not actively connect to switch %s", |
| 510 | self.switch ) |
Dan Talayco | 69ca4d6 | 2012-11-15 11:50:22 -0800 | [diff] [blame] | 511 | self.active = False |
| 512 | else: |
| 513 | with self.connect_cv: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 514 | ofutils.timed_wait( self.connect_cv, lambda: self.switch_socket, |
| 515 | timeout=timeout ) |
Dan Talayco | 69ca4d6 | 2012-11-15 11:50:22 -0800 | [diff] [blame] | 516 | |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 517 | return self.switch_socket is not None |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 518 | |
| 519 | def disconnect( self, timeout=-1 ): |
Ken Chiang | adc950f | 2012-10-05 13:50:03 -0700 | [diff] [blame] | 520 | """ |
| 521 | If connected to a switch, disconnect. |
| 522 | """ |
| 523 | if self.switch_socket: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 524 | self.switch_socket.close( ) |
Ken Chiang | adc950f | 2012-10-05 13:50:03 -0700 | [diff] [blame] | 525 | self.switch_socket = None |
| 526 | self.switch_addr = None |
Ken Chiang | 74be472 | 2012-12-21 13:07:03 -0800 | [diff] [blame] | 527 | with self.packets_cv: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 528 | self.packets = [ ] |
Ken Chiang | e875baf | 2012-10-09 15:24:40 -0700 | [diff] [blame] | 529 | with self.connect_cv: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 530 | self.connect_cv.notifyAll( ) |
Ken Chiang | adc950f | 2012-10-05 13:50:03 -0700 | [diff] [blame] | 531 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 532 | def wait_disconnected( self, timeout=-1 ): |
Ken Chiang | adc950f | 2012-10-05 13:50:03 -0700 | [diff] [blame] | 533 | """ |
| 534 | @param timeout Block for up to timeout seconds. Pass -1 for the default. |
| 535 | @return Boolean, True if disconnected |
| 536 | """ |
| 537 | |
Ken Chiang | e875baf | 2012-10-09 15:24:40 -0700 | [diff] [blame] | 538 | with self.connect_cv: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 539 | ofutils.timed_wait( self.connect_cv, |
| 540 | lambda: True if not self.switch_socket else None, |
| 541 | timeout=timeout ) |
Ken Chiang | adc950f | 2012-10-05 13:50:03 -0700 | [diff] [blame] | 542 | return self.switch_socket is None |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 543 | |
| 544 | def kill( self ): |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 545 | """ |
| 546 | Force the controller thread to quit |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 547 | """ |
| 548 | self.active = False |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 549 | self.wakeup( ) |
| 550 | self.join( ) |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 551 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 552 | def shutdown( self ): |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 553 | """ |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 554 | Shutdown the controller closing all sockets |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 555 | |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 556 | @todo Might want to synchronize shutdown with self.sync... |
| 557 | """ |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 558 | |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 559 | self.active = False |
| 560 | try: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 561 | self.switch_socket.shutdown( socket.SHUT_RDWR ) |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 562 | except: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 563 | self.logger.info( "Ignoring switch soc shutdown error" ) |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 564 | self.switch_socket = None |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 565 | |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 566 | try: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 567 | self.listen_socket.shutdown( socket.SHUT_RDWR ) |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 568 | except: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 569 | self.logger.info( "Ignoring listen soc shutdown error" ) |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 570 | self.listen_socket = None |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 571 | |
Rich Lane | ee3586c | 2012-07-11 17:26:02 -0700 | [diff] [blame] | 572 | # Wakeup condition variables on which controller may be wait |
| 573 | with self.xid_cv: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 574 | self.xid_cv.notifyAll( ) |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 575 | |
Rich Lane | ee3586c | 2012-07-11 17:26:02 -0700 | [diff] [blame] | 576 | with self.connect_cv: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 577 | self.connect_cv.notifyAll( ) |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 578 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 579 | self.wakeup( ) |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 580 | self.dbg_state = "down" |
| 581 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 582 | def register( self, msg_type, handler ): |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 583 | """ |
| 584 | Register a callback to receive a specific message type. |
| 585 | |
| 586 | Only one handler may be registered for a given message type. |
Dan Talayco | d12b661 | 2010-03-07 22:00:46 -0800 | [diff] [blame] | 587 | |
| 588 | WARNING: A lock is held during the handler call back, so |
| 589 | the handler should not make any blocking calls |
| 590 | |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 591 | @param msg_type The type of message to receive. May be DEFAULT |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 592 | for all non-handled packets. The special type, the string "all" |
| 593 | will send all packets to the handler. |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 594 | @param handler The function to call when a message of the given |
| 595 | type is received. |
| 596 | """ |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 597 | # Should check type is valid |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 598 | if not handler and msg_type in self.handlers.keys( ): |
| 599 | del self.handlers[ msg_type ] |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 600 | return |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 601 | self.handlers[ msg_type ] = handler |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 602 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 603 | def poll( self, exp_msg=None, timeout=-1 ): |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 604 | """ |
| 605 | Wait for the next OF message received from the switch. |
| 606 | |
| 607 | @param exp_msg If set, return only when this type of message |
Dan Talayco | 4837010 | 2010-03-03 15:17:33 -0800 | [diff] [blame] | 608 | is received (unless timeout occurs). |
Rich Lane | b64ce3d | 2012-07-26 15:37:57 -0700 | [diff] [blame] | 609 | |
| 610 | @param timeout Maximum number of seconds to wait for the message. |
| 611 | Pass -1 for the default timeout. |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 612 | |
Dan Talayco | d7e2dbe | 2010-02-13 21:51:15 -0800 | [diff] [blame] | 613 | @retval A pair (msg, pkt) where msg is a message object and pkt |
| 614 | the string representing the packet as received from the socket. |
| 615 | This allows additional parsing by the receiver if necessary. |
| 616 | |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 617 | The data members in the message are in host endian order. |
Dan Talayco | 4837010 | 2010-03-03 15:17:33 -0800 | [diff] [blame] | 618 | If an error occurs, (None, None) is returned |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 619 | """ |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 620 | |
Rich Lane | e9d3691 | 2014-01-31 12:46:05 -0800 | [diff] [blame] | 621 | if exp_msg is None: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 622 | self.logger.warn( "DEPRECATED polling for any message class" ) |
Rich Lane | e9d3691 | 2014-01-31 12:46:05 -0800 | [diff] [blame] | 623 | klass = None |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 624 | elif isinstance( exp_msg, int ): |
| 625 | klass = cfg_ofp.message.message.subtypes[ exp_msg ] |
| 626 | elif issubclass( exp_msg, loxi.OFObject ): |
Rich Lane | e9d3691 | 2014-01-31 12:46:05 -0800 | [diff] [blame] | 627 | klass = exp_msg |
Ed Swierk | 9e55e28 | 2012-08-22 06:57:28 -0700 | [diff] [blame] | 628 | else: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 629 | raise ValueError( "Unexpected exp_msg argument %r" % exp_msg ) |
Rich Lane | e9d3691 | 2014-01-31 12:46:05 -0800 | [diff] [blame] | 630 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 631 | self.logger.debug( "Polling for %s", klass.__name__ ) |
Rich Lane | b64ce3d | 2012-07-26 15:37:57 -0700 | [diff] [blame] | 632 | |
| 633 | # Take the packet from the queue |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 634 | def grab( ): |
| 635 | for i, (msg, pkt) in enumerate( self.packets ): |
| 636 | if klass is None or isinstance( msg, klass ): |
| 637 | self.logger.debug( "Got %s message", |
| 638 | msg.__class__.__name__ ) |
| 639 | return self.packets.pop( i ) |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 640 | # Not found |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 641 | self.logger.debug( "%s message not in queue", klass.__name__ ) |
Rich Lane | b64ce3d | 2012-07-26 15:37:57 -0700 | [diff] [blame] | 642 | return None |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 643 | |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 644 | with self.packets_cv: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 645 | ret = ofutils.timed_wait( self.packets_cv, grab, timeout=timeout ) |
Rich Lane | c4f071b | 2012-07-11 17:25:57 -0700 | [diff] [blame] | 646 | |
Rich Lane | b64ce3d | 2012-07-26 15:37:57 -0700 | [diff] [blame] | 647 | if ret != None: |
| 648 | (msg, pkt) = ret |
Rich Lane | b64ce3d | 2012-07-26 15:37:57 -0700 | [diff] [blame] | 649 | return (msg, pkt) |
| 650 | else: |
| 651 | return (None, None) |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 652 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 653 | def transact( self, msg, timeout=-1 ): |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 654 | """ |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 655 | Run a message transaction with the switch |
Dan Talayco | e37999f | 2010-02-09 15:27:12 -0800 | [diff] [blame] | 656 | |
| 657 | Send the message in msg and wait for a reply with a matching |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 658 | transaction id. Transactions have the highest priority in |
| 659 | received message handling. |
Dan Talayco | e37999f | 2010-02-09 15:27:12 -0800 | [diff] [blame] | 660 | |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 661 | @param msg The message object to send; must not be a string |
Rich Lane | e1da7ea | 2012-07-26 15:58:45 -0700 | [diff] [blame] | 662 | @param timeout The timeout in seconds; if -1 use default. |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 663 | """ |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 664 | |
Rich Lane | 8fbfd66 | 2013-03-11 15:30:44 -0700 | [diff] [blame] | 665 | if msg.xid == None: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 666 | msg.xid = ofutils.gen_xid( ) |
Dan Talayco | d7e2dbe | 2010-02-13 21:51:15 -0800 | [diff] [blame] | 667 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 668 | self.logger.debug( "Running transaction %d" % msg.xid ) |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 669 | |
Rich Lane | 9aca199 | 2012-07-11 17:26:31 -0700 | [diff] [blame] | 670 | with self.xid_cv: |
| 671 | if self.xid: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 672 | self.logger.error( "Can only run one transaction at a time" ) |
Rich Lane | 9aca199 | 2012-07-11 17:26:31 -0700 | [diff] [blame] | 673 | return (None, None) |
Dan Talayco | f8de518 | 2012-04-12 22:38:41 -0700 | [diff] [blame] | 674 | |
Rich Lane | b73808c | 2013-03-11 15:22:23 -0700 | [diff] [blame] | 675 | self.xid = msg.xid |
Dan Talayco | d12b661 | 2010-03-07 22:00:46 -0800 | [diff] [blame] | 676 | self.xid_response = None |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 677 | self.message_send( msg ) |
Rich Lane | 9aca199 | 2012-07-11 17:26:31 -0700 | [diff] [blame] | 678 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 679 | self.logger.debug( "Waiting for transaction %d" % msg.xid ) |
| 680 | ofutils.timed_wait( self.xid_cv, lambda: self.xid_response, |
| 681 | timeout=timeout ) |
Rich Lane | 9aca199 | 2012-07-11 17:26:31 -0700 | [diff] [blame] | 682 | |
| 683 | if self.xid_response: |
| 684 | (resp, pkt) = self.xid_response |
| 685 | self.xid_response = None |
| 686 | else: |
| 687 | (resp, pkt) = (None, None) |
| 688 | |
Dan Talayco | 09c2c59 | 2010-05-13 14:21:52 -0700 | [diff] [blame] | 689 | if resp is None: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 690 | self.logger.warning( "No response for xid " + str( self.xid ) ) |
Dan Talayco | 09c2c59 | 2010-05-13 14:21:52 -0700 | [diff] [blame] | 691 | return (resp, pkt) |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 692 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 693 | def message_send( self, msg ): |
Dan Talayco | 3408952 | 2010-02-07 23:07:41 -0800 | [diff] [blame] | 694 | """ |
| 695 | Send the message to the switch |
Dan Talayco | e37999f | 2010-02-09 15:27:12 -0800 | [diff] [blame] | 696 | |
Dan Talayco | 11c26e7 | 2010-03-07 22:03:57 -0800 | [diff] [blame] | 697 | @param msg A string or OpenFlow message object to be forwarded to |
| 698 | the switch. |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 699 | """ |
| 700 | |
Dan Talayco | 1b3f690 | 2010-02-15 14:14:19 -0800 | [diff] [blame] | 701 | if not self.switch_socket: |
| 702 | # Sending a string indicates the message is ready to go |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 703 | raise Exception( "no socket" ) |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 704 | |
Rich Lane | 1fd43e3 | 2014-01-06 15:22:50 -0800 | [diff] [blame] | 705 | if msg.xid == None: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 706 | msg.xid = ofutils.gen_xid( ) |
Rich Lane | 1fd43e3 | 2014-01-06 15:22:50 -0800 | [diff] [blame] | 707 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 708 | outpkt = msg.pack( ) |
Rich Lane | 1fd43e3 | 2014-01-06 15:22:50 -0800 | [diff] [blame] | 709 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 710 | self.logger.debug( "Msg out: version %d class %s len %d xid %d", |
| 711 | msg.version, type( msg ).__name__, len( outpkt ), |
| 712 | msg.xid ) |
Rich Lane | c9d3edd | 2013-10-09 00:21:01 -0700 | [diff] [blame] | 713 | |
| 714 | with self.tx_lock: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 715 | if self.switch_socket.sendall( outpkt ) is not None: |
| 716 | raise AssertionError( "failed to send message to switch" ) |
Dan Talayco | 710438c | 2010-02-18 15:16:07 -0800 | [diff] [blame] | 717 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 718 | return 0 # for backwards compatibility |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 719 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 720 | def clear_queue( self ): |
Dan Talayco | dd6b6ff | 2013-04-12 08:20:18 -0700 | [diff] [blame] | 721 | """ |
| 722 | Clear the input queue and report the number of messages |
| 723 | that were in it |
| 724 | """ |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 725 | enqueued_pkt_count = len( self.packets ) |
Dan Talayco | dd6b6ff | 2013-04-12 08:20:18 -0700 | [diff] [blame] | 726 | with self.packets_cv: |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 727 | self.packets = [ ] |
Dan Talayco | 7071cf1 | 2013-04-16 11:02:13 -0700 | [diff] [blame] | 728 | return enqueued_pkt_count |
Dan Talayco | dd6b6ff | 2013-04-12 08:20:18 -0700 | [diff] [blame] | 729 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 730 | def __str__( self ): |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 731 | string = "Controller:\n" |
Dan Talayco | d7e2dbe | 2010-02-13 21:51:15 -0800 | [diff] [blame] | 732 | string += " state " + self.dbg_state + "\n" |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 733 | string += " switch_addr " + str( self.switch_addr ) + "\n" |
| 734 | string += " pending pkts " + str( len( self.packets ) ) + "\n" |
| 735 | string += " total pkts " + str( self.packets_total ) + "\n" |
| 736 | string += " expired pkts " + str( self.packets_expired ) + "\n" |
| 737 | string += " handled pkts " + str( self.packets_handled ) + "\n" |
| 738 | string += " poll discards " + str( self.poll_discards ) + "\n" |
| 739 | string += " parse errors " + str( self.parse_errors ) + "\n" |
| 740 | string += " sock errrors " + str( self.socket_errors ) + "\n" |
| 741 | string += " max pkts " + str( self.max_pkts ) + "\n" |
| 742 | string += " target switch " + str( self.switch ) + "\n" |
| 743 | string += " host " + str( self.host ) + "\n" |
| 744 | string += " port " + str( self.port ) + "\n" |
| 745 | string += " keep_alive " + str( self.keep_alive ) + "\n" |
| 746 | string += " pkt_in_run " + str( self.pkt_in_run ) + "\n" |
| 747 | string += " pkt_in_dropped " + str( self.pkt_in_dropped ) + "\n" |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 748 | return string |
| 749 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 750 | def show( self ): |
| 751 | print str( self ) |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 752 | |
Flavio Castro | b01d0aa | 2016-07-20 16:14:48 -0700 | [diff] [blame] | 753 | |
| 754 | def sample_handler( controller, msg, pkt ): |
Dan Talayco | 21c75c7 | 2010-02-12 22:59:24 -0800 | [diff] [blame] | 755 | """ |
| 756 | Sample message handler |
| 757 | |
| 758 | This is the prototype for functions registered with the controller |
| 759 | class for packet reception |
| 760 | |
| 761 | @param controller The controller calling the handler |
| 762 | @param msg The parsed message object |
| 763 | @param pkt The raw packet that was received on the socket. This is |
| 764 | in case the packet contains extra unparsed data. |
| 765 | @returns Boolean value indicating if the packet was handled. If |
| 766 | not handled, the packet is placed in the queue for pollers to received |
| 767 | """ |
| 768 | pass |