ody_sbe_dump: Add dump content type for odyssey dumps

Odyssey dumps are sbe dumps, add dump content type into dump header
so that dump extraction tools can classify it as a Odyssey sbe dump.

Verifi

ody_sbe_dump: Add dump content type for odyssey dumps

Odyssey dumps are sbe dumps, add dump content type into dump header
so that dump extraction tools can classify it as a Odyssey sbe dump.

Verified:
```
BEFORE - ODY_SBE_DUMP
00000000 46 49 4c 45 20 20 20 20 00 40 00 00 00 00 00 00 |FILE .@......|
00000010 00 00 00 00 00 01 00 0f 53 59 53 44 55 4d 50 2e |........SYSDUMP.|
00000020 31 33 39 46 32 33 30 2e 34 30 30 30 30 30 30 46 |139F230.4000000F|
00000030 2e 32 30 32 35 30 36 31 39 31 36 35 39 33 34 00 |.20250619165934.|
00000040 53 45 43 54 49 4f 4e 20 00 30 00 00 00 00 00 00 |SECTION .0......|
00000050 00 00 00 00 00 02 00 00 00 00 00 00 00 00 04 00 |................|
00000060 44 55 4d 50 20 53 55 4d 4d 41 52 59 00 00 00 00 |DUMP SUMMARY....|
00000070 53 45 43 54 49 4f 4e 20 00 30 00 02 00 00 00 00 |SECTION .0......|
00000080 00 00 00 00 00 02 00 00 00 00 00 00 00 03 78 1d |..............x.|
00000090 48 41 52 44 57 41 52 45 20 44 41 54 41 00 00 00 |HARDWARE DATA...|
000000a0 53 45 43 54 49 4f 4e 20 00 30 00 02 00 00 00 00 |SECTION .0......|
000000b0 00 00 00 01 00 02 00 00 00 00 00 00 00 00 00 00 |................|
000000c0 48 59 50 45 52 56 49 53 4f 52 20 44 41 54 41 00 |HYPERVISOR DATA.|
000000d0 53 59 53 20 44 55 4d 50 20 25 06 19 16 59 34 00 |SYS DUMP %...Y4.|
000000e0 40 00 00 0f 02 21 04 00 00 00 00 00 00 03 7c 1d |@....!........|.|
000000f0 39 31 30 35 2d 32 32 41 00 00 00 00 00 00 00 00 |9105-22A........|
00000100 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
00000110 70 31 30 62 6d 63 00 00 00 00 00 00 00 00 00 00 |p10bmc..........|
00000120 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
00000130 31 33 39 46 32 33 30 01 00 00 00 00 00 d0 00 00 |139F230.........|
00000140 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
*
00000360 01 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
00000370 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|

AFTER FIX- ODY_SBE_DUMP
00000000 46 49 4c 45 20 20 20 20 00 40 00 00 00 00 00 00 |FILE .@......|
00000010 00 00 00 00 00 01 00 0f 53 59 53 44 55 4d 50 2e |........SYSDUMP.|
00000020 31 33 39 46 32 33 30 2e 34 30 30 30 30 30 31 33 |139F230.40000013|
00000030 2e 32 30 32 35 30 36 31 39 31 37 33 30 35 30 00 |.20250619173050.|
00000040 53 45 43 54 49 4f 4e 20 00 30 00 00 00 00 00 00 |SECTION .0......|
00000050 00 00 00 00 00 02 00 00 00 00 00 00 00 00 04 00 |................|
00000060 44 55 4d 50 20 53 55 4d 4d 41 52 59 00 00 00 00 |DUMP SUMMARY....|
00000070 53 45 43 54 49 4f 4e 20 00 30 00 02 00 00 00 00 |SECTION .0......|
00000080 00 00 00 00 00 02 00 00 00 00 00 00 00 03 78 3d |..............x=|
00000090 48 41 52 44 57 41 52 45 20 44 41 54 41 00 00 00 |HARDWARE DATA...|
000000a0 53 45 43 54 49 4f 4e 20 00 30 00 02 00 00 00 00 |SECTION .0......|
000000b0 00 00 00 01 00 02 00 00 00 00 00 00 00 00 00 00 |................|
000000c0 48 59 50 45 52 56 49 53 4f 52 20 44 41 54 41 00 |HYPERVISOR DATA.|
000000d0 53 59 53 20 44 55 4d 50 20 25 06 19 17 30 50 00 |SYS DUMP %...0P.|
000000e0 40 00 00 13 02 21 04 00 00 00 00 00 00 03 7c 3d |@....!........|=|
000000f0 39 31 30 35 2d 32 32 41 00 00 00 00 00 00 00 00 |9105-22A........|
00000100 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
00000110 70 31 30 62 6d 63 00 00 00 00 00 00 00 00 00 00 |p10bmc..........|
00000120 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
00000130 31 33 39 46 32 33 30 01 00 00 00 00 00 d0 00 00 |139F230.........|
00000140 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
*
00000330 02 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
00000340 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
*
00000360 01 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
00000370 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|

AFTER FIX (PROC SBE DUMP REMAINS UNAFFECTED)
00000000 46 49 4c 45 20 20 20 20 00 40 00 00 00 00 00 00 |FILE .@......|
00000010 00 00 00 00 00 01 00 0f 53 59 53 44 55 4d 50 2e |........SYSDUMP.|
00000020 31 33 39 46 32 33 30 2e 33 30 30 30 30 30 31 34 |139F230.30000014|
00000030 2e 32 30 32 35 30 36 31 39 31 37 33 37 33 37 00 |.20250619173737.|
00000040 53 45 43 54 49 4f 4e 20 00 30 00 00 00 00 00 00 |SECTION .0......|
00000050 00 00 00 00 00 02 00 00 00 00 00 00 00 00 04 00 |................|
00000060 44 55 4d 50 20 53 55 4d 4d 41 52 59 00 00 00 00 |DUMP SUMMARY....|
00000070 53 45 43 54 49 4f 4e 20 00 30 00 02 00 00 00 00 |SECTION .0......|
00000080 00 00 00 00 00 02 00 00 00 00 00 00 00 04 04 ac |................|
00000090 48 41 52 44 57 41 52 45 20 44 41 54 41 00 00 00 |HARDWARE DATA...|
000000a0 53 45 43 54 49 4f 4e 20 00 30 00 02 00 00 00 00 |SECTION .0......|
000000b0 00 00 00 01 00 02 00 00 00 00 00 00 00 00 00 00 |................|
000000c0 48 59 50 45 52 56 49 53 4f 52 20 44 41 54 41 00 |HYPERVISOR DATA.|
000000d0 53 59 53 20 44 55 4d 50 20 25 06 19 17 37 37 00 |SYS DUMP %...77.|
000000e0 30 00 00 14 02 21 04 00 00 00 00 00 00 04 08 ac |0....!..........|
000000f0 39 31 30 35 2d 32 32 41 00 00 00 00 00 00 00 00 |9105-22A........|
00000100 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
00000110 70 31 30 62 6d 63 00 00 00 00 00 00 00 00 00 00 |p10bmc..........|
00000120 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
00000130 31 33 39 46 32 33 30 01 00 00 00 00 00 d0 00 00 |139F230.........|
00000140 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
*
00000330 02 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
00000340 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
*
00000360 01 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................|
```

