> On Wed, 26 Mar 2014 15:39:30 +0900 (JST) > [email protected] (YAMAMOTO Takashi) wrote: > >>> On Wed, 26 Mar 2014 14:13:24 +0900 (JST) >>> [email protected] (YAMAMOTO Takashi) wrote: >>> >>>>> On Wed, 26 Mar 2014 14:00:07 +0900 (JST) >>>>> [email protected] (YAMAMOTO Takashi) wrote: >>>>> >>>>>>> On Wed, 26 Mar 2014 13:04:17 +0900 >>>>>>> YAMAMOTO Takashi <[email protected]> wrote: >>>>>>> >>>>>>>> this will be used by ofctl app which registers handlers dynamically. >>>>>>> >>>>>>> Why ofctl needs to register handlers dynamically? >>>>>> >>>>>> because the interested set of OF replies events (EventOFPxxxReply) >>>>>> depends on the consumer of the ofctl.api. >>>>> >>>>> EventOFPxxxReply is sent to ofctl only when ofctl sent the >>>>> request. Having a handler to get all kinds of events from is simpler? >>>> >>>> ofctl app acts as a proxy which sends a request for ofctl.api consumer >>>> app. ie. ofctl app does not know what requests it wants to send >>>> (thus what replies it wants to listen) until actually requested by >>>> the consumer app. >>>> >>>> we can have a handler with set_ev_cls listing all possible >>>> EventOFPxxxReply and update the list whenever we implement >>>> new OFP messages. but i don't consider it simpler. >>> >>> I meant that we can add a handler to all the kinds of handlers from >>> ofp like: >>> >>> set_ev_cls(OFPAllEvents) >>> def hoge() >> >> there are two steps to listen an event. >> >> a. put the consumer app into the provider's RyuApp.observers >> b. set up the consumer's RyuApp.event_handlers >> >> which is your 'set_ev_cls(OFPAllEvents)' supposed to perform? >> both? > > both only once at the startup.
as i explained off-line, depending combination of apps, it can affect performance badly. i can implement your suggested way if you still prefer it, though. YAMAMOTO Takashi > >> i'm hesitate to make it do a. because it might cause a lot of >> unnecesary thread switches. >> on the other hand, if it does only b., i'm not sure if it will >> look so simpler than the current patches. >> >> YAMAMOTO Takashi >> >>> >>> ------------------------------------------------------------------------------ >>> Learn Graph Databases - Download FREE O'Reilly Book >>> "Graph Databases" is the definitive new guide to graph databases and their >>> applications. Written by three acclaimed leaders in the field, >>> this first edition is now available. Download your free book today! >>> http://p.sf.net/sfu/13534_NeoTech >>> _______________________________________________ >>> Ryu-devel mailing list >>> [email protected] >>> https://lists.sourceforge.net/lists/listinfo/ryu-devel >> >> ------------------------------------------------------------------------------ >> Learn Graph Databases - Download FREE O'Reilly Book >> "Graph Databases" is the definitive new guide to graph databases and their >> applications. Written by three acclaimed leaders in the field, >> this first edition is now available. Download your free book today! >> http://p.sf.net/sfu/13534_NeoTech >> _______________________________________________ >> Ryu-devel mailing list >> [email protected] >> https://lists.sourceforge.net/lists/listinfo/ryu-devel > > ------------------------------------------------------------------------------ > Learn Graph Databases - Download FREE O'Reilly Book > "Graph Databases" is the definitive new guide to graph databases and their > applications. Written by three acclaimed leaders in the field, > this first edition is now available. Download your free book today! > http://p.sf.net/sfu/13534_NeoTech > _______________________________________________ > Ryu-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/ryu-devel ------------------------------------------------------------------------------ _______________________________________________ Ryu-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ryu-devel
