Re: RE: TX-RX gain in Sig MIB Draft

"Randy Presuhn" <[email protected]> Wed, 22 Jun 2005 22:00:03 -0700
Newsgroups gmane.ietf.ipcdn
Message-ID <002a01c577b0$6b4412e0$7f1afea9@oemcomputer>
Hi -

> From: "Sumanth Channabasappa" <[email protected]>
> To: "Satish Kumar at Texas Instruments" <[email protected]>; <[email protected]>
> Sent: Wednesday, June 22, 2005 3:06 PM
> Subject: [ipcdn] RE: TX-RX gain in Sig MIB Draft
...
> While I would agree in general regarding the DEFVAL, this MIB object
> would need to go by recommendations (or RECOMMENDATIONS as you pointed
> out):
>
>           "The default values are based on the deployed markets. Some
>           recommendations are made as follows:
>           - Based on the default G.711 Vocoder maximum of 3.14 or 3.17
>
>             dBm a default value of '-4 dB' provides a maximum analog
>             signal level at the a-b (T/R) termination point
>           - Based on [ETSI TS 101 909-4], which provides guidance of
>             11 dB loss (-11 dB gain), a default value of '-11 dB' is
>             recommended."
>
>Is that ok?

The DESCRIPTION that is there clearly says that the RECOMMENDED
default value is -11.  If you mean something else, then the DESCRIPTION
needs work.

> Also, thanks for the pointer on '[email protected]'. I got the
> following
> Warning messages for this MIB:
>
> W1. mailbody:1173: [5] {integer-misuse} warning: use Integer32 instead
> of INTEGER in SMIv2
> W2. mailbody:1136: [5] {index-element-not-column} warning: index element
> `pktcSigDevToneType' of row `pktcSigDevToneEntry' must be a column
> W3/ mailbody:1136: [5] {index-element-not-column} warning: index element
> `pktcSigDevToneType' of row `pktcSigDevMultiFreqToneEntry' must be a
> column
>
> W1 is due to the fact the we use enumerations at various places and W2

But the definition of, for example, pktcSigDevToneNumFrequencies, the
RECOMMENDED type would be Unsigned32.  See
http://www.ietf.org/internet-drafts/draft-ietf-ops-mib-review-guidelines-04.txt
section 4.6.1.1

> and W3 are due to the fact that
> 'pktcSigDevToneType' is defined outside of the two tables, but binds
> them together.
...

This definition is odd, and I agree that it merits at least a warning.
 A not-accessible scalar being used as an index is quite strange.
It should be a columnar object in one of the tables.  Note the requirements
in the guidelines section 4.6.4 .

BTW, for all the RowStatus objects in the mib, note that the guidelines say:

|     - The DESCRIPTION clause of the status column MUST specify which
|       columnar objects (if any) have to be set to valid values before
|       the row can be activated.  If any objects in cascading tables
|       have to be populated with related data before the row can be
|       activated, then this MUST also be specified.
|
|     - The DESCRIPTION clause of the status column MUST specify whether
|       or not it is possible to modify other columns in the same
|       conceptual row when the status value is active(1).  Note that in
|       many cases it will be possible to modify some writeable columns
|       when the row is active but not others.  In such cases the
|       DESCRIPTION clause for each writeable column SHOULD state whether
|       or not that column can be modified when the row is active, and
|       the DESCRIPTION clause for the status column SHOULD state that
|       modifiability of other columns when the status value is active(1)
|       is specified in the DESCRIPTION clauses for those columns (rather
|       than listing the modifiable columns individually).

Randy