Change-Id: I3e032545ffe586c29dc677b5d93bd461bedf3288
Signed-off-by: Gopichand Paturi <gopichandpaturi@gmail.com>

show more ...

clang-tidy: Initial commit

Added initial .clang-tidy configuration file with recommended C++ checks
aligned with the OpenBMC-approved checklist. This setup enforces modern
C++ best practices and hel

clang-tidy: Initial commit

Added initial .clang-tidy configuration file with recommended C++ checks
aligned with the OpenBMC-approved checklist. This setup enforces modern
C++ best practices and helps catch common issues early in development.

Change-Id: Ieca63e81a60027d2ab18d9f2d1496cd0cd66851a
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

bmcdump: Fix fetch serial number function in shell script

An extra space caused the shell to interpret the statement incorrectly,
leading to unexpected behavior. The extra space was removed.

```
Te

bmcdump: Fix fetch serial number function in shell script

An extra space caused the shell to interpret the statement incorrectly,
leading to unexpected behavior. The extra space was removed.

```
Test Results :

Before:
Nov 26 07:30:18 p10bmc phosphor-dump-manager[2873]: /usr/share/dreport.d/include.d/opfunctions: line 15: [: missing `]'
Nov 26 07:30:18 p10bmc phosphor-dump-manager[2878]: /usr/share/dreport.d/include.d/opfunctions: line 15: 137A791: command not found
Nov 26 07:30:18 p10bmc phosphor-dump-manager[2873]: performing dump compression /tmp/BMCDUMP.0000000.00000051.20251126072948
Nov 26 07:30:21 p10bmc phosphor-dump-manager[2873]: Adding Dump Header :/usr/share/dreport.d/include.d/gendumpheader
Nov 26 07:30:21 p10bmc phosphor-dump-manager[2902]: /usr/share/dreport.d/include.d/opfunctions: line 15: [: missing `]'
Nov 26 07:30:21 p10bmc phosphor-dump-manager[2924]: /usr/share/dreport.d/include.d/opfunctions: line 15: 137A791: command not found
Nov 26 07:30:39 p10bmc phosphor-dump-manager[2873]: Wed Nov 26 07:30:39 UTC 2025 Report is available in /var/lib/phosphor-debug-collector/dumps/51

After:
Nov 26 07:39:43 p10bmc phosphor-dump-manager[3487]: performing dump compression /tmp/BMCDUMP.137A791.00000052.20251126073911
Nov 26 07:39:45 p10bmc phosphor-dump-manager[3487]: Adding Dump Header :/usr/share/dreport.d/include.d/gendumpheader
Nov 26 07:40:03 p10bmc phosphor-dump-manager[3487]: Wed Nov 26 07:40:03 UTC 2025 Report is available in /var/lib/phosphor-debug-collector/dumps/52
Nov 26 07:40:03 p10bmc phosphor-dump-manager[3101]: Wed Nov 26 07:40:03 UTC 2025 Successfully completed
```

Change-Id: Ic1b4c3d5ec47efd664f4f610b7c40075a389fc8b
Signed-off-by: Sera Susan Koshy <Sera.Koshy@ibm.com>

show more ...

opfunctions: Fix serial number check

If the serial number contains invalid characters, the BMC dump filename
and header may be incorrect, causing extraction errors. Check that
the serial number is

opfunctions: Fix serial number check

If the serial number contains invalid characters, the BMC dump filename
and header may be incorrect, causing extraction errors. Check that
the serial number is an alphanumeric string exactly 7 characters long;
if it is not, assign 0000000 as a default value.

Test Results:

```
Before:
Nov 23 01:04:53 p10bmc phosphor-dump-manager[10552]: serial no ''
Nov 23 01:04:53 p10bmc phosphor-dump-manager[10552]: performing dump compression /tmp/BMCDUMP.''.00000016.20261123010343
Nov 23 01:04:56 p10bmc phosphor-dump-manager[10552]: Adding Dump Header :/usr/share/dreport.d/include.d/gendumpheader
Nov 23 01:05:03 p10bmc phosphor-dump-manager[10552]: Mon Nov 23 01:05:03 UTC 2026 Report is available in /var/lib/phosphor-debug-collector/dumps/16
Nov 23 01:05:03 p10bmc phosphor-dump-manager[504]: Invalid Dump file name, FILENAME: /var/lib/phosphor-debug-collector/dumps/16/BMCDUMP.''.00000016.20261123010343

