Re: draft-ietf-ipoib-subnet-manager-mib-00.txt

Hi Cheng,

Looks good. I have some comments on this MIB:

Here are some general comments and questions:

It is not just a subnet manager but also subnet administration MIB. Should
it be noted that it includes support for SA as well as SM in the overview
(section 4) ?

Should Reversible be included in (both) the path and multipath request ?

It appears that service associations are creatable and deletable via SNMP
management. Isn't this at odds with IB ?

Should the ability to create/delete multicast groups/members be supported by
management (proxy) ?

An inform info information group to see what nodes are subscribed to what
traps/notices might be useful.

Should there be a LEDInfo information group ?

Despite the current lack of interest, shouldn't the CA SL to VL map table,
Switch SL to VL map table and VL arbitration table be included so this MIB
won't need to be rev'd in the future when these become of interest ?

Here are some nits:

Should ibSmPortInfoCapMask (p.16,17) isConnectionManagerSupported be named
IsCommunicationManagementSupported ?

Should ibSmPortInfoOperVL (p. 23) have a 0 (no change) value ?

Should valid values for ibSmSwitchInfoLinearFdbCap and
ibSmSwitchInfoRandomFdbCap (p. 28) be 0...49151 rather than 0..65535 ?
Similarly for ibSmSwitchInfoLinearFdbTop (p. 29)

Should valid values for ibSmSwitchInfoMcastFdbCap (p.29) be 0...16383 rather
than 0...65535 ?

typo comparion should be comparison in ibSmPartitionLastChange (p. 33)

Should the description of ibSmPathReqTable (p. 45) say
"A table used to list all the path entries that satisfy the path request"
rather than
"A table used to list all the path entries that satisfy the multi path
request"

-- Hal

----- Original Message -----
From: "Cheng Yang" <[email protected]>
To: <[email protected]>; <[email protected]>
Cc: "Edwin Tsang" <[email protected]>; "Carl Yang" <[email protected]>; "Bill
Strahm" <[email protected]>
Sent: Friday, March 05, 2004 1:15 PM
Subject: [Ipoverib] draft-ietf-ipoib-subnet-manager-mib-00.txt


> Dear Internet-Drafts Administrator,
>
> Attached is draft-ietf-ipoib-subnet-manager-mib-00.txt as an
> IP over IB workgroup submission.
>
> Please publish it.
>
> Thank you,
>
> Cheng Yang
> Edwin Tsang
> Carl Yang
>
>