kernel_7.10.6 test procedures

- The following issues were identified in oversight reviews of
  chino and of the libraries/tools updated for it.
- Issues which have already been fixed in other PCRs (while the
  oversight review process occurred) have been removed (when
  spotted).
- Issues marked with a number -- "(N)" -- are likely
  enhancements/editorial.  Issues marked with a number and 't' --
  "(Nt)" -- are likely technical defects or checklist violations.
  Issues marked with a number and 'p' -- "(Np)" -- are likely process
  issues.


- File: /scm/products/kernel/kernel/branches/mainline/tests/code/x86/tpk510/tpk510b/tpk510b1.cpp
  - Issues identified in oversight review of packet tr281 in
    kernel_7.10.6 (extracted from ASPIRE)
  - URL: https://deos.ddci.com/scm/Deos/products/kernel/kernel/branches/mainline/tests/code/x86/tpk510/tpk510b/tpk510b1.cpp?r=29204  //o-review:23337
  - Function GPFaultError(), comment "the LMSW instruction is"
    - (1) "priviliged" should be "privileged"
  - Function PageFaultError(), comment on the "movl"
    - (2) "Refrence" should be "reference"
    - On line below, "refrence" should be "reference"
  - Function FloatingPointError(), comment on "fdiv"
    - (3) "currupting" should be "corrupting"
  - Function SystemCall()
    - Function header comment block missing

- File: /scm/products/kernel/kernel/branches/mainline/tests/code/ppc/tpk501/tpk501b/tpk501b1.cpp
  - Issues identified in oversight review of packet tr210 in
    kernel_7.10.6 (extracted from ASPIRE)
  - URL: https://deos.ddci.com/scm/Deos/products/kernel/kernel/branches/mainline/tests/code/ppc/tpk501/tpk501b/tpk501b1.cpp?r=26439  //o-review:23109
  - Function main(), in 'ifdef displayDebug' under createThread() call
    - (4) "Succesful" should be "Successful"
  - Function PrivInstException(), "Purpose:"
    - (5) "Priviledged" should be "privileged"
  - Function TrapException(), "Purpose:"
    - (6) "intruction" should be "instruction"
  - Function GeneralExceptionHandler(), comment "//First thing"
    - (7) "pervent castcading" should be "prevent cascading"
  - Function GeneralExceptionHandler(), comment "Ensure this exception handler touches", under "Note:"
    - (8) "incase" should be "in case"


- File: /scm/products/kernel/kernel/branches/mainline/tests/code/tpk455/tpk455_specializeIPal.cpp
  - Issues identified in oversight review of packet tr238 in
    kernel_7.10.6 (extracted from ASPIRE)
  - URL: https://deos.ddci.com/scm/Deos/products/kernel/kernel/branches/mainline/tests/code/tpk455/tpk455_specializeIPal.cpp?r=27100  //o-review:23238
  - In file header comments:
    - (9) Missing module name

- File: /scm/products/kernel/kernel/branches/mainline/tests/code/x86/tpk510/tpk510a/tpk510a1.cpp
  - Issues identified in oversight review of packet tr281 in
    kernel_7.10.6 (extracted from ASPIRE)
  - URL: https://deos.ddci.com/scm/Deos/products/kernel/kernel/branches/mainline/tests/code/x86/tpk510/tpk510a/tpk510a1.cpp?r=29232  //o-review:23337
  - Near top of tile, under "Test as many bits in the EFLAGS as possible", final two comments on code lines
    - In both comments, "becuase" should be "because"
  - Function GetGDTaddr()
    - (10) Header comment block for function missing (in contrast to
      earlier functions).
  - Function storeGDTSup()
    - (11) Header comment block for function missing (in contrast to
      earlier functions).
  - Function storeGDT()
    - (12) Header comment block for function missing (in contrast to
      earlier functions).
  - Function allocateIdentityMappedMemory()
    - (13) Header comment block for function missing (in contrast to
      earlier functions).
  - Function allocateIdentityMappedMemory(), in pTestIO->out of "unsufficent room to allocate"
    - (14) "insufficent" shoujld be "insufficient"
  - Function initTestGDT()
    - (15) Header comment block for function missing (in contrast to
      earlier functions).
  - Function testCase220() "Purpose:"
    - (16) "mechinism" should be "mechanism"
  - Function testCase220(), comments on assignment to EFLAGS_UserWriteableMask
    - (17) Last two comments on code lines, "becuase" should be
      "because"
  - Function testCase220(), near end
    - (18) Both "tout << Pass" lines: "befor" should be "before"

- File: /scm/products/kernel/kernel/branches/mainline/tests/code/tpk492/tpk492a/tpk492a1.cpp
  - Issues identified in oversight review of packet tr124 in
    kernel_7.10.6 (extracted from ASPIRE)
  - URL: https://deos.ddci.com/scm/Deos/products/kernel/kernel/branches/mainline/tests/code/tpk492/tpk492a/tpk492a1.cpp  //o-review:23086
  - At Top of file "Purpose:"
    - (19) "dependancies" should be "dependencies"
  - Function TestCase110(), header comment "Purpose"
    - (20) "willl" should be "will"