After:
Nov 23 01:07:25 p10bmc phosphor-dump-manager[11104]: serial no ''
Nov 23 01:07:25 p10bmc phosphor-dump-manager[11104]: performing dump compression /tmp/BMCDUMP.0000000.00000017.20261123010614
Nov 23 01:07:27 p10bmc phosphor-dump-manager[11104]: Adding Dump Header :/usr/share/dreport.d/include.d/gendumpheader
Nov 23 01:07:35 p10bmc phosphor-dump-manager[11104]: Mon Nov 23 01:07:35 UTC 2026 Report is available in /var/lib/phosphor-debug-collector/dumps/17
Nov 23 01:07:35 p10bmc phosphor-dump-manager[10704]: Mon Nov 23 01:07:35 UTC 2026 Successfully completed
```

Change-Id: Ic4bbc83c7ca30c4b51aca10268188811b7584897
Signed-off-by: Swetha Parasa <parasa.swetha1@ibm.com>

show more ...

use sdbusplus unpack syntax

Rather than defining a variable and then reading it from a message,
sdbusplus also supports directly unpack-ing from the message. Use
this syntax instead as it is more e

use sdbusplus unpack syntax

Rather than defining a variable and then reading it from a message,
sdbusplus also supports directly unpack-ing from the message. Use
this syntax instead as it is more efficient and succinct.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I311c0a4a3aa8b68ea2814c00eaff11e2e8803fb1

show more ...

ignore beautysh

Add dumpdelete to beautysh-ignore because it can't seem to handle
the syntax in that file and ends up reformatting into an invalid
shellscript.

Signed-off-by: Patrick Williams <patr

ignore beautysh

Add dumpdelete to beautysh-ignore because it can't seem to handle
the syntax in that file and ends up reformatting into an invalid
shellscript.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I5c2c7cc9756ce4f61f65b25c5e342a0bb8bcd1ba

show more ...

Move collectSBEDump to openpower-debug-collector

Hardware and Hostboot dump collection are being handled in
openpower-debug-collector. SBE dump collection should also be performed
from openpower-

Move collectSBEDump to openpower-debug-collector

Hardware and Hostboot dump collection are being handled in
openpower-debug-collector. SBE dump collection should also be performed
from openpower-debug-collector instead of from IPL.This change
implements the collectSBEDump function as a SbeDumpCollector member
function which executes the SBE dump collection process. collectSBEDump
function is being removed from IPL in this commit[1]

Declared functions which will be called from collectSBEDump
in libphal.H file via commit[1]

Implemented collectSBEDump(for SBE dump collection) and
collectHWHBDump(for Hardware and Hostboot dump collection)
as private member functions in SbeDumpCollector class. These
private member functions will now be called via collectDump
public method based on the type of dump that needs to be collected.

Tested on a Everest DDR5 machine. Able to generate Proc and
Odyssey SBE dumps

[1] https://github.com/open-power/ipl/pull/96

Change-Id: I7919fdf31dc9a985a72bc06444d9d55fe78aa13b
Signed-off-by: Nabil Ananthamangalath <nabilmanjeri@gmail.com>

show more ...

bmcdump: Add occ-control data to plugin

Send USR1 signal to openpower-occ-control app to collect it's dump data
to be added to a BMC dump.

Tested on Rainier:
OCCs not running:
```
$ cat BMCDUMP.139

bmcdump: Add occ-control data to plugin

Send USR1 signal to openpower-occ-control app to collect it's dump data
to be added to a BMC dump.

Tested on Rainier:
OCCs not running:
```
$ cat BMCDUMP.139F210.00000000.20250801210223_out/archive/occ_control_dump.json
{
"objectCount": "4 OCC objects",
"occ0": {
"occState": "NOT ACTIVE"
},
"occ1": {
"occState": "NOT ACTIVE"
},
"occ2": {
"occState": "NOT ACTIVE"
},
"occ3": {
"occState": "NOT ACTIVE"
}
}
```
OCCs running:
```
$ cat BMCDUMP.139F210.00000000.20250801191023_out/archive/occ_control_dump.json
{
"objectCount": "4 OCC objects",
"occ0": {
"occHwmonPath": "/sys/bus/platform/drivers/occ-hwmon/occ-hwmon.1/hwmon/hwmon13",
"occRole": "MASTER",
"occState": "ACTIVE",
"pollResponse": [
"0000: 93000F00 030C0100 00000000 00000000",
"0010: 6F705F70 31315F32 35303432 38610000",
"0020: 53454E53 4F520501 54454D50 0010081C",
"0030: C0000004 0025555F C0000005 0025555F",
"0040: C0000006 0023555F C0000007 0024555F",
"0050: C0000008 0024555F C0000009 0023555F",
"0060: C000000A 0023555F C000000B 0024555F",
...
```

