On Mon, 11 Mar 2013 19:09:01 +0900 Yoshihiro Kaneko <[email protected]> wrote:
> I suppose that the list of handler to be called for the OFP message > should be kept even if the state is changed in a handler. > Is this correct? > > diff --git a/ryu/controller/controller.py b/ryu/controller/controller.py > index c753f01..79c6316 100644 > --- a/ryu/controller/controller.py > +++ b/ryu/controller/controller.py > @@ -172,10 +172,11 @@ class Datapath(object): > ev = ofp_event.ofp_msg_to_ev(msg) > self.ofp_brick.send_event_to_observers(ev, self.state) > > - handlers = self.ofp_brick.get_handlers(ev) > + handlers = [handler for handler in > + self.ofp_brick.get_handlers(ev) if self.state in > + handler.dispatchers] > for handler in handlers: > - if self.state in handler.dispatchers: > - handler(ev) > + handler(ev) > > buf = buf[required_len:] > required_len = ofproto_common.OFP_HEADER_SIZE Looks good. You tried this with our OpenStack testings? ------------------------------------------------------------------------------ Everyone hates slow websites. So do we. Make your web apps faster with AppDynamics Download AppDynamics Lite for free today: http://p.sf.net/sfu/appdyn_d2d_mar _______________________________________________ Ryu-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ryu-devel