- File: /scm/products/kernel/kernel/branches/mainline/tests/code/ppc/tpk501/tpk501a/tpk501a1.cpp
  - Issues identified in oversight review of packet tr210 in
    kernel_7.10.6 (extracted from ASPIRE)
  - URL: https://deos.ddci.com/scm/Deos/products/kernel/kernel/branches/mainline/tests/code/ppc/tpk501/tpk501a/tpk501a1.cpp?r=17680  //o-review:23109
  - In Function ReadExcRegisters() "Purpose:"
    - (21) "ino" should be "into"
  - In Function TestFunction() "Purpose:"
    - (22) "excutes" should be "executes"

- File: /scm/products/kernel/kernel/branches/mainline/tests/code/ppc/tpk502/tpk502a/tpk502a1.cpp
  - Issues identified in oversight review of packet tr219 in
    kernel_7.10.6 (extracted from ASPIRE)
  - URL: https://deos.ddci.com/scm/Deos/products/kernel/kernel/branches/mainline/tests/code/ppc/tpk502/tpk502a/tpk502a1.cpp?r=26927  //o-review:23231
  - At Top of file, prototype for TestCase100()
    - (23) "fram" should be "frame"
  - At Top of file, prototype for TestCase110()
    - (24) "fram" should be "frame"
  - Function saveSegmentRegisters()
    - (25) Missing function header comments
  - Function saveSPRG7()
    - (26) Missing function header comments
  - Function Thread100Function() header comments
    - (27) "accessable" should be "accessible"
  - Function Thread100Function(), comment "check that the required user mode"
    - (28) "accessable" should be "accessible"
  - Function Thread100Function(), comment "check that the required supervisor mode"
    - (29) "accessable" should be "accessible"
  - Function TestCase110(), comment "Note: The clobber register lists"
    - (30) "incase" should be "in case"

- File: /scm/products/kernel/kernel/branches/mainline/tests/code/tpk230/tpk230a/tpk230a1.cpp
  - Issues identified in oversight review of packet tr96 in
    kernel_7.10.6 (extracted from ASPIRE)
  - URL: https://deos.ddci.com/scm/Deos/products/kernel/kernel/branches/mainline/tests/code/tpk230/tpk230a/tpk230a1.cpp  //o-review:22972
  - At Top of file "Purpose:"
    - (31) suggest add this table to the tck230 comments or here, so
      the procA, B, C, D registry info and purpose are more clear
     |       | inDebugSet | breakAtStartup | exceptions     |
     | ProcA | no         | no             | n/a            |
     | ProcB | yes        | no             | Illegal access |
     | ProcC | no         | yes            | Divide by 0    |
     | ProcD | yes        | yes            | n/a            |
  - Function workerEntryPoint(), case 160, comment
    - (32) suggest add comment that causes processorSpecificEx
  - Function workerEntryPoint(), case 180
    - (33) There is no SRS in tck230 to support this test. But in the
      ddd.htm, at 8.14.2.2 Warning/Retrictions it is mentioned. So
      suggest : modify tck230
  - Function workerEntryPoint(), case 190
    - (34) There is no SRS in tck230 to support this test. But in the
      ddd.htm, at 8.14.2.2 Warning/Retrictions it is mentioned. So
      suggest : modify tck230


- File: /scm/products/kernel/kernel/branches/mainline/tests/code/tpk455/tpk455a/tpk455a1.cpp
  - Issues identified in oversight review of packet tr238 in
    kernel_7.10.6 (extracted from ASPIRE)
  - URL: https://deos.ddci.com/scm/Deos/products/kernel/kernel/branches/mainline/tests/code/tpk455/tpk455a/tpk455a1.cpp?r=27100  //o-review:23238
  - Function TestCase130(), under "First, did the kernelModeError function get called?", the "if" statement
    - (35t) The logic here should be: if ((results->errorCode ==
      NO_KERNEL_MODE_ERROR) or (results->data1 ==
      NO_KERNEL_MODE_ERROR) or (results->data2 ==
      NO_KERNEL_MODE_ERROR)) due to tpk455_specializeIPal_27100.cpp
      line 262-270 set the error codes. If the error handler get
      called, it is the failed test.
  - Function TestCase150(), under "First, did the kernelModeError function get called?", the "if" statement
    - (36t) The logic here should be: if ((results->errorCode ==
      NO_KERNEL_MODE_ERROR) or (results->data1 ==
      NO_KERNEL_MODE_ERROR) or (results->data2 ==
      NO_KERNEL_MODE_ERROR)) due to tpk455_specializeIPal_27100.cpp
      line 262-270 set the error codes. If the error handler get
      called, it is the failed test.