Change-Id: I792cb5554614f28c4b5bc5bf43381bc5ec142e56
Signed-off-by: Chris Cain <cjcain@us.ibm.com>

show more ...

opdreport: Remove duplicate fetch_serial_number function

fetch_serial_number is moved to opfunctions script,
it is being sourced in opdreport, hence this function
is no longer needed in opdreport.

opdreport: Remove duplicate fetch_serial_number function

fetch_serial_number is moved to opfunctions script,
it is being sourced in opdreport, hence this function
is no longer needed in opdreport.

Change-Id: Ibbdff9ac10d1ee47827c276d54e78172286035d4
Signed-off-by: Gopichand Paturi <gopichandpaturi@gmail.com>

show more ...

opdreport: Fix script errors while using fetch_serial_number function

readonly type variables are not passed to child processes/scripts,
this causes broken file paths when these variables are used i

opdreport: Fix script errors while using fetch_serial_number function

readonly type variables are not passed to child processes/scripts,
this causes broken file paths when these variables are used in
child processes/scripts. Use declare -rx instead.

```
Verified:

Before
openpower-dump-monitor[4810]: Adding Dump Header: /usr/share/dreport.d/include.d/gendumpheader
openpower-dump-monitor[4948]: /usr/share/dreport.d/include.d/gendumpheader: line 48: /opfunctions: No such file or directory
openpower-dump-monitor[4969]: /usr/share/dreport.d/include.d/gendumpheader: line 564: fetch_serial_number: command not found
phosphor-dump-manager[5055]: performing dump compression /tmp/BMCDUMP.139F210.00000016.20250605083005
phosphor-dump-manager[520]: A new dump file found /var/lib/phosphor-debug-collector/opdump/3000000E/SYSDUMP.139F210.3000000E.20250605083051

After
openpower-dump-monitor[1238]: Adding Dump Header: /usr/share/dreport.d/include.d/gendumpheader
phosphor-dump-manager[526]: A new dump file found /var/lib/phosphor-debug-collector/opdump/30000012/SYSDUMP.139F210.30000012.20250605093640
```

Change-Id: I0544efd674d4abdefe43a36a46c76d0f31fb0f8b
Signed-off-by: Gopichand Paturi <gopichandpaturi@gmail.com>

show more ...

clang-tidy: fix missing include for uint32_t

Fixed the following error by including the <cstdint> header file.

'''
watchdog/watchdog_logging.hpp:23:33: error: unknown type name 'uint32_t' [clang-di

clang-tidy: fix missing include for uint32_t

Fixed the following error by including the <cstdint> header file.

'''
watchdog/watchdog_logging.hpp:23:33: error: unknown type name 'uint32_t' [clang-diagnostic-error]
23 | void eventWatchdogTimeout(const uint32_t timeout);
'''

Change-Id: I3cee346296fa0c9925e920f5abc04470b15ad278
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

clang-tidy: correct include directives

Replaced angled brackets with quotes in include directives
for local headers in watchdog folder to resolve clang-tidy errors

Sample error:
'''
openpower-debu

clang-tidy: correct include directives

Replaced angled brackets with quotes in include directives
for local headers in watchdog folder to resolve clang-tidy errors

Sample error:
'''
openpower-debug-collector/watchdog/watchdog_logging.cpp:4:10: error: 'watchdog_common.hpp'
file not found with <angled> include; use "quotes" instead [clang-diagnostic-error]
4 | #include <watchdog_common.hpp>
| ^~~~~~~~~~~~~~~~~~~~~
| "watchdog_common.hpp"
'''

