Skip to content

Commit 2f5846c

Browse files
grom72janekmi
andauthored
DAOS-16660 No error message on intentional pmemobj transaction abort (#43)
* DAOS-16660 common: no-error-msg-on-user-abort It also fixes: DAOS-16758 - Unnecessary warning: "Cannot find any matching device, no bad blocks found" for non-pmem HW Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com> Co-authored-by: Jan Michalski <jan-marian.michalski@hpe.com>
1 parent 7fe78c0 commit 2f5846c

File tree

4 files changed

+30
-3
lines changed

4 files changed

+30
-3
lines changed

debian/changelog

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,10 @@
1+
pmdk (2.1.0-3) stable; urgency=medium
2+
* Apply patches to silence annoying error messages on:
3+
* an intentional transaction abort and
4+
* PMDK being used with non-PMem HW.
5+
6+
-- Tomasz Gromadzki <tomasz.gromadzki@intel.com> Wed, 06 Nov 2024 10:00:00 +0000
7+
18
pmdk (2.1.0-2) stable; urgency=medium
29

310
* Enable NDCTL on the top of PMDK 2.1.0

packaging/Makefile_distro_vars.mk

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,14 @@ DISTRO_VERSION ?= $(VERSION_ID)
9191
ORIG_TARGET_VER := 15.5
9292
SED_EXPR := 1p
9393
endif
94+
ifeq ($(CHROOT_NAME),opensuse-leap-15.6-x86_64)
95+
VERSION_ID := 15.6
96+
DISTRO_ID := sl15.6
97+
DISTRO_BASE := LEAP_15
98+
DISTRO_VERSION ?= $(VERSION_ID)
99+
ORIG_TARGET_VER := 15.6
100+
SED_EXPR := 1p
101+
endif
94102
endif
95103
ifeq ($(ID),centos)
96104
ID = el

packaging/get_base_branch

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
set -eux -o pipefail
66
IFS=' ' read -r -a add_bases <<< "${1:-}"
7-
origin=origin
7+
origin="${ORIGIN:-origin}"
88
mapfile -t all_bases < <(echo "master"
99
git branch -r | sed -ne "/^ $origin\\/release\\/[0-9]/s/^ $origin\\///p")
1010
all_bases+=("${add_bases[@]}")

pmdk.spec

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
%global minor 1
1717
%global bugrelease 0
1818
#%%global prerelease rc1
19-
%global buildrelease 2
19+
%global buildrelease 3
2020

2121
%global _hardened_build 1
2222

@@ -37,6 +37,13 @@ URL: https://github.com/pmem/pmdk
3737
}
3838

3939
Source: https://github.com/pmem/%{name}/releases/download/%{upstream_version}/%{name}-%{upstream_version}.tar.gz
40+
%if "%{?commit}" != ""
41+
Patch0: %{version}..%{commit}.patch
42+
%endif
43+
# Fix https://github.com/pmem/pmdk/issues/6107 : Annoying error message on user intentional transaction abort
44+
Patch1: https://github.com/pmem/pmdk/commit/61e32285370e629e2b36bbb991b919e44f87d915.patch
45+
# Fix https://github.com/pmem/pmdk/issues/6126 : Unnecessary warning: "Cannot find any matching device, no bad blocks found" for non-pmem HW
46+
Patch2: https://github.com/pmem/pmdk/commit/518b7426a13b21f98b2d2c435fa645770899446a.patch
4047

4148
BuildRequires: gcc
4249
BuildRequires: make
@@ -403,11 +410,16 @@ make %{make_common_args} check
403410

404411

405412
%changelog
413+
* Wed Nov 06 2024 Tomasz Gromadzki <tomasz.gromadzki@intel.com> - 2.1.0-3
414+
- Apply patches to silence annoying error messages on:
415+
- an intentional transaction abort and
416+
- PMDK being used with non-PMem HW.
417+
406418
* Wed Sep 04 2024 Tomasz.Gromadzki <tomasz.gromadzki@intel.com> - 2.1.0-2
407419
- Enable NDCTL on the top of PMDK 2.1.0
408420
- remove an option to build PMDK w/o NDCTL.
409421

410-
* Tue Aug 06 2024 Tomasz.Gromadzki <tomasz.gromadzki@intel.com> - 2.1.0-1
422+
* Tue Aug 06 2024 Tomasz Gromadzki <tomasz.gromadzki@intel.com> - 2.1.0-1
411423
- Update to release 2.1.0 w/o NDCTL support which:
412424
- Introduces the new logging subsystem in the release build for all libraries.
413425
- Messages by default are printed to syslog and stderr but might be redirected to a user-defined function, see pmem(obj)_log_set_function() for details.

0 commit comments

Comments
 (0)