Thanks Ben for your help/reviews. I have added this test.
+# actions=dec_ttl(32768,12345,90,765,1024) +ffff 0020 00002320 0015 000500000000 80003039005A02fd 0400000000000000 + I will do a quick retest and will push the patch shortly. thanx! mehak On Thu, Aug 16, 2012 at 1:41 PM, Ben Pfaff <b...@nicira.com> wrote: > On Thu, Aug 16, 2012 at 01:36:47PM -0700, Mehak Mahajan wrote: > > Currently, if a controller having a nonzero id registers to get a > > OFPR_INVALID_TTL async message, it will not receive it. This is because > > compose_dec_ttl() only sent the invalid ttl packets to the default > controller > > id. NXAST_DEC_TTL_CNT_IDS is a new action that accepts a list of > controller > > ids, each separated by `,', to which the OFPR_INVALID_TTL packets must > be sent. > > The earlier requirement of the controller having to explicitly register > to > > receive these asynchronous messages is retained. > > The syntax of this action is: > > dec_ttl(id1,id2) > > where id1, id2 are valid controller ids. > > > > Signed-off-by: Mehak Mahajan <mmaha...@nicira.com> > > It would be nice to add a test that uses more than one controller id. > > Otherwise, thanks, this looks good to me. >
_______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
