> So, my interpretation of the database has been that cfm_fault says > whether there is a fault and cfm_fault_status gives the reason for the > fault. How should a controller interpret it if there is a fault > (cfm_fault=true) but no reason, or almost equivalently if there is a > fault but the reason is just "override"? Especially if the goal of > providing the ability to override is to allow for testing (that is the > reason, right?) then I'd expect that one would want to simulate a > specific kind of fault. > > Perhaps I just misunderstand.
Right sorry perhaps I wasn't clear, my plan is to add the override fault status to the database. > Right, recv_fault would be a CFM_FAULT_* bitmap. On an unexpected > MAID, recv_fault |= CFM_FAULT_MAID. On an overflow, recv_fault |= > CFM_FAULT_OVERFLOW. Oh, I had misunderstood you. This makes sense to me I'll go ahead and do it. > Not sure you saw my other comments since I didn't see any replies. Your other comments seemed reasonable and will be folded into the incremental I intend to send shortly. Ethan > > Thanks, > > Ben. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev