clang-format: re-format for clang-19

clang-format-19 isn't compatible with the clang-format-18 output, so we
need to reformat the code with the latest version. A few parameters
in clang-tidy have b

clang-format: re-format for clang-19

clang-format-19 isn't compatible with the clang-format-18 output, so we
need to reformat the code with the latest version. A few parameters
in clang-tidy have been deprecated, so adjust the style file
accordingly.

See Ie2f6eb3b043f2d655c9df806815afd7971fd0947 for updated style.
See I88192b41ab7a95599a90915013579608af7bc56f for clang-19 enablement.

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

show more ...

ncsid: skip setting mac address if it matches

There are cases where NIC keeps trying to set MAC
address of already set interface, which can lead
to ncsid turning interface down/up non-stop. A
worka

ncsid: skip setting mac address if it matches

There are cases where NIC keeps trying to set MAC
address of already set interface, which can lead
to ncsid turning interface down/up non-stop. A
workaround is to check if mac matches before setting.

Change-Id: I376cfdde62d5e6a1021fd3410d48027644c7ddde
Signed-off-by: Mo Elbadry <elbadrym@google.com>

show more ...

host-gpio-monitor: Use systemd template

The systemd target gbmc-bare-metal-active.target is being changed to a
template, in order to support multi-host systems. The commit updates the
host-gpio-moni

host-gpio-monitor: Use systemd template

The systemd target gbmc-bare-metal-active.target is being changed to a
template, in order to support multi-host systems. The commit updates the
host-gpio-monitor functionality accordingly. The host number now gets
passed in as an argument.

Tested:
Set the bare metal flag /var/google/config-package/enable-bm.flag on the
BMC. Started host-gpio-monitor@0.service, which then started
gbmc-bare-metal-active@0.target. Then rebooted the host, at which point,
the host-gpio-monitor@0.service disabled
gbmc-bare-metal-active@0.target.

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

show more ...

ncsid: Set MAC address using ioctl calls

Set the MAC address directly via ioctl calls, removing the
dependency on phosphor-networkd. This simplifies the process
and reduces overhead as phosphor-netw

ncsid: Set MAC address using ioctl calls

Set the MAC address directly via ioctl calls, removing the
dependency on phosphor-networkd. This simplifies the process
and reduces overhead as phosphor-networkd is not extensively
utilized in this context.

Change-Id: If3465f52834ba38c551d5152ef354cbf455f6a90
Signed-off-by: Mo Elbadry <elbadrym@google.com>

show more ...

metric-ipmi-blobs: adding tmpfs

Adding tmpfs metric coverage to pre-ttf machines

Tested:
```
df -h
Filesystem Size Used Available Use% Mounted on
dev 468.9M

metric-ipmi-blobs: adding tmpfs

Adding tmpfs metric coverage to pre-ttf machines

Tested:
```
df -h
Filesystem Size Used Available Use% Mounted on
dev 468.9M 0 468.9M 0% /dev
tmpfs 469.9M 12.6M 457.3M 3% /run
/dev/mtdblock4 20.6M 20.6M 0 100% /run/initramfs/ro
/dev/mtdblock6 3.0M 436.0K 2.6M 14% /run/initramfs/rw
cow 3.0M 436.0K 2.6M 14% /
tmpfs 469.9M 8.0K 469.9M 0% /dev/shm
tmpfs 469.9M 0 469.9M 0% /tmp
tmpfs 469.9M 40.0K 469.8M 0% /var/volatile
tmpfs 469.9M 12.6M 457.3M 3% /var/lib/systemd/coredump
```

Change-Id: I50c763707268252ed51a903d8b7e02543dab6166
Signed-off-by: Abby <buchmana@google.com>

show more ...

bare-metal-gpio-monitor: don't start on non-bm mode

This adds a condition check for the bm flag. So that the service will
not take effect on non-bm mode.

Change-Id: I0d15247a6e05aa5edf93468793cd74b

bare-metal-gpio-monitor: don't start on non-bm mode

This adds a condition check for the bm flag. So that the service will
not take effect on non-bm mode.

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

show more ...

meson: switch code dependency to libsystemd

For a while now systemd has packaged a separate `libsystemd`. For
meson dependency, `systemd` should be used when looking for systemd
service file paths

meson: switch code dependency to libsystemd

For a while now systemd has packaged a separate `libsystemd`. For
meson dependency, `systemd` should be used when looking for systemd
service file paths and `libsystemd` should be used for `sd_*` functions.

Update the dependency accordingly.

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

show more ...

clang-format: re-format for clang-18

clang-format-18 isn't compatible with the clang-format-17 output, so we
need to reformat the code with the latest version. The way clang-18
handles lambda forma

clang-format: re-format for clang-18

clang-format-18 isn't compatible with the clang-format-17 output, so we
need to reformat the code with the latest version. The way clang-18
handles lambda formatting also changed, so we have made changes to the
organization default style format to better handle lambda formatting.

See I5e08687e696dd240402a2780158664b7113def0e for updated style.
See Iea0776aaa7edd483fa395e23de25ebf5a6288f71 for clang-18 enablement.

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

show more ...

meta-google: dhcp-done: Adding status report

Previously dhcp-done only sends status code, this one provides the
capability to send status code + status message for better
troubleshooting.
Provide a

meta-google: dhcp-done: Adding status report

Previously dhcp-done only sends status code, this one provides the
capability to send status code + status message for better
troubleshooting.
Provide a way to let other process upgrade the status.

Tested: Unit test passed.

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

show more ...

ncsid: fix default wait time

The default wait time needs to shift with current timestamp, otherwise
the script will keep looping with 0 intervals and 0 wait time in RA, if
the initial loop does not

ncsid: fix default wait time

The default wait time needs to shift with current timestamp, otherwise
the script will keep looping with 0 intervals and 0 wait time in RA, if
the initial loop does not capture any RA data.

Tested: bring down the interface to simulate the nic not ready, start
this script and verified that it loops every 10s. Then bring up the
interface, and see the route is setup with no issue

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

show more ...

ncsid: update_ra_gw: Fix old_rtr check

An iteration of a previous commit changed the default value of `old_rtr`
from empty string to explicitly `invalid`. We need to use the correct
comparison for e

ncsid: update_ra_gw: Fix old_rtr check

An iteration of a previous commit changed the default value of `old_rtr`
from empty string to explicitly `invalid`. We need to use the correct
comparison for early router setting.

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

show more ...

Remove Ed from Google-misc maintainers

Change-Id: I92a9e7ece6a2ae170989d0022376d5b0a1e7f12b
Signed-off-by: Ed Tanous <ed@tanous.net>

ncsid: Make fixup run for all RA

We want to ensure we get regular fixes to the router state without
switching to different routers when our route accidentally disappears.

Change-Id: I14c4ea63b51474

ncsid: Make fixup run for all RA

We want to ensure we get regular fixes to the router state without
switching to different routers when our route accidentally disappears.

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

show more ...

ncsid: Reduce networkd reloads

This should help to avoid race conditions with phosphor-networkd. We
still write the networkd files so that when reloads occur the
information is updated correctly.

C

ncsid: Reduce networkd reloads

This should help to avoid race conditions with phosphor-networkd. We
still write the networkd files so that when reloads occur the
information is updated correctly.

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

show more ...

ncsid: Prefer fe80::1 gateway

Our labs and production environment normally use this as the gateway
address. We often have noisy lab networks with many bad gateways
advertising their presence. We kno

ncsid: Prefer fe80::1 gateway

Our labs and production environment normally use this as the gateway
address. We often have noisy lab networks with many bad gateways
advertising their presence. We know we generally prefer fe80::1 so we
switch to it when available.

Tested: On a noisy lab network and verified that the script correctly
locks onto fe80::1 and switches away from other gateways it receives.

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

show more ...

ncsid: Find IPv6 gateway stably

Some environments have multiple gateways for the node to use. We want to
ensure that we aren't flip flopping between multiple gateways and lock
on to a specific gatew

ncsid: Find IPv6 gateway stably

Some environments have multiple gateways for the node to use. We want to
ensure that we aren't flip flopping between multiple gateways and lock
on to a specific gateway until it goes away.

Tested: Ran in an environment with multiple gateways and verified that
it did not switch after it acquired the first one.

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

show more ...

ncsid: Remove unused services

We no longer need IPv4 support or any of the neighbor finding logic, as
we have all of the neighbor information updated via the primary RA from
the router.

Tested: Ran

ncsid: Remove unused services

We no longer need IPv4 support or any of the neighbor finding logic, as
we have all of the neighbor information updated via the primary RA from
the router.

Tested: Ran on a lab and prod machine to verify the networking behavior
is still consistent and gateway information is updated accurately.

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

show more ...

ncsid: reconfigure gateway when default route is missing

This will ensure that default gateway can be reconfigured if any failure
happens at the first time.

Tested:
On a machine that has default ro

ncsid: reconfigure gateway when default route is missing

This will ensure that default gateway can be reconfigured if any failure
happens at the first time.

Tested:
On a machine that has default route configured, manually deleted the
route, without this change it will stay broken. This change will bring
the route back in 1mins or so.

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

show more ...

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 ...