Proposed changes to draft ADSL2 MIB

Date    : 2005-june-16
Authors : Scott Baillie, Umberto Bonollo.
Company : NEC Australia.


This file contains a list of the changes made to the draft MIB
draft-morgenstern-ngadsl-mib-00.txt.
An explanation for why the change was made is also included
in this file.


Change 1
--------
Description of change:
  Moved the 4 channel profile pointers from the adsl2LineConfTemplateTable table
  to the adsl2LineConfProfTable.
Reason for change:
  The adsl2LineConfTemplateTable introduces an unnecessarily level
  of indirection.
  The adsl2LineConfTemplateTable table does not follow the pattern
  for profiles that is implemented in RFC2662, VDSL-LINE-MIB and
  SHDSL-LINE-MIB.


Change 2
--------
Description of change:
  Moved the us & ds rate adaption ratio attributes from the
  adsl2LineConfTemplateTable table to the adsl2ChConfProfileTable.
Reason for change:
  The rate adaption ratios are per channel attributes. Therefore,
  we have moved these attributes to the channel profile table.
  The constraint that the ratios must sum to 100 percent can
  still be checked and enforced.


Change 3
--------
Description of change:
  Removed the adsl2LineConfTemplateTable table.
Reason for change:
  After changes 1 and 2 above, the adsl2LineConfTemplateTable
  table is now empty.


Change 4
--------
Description of change:
  Changed comments for the attributes :
  adsl2LConfProfChan1ConfProfile,
  adsl2LConfProfChan2ConfProfile,
  adsl2LConfProfChan3ConfProfile,
  adsl2LConfProfChan4ConfProfile,
Reason for change:
  Describe the constraints that apply for these attributes.
  The comments specify that bearer channels should be
  allocated sequentially.


Change 5
--------
Description of change:
  Merge the contents of the tables :
  adsl2LineStatusTable,
  adsl2LineCommandTable,
  adsl2LineConfigTable
  into one single table ( adsl2LineTable ).
Reason for change:
  All of the above mentioned tables share the same index and
  it is not necessary to separate read-only and read-write
  attributes. Reducing the number of tables is usually a 
  good thing especially for resource limited SNMP agents.
  Also, some of the old tables were really quite small,
  having only one or two attributes in them.
  For example, see the adslLineTable in RFC2662.


Change 6
--------
Description of change:
  Add adsl2LineAlarmConfProfileTable table for performance
  thresholds and data rate change thresholds.
Reason for change:
  Follow the pattern established by RFC2662, which practically
  means that we require notifications when performance thresholds
  are exceeded.


Change 7
--------
Description of change:
  Add threshold and data rate traps.
Reason for change:
  Follow the pattern established by RFC2662, as per previous change.


Change 8
--------
Description of change:
  Use standard Textual conventions such as TruthValue instead
  of re-inventing these Textual conventions.
Reason for change:
  Use standard Textual conventions.


Change 9
--------
Description of change:
  Use a OCTET STRING instead of a huge BITS type for the
  sub carrier mask attributes.
Reason for change:
  A textual convention of type BITS that defines 512 bits
  is un-necessarily verbose, where the name of each bit
  conveys no further information.


Change 10
---------
Description of change:
  Use the standard index ifIndex rather than defining a
  index of type Unsigned32.
  In the comments, specify the ifType of the interface
  where ifIndex is used.
Reason for change:
  Conform to standard MIB design rules.



Change 11
---------
Description of change:
  Add an extra index to the subcarrier table adsl2SCStatusTable.
Reason for change:
  The old table was very large:
  No need to specify an upstream and downstream variant
  for each attribute, instead, define an additional index
  to specify the direction ( upstream, downstream ).


Change 12
---------
Description of change:
  Removed attributes from from adsl2SCStatusTable 
  adsl2SCStatusLnAttenUs
  adsl2SCStatusLnAttenDs
  adsl2SCStatusSigAttenUs
  adsl2SCStatusSigAttenDs
  adsl2SCStatusSnrMarginUs
  adsl2SCStatusSnrMarginDs
  adsl2SCStatusAttainableRateUs
  adsl2SCStatusAttainableRateDs
  adsl2SCStatusActAtpUs
  adsl2SCStatusActAtpDs
Reason for change:
  These attributes already exist in the new table adsl2LineTable.
  These attributes will be updated after a full initialization
  and after a DELT test.


Change 13
---------
Description of change:
  Add a row status attribute to the adsl2SCStatusTable table.
Reason for change:
  The adsl2SCStatusTable table has a slightly different
  meaning than before. Now it is used to hold DELT-specific
  test results which are not also mentioned in the new
  table adsl2LineTable.
  Because of this new meaning, the row status attribute
  is used to help reduce the storage requirements because
  rows can be deleted when they are no longer needed for
  DELT-specific results.


Change 14
---------
Description of change:
  Added TEXTUAL CONVENTIONS :
  Adsl2Direction
  Adsl2LastTransmittedState
  Adsl2LineLdsf
  Adsl2LineStatus
  Adsl2ChAtmStatus
  Adsl2LConfProfPmMode
Reason for change:
  Use symbolic labels to define the enumerated values of
  MIB attributes rather than using an Unsigned32 and then
  specifying the valid values in the comments.


Comment 1
---------
   In general, the wording for DESCRIPTION fields of attributes
   should be cut and pasted and edited from G.997.1


Comment 2
---------
   Need to add extra clauses to most MIB attributes.
   Clauses needed are :
   UNITS
   REFERENCE
   DEFVAL

--end of change list--