Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/79256?usp=email )
Change subject: configs: Rename UART_DEBUG to INTEL_LPSS_UART_FOR_CONSOLE
......................................................................
configs: Rename UART_DEBUG to INTEL_LPSS_UART_FOR_CONSOLE
This follows commit a96e66a76f2 which did the rename across the tree
except here. Since Kconfig is going to become more strict about unknown
symbols, fix it.
Change-Id: I3b855085d4be13622e8f38ff651d576e719b682c
Signed-off-by: Patrick Georgi <patrick(a)coreboot.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/79256
Reviewed-by: Arthur Heymans <arthur(a)aheymans.xyz>
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
Reviewed-by: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
---
M configs/config.google_meep_cros
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Arthur Heymans: Looks good to me, approved
build bot (Jenkins): Verified
Felix Singer: Looks good to me, approved
diff --git a/configs/config.google_meep_cros b/configs/config.google_meep_cros
index c8e3227..5dfd8e3 100644
--- a/configs/config.google_meep_cros
+++ b/configs/config.google_meep_cros
@@ -33,5 +33,5 @@
CONFIG_CONSOLE_SERIAL_115200=y
# CONFIG_DRIVERS_UART_8250IO is not set
# GLK specific setting to auto select all the correct settings.
-CONFIG_UART_DEBUG=y
+CONFIG_INTEL_LPSS_UART_FOR_CONSOLE=y
CONFIG_NO_BOOTBLOCK_CONSOLE=y
--
To view, visit https://review.coreboot.org/c/coreboot/+/79256?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I3b855085d4be13622e8f38ff651d576e719b682c
Gerrit-Change-Number: 79256
Gerrit-PatchSet: 2
Gerrit-Owner: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Reviewer: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: merged
Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/79255?usp=email )
Change subject: configs: Drop references to MRC_CLEAR_NORMAL_CACHE_ON_RECOVERY_RETRAIN
......................................................................
configs: Drop references to MRC_CLEAR_NORMAL_CACHE_ON_RECOVERY_RETRAIN
This follows commit 6615c6eaf79 which removed the symbol. Since Kconfig
is going to become more strict about unknown symbols, fix it.
Change-Id: I7b7f2e4c0774919a55083f7c5348f2b5031c8287
Signed-off-by: Patrick Georgi <patrick(a)coreboot.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/79255
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
Reviewed-by: Arthur Heymans <arthur(a)aheymans.xyz>
Reviewed-by: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
---
M configs/config.google_meep_cros
1 file changed, 0 insertions(+), 1 deletion(-)
Approvals:
build bot (Jenkins): Verified
Arthur Heymans: Looks good to me, approved
Felix Singer: Looks good to me, approved
diff --git a/configs/config.google_meep_cros b/configs/config.google_meep_cros
index 97e23fd..c8e3227 100644
--- a/configs/config.google_meep_cros
+++ b/configs/config.google_meep_cros
@@ -10,7 +10,6 @@
# ChromeOS
CONFIG_CHROMEOS=y
CONFIG_HAS_RECOVERY_MRC_CACHE=y
-CONFIG_MRC_CLEAR_NORMAL_CACHE_ON_RECOVERY_RETRAIN=y
# Event Logging
CONFIG_CMOS_POST=y
--
To view, visit https://review.coreboot.org/c/coreboot/+/79255?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I7b7f2e4c0774919a55083f7c5348f2b5031c8287
Gerrit-Change-Number: 79255
Gerrit-PatchSet: 2
Gerrit-Owner: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Reviewer: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: merged
Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/79254?usp=email )
Change subject: configs: Drop references to CPU_QEMU_X86_SMMLOADERV2
......................................................................
configs: Drop references to CPU_QEMU_X86_SMMLOADERV2
This follows commit 88407bcd which removed the symbol. Since Kconfig
is going to become more strict about unknown symbols, fix it.
Change-Id: I19d26de8003c51437ea62e04083a14c3587a4665
Signed-off-by: Patrick Georgi <patrick(a)coreboot.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/79254
Reviewed-by: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
Reviewed-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M configs/config.emulation_qemu_x86_q35_smm_tseg
1 file changed, 0 insertions(+), 1 deletion(-)
Approvals:
build bot (Jenkins): Verified
Felix Singer: Looks good to me, approved
Arthur Heymans: Looks good to me, approved
diff --git a/configs/config.emulation_qemu_x86_q35_smm_tseg b/configs/config.emulation_qemu_x86_q35_smm_tseg
index 4b3e6e5..0ec233b 100644
--- a/configs/config.emulation_qemu_x86_q35_smm_tseg
+++ b/configs/config.emulation_qemu_x86_q35_smm_tseg
@@ -1,4 +1,3 @@
CONFIG_VENDOR_EMULATION=y
CONFIG_BOARD_EMULATION_QEMU_X86_Q35=y
CONFIG_CPU_QEMU_X86_TSEG_SMM=y
-CONFIG_CPU_QEMU_X86_SMMLOADERV2=y
--
To view, visit https://review.coreboot.org/c/coreboot/+/79254?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I19d26de8003c51437ea62e04083a14c3587a4665
Gerrit-Change-Number: 79254
Gerrit-PatchSet: 2
Gerrit-Owner: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Reviewer: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: merged
Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/79253?usp=email )
Change subject: configs: Drop references to CPU_QEMU_X86_PARALLEL_MP
......................................................................
configs: Drop references to CPU_QEMU_X86_PARALLEL_MP
This follows commit e2d291b5 which removed the symbol. Since Kconfig
is going to become more strict about unknown symbols, fix it.
Change-Id: I838f98d07fc0448dda6c02b58d7c5639992c77a2
Signed-off-by: Patrick Georgi <patrick(a)coreboot.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/79253
Reviewed-by: Arthur Heymans <arthur(a)aheymans.xyz>
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
Reviewed-by: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
---
M configs/config.emulation_qemu_x86_i440fx_debug
M configs/config.emulation_qemu_x86_q35_smm_tseg
2 files changed, 0 insertions(+), 2 deletions(-)
Approvals:
Arthur Heymans: Looks good to me, approved
build bot (Jenkins): Verified
Felix Singer: Looks good to me, approved
diff --git a/configs/config.emulation_qemu_x86_i440fx_debug b/configs/config.emulation_qemu_x86_i440fx_debug
index 3cff033..85f5499 100644
--- a/configs/config.emulation_qemu_x86_i440fx_debug
+++ b/configs/config.emulation_qemu_x86_i440fx_debug
@@ -8,4 +8,3 @@
CONFIG_DEBUG_MALLOC=y
CONFIG_DEBUG_BOOT_STATE=y
CONFIG_DEBUG_ADA_CODE=y
-CONFIG_CPU_QEMU_X86_PARALLEL_MP=y
diff --git a/configs/config.emulation_qemu_x86_q35_smm_tseg b/configs/config.emulation_qemu_x86_q35_smm_tseg
index 6d64715..4b3e6e5 100644
--- a/configs/config.emulation_qemu_x86_q35_smm_tseg
+++ b/configs/config.emulation_qemu_x86_q35_smm_tseg
@@ -1,5 +1,4 @@
CONFIG_VENDOR_EMULATION=y
CONFIG_BOARD_EMULATION_QEMU_X86_Q35=y
-CONFIG_CPU_QEMU_X86_PARALLEL_MP=y
CONFIG_CPU_QEMU_X86_TSEG_SMM=y
CONFIG_CPU_QEMU_X86_SMMLOADERV2=y
--
To view, visit https://review.coreboot.org/c/coreboot/+/79253?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I838f98d07fc0448dda6c02b58d7c5639992c77a2
Gerrit-Change-Number: 79253
Gerrit-PatchSet: 2
Gerrit-Owner: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Reviewer: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: merged
Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/79252?usp=email )
Change subject: configs: Rename PXE_SERIAL_CONSOLE to IPXE_SERIAL_CONSOLE
......................................................................
configs: Rename PXE_SERIAL_CONSOLE to IPXE_SERIAL_CONSOLE
This follows commit 238ff1e9c which did the rename across the tree
except here. Since Kconfig is going to become more strict about unknown
symbols, fix it.
Change-Id: Ic31b8ae353ec07e8b8adab46b604365be4be44d9
Signed-off-by: Patrick Georgi <patrick(a)coreboot.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/79252
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
Reviewed-by: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Reviewed-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M configs/config.pcengines_apu1
M configs/config.pcengines_apu2
M configs/config.pcengines_apu3
M configs/config.pcengines_apu4
M configs/config.pcengines_apu5
5 files changed, 5 insertions(+), 5 deletions(-)
Approvals:
build bot (Jenkins): Verified
Arthur Heymans: Looks good to me, approved
Felix Singer: Looks good to me, approved
diff --git a/configs/config.pcengines_apu1 b/configs/config.pcengines_apu1
index e511d65..a3e34b8 100644
--- a/configs/config.pcengines_apu1
+++ b/configs/config.pcengines_apu1
@@ -6,5 +6,5 @@
CONFIG_PXE=y
CONFIG_BUILD_IPXE=y
CONFIG_PXE_ROM_ID="10ec,8168"
-# CONFIG_PXE_SERIAL_CONSOLE is not set
+# CONFIG_IPXE_SERIAL_CONSOLE is not set
CONFIG_MEMTEST_SECONDARY_PAYLOAD=y
diff --git a/configs/config.pcengines_apu2 b/configs/config.pcengines_apu2
index 08d4d4e..fe0a9d7 100644
--- a/configs/config.pcengines_apu2
+++ b/configs/config.pcengines_apu2
@@ -8,5 +8,5 @@
CONFIG_PXE=y
CONFIG_BUILD_IPXE=y
CONFIG_PXE_ROM_ID="8086,157b"
-# CONFIG_PXE_SERIAL_CONSOLE is not set
+# CONFIG_IPXE_SERIAL_CONSOLE is not set
CONFIG_MEMTEST_SECONDARY_PAYLOAD=y
diff --git a/configs/config.pcengines_apu3 b/configs/config.pcengines_apu3
index 7acf014..bf37715 100644
--- a/configs/config.pcengines_apu3
+++ b/configs/config.pcengines_apu3
@@ -8,5 +8,5 @@
CONFIG_PXE=y
CONFIG_BUILD_IPXE=y
CONFIG_PXE_ROM_ID="8086,1539"
-# CONFIG_PXE_SERIAL_CONSOLE is not set
+# CONFIG_IPXE_SERIAL_CONSOLE is not set
CONFIG_MEMTEST_SECONDARY_PAYLOAD=y
diff --git a/configs/config.pcengines_apu4 b/configs/config.pcengines_apu4
index 9e7ca5a..17db794 100644
--- a/configs/config.pcengines_apu4
+++ b/configs/config.pcengines_apu4
@@ -8,5 +8,5 @@
CONFIG_PXE=y
CONFIG_BUILD_IPXE=y
CONFIG_PXE_ROM_ID="8086,1539"
-# CONFIG_PXE_SERIAL_CONSOLE is not set
+# CONFIG_IPXE_SERIAL_CONSOLE is not set
CONFIG_MEMTEST_SECONDARY_PAYLOAD=y
diff --git a/configs/config.pcengines_apu5 b/configs/config.pcengines_apu5
index e5ddcee..b004e89 100644
--- a/configs/config.pcengines_apu5
+++ b/configs/config.pcengines_apu5
@@ -8,5 +8,5 @@
CONFIG_PXE=y
CONFIG_BUILD_IPXE=y
CONFIG_PXE_ROM_ID="8086,1539"
-# CONFIG_PXE_SERIAL_CONSOLE is not set
+# CONFIG_IPXE_SERIAL_CONSOLE is not set
CONFIG_MEMTEST_SECONDARY_PAYLOAD=y
--
To view, visit https://review.coreboot.org/c/coreboot/+/79252?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ic31b8ae353ec07e8b8adab46b604365be4be44d9
Gerrit-Change-Number: 79252
Gerrit-PatchSet: 2
Gerrit-Owner: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Reviewer: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: merged
Martin L Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/79147?usp=email )
Change subject: Docs/releases: Finalize 4.22/4.22.01 release notes
......................................................................
Docs/releases: Finalize 4.22/4.22.01 release notes
Now that the 4.22 release tag has been added to git, update the release
notes with the final statistics and wording.
We also decided to add a fix submitted immediately after the 4.22
release was tagged into the release package and do a point release.
This also adds an expected date for the next release
Signed-off-by: Martin Roth <gaumless(a)gmail.com>
Change-Id: Iae9653a275fcc1d11efbb88e12676f332be0a5dc
Reviewed-on: https://review.coreboot.org/c/coreboot/+/79147
Reviewed-by: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
---
M Documentation/releases/coreboot-4.22-relnotes.md
1 file changed, 22 insertions(+), 11 deletions(-)
Approvals:
build bot (Jenkins): Verified
Felix Singer: Looks good to me, approved
diff --git a/Documentation/releases/coreboot-4.22-relnotes.md b/Documentation/releases/coreboot-4.22-relnotes.md
index e32607a..ff978c2 100644
--- a/Documentation/releases/coreboot-4.22-relnotes.md
+++ b/Documentation/releases/coreboot-4.22-relnotes.md
@@ -1,7 +1,7 @@
-Upcoming release - coreboot 4.22
+coreboot 4.22 & 4.22.01 releases
========================================================================
-The next release is planned for mid-February, 2024
+The next release is planned for the 19th of February, 2024
These notes cover the latest updates and improvements to coreboot over
the past three months. A big thank you to the returning contributors as
@@ -11,6 +11,17 @@
security enhancements, and expanding hardware support.
+### 4.22.01 release
+
+The week between tagging a release and announcing it publicly is used
+to test the tagged version and make sure everything is working as we
+expect. This is done instead of freezing the tree and doing release
+candidates before the release.
+
+For the 4.22 release cycle we found an uninitialized variable error on
+the sandybridge/ivybridge platforms and rolled that into the 4.22.01
+release package.
+
### coreboot version naming update
This release is the last release to use the incrementing 4.xx release
@@ -235,15 +246,15 @@
Statistics from the 4.21 to the 4.22 release
--------------------------------------------
-* Total Commits: 885
-* Average Commits per day: 10.72
-* Total lines added: 58276
-* Average lines added per commit: 65.85
-* Number of patches adding more than 100 lines: 54
-* Average lines added per small commit: 37.77
-* Total lines removed: 27790
-* Average lines removed per commit: 31.40
-* Total difference between added and removed: 30486
+* Total Commits: 977
+* Average Commits per day: 10.98
+* Total lines added: 62993
+* Average lines added per commit: 64.48
+* Number of patches adding more than 100 lines: 60
+* Average lines added per small commit: 37.55
+* Total lines removed: 30042
+* Average lines removed per commit: 30.75
+* Total difference between added and removed: 32951
* Total authors: 135
* New authors: 14
--
To view, visit https://review.coreboot.org/c/coreboot/+/79147?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Iae9653a275fcc1d11efbb88e12676f332be0a5dc
Gerrit-Change-Number: 79147
Gerrit-PatchSet: 3
Gerrit-Owner: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Reviewer: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Reviewer: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: merged
Martin L Roth has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/59636?usp=email )
Change subject: Revert "util/crossgcc: Update gcc to 11.2"
......................................................................
Abandoned
These patches aren't for merge, so move them to an abandoned state. They can always be brought back to be rebased on the main branch.
--
To view, visit https://review.coreboot.org/c/coreboot/+/59636?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I09a789f9ccb284ac7cad32b12bbf60ecb82efb44
Gerrit-Change-Number: 59636
Gerrit-PatchSet: 3
Gerrit-Owner: Mike Banon <mikebdp2(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: abandon
Martin L Roth has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/63679?usp=email )
Change subject: Revert "crossgcc/buildgcc: Remove unused GCC_AUTOCONF_VERSION"
......................................................................
Abandoned
These patches aren't for merge, so move them to an abandoned state. They can always be brought back to be rebased on the main branch.
--
To view, visit https://review.coreboot.org/c/coreboot/+/63679?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I0bc6772fc68ce8997c463e67ccfa8a911c076b94
Gerrit-Change-Number: 63679
Gerrit-PatchSet: 2
Gerrit-Owner: Mike Banon <mikebdp2(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-CC: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-MessageType: abandon
Attention is currently required from: Elyes Haouas, Felix Singer, Julius Werner, Nico Huber, Patrick Georgi.
Martin L Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/70771?usp=email )
Change subject: crossgcc: Upgrade GCC from 11.4.0 to 13.2.0
......................................................................
Patch Set 32: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/70771?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I4f2ed4de4811abaa13528906de71eee29a8f2910
Gerrit-Change-Number: 70771
Gerrit-PatchSet: 32
Gerrit-Owner: Elyes Haouas <ehaouas(a)noos.fr>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Reviewer: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Reviewer: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Julius Werner <jwerner(a)chromium.org>
Gerrit-CC: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-CC: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-CC: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-CC: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Attention: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Attention: Nico Huber <nico.h(a)gmx.de>
Gerrit-Attention: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Attention: Julius Werner <jwerner(a)chromium.org>
Gerrit-Attention: Elyes Haouas <ehaouas(a)noos.fr>
Gerrit-Comment-Date: Fri, 24 Nov 2023 23:30:39 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment