Subject: [mpls] Comments to draft-ietf-mpls-tp-oam-id-mib-02

From: Muly Ilan <[email-masked]>
Date: Mon, 11 Feb 2013 11:38:50 +0000

Hi,

In this version you added the bit notApplicable(4) to the object mplsOamIdMegSubOperStatus.
It seems that this new bit is redundant. In case the MEG is up all the other bits would be clear i.e. a value of zero for the object mplsOamIdMegSubOperStatus.
This is the added value of the BITS syntax over INTEGER. With BITS there is no need to explicitly define a dedicated bit for the normal/idle case.
I suggest to revert to the previous definition and to remove this bit.

Another issue is to explain the relationship between mplsOamIdMegOperStatus  and mplsOamIdMegSubOperStatus.
I suggest to add the following description to mplsOamIdMegSubOperStatus:
"
When the value of mplsOamIdMegOperStatus is up(1), all the bits of mplsOamIdMegSubOperStatus must be cleared.
When the value of mplsOamIdMegOperStatus is down(2), at least one bit of mplsOamIdMegSubOperStatus must be set.
"

Regards,

Muly