Oops, forgot the in-reply-to. This is in response to: [PATCH] cfm: No longer keep track of bad remote MPs and MAIDS.
I'll resend the email. On Mon, Mar 28, 2011 at 3:37 PM, Ethan Jackson <et...@nicira.com> wrote: > --- > I've applied the following incremental to the patch. > --- > lib/cfm.c | 6 ++++++ > 1 files changed, 6 insertions(+), 0 deletions(-) > > diff --git a/lib/cfm.c b/lib/cfm.c > index 00365e6..5cbbd2b 100644 > --- a/lib/cfm.c > +++ b/lib/cfm.c > @@ -345,10 +345,12 @@ cfm_process_heartbeat(struct cfm *cfm, const struct > ofpbuf *p) > uint32_t ccm_seq; > uint8_t ccm_interval; > struct remote_mp *rmp; > + struct eth_header *eth; > > struct cfm_internal *cfmi = cfm_to_internal(cfm); > static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(5, 20); > > + eth = ofpbuf_at(p, (uint8_t *)p->l2 - (uint8_t *)p->data, > ETH_HEADER_LEN); > ccm = ofpbuf_at(p, (uint8_t *)p->l3 - (uint8_t *)p->data, CCM_LEN); > > if (!ccm) { > @@ -365,6 +367,8 @@ cfm_process_heartbeat(struct cfm *cfm, const struct > ofpbuf *p) > if (memcmp(ccm->maid, cfm->maid, sizeof ccm->maid)) { > cfmi->x_recv_time = time_msec(); > cfm->fault = true; > + VLOG_WARN_RL(&rl, "Received unexpected remote MAID from MAC " > + ETH_ADDR_FMT, ETH_ADDR_ARGS(eth->eth_src)); > } else { > ccm_mpid = ntohs(ccm->mpid); > ccm_seq = ntohl(ccm->seq); > @@ -379,6 +383,8 @@ cfm_process_heartbeat(struct cfm *cfm, const struct > ofpbuf *p) > } else { > cfmi->x_recv_time = time_msec(); > cfm->fault = true; > + VLOG_WARN_RL(&rl, "Received unexpected remote MPID %d from MAC " > + ETH_ADDR_FMT, ccm_mpid, > ETH_ADDR_ARGS(eth->eth_src)); > } > } > } > -- > 1.7.4.1 > > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev