Script 'mail_helper' called by obssrc
Hello community,
here is the log from the commit of package powerpc-utils for openSUSE:Factory
checked in at 2022-01-14 23:12:45
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/powerpc-utils (Old)
and /work/SRC/openSUSE:Factory/.powerpc-utils.new.1892 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "powerpc-utils"
Fri Jan 14 23:12:45 2022 rev:122 rq:945829 version:1.3.9
Changes:
--------
--- /work/SRC/openSUSE:Factory/powerpc-utils/powerpc-utils.changes
2022-01-07 12:45:47.807821107 +0100
+++ /work/SRC/openSUSE:Factory/.powerpc-utils.new.1892/powerpc-utils.changes
2022-01-14 23:13:20.562635738 +0100
@@ -1,0 +2,12 @@
+Wed Jan 12 12:37:04 UTC 2022 - Michal Suchanek <msucha...@suse.com>
+
+- errinjct: sanitize devspec output of a newline if one is present
(bsc#1194579 ltc#195746).
+ + errinjct-sanitize-devspec-output-of-a-newline-if-one.patch
+
+-------------------------------------------------------------------
+Wed Jan 5 21:45:23 UTC 2022 - Michal Suchanek <msucha...@suse.com>
+
+- Fix reported online memory in legacy format (bsc#1191147).
+ + lparstat-Fix-reported-online-memory-in-legacy-format.patch
+
+-------------------------------------------------------------------
New:
----
errinjct-sanitize-devspec-output-of-a-newline-if-one.patch
lparstat-Fix-reported-online-memory-in-legacy-format.patch
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ powerpc-utils.spec ++++++
--- /var/tmp/diff_new_pack.hcmVSf/_old 2022-01-14 23:13:21.110636092 +0100
+++ /var/tmp/diff_new_pack.hcmVSf/_new 2022-01-14 23:13:21.114636094 +0100
@@ -34,6 +34,8 @@
Patch8: 0002-hcnmgr-Remove-some-dead-code.patch
Patch9: 0003-ofpathname-Fix-nvme-support-in-ANA-mode.patch
Patch10: 0004-ofpathname-Add-support-for-NVMf-devices.patch
+Patch11: lparstat-Fix-reported-online-memory-in-legacy-format.patch
+Patch12: errinjct-sanitize-devspec-output-of-a-newline-if-one.patch
Patch14: fix_kexec_service_name_for_suse.patch
BuildRequires: autoconf
BuildRequires: automake
++++++ errinjct-sanitize-devspec-output-of-a-newline-if-one.patch ++++++
>From d8408b429ff022ff446ac2607bc26eaf60627658 Mon Sep 17 00:00:00 2001
From: Tyrel Datwyler <tyr...@linux.ibm.com>
Date: Mon, 10 Jan 2022 16:55:58 -0500
Subject: [PATCH] errinjct: sanitize devspec output of a newline if one is
present
References: bsc#1194579 ltc#195746
Upstream: accepted (expected 1.3.10)
Git-commit: d8408b429ff022ff446ac2607bc26eaf60627658
Linux Kernel Commit: 14c19b2a40b6 ("PCI/sysfs: Add 'devspec' newline")
introduced a newline character at the end of the devspec result which
historically did not have one before. When a newline is present the
errinjct utility constructs an invalid pathname for a PCI devices /reg
property in the device tree. As a result the tool fails to provide a
valid config address to RTAS resulting in a (-3) Parameter Error.
errinjct eeh -v -f 6 -s net/eth1 -a 0xe0800000 -m 0xff800000
errinjct: Could not open file
/proc/device-tree//pci@800000020000154/ethernet@0,1
/reg, No such file or directory
Injecting an ioa-bus-error with the following data:
BUS ADDR: e0800000
ADDR MASK: ff800000
CONFIG ADDR: 1
PHB UNIT_ID: 800000020000154
FUNCTION: 6
Store to PCI Memory Address Space - inject an Address Parity Error
errinjct: RTAS error injection failed!
errinjct: RTAS: ioa-bus-error: Argument error (-3)
Fix this issue by nul terminating the devspec output string at the newline
if present.
Signed-off-by: Tyrel Datwyler <tyr...@linux.ibm.com>
---
src/errinjct/ioa_bus_error.c | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/src/errinjct/ioa_bus_error.c b/src/errinjct/ioa_bus_error.c
index 281b56b..fc38037 100644
--- a/src/errinjct/ioa_bus_error.c
+++ b/src/errinjct/ioa_bus_error.c
@@ -232,7 +232,7 @@ static int parse_sysfsname(void)
{
char path[BUFSZ];
char *devspec;
- char *at;
+ char *at, *nl;
uint32_t addr;
uint64_t phb_id;
@@ -247,6 +247,14 @@ static int parse_sysfsname(void)
if (!devspec)
return 1;
+ /* Linux Kernel Commit: 14c19b2a40b6 ("PCI/sysfs: Add 'devspec'
newline")
+ * began reporting the devspec value for pci devices with a trailing
newline.
+ * Remove the newline if present to ensure our built pathname for
obtaining
+ * the config address is valid. */
+ nl = strchr(devspec, '\n');
+ if (nl)
+ *nl = '\0';
+
/* Now we parse something like /pci@400000000112/pci@2/ethernet@1 for
* BUID HI =4000 and LOW 00000112 */
at = strchr(devspec, '@');
--
2.34.1
++++++ lparstat-Fix-reported-online-memory-in-legacy-format.patch ++++++
>From 5d2d0a0b50ff386b7d9416e122d05214846573b6 Mon Sep 17 00:00:00 2001
From: Thomas Abraham <tabra...@suse.com>
Date: Tue, 12 Oct 2021 09:21:38 -0400
Subject: [PATCH] lparstat: Fix reported online memory in legacy format
References: bsc#1191147
Upstream: merged - expected in 1.3.10
Git-commit: 5d2d0a0b50ff386b7d9416e122d05214846573b6
On systems with more than 2TB of online memory, legacy mode
will calculate and report a negative value due to exceeding
the maximum value for a signed 32-bit integer:
$ lparstat -i | grep "Online Memory"
Online Memory : 3771084032 kB
$ lparstat -il | grep "Online Memory"
Online Memory : -511604 MB
Signed-off-by: Thomas Abraham <tabra...@suse.com>
Signed-off-by: Tyrel Datwyler <tyr...@linux.ibm.com>
---
src/lparstat.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/lparstat.c b/src/lparstat.c
index 00922c4..5ea1a62 100644
--- a/src/lparstat.c
+++ b/src/lparstat.c
@@ -834,7 +834,7 @@ void get_mem_total(struct sysentry *se, char *buf)
*nl = '\0';
if (o_legacy) {
- sprintf(buf, "%d %s", atoi(mem) / 1024, "MB");
+ sprintf(buf, "%lld %s", atoll(mem) / 1024, "MB");
} else {
sprintf(buf, "%s %s", mem, unit);
}
--
2.34.1
