Hello community,

here is the log from the commit of package ndctl for openSUSE:Factory checked 
in at 2019-11-15 22:32:27
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/ndctl (Old)
 and      /work/SRC/openSUSE:Factory/.ndctl.new.26869 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "ndctl"

Fri Nov 15 22:32:27 2019 rev:25 rq:747514 version:64.1

Changes:
--------
--- /work/SRC/openSUSE:Factory/ndctl/ndctl.changes      2019-10-25 
18:40:30.683805096 +0200
+++ /work/SRC/openSUSE:Factory/.ndctl.new.26869/ndctl.changes   2019-11-15 
22:32:28.604068052 +0100
@@ -1,0 +2,9 @@
+Fri Nov  8 14:42:49 UTC 2019 - Michal Suchanek <msucha...@suse.de>
+
+- ndctl/dimm: Fix init-labels success reporting (bsc#1156308 ltc#181441).
+  + ndctl-dimm-Fix-init-labels-success-reporting.patch
+- Refresh patches with upstream status:
+   ndctl-Use-the-same-align-value-as-original-namespace-on-reconfigure.patch
+   ndctl-ndctl-lib-Fix-duplicate-bus-detection.patch
+
+-------------------------------------------------------------------

New:
----
  ndctl-dimm-Fix-init-labels-success-reporting.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ ndctl.spec ++++++
--- /var/tmp/diff_new_pack.vNMZ90/_old  2019-11-15 22:32:29.412067698 +0100
+++ /var/tmp/diff_new_pack.vNMZ90/_new  2019-11-15 22:32:29.416067696 +0100
@@ -33,6 +33,7 @@
 Patch3:         
%{name}-Use-the-same-align-value-as-original-namespace-on-reconfigure.patch
 Patch4:         %{name}-ndctl-lib-Fix-duplicate-bus-detection.patch
 Patch5:         
%{name}-namespace-Handle-create-namespace-in-label-less-mode.patch
+Patch6:         %{name}-dimm-Fix-init-labels-success-reporting.patch
 BuildRequires:  autoconf
 BuildRequires:  automake
 BuildRequires:  keyutils-devel

++++++ 
ndctl-Use-the-same-align-value-as-original-namespace-on-reconfigure.patch ++++++
--- /var/tmp/diff_new_pack.vNMZ90/_old  2019-11-15 22:32:29.448067682 +0100
+++ /var/tmp/diff_new_pack.vNMZ90/_new  2019-11-15 22:32:29.448067682 +0100
@@ -4,7 +4,7 @@
 Subject: [PATCH] ndctl: Reuse the align value from the original namespace on
  reconfiguration
 
-Patch-mainline: expected v67
+Patch-mainline: v67
 Git-commit: e81ccd7f062beeae7498d04785fe55c55c857d4b
 
 When using reconfigure command to add a 'name' to the namespace we end

++++++ ndctl-dimm-Fix-init-labels-success-reporting.patch ++++++
>From 4f7cad25e06483ae20b9bebabefb05fb1e55a2a7 Mon Sep 17 00:00:00 2001
From: Dan Williams <dan.j.willi...@intel.com>
Date: Sat, 27 Jul 2019 14:40:42 -0700
Subject: [PATCH] ndctl/dimm: Fix init-labels success reporting

Patch-mainline: v66
Git-commit: 4f7cad25e06483ae20b9bebabefb05fb1e55a2a7

When a DIMM is disabled due to a label parsing issue "ndctl init-labels"
mis-reports the status of the init-labels command:

    # ndctl init-labels all -f
    initialized 1 nmem
    [root@dwillia2-dev ndctl]# ndctl list -Di
    [
      {
        "dev":"nmem1",
        "id":"8680-57341200",
        "handle":2,
        "phys_id":0,
        "state":"disabled"
      },
      {
        "dev":"nmem0",
        "id":"8680-56341200",
        "handle":1,
        "phys_id":0
      }
    ]
    # ndctl init-labels nmem1 -f
    initialized 1020 nmems

Catch any positive return from action_init() as success:

    # ndctl init-labels all -f
    initialized 2 nmems
    # ndctl init-labels nmem1 -f
    initialized 1 nmem

Link: https://github.com/pmem/ndctl/issues/60
Reported-by: Jane Chu <jane....@oracle.com>
Reported-by: Steve Scargal <steve.scarg...@intel.com>
Signed-off-by: Dan Williams <dan.j.willi...@intel.com>
Signed-off-by: Vishal Verma <vishal.l.ve...@intel.com>
---
 ndctl/dimm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ndctl/dimm.c b/ndctl/dimm.c
index 5f05a75f00eb..db91f42421e4 100644
--- a/ndctl/dimm.c
+++ b/ndctl/dimm.c
@@ -1027,7 +1027,7 @@ static int __action_init(struct ndctl_dimm *dimm,
 
  out:
        ndctl_cmd_unref(cmd_read);
-       return rc;
+       return rc >= 0 ? 0 : rc;
 }
 
 static int action_init(struct ndctl_dimm *dimm, struct action_context *actx)
-- 
2.23.0

++++++ ndctl-ndctl-lib-Fix-duplicate-bus-detection.patch ++++++
--- /var/tmp/diff_new_pack.vNMZ90/_old  2019-11-15 22:32:29.460067677 +0100
+++ /var/tmp/diff_new_pack.vNMZ90/_new  2019-11-15 22:32:29.464067675 +0100
@@ -3,7 +3,7 @@
 Date: Wed, 7 Aug 2019 10:51:39 -0700
 Subject: [PATCH] ndctl/lib: Fix duplicate bus detection
 
-Patch-mainline: expected v67
+Patch-mainline: v67
 Git-commit: d961a474be65c8195bb069c470b01f951bd2eb90
 
 After an ndctl_invalidate() event the next add_bus() invocation attempts


Reply via email to