Hello community, here is the log from the commit of package ppc64-diag for openSUSE:Factory checked in at 2014-09-29 12:40:17 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/ppc64-diag (Old) and /work/SRC/openSUSE:Factory/.ppc64-diag.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "ppc64-diag" Changes: -------- --- /work/SRC/openSUSE:Factory/ppc64-diag/ppc64-diag.changes 2014-09-10 17:02:48.000000000 +0200 +++ /work/SRC/openSUSE:Factory/.ppc64-diag.new/ppc64-diag.changes 2014-09-29 12:40:18.000000000 +0200 @@ -1,0 +2,9 @@ +Fri Sep 26 09:20:31 UTC 2014 - stefan.f...@suse.com + +- fix bsc #898480 ppc64-diag: fixes for LE support +- patches added: + - ppc64-diag.opal-dump-parse-Add-printk-log-in-the-list-of-dump-s.patch + - ppc64-diag.rtas_errd-Fix-PRRN-Event-handling-on-LE.patch + - ppc64-diag.rtas_errd-LE-fixes-for-extract_platdump.patch + +------------------------------------------------------------------- New: ---- ppc64-diag.opal-dump-parse-Add-printk-log-in-the-list-of-dump-s.patch ppc64-diag.rtas_errd-Fix-PRRN-Event-handling-on-LE.patch ppc64-diag.rtas_errd-LE-fixes-for-extract_platdump.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ ppc64-diag.spec ++++++ --- /var/tmp/diff_new_pack.bEDdEm/_old 2014-09-29 12:40:19.000000000 +0200 +++ /var/tmp/diff_new_pack.bEDdEm/_new 2014-09-29 12:40:19.000000000 +0200 @@ -56,6 +56,9 @@ Patch3: %{name}.opal_errd.patch Patch4: ppc64-diag.add_ncurses_include_path.patch Patch5: ppc64-diag-tmpraces.patch +Patch6: %{name}.opal-dump-parse-Add-printk-log-in-the-list-of-dump-s.patch +Patch7: %{name}.rtas_errd-LE-fixes-for-extract_platdump.patch +Patch8: %{name}.rtas_errd-Fix-PRRN-Event-handling-on-LE.patch %description This package contains various diagnostic tools for PowerLinux. @@ -78,6 +81,9 @@ %patch3 -p1 %patch4 -p1 %patch5 -p1 +%patch6 -p1 +%patch7 -p1 +%patch8 -p1 %build %if 0%{?has_systemd} ++++++ ppc64-diag.opal-dump-parse-Add-printk-log-in-the-list-of-dump-s.patch ++++++ >From f5a0a711ad79e6d1ec8bbf5f561a55582ae8ea03 Mon Sep 17 00:00:00 2001 From: Aruna Balakrishnaiah <ar...@linux.vnet.ibm.com> Date: Thu, 4 Sep 2014 11:09:18 +0530 Subject: [PATCH 1/3] opal-dump-parse: Add printk log in the list of dump sections Signed-off-by: Aruna Balakrishnaiah <ar...@linux.vnet.ibm.com> --- opal-dump-parse/opal-dump-parse.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/opal-dump-parse/opal-dump-parse.h b/opal-dump-parse/opal-dump-parse.h index 1d13029..8fbd928 100644 --- a/opal-dump-parse/opal-dump-parse.h +++ b/opal-dump-parse/opal-dump-parse.h @@ -28,7 +28,8 @@ #define DUMP_SECTION_DESC \ {0, "Unknown"}, \ {1, "Opal-log"}, \ - {2, "HostBoot-Runtime-log"} + {2, "HostBoot-Runtime-log"}, \ + {128, "printk"} #define MDST_SECTION_DESC_LEN 128 -- 1.8.3.1 ++++++ ppc64-diag.rtas_errd-Fix-PRRN-Event-handling-on-LE.patch ++++++ >From f53bb968306ee868de781911e1d906879698c3cd Mon Sep 17 00:00:00 2001 From: "Suzuki K. Poulose" <suz...@in.ibm.com> Date: Tue, 9 Sep 2014 15:24:50 -0400 Subject: [PATCH 3/3] rtas_errd: Fix PRRN Event handling on LE Fixes for PRRN event handling on LE systems 1) Convert the number of properties to host endian in i update_properties(). 2) Use phandle in host endian while issuing an ofdt command, since we pass the phandle as a numeric string to the kernel. With this patch the PRRN event handling works fine, however, drmgr needs to support LE systems for a seemless operation of the rtas_errd, since we depend on drmgr Signed-off-by: Suzuki K. Poulose <suz...@in.ibm.com> Acked-by: Nathan Fontenot <nf...@linux.vnet.ibm.com> Signed-off-by: Vasant Hegde <hegdevas...@linux.vnet.ibm.com> --- rtas_errd/prrn.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/rtas_errd/prrn.c b/rtas_errd/prrn.c index 7a791cb..8226ec1 100644 --- a/rtas_errd/prrn.c +++ b/rtas_errd/prrn.c @@ -342,7 +342,7 @@ static int update_properties(uint32_t phandle) dbg("successful rtas_update_properties (more %d)", rc); op = wa+4; - nprop = *op++; + nprop = be32toh(*op++); /* Should just be on property to update, the affinity. Except * for reconfig memory, that is a single property for @@ -374,7 +374,7 @@ static int update_properties(uint32_t phandle) dbg("%s - delete property %s", pms->name, pname); sprintf(cmd,"remove_property %u %s", - htobe32(phandle), pname); + phandle, pname); do_update(cmd, strlen(cmd)); break; @@ -412,7 +412,7 @@ static int update_properties(uint32_t phandle) */ lenpos = sprintf(longcmd, "update_property %u " - "%s ", htobe32(phandle), + "%s ", phandle, pname); strcat(longcmd, "000000 "); cmdlen = strlen(longcmd); -- 1.8.3.1 ++++++ ppc64-diag.rtas_errd-LE-fixes-for-extract_platdump.patch ++++++ >From 852c42d36ee94f41d9a314eb7fde7c3486a89ffd Mon Sep 17 00:00:00 2001 From: "Suzuki K. Poulose" <suz...@in.ibm.com> Date: Tue, 9 Sep 2014 15:24:49 -0400 Subject: [PATCH 2/3] rtas_errd: LE fixes for extract_platdump Exit with error when we can't retrieve a dump for an invalid dump ID. Signed-off-by: Suzuki K. Poulose <suz...@in.ibm.com> Signed-off-by: Vasant Hegde <hegdevas...@linux.vnet.ibm.com> --- rtas_errd/extract_platdump.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/rtas_errd/extract_platdump.c b/rtas_errd/extract_platdump.c index 75be727..b9afdde 100644 --- a/rtas_errd/extract_platdump.c +++ b/rtas_errd/extract_platdump.c @@ -226,6 +226,13 @@ extract_platform_dump(uint64_t dump_tag) ret = 1; goto platdump_error_out; } + /* If we didn't read enough bytes, handle it */ + if (bytes <= 0) { + msg("Platform dump with id: 0x%016LX is either invalid " + "or empty\n", dump_tag); + ret = 1; + goto platdump_error_out; + } seq = seq_next; -- 1.8.3.1 -- To unsubscribe, e-mail: opensuse-commit+unsubscr...@opensuse.org For additional commands, e-mail: opensuse-commit+h...@opensuse.org