Fix spelling mistakes using codespell

This commit corrects various spelling mistakes throughout the
repository. The corrections were made automatically using `codespell`[1]
tool.

[1]: https://githu

Fix spelling mistakes using codespell

This commit corrects various spelling mistakes throughout the
repository. The corrections were made automatically using `codespell`[1]
tool.

[1]: https://github.com/codespell-project/codespell

Change-Id: I539a0c5baa63be240fc1c76367c0af338dd89c7b
Signed-off-by: Manojkiran Eda <manojkiran.eda@gmail.com>

show more ...

oem-ibm: Add util API to get the connector object paths

This commit adds a common API to find all the port object paths
if present under a connected adapter.

Change-Id: I340838570a3c5add02ef92ecac3

oem-ibm: Add util API to get the connector object paths

This commit adds a common API to find all the port object paths
if present under a connected adapter.

Change-Id: I340838570a3c5add02ef92ecac3df9278b0faffb
Signed-off-by: Pavithra Barithaya <pavithrabarithaya07@gmail.com>

show more ...

oem-ibm: Check if a FRU is IBM specific

The major difference between the standard FRUs
and the IBM specific FRUs is the format in which
the VPD data is written and its model type.
If the model prope

oem-ibm: Check if a FRU is IBM specific

The major difference between the standard FRUs
and the IBM specific FRUs is the format in which
the VPD data is written and its model type.
If the model property is filled it is a IBM specific FRU,
if not then that can be an industry standard FRU.

Change-Id: I9c01b99e37107bc3caeb88336626bc9e514e78f0
Signed-off-by: Sampa Misra <sampmisr@in.ibm.com>

show more ...

Fix pldm bump fail

Fix for the bump CI failure on yosemite4 systems
https://gerrit.openbmc.org/c/openbmc/openbmc/+/68035

TESTED: Ran Bitbake for a yosemite4 obmc-phosphor-image

Signed-off-by: Saga

Fix pldm bump fail

Fix for the bump CI failure on yosemite4 systems
https://gerrit.openbmc.org/c/openbmc/openbmc/+/68035

TESTED: Ran Bitbake for a yosemite4 obmc-phosphor-image

Signed-off-by: Sagar Srinivas <sagar.srinivas@ibm.com>
Change-Id: I5f8766d31b6841bd5e337584779bbd3d59131ac0

show more ...

oem_ibm: Persist the remote PDRs before merging

This commit persists the remote range PDRs in the repo
before they are merged. This change was needed as the
remote terminus would not send down to BM

oem_ibm: Persist the remote PDRs before merging

This commit persists the remote range PDRs in the repo
before they are merged. This change was needed as the
remote terminus would not send down to BMC the entity
Association PDRs of the remote terminus after the BMC
takes a reset after the PDR exchange as the PDRs will be
in BMC range of record handles.

TESTED: Tested by triggering a PDR exchange with the
remote terminus by doing poweron test and also reset the
BMC after the PDR exchange.

Change-Id: Iec2109073f984e5980d9b6e445ce0fa4198d4a0f
Signed-off-by: Sagar Srinivas <sagar.srinivas@ibm.com>

show more ...

pldmd: instance_id: Fix compilation under GCC 13

GCC 13 appears to have refactored some system headers so make sure to
include what we use[1].

[1]: https://include-what-you-use.org/

Resolves the f

pldmd: instance_id: Fix compilation under GCC 13

GCC 13 appears to have refactored some system headers so make sure to
include what we use[1].

[1]: https://include-what-you-use.org/

Resolves the following errors:

```
[8/113] Compiling C++ object libpldmresponder/test/libpldmresponder_base_test.p/.._.._pldmd_instance_id.cpp.o
FAILED: libpldmresponder/test/libpldmresponder_base_test.p/.._.._pldmd_instance_id.cpp.o
ccache c++ -Ilibpldmresponder/test/libpldmresponder_base_test.p -Irequester -I../requester -Isubprojects/libpldm/include -I../subprojects/libpldm/include -Isubprojects/libpldm/src -I../subprojects/libpldm/src -Isubprojects/libpldm/include/libpldm -I../subprojects/libpldm/include/libpldm -Ilibpldmresponder -I../libpldmresponder -I. -I.. -I../subprojects/googletest/googletest/include -I../subprojects/googletest/googletest -Isubprojects/googletest/__CMake_build -I../subprojects/googletest/__CMake_build -Isubprojects/googletest -I../subprojects/googletest -I../subprojects/googletest/googlemock/include -I../subprojects/googletest/googlemock -I../subprojects/nlohmann-json/single_include -Isubprojects/phosphor-dbus-interfaces/gen -I../subprojects/phosphor-dbus-interfaces/gen -I../subprojects/sdbusplus/include -Isubprojects/phosphor-logging/lib/include -I../subprojects/phosphor-logging/lib/include -Isubprojects/sdeventplus/src -I../subprojects/sdeventplus/src -Isubprojects/stdplus/include -I../subprojects/stdplus/include -Isubprojects/stdplus/include-fd -I../subprojects/stdplus/include-fd -I../subprojects/fmt/include -Isubprojects/fmt/__CMake_build -I../subprojects/fmt/__CMake_build -Isubprojects/fmt -I../subprojects/fmt -I../subprojects/function2/include -Isubprojects/function2/__CMake_build -I../subprojects/function2/__CMake_build -Isubprojects/function2 -I../subprojects/function2 -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -Wpedantic -Werror -std=c++20 -O2 -g -Wno-psabi -DLIBPLDMRESPONDER -DFMT_LOCALE -DBOOST_ASIO_DISABLE_THREADS -DBOOST_ALL_NO_LIB -DBOOST_SYSTEM_NO_DEPRECATED -DBOOST_ERROR_CODE_HEADER_ONLY -DBOOST_COROUTINES_NO_DEPRECATION_WARNING -pthread -isystem../subprojects/googletest/googletest -isystem../subprojects/googletest/googletest/include -MD -MQ libpldmresponder/test/libpldmresponder_base_test.p/.._.._pldmd_instance_id.cpp.o -MF libpldmresponder/test/libpldmresponder_base_test.p/.._.._pldmd_instance_id.cpp.o.d -o libpldmresponder/test/libpldmresponder_base_test.p/.._.._pldmd_instance_id.cpp.o -c ../pldmd/instance_id.cpp
In file included from ../pldmd/instance_id.cpp:1:
../pldmd/instance_id.hpp:19:5: error: ‘uint8_t’ does not name a type
19 | uint8_t next();
| ^~~~~~~
../pldmd/instance_id.hpp:4:1: note: ‘uint8_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
3 | #include <bitset>
+++ |+#include <cstdint>
4 |
../pldmd/instance_id.hpp:24:19: error: ‘uint8_t’ has not been declared
24 | void markFree(uint8_t instanceId)
| ^~~~~~~
../pldmd/instance_id.cpp:8:1: error: ‘uint8_t’ does not name a type
8 | uint8_t InstanceId::next()
| ^~~~~~~
../pldmd/instance_id.cpp:4:1: note: ‘uint8_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
3 | #include <stdexcept>
+++ |+#include <cstdint>
4 |
[9/113] Compiling C++ object libpldmresponder/test/libpldmresponder_base_test.p/.._.._pldmd_dbus_impl_requester.cpp.o
FAILED: libpldmresponder/test/libpldmresponder_base_test.p/.._.._pldmd_dbus_impl_requester.cpp.o
ccache c++ -Ilibpldmresponder/test/libpldmresponder_base_test.p -Irequester -I../requester -Isubprojects/libpldm/include -I../subprojects/libpldm/include -Isubprojects/libpldm/src -I../subprojects/libpldm/src -Isubprojects/libpldm/include/libpldm -I../subprojects/libpldm/include/libpldm -Ilibpldmresponder -I../libpldmresponder -I. -I.. -I../subprojects/googletest/googletest/include -I../subprojects/googletest/googletest -Isubprojects/googletest/__CMake_build -I../subprojects/googletest/__CMake_build -Isubprojects/googletest -I../subprojects/googletest -I../subprojects/googletest/googlemock/include -I../subprojects/googletest/googlemock -I../subprojects/nlohmann-json/single_include -Isubprojects/phosphor-dbus-interfaces/gen -I../subprojects/phosphor-dbus-interfaces/gen -I../subprojects/sdbusplus/include -Isubprojects/phosphor-logging/lib/include -I../subprojects/phosphor-logging/lib/include -Isubprojects/sdeventplus/src -I../subprojects/sdeventplus/src -Isubprojects/stdplus/include -I../subprojects/stdplus/include -Isubprojects/stdplus/include-fd -I../subprojects/stdplus/include-fd -I../subprojects/fmt/include -Isubprojects/fmt/__CMake_build -I../subprojects/fmt/__CMake_build -Isubprojects/fmt -I../subprojects/fmt -I../subprojects/function2/include -Isubprojects/function2/__CMake_build -I../subprojects/function2/__CMake_build -Isubprojects/function2 -I../subprojects/function2 -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -Wpedantic -Werror -std=c++20 -O2 -g -Wno-psabi -DLIBPLDMRESPONDER -DFMT_LOCALE -DBOOST_ASIO_DISABLE_THREADS -DBOOST_ALL_NO_LIB -DBOOST_SYSTEM_NO_DEPRECATED -DBOOST_ERROR_CODE_HEADER_ONLY -DBOOST_COROUTINES_NO_DEPRECATION_WARNING -pthread -isystem../subprojects/googletest/googletest -isystem../subprojects/googletest/googletest/include -MD -MQ libpldmresponder/test/libpldmresponder_base_test.p/.._.._pldmd_dbus_impl_requester.cpp.o -MF libpldmresponder/test/libpldmresponder_base_test.p/.._.._pldmd_dbus_impl_requester.cpp.o.d -o libpldmresponder/test/libpldmresponder_base_test.p/.._.._pldmd_dbus_impl_requester.cpp.o -c ../pldmd/dbus_impl_requester.cpp
In file included from ../pldmd/dbus_impl_requester.hpp:3,
from ../pldmd/dbus_impl_requester.cpp:1:
../pldmd/instance_id.hpp:19:5: error: ‘uint8_t’ does not name a type
19 | uint8_t next();
| ^~~~~~~
../pldmd/instance_id.hpp:4:1: note: ‘uint8_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
3 | #include <bitset>
+++ |+#include <cstdint>
4 |
../pldmd/instance_id.hpp:24:19: error: ‘uint8_t’ has not been declared
24 | void markFree(uint8_t instanceId)
| ^~~~~~~
../pldmd/dbus_impl_requester.cpp: In member function ‘virtual uint8_t pldm::dbus_api::Requester::getInstanceId(uint8_t)’:
../pldmd/dbus_impl_requester.cpp:25:23: error: ‘std::map<unsigned char, pldm::InstanceId>::mapped_type’ {aka ‘class pldm::InstanceId’} has no member named ‘next’
25 | id = ids[eid].next();
| ^~~~
[19/113] Compiling C++ object libpldmresponder/libpldmresponder.so.0.1.p/.._host-bmc_dbus_to_host_effecters.cpp.o
ninja: build stopped: subcommand failed.
```

Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Change-Id: I8aab8ee5c9b9fe66e868b829342f4c2b071f0127

show more ...

oem-ibm: Modified to write system dump data on unix domain socket.

Currently pldm writes dump data on nbd device.

This commit is to enable dump offload using UNIX socket
On dump

oem-ibm: Modified to write system dump data on unix domain socket.

Currently pldm writes dump data on nbd device.

This commit is to enable dump offload using UNIX socket
On dump offload request, pldm setup a UNIX socket and
write data on socket, webserver connects to this socket
and reads data to offload

Tested By:
1. Offloaded system dump
2. Ran below pldmtool commands for performance test
while offloading system dump, i don't see an delay
pldmtool bios GetBIOSTable -t 1
pldmtool bios GetBIOSTable -t 2

Signed-off-by: Ravi Teja <raviteja28031990@gmail.com>
Change-Id: Iad8863d87c3b04a8dd588f1f0239f65fcb59f38b

show more ...