metrics-ipmi-blobs: Support ECC Error Counts

Export the ECC Error Cournter from
http://github.com/openbmc/phosphor-ecc to ipmi blob.

Tested:
The blob output is updated if MemoryECC dbus object exis

metrics-ipmi-blobs: Support ECC Error Counts

Export the ECC Error Cournter from
http://github.com/openbmc/phosphor-ecc to ipmi blob.

Tested:
The blob output is updated if MemoryECC dbus object exists compared to
the blob output if MemoryECC doesn't exists.

Change-Id: I2c63dbcd0970afc587f5c2ee01f8c261909f0b08
Signed-off-by: Willy Tu <wltu@google.com>

show more ...

ncsid: lib: Fix neighbor path

It no longer has its own hierarchy. Instead, it's directly below the
ethernet object now.

Tested: Verified that neighbors are now correctly identified when they
need t

ncsid: lib: Fix neighbor path

It no longer has its own hierarchy. Instead, it's directly below the
ethernet object now.

Tested: Verified that neighbors are now correctly identified when they
need to be deleted.

Change-Id: Ibd319dcc84fcff99d7c45a5e2fb2de58b6f84406
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

ncsid: update_ra_gw: Improve gateway discovery speed

The rdisc6 binary when run in a subshell buffers all of the output
instead of sending each line. This results in a long delay to figure out
the g

ncsid: update_ra_gw: Improve gateway discovery speed

The rdisc6 binary when run in a subshell buffers all of the output
instead of sending each line. This results in a long delay to figure out
the gateway address.

We can fix this by running it through `script`, which removes the
buffering and allows us to nearly instantly update our route.

Tested: Ran on our AMD Rome system and verified that the gateway was
quickly being updated.

Change-Id: I22350d6bd95665ab853c3f1273c0d6e4ff60718e
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

ncsid: update_ra_gw: Don't depend on phosphor-networkd

The daemon is currently racy for setting the GW settings, don't depend
on it if we don't need to. We always prefer the RA discovered gateway
se

ncsid: update_ra_gw: Don't depend on phosphor-networkd

The daemon is currently racy for setting the GW settings, don't depend
on it if we don't need to. We always prefer the RA discovered gateway
settings, so override them.

Tested: Ran on our AMD Rome system and verified that the gateway files
get generated correctly and the system was using the right routes.

Change-Id: Ibeea9f02370ee8eefdee3eff29ea5c528890951d
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

metrics-ipmi-blobs: Convert to nanopb

Tested: Decoded a metric proto from a host and verified that the fields
all looked as expected and "normal" with decode successfully identifying
everything.

Ch

metrics-ipmi-blobs: Convert to nanopb

Tested: Decoded a metric proto from a host and verified that the fields
all looked as expected and "normal" with decode successfully identifying
everything.

Change-Id: Iaa1b5500db6d1b93fae95d44a1045a858624cc51
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

subprojects: Fix wrapfile provides

Our packages depend on being able to map the dependencies so we should
add them to the wrap.

Change-Id: Ie4500329c28d18bbde8d8c7e51015f5a3ad48466
Signed-off-by: W

subprojects: Fix wrapfile provides

Our packages depend on being able to map the dependencies so we should
add them to the wrap.

Change-Id: Ie4500329c28d18bbde8d8c7e51015f5a3ad48466
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

bare-metal-host-monitor: capture state and action

state/action needs to be captured by value instead of reference.

Change-Id: I68dbc5bd91979b9fc21e7911a4b6b1433a9852ba
Signed-off-by: Yuxiao Zhang <

bare-metal-host-monitor: capture state and action

state/action needs to be captured by value instead of reference.

Change-Id: I68dbc5bd91979b9fc21e7911a4b6b1433a9852ba
Signed-off-by: Yuxiao Zhang <yuxiaozhang@google.com>

show more ...

Bare-metal: post complete GPIO monitor

This creates a service that monitors the post complete gpio, it
reenabled ipmi when gpio is deasserted. And at the startup it checks the
gpio and disable ipmi

Bare-metal: post complete GPIO monitor

This creates a service that monitors the post complete gpio, it
reenabled ipmi when gpio is deasserted. And at the startup it checks the
gpio and disable ipmi if it is asserted.

Tested:
1. reset host, ipmi is re-enabled
2. reboot bmc, ipmi is disabled after bmc booting up.

Change-Id: If52c72ce57e10f6efbb94fd3f0bdcb7655b48d61
Signed-off-by: John Wedig <johnwedig@google.com>
Signed-off-by: Yuxiao Zhang <yuxiaozhang@google.com>

show more ...

update libcr51sign to support MAUV

Add feature support to manage MAUV with cr51 descriptor.

Change-Id: I3795259f81b1dbd3016d1c41c68895be00455061
Signed-off-by: Willy Tu <wltu@google.com>

build: Remove dependencies on fmtlib

With C++23, we can replace fmt::format with std::format and fmt::print
with stdplus::print. This allows us to migrate to std::print once the
support is available

build: Remove dependencies on fmtlib

With C++23, we can replace fmt::format with std::format and fmt::print
with stdplus::print. This allows us to migrate to std::print once the
support is available.

Change-Id: I790901fb3b96d815275a3919e1fb90973189fc02
Signed-off-by: Willy Tu <wltu@google.com>

show more ...

build: use allowed over enabled

Meson feature options are typically in a tri-state of enabled, disabled,
or auto. The enabled and disabled functions on an option (from
`get_option`) no longer retur

build: use allowed over enabled

Meson feature options are typically in a tri-state of enabled, disabled,
or auto. The enabled and disabled functions on an option (from
`get_option`) no longer return true for auto features. Instead, the
expectation is to use `allowed()` which is true for both enabled and auto.

Switch all uses of `enabled` to `allowed`.

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

show more ...

clang-format: copy latest and re-format

clang-format-17 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest

clang-format: copy latest and re-format

clang-format-17 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest .clang-format from the docs repository and reformat the
repository.

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

show more ...

ncsid: Fix if_packet warnings

The new Docker CI now hits the following error.
```
/usr/include/linux/if_packet.h:24:17: error: flexible array member 'sockaddr_ll::<unnamed union>::<unnamed struct>::

ncsid: Fix if_packet warnings

The new Docker CI now hits the following error.
```
/usr/include/linux/if_packet.h:24:17: error: flexible array member 'sockaddr_ll::<unnamed union>::<unnamed struct>::sll_addr_flex' not at end of 'class ncsi::SockIO'
24 | __DECLARE_FLEX_ARRAY(unsigned char, sll_addr_flex);
| ^~~~~~~~~~~~~~~~~~~~
../subprojects/ncsid/src/ncsi_sockio.h:55:15: note: next member 'const int ncsi::SockIO::kpoll_timeout_' declared here
55 | const int kpoll_timeout_ = 10;
| ^~~~~~~~~~~~~~
../subprojects/ncsid/src/ncsi_sockio.h:29:7: note: in the definition of 'class ncsi::SockIO'
29 | class SockIO : public net::SockIO
```

Removed the sockaddr_ll variable.

Change-Id: I67e700f6cd728bfbc822c1b8661915f8306711a1
Signed-off-by: Willy Tu <wltu@google.com>

show more ...

ncsid: Fix build issue

IWYU

Change-Id: Ibd54f428ea8e6e5c9f9dfd2ec8da4fcfbafdb572
Signed-off-by: William A. Kennington III <wak@google.com>

espi-control: Add program to disable eSPI

This commit introduces a program that can disable and re-enable eSPI on
a Nuvoton BMC.

Tested:
- Ran "npcm7xx_espi_control -d", and confirmed that IPMI sto

espi-control: Add program to disable eSPI

This commit introduces a program that can disable and re-enable eSPI on
a Nuvoton BMC.

Tested:
- Ran "npcm7xx_espi_control -d", and confirmed that IPMI stopped
working.
- Ran "npcm7xx_espi_control", and confirmed that IPMI started working
again.

Change-Id: I9637ac4d84b82931f107946ff53e7ef16a965d25
Signed-off-by: John Wedig <johnwedig@google.com>

show more ...

Add dhcp-done to base meson.build

Allow dhcp-done to be part of the presubmit builds.

Change-Id: Ic80658916a2c6de75f279ebc9698a8f189fa36de
Signed-off-by: Willy Tu <wltu@google.com>

iwyu: Cleanup include headers

Triggered iwyu and add/remove used headers.

Change-Id: Iadbdb459563b3c89baf2d56f90a540342d020cd5
Signed-off-by: Willy Tu <wltu@google.com>

cleanup fmt::format for std::format

Update the code to use std::format since gcc13 supports it now. Also
cleaned up some unused headers.

Change-Id: Ief2ad4df9e8b8227168f4f98213da8b3978fda03
Signed-

cleanup fmt::format for std::format

Update the code to use std::format since gcc13 supports it now. Also
cleaned up some unused headers.

Change-Id: Ief2ad4df9e8b8227168f4f98213da8b3978fda03
Signed-off-by: Willy Tu <wltu@google.com>

show more ...

build: upgrade to C++23

Meson 1.1.1 and GCC-13 both support C++23 and a sufficient portion of
the standard has been implemented. Upgrade the build to leverage it.

Change-Id: Ibb8913a93032d388399b6

build: upgrade to C++23

Meson 1.1.1 and GCC-13 both support C++23 and a sufficient portion of
the standard has been implemented. Upgrade the build to leverage it.

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

show more ...

build: Fix build warnings

```
../subprojects/metrics-ipmi-blobs/util.hpp:32:5: error: 'uint64_t' does not name a type
32 | uint64_t firmwareTime = 0;
| ^~~~~~~~
../subprojects/metri

build: Fix build warnings

```
../subprojects/metrics-ipmi-blobs/util.hpp:32:5: error: 'uint64_t' does not name a type
32 | uint64_t firmwareTime = 0;
| ^~~~~~~~
../subprojects/metrics-ipmi-blobs/util.hpp:18:1: note: 'uint64_t' is defined in header '<cstdint>'; did you forget to '#include <cstdint>'?
17 | #include <string>
+++ |+#include <cstdint>
18 | #include <string_view>
...
../subprojects/nemora-postd/src/serializer.cpp:66:28: error: call of overloaded 'format(const char [15], std::string)' is ambiguous
66 | log<level::INFO>(format("NemoraEvent {}", pb.DebugString()).c_str());
```

Change-Id: Id7da1567cdf8cd74d94bd71fa338099b1f0b861e
Signed-off-by: Willy Tu <wltu@google.com>

show more ...

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest

clang-format: copy latest and re-format

clang-format-16 has some backwards incompatible changes that require
additional settings for best compatibility and re-running the formatter.
Copy the latest .clang-format from the docs repository and reformat the
repository.

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

show more ...

meson: remove deprecated get_pkgconfig_variable

Since meson 0.56, the `get_pkgconfig_variable` has been deprecated. In
meson 0.58 the `get_variable` was enhanced to no longer require the
`pkgconfig

meson: remove deprecated get_pkgconfig_variable

Since meson 0.56, the `get_pkgconfig_variable` has been deprecated. In
meson 0.58 the `get_variable` was enhanced to no longer require the
`pkgconfig` keyword argument. Ensure meson 0.58 is required and update
the usage of all `get_pkgconfig_variable` and `get_variable` to be the
modern variant.

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

show more ...

dhcp-done: send status to notify netboot service

This enables the daemon for sending boot status so that netboot service
can take actions accordingly.

Tested with nc and verified that the data byte

dhcp-done: send status to notify netboot service

This enables the daemon for sending boot status so that netboot service
can take actions accordingly.

Tested with nc and verified that the data byte is present.

Change-Id: I570bbd7c05028b6c007ff9d5a00a8589da73a8ff
Signed-off-by: Yuxiao Zhang <yuxiaozhang@google.com>

show more ...

ncsid: Only delete static addresses

Technically this is safe, but it causes errors to be logged.

Change-Id: I5872e2dd7c9da4ce0dc82b80dd642c715d76b0be
Signed-off-by: William A. Kennington III <wak@g

ncsid: Only delete static addresses

Technically this is safe, but it causes errors to be logged.

Change-Id: I5872e2dd7c9da4ce0dc82b80dd642c715d76b0be
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...

ncsid: UpdateIP should not add the nul address

It's technically safe, but it produces errors unnecessarily.

Change-Id: I48aee2560f9bf58553147febe6b3abaf49c7414d
Signed-off-by: William A. Kennington

ncsid: UpdateIP should not add the nul address

It's technically safe, but it produces errors unnecessarily.

Change-Id: I48aee2560f9bf58553147febe6b3abaf49c7414d
Signed-off-by: William A. Kennington III <wak@google.com>

show more ...