Change-Id: I99810f37d74db2fda9b59310b0ffe6a8456e4239
Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>

show more ...

bmcdump plugin: Install hostboot plugin

Verified:
Verified that hostboot plugin is installed.

Change-Id: I3573d94551208eee540e9ad158ed84c947729282
Signed-off-by: Gopichand Paturi <gopichandpaturi@g

bmcdump plugin: Install hostboot plugin

Verified:
Verified that hostboot plugin is installed.

Change-Id: I3573d94551208eee540e9ad158ed84c947729282
Signed-off-by: Gopichand Paturi <gopichandpaturi@gmail.com>

show more ...

Remove unused bmcdump VPD plugin entry

This commit updates the VPD plugin in bmcdump to remove the unused bad
vpd directory in /tmp. vpd-manager now dumps bad VPD files into
/var/lib/vpd/dumps and /

Remove unused bmcdump VPD plugin entry

This commit updates the VPD plugin in bmcdump to remove the unused bad
vpd directory in /tmp. vpd-manager now dumps bad VPD files into
/var/lib/vpd/dumps and /var/lib/vpd is already handled by plugin entry
"VPD persistent data".

Test:
```
1. Ensure vpd-manager has dumped 3 bad VPD files in /var/lib/vpd/dumps
2. Initiate a user generated BMC dump
busctl --verbose call xyz.openbmc_project.Dump.Manager
/xyz/openbmc_project/dump/bmc xyz.openbmc_project.Dump.Create
CreateDump a{sv} 0
3. Once dump entry status is shown as completed, extract the dump and
check vpd folder to see a dumps directory is present with the
expected bad VPD files inside.
```

Change-Id: I34d108ebba69e43b9fbe2fddd2a83cb8dd2b1174
Signed-off-by: Souvik Roy <souvikroyofficial10@gmail.com>

show more ...

sdbusplus: use shorter type aliases

The sdbusplus headers provide shortened aliases for many types.
Switch to using them to provide better code clarity and shorter
lines. Possible replacements are

sdbusplus: use shorter type aliases

The sdbusplus headers provide shortened aliases for many types.
Switch to using them to provide better code clarity and shorter
lines. Possible replacements are for:
* bus_t
* exception_t
* manager_t
* match_t
* message_t
* object_t
* slot_t

Change-Id: Id4aafe0272f14a786641f2e67380cb206a60255c
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...

dumpheader: Add default serial number in the header

Whenever the serialNo is empty, use a default serial
number (all zeroes) so that scripts/apps that decode
the dump header would not throw errors.

dumpheader: Add default serial number in the header

Whenever the serialNo is empty, use a default serial
number (all zeroes) so that scripts/apps that decode
the dump header would not throw errors.

Use fetch_serial_number() so that default serialNo
gets populated whenever the serialNo is empty.

Verified that the default serial number is utilized.

Change-Id: I0d9a8cc9e257ed0a852aaca5e6cb0c1833abd41e
Signed-off-by: Gopichand Paturi <gopichandpaturi@gmail.com>

show more ...

gendumpheader: Originator details info added

There are two parameters available for each dump,
Originator ID and Originator Type. This change
aims to include these fields into the BMC dump
header.

gendumpheader: Originator details info added

There are two parameters available for each dump,
Originator ID and Originator Type. This change
aims to include these fields into the BMC dump
header.

If for any reason either the originator ID or
originator type or both are unavailable then
they are been replaced by their allocated null
bytes.

Test Results:
Tested on generated BMC dumps and verified.

Signed-off-by: Swarnendu Roy Chowdhury <swarnendu.roy.chowdhury@ibm.com>
Signed-off-by: Gopichand Paturi <gopichandpaturi@gmail.com>
Change-Id: I611cce4f19aaef13c3ee68c16cf7b4fb2f841591

show more ...

package: Support customized dump naming

Customize a dump name by renaming the original default name.

Tested:
Verified that BMC Dump is getting generated and renamed as follows.
BMCDUMP.<serialNo>.<

package: Support customized dump naming

Customize a dump name by renaming the original default name.

Tested:
Verified that BMC Dump is getting generated and renamed as follows.
BMCDUMP.<serialNo>.<dump_id>.<dDay>

Change-Id: I5d51693495ef4bf3fc0eff04c03b11cb9e4c225b
Signed-off-by: Gopichand Paturi <gopichandpaturi@gmail.com>

show more ...

package dump: Custom packaging for faultlog dump

Test Results:
Verified that custom_package function is working as expected.

Change-Id: I7fe40512732f0f5cc27265ab8cce209da5a939cd
Signed-off-by: Gopi

package dump: Custom packaging for faultlog dump

Test Results:
Verified that custom_package function is working as expected.

Change-Id: I7fe40512732f0f5cc27265ab8cce209da5a939cd
Signed-off-by: Gopichand Paturi <gopichandpaturi@gmail.com>

show more ...

Add error info to dump on collection failures

There is a possibility that dump collection may fail due to hardware
procedure execution errors or other issues. Modified info.yaml to
store error infor

Add error info to dump on collection failures

There is a possibility that dump collection may fail due to hardware
procedure execution errors or other issues. Modified info.yaml to
store error information along with driver details in case of dump
collection failures.

Change-Id: I74c2fb8ebc31d1c3bd60feffa3f0467faa435d5c
Signed-off-by: Swetha Parasa <parasa.swetha1@ibm.com>

show more ...

opdreport: add host preserved partitions to BMC dump

This change is picked from
https://gerrit.openbmc.org/c/openbmc/phosphor-debug-collector/+/52594

HBEL (Hostboot error logs)
HBD_RW (Hostboot

opdreport: add host preserved partitions to BMC dump

This change is picked from
https://gerrit.openbmc.org/c/openbmc/phosphor-debug-collector/+/52594

HBEL (Hostboot error logs)
HBD_RW (Hostboot preserved attributes)
EECACHE (Hostboot VPD cache)

Tested:
```
Wed Apr 13 08:13:31 UTC 2022 INFO: Copied HBEL /media/hostfw/running/HBEL
Wed Apr 13 08:13:31 UTC 2022 INFO: Copied HBD_RW /media/hostfw/running/HBD_RW
Wed Apr 13 08:13:31 UTC 2022 INFO: Copied EECACHE /media/hostfw/running/EECACHE

root@rain104bmc:/tmp/test/obmcdump_00000000_1649837600# ls -la HBEL
-rw-r----- 1 root root 147456 Apr 13 08:13 HBEL
root@rain104bmc:/tmp/test/obmcdump_00000000_1649837600# ls -lah HBEL
-rw-r----- 1 root root 144.0K Apr 13 08:13 HBEL
-rw-r----- 1 root root 2.0M Apr 13 08:13 HBD_RW
root@rain104bmc:/tmp/test/obmcdump_00000000_1649837600# ls -lah EECACHE
-rw-r----- 1 root root 2.8M Apr 13 08:13 EECACHE
```

Change-Id: Icaa065540e940f26a930e27db8d94fa4c4a19c88
Signed-off-by: Gopichand Paturi <gopichandpaturi@gmail.com>

show more ...

watchdog:Trigger System Dump when host load fails

Using the xyz.openbmc_project.State.Host interface to determine the
operational status of the host becomes less precise when shifting from
hostboot

watchdog:Trigger System Dump when host load fails

Using the xyz.openbmc_project.State.Host interface to determine the
operational status of the host becomes less precise when shifting from
hostboot to host. In the interim phase when host is initializing and
hasn't reached full functionality, the host's state is inaccurately
assumed to be in hostboot. In cases where host encounters initial
boot challenges and the watchdog timer triggers because the boot
process hasn't finished within the set time, this watchdog
misinterprets the situation as a hostboot problem.

To address this, there exists a core scratch register that undergoes
an update by hostboot just before transferring control to host.
We have devised a method that leverages this register to determine
whether the transition to host has already occurred.

By implementing this functionality we can determine which booting
subsystem is failed or stopped responding, and the dump can be
extracted from the right subsystem

Tested:
Oct 03 09:35:36 p10bmc watchdog_timeout[7099]: Host did not respond
within watchdog timeout interval
Oct 03 09:35:36 p10bmc watchdog_timeout[7099]: PHYP boot failure,
triggering system dump
Oct 03 09:35:37 p10bmc phosphor-log-manager[372]: Created PEL
0x50001924 (BMC ID 252) with SRC BD5EC101
Oct 03 09:35:37 p10bmc ibm-panel[1208]: Resolution is empty for
PEL = /xyz/openbmc_project/logging/entry/252
Oct 03 09:35:37 p10bmc phosphor-host-state-manager[795]: Received
signal that host has crashed, decrement reboot count
...
...
Oct 03 09:35:38 p10bmc systemd[1]: Starting Start memory preserving
reboot host0...
Oct 03 09:35:38 p10bmc pldmd[719]: BIOS:pvm_sys_dump_active, updated
to value: Enabled(16), by BMC: true
Oct 03 09:35:38 p10bmc phosphor-dump-manager[473]: OriginatorId is
not provided
Oct 03 09:35:38 p10bmc phosphor-dump-manager[473]: OriginatorType is
not provided. Replacing the string with the default value
Oct 03 09:35:38 p10bmc sh[7151]:
o "/xyz/openbmc_project/dump/bmc/entry/2"
Oct 03 09:35:38 p10bmc openpower-proc-control[7153]: Starting memory
preserving reboot

Change-Id: I312ad7201e9258d23f6e784fab504d0fb8f0f712
Signed-off-by: Deepa Karthikeyan <deepakala.karthikeyan@ibm.com>

show more ...

Move clear host dump service to openpower-debug-collector

Move the clear_hostdumps_poweroff.service to openpower-debug-collector
as it is needed only on IBM systems. Additionally added the scripts
t

Move clear host dump service to openpower-debug-collector

Move the clear_hostdumps_poweroff.service to openpower-debug-collector
as it is needed only on IBM systems. Additionally added the scripts
to selectively delete all entries based on the dump interface to
delete only resource dump and system dump during power off.

Each type of dump will have separate service files due to difference
in activation, system dump entries are not cleared during an mpreboot

Change-Id: Ibf20b097862acb3dc7887cd75d4d5ba6bc1ff5c3
Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>

show more ...

Add entity-objects D-Bus objects to dump

There are some associations that are useful for debug and just getting
them from entity-manager as opposed to the mapper is less resource
intensive and takes

Add entity-objects D-Bus objects to dump

There are some associations that are useful for debug and just getting
them from entity-manager as opposed to the mapper is less resource
intensive and takes up a smaller amount of space in the dump.

Tested:
The file and the data are in the dump:
```
~/BMCDUMP.XXXXXXX.00000000.20250226205426_out/archive$ head emobjects.log
MESSAGE "a{oa{sa{sv}}}" {
ARRAY "{oa{sa{sv}}}" {
DICT_ENTRY "oa{sa{sv}}" {
OBJECT_PATH "/xyz/openbmc_project/inventory/system/board/Ingraham_Board";
ARRAY "{sa{sv}}" {
DICT_ENTRY "sa{sv}" {
STRING "org.freedesktop.DBus.Peer";
ARRAY "{sv}" {
};
};

```

Change-Id: I612b59b1a0d9483b33beae05f04ab82eafbe72cf
Signed-off-by: Matt Spinler <spinler@us.ibm.com>

show more ...

clang-format: update latest spec and reformat

Copy the latest format file from the docs repository and apply.

Change-Id: Id6ec8ebfc4a679df846e80b35274848b5a8a8866
Signed-off-by: Patrick Williams <p

clang-format: update latest spec and reformat

Copy the latest format file from the docs repository and apply.

Change-Id: Id6ec8ebfc4a679df846e80b35274848b5a8a8866
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

show more ...