Skip to content

Conversation

@SwethaParasa
Copy link
Contributor

@SwethaParasa SwethaParasa commented Mar 24, 2025

For POWER11 systems, NagDump contained POWER10 prefix for all the targets. The prefix was updated to POWER for fapi targets in earlier commit but chip targets and htm targets were missed. Modifying the prefix for core, thread and htm targets to maintain consistency.

Test Results:

Before:

{
   "MANUAL_ISOLATION": {
     "CURRENT_STATE": "DECONFIGURED",
     "LOCATION_CODE": "Ufcs-P0-C15",
     "PHYS_PATH": "physical:sys-0/node-0/proc-1/eq-0/fc-1/core-1",
     "REASON_DESCRIPTION": "MANUAL",
     "TYPE": "POWER10 Core"
   }

After:

{
   "MANUAL_ISOLATION": {
     "CURRENT_STATE": "DECONFIGURED",
     "LOCATION_CODE": "Ufcs-P0-C15",
     "PHYS_PATH": "physical:sys-0/node-0/proc-1/eq-0/fc-1/core-1",
     "REASON_DESCRIPTION": "MANUAL",
     "TYPE": "POWER Core"
   }

Change-Id: If4180988f2bd6417ed1e8b67ef8e5000e4bee8b5

For POWER11 systems, NagDump contained POWER10 prefix for all the
targets. The prefix was updated to POWER for fapi targets in earlier
commit but chip targets and htm targets were missed. Modifying the
prefix for core, thread and htm targets to maintain consistency.

Test Results:

Before:
  {
    "MANUAL_ISOLATION": {
      "CURRENT_STATE": "DECONFIGURED",
      "LOCATION_CODE": "Ufcs-P0-C15",
      "PHYS_PATH": "physical:sys-0/node-0/proc-1/eq-0/fc-1/core-1",
      "REASON_DESCRIPTION": "MANUAL",
      "TYPE": "POWER10 Core"
    }

After:
  {
    "MANUAL_ISOLATION": {
      "CURRENT_STATE": "DECONFIGURED",
      "LOCATION_CODE": "Ufcs-P0-C15",
      "PHYS_PATH": "physical:sys-0/node-0/proc-1/eq-0/fc-1/core-1",
      "REASON_DESCRIPTION": "MANUAL",
      "TYPE": "POWER Core"
    }

Change-Id: If4180988f2bd6417ed1e8b67ef8e5000e4bee8b5
Signed-off-by: Parasa-Swetha1 <[email protected]>
@dhruvibm dhruvibm merged commit f8713b3 into open-power:master Mar 28, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants