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

Reply via email to