diff options
author | Alexander Kanavin <alex.kanavin@gmail.com> | 2022-01-09 23:27:25 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2022-01-11 10:53:44 +0000 |
commit | b93bd0e85ec9e5646e08639771e8d787820df8f8 (patch) | |
tree | 8ba72528c9369573eca192e646baf4e49595c198 /meta/recipes-extended/mdadm/files | |
parent | 633b403b6db40cf1a88a4e952615c03f290227ea (diff) | |
download | poky-b93bd0e85ec9e5646e08639771e8d787820df8f8.tar.gz |
mdadm: update 4.1 -> 4.2
Drop 0001-Compute-abs-diff-in-a-standard-compliant-way.patch
(upstream refactored code)
mdadm-fix-ptest-build-errors.patch
(upstream fixed the issue)
(From OE-Core rev: acf82e36996c082f52c02d50b06965f74ef87430)
Signed-off-by: Alexander Kanavin <alex@linutronix.de>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/recipes-extended/mdadm/files')
3 files changed, 12 insertions, 81 deletions
diff --git a/meta/recipes-extended/mdadm/files/0001-Compute-abs-diff-in-a-standard-compliant-way.patch b/meta/recipes-extended/mdadm/files/0001-Compute-abs-diff-in-a-standard-compliant-way.patch deleted file mode 100644 index 803a59b3ec..0000000000 --- a/meta/recipes-extended/mdadm/files/0001-Compute-abs-diff-in-a-standard-compliant-way.patch +++ /dev/null | |||
@@ -1,31 +0,0 @@ | |||
1 | From b273e892bd6aaafe448f84356abb78a6d463e784 Mon Sep 17 00:00:00 2001 | ||
2 | From: Khem Raj <raj.khem@gmail.com> | ||
3 | Date: Fri, 7 Dec 2018 17:22:39 -0800 | ||
4 | Subject: [PATCH] Compute abs diff in a standard compliant way | ||
5 | |||
6 | This make it a bit less implementation defined and silences clang | ||
7 | warning -Wabsolute-value | ||
8 | |||
9 | | super-intel.c:2822:20: error: taking the absolute value of unsigned type 'unsi | ||
10 | gned long long' has no effect [-Werror,-Wabsolute-value] | ||
11 | |||
12 | Upstream-Status: Pending | ||
13 | |||
14 | Signed-off-by: Khem Raj <raj.khem@gmail.com> | ||
15 | --- | ||
16 | super-intel.c | 2 +- | ||
17 | 1 file changed, 1 insertion(+), 1 deletion(-) | ||
18 | |||
19 | diff --git a/super-intel.c b/super-intel.c | ||
20 | index 6438987..10d7218 100644 | ||
21 | --- a/super-intel.c | ||
22 | +++ b/super-intel.c | ||
23 | @@ -2819,7 +2819,7 @@ static unsigned long long calc_component_size(struct imsm_map *map, | ||
24 | * 2048 blocks per each device. If the difference is higher it means | ||
25 | * that array size was expanded and num_data_stripes was not updated. | ||
26 | */ | ||
27 | - if ((unsigned int)abs(calc_dev_size - dev_size) > | ||
28 | + if ((unsigned int)abs((int)calc_dev_size - (int)dev_size) > | ||
29 | (1 << SECT_PER_MB_SHIFT) * member_disks) { | ||
30 | component_size = dev_size / member_disks; | ||
31 | dprintf("Invalid num_data_stripes in metadata; expected=%llu, found=%llu\n", | ||
diff --git a/meta/recipes-extended/mdadm/files/debian-no-Werror.patch b/meta/recipes-extended/mdadm/files/debian-no-Werror.patch index e66a15cd79..fa90647489 100644 --- a/meta/recipes-extended/mdadm/files/debian-no-Werror.patch +++ b/meta/recipes-extended/mdadm/files/debian-no-Werror.patch | |||
@@ -1,5 +1,7 @@ | |||
1 | From: martin f. krafft <madduck@debian.org> | 1 | From adb75f0bdec97dbe4aa15cc988d349775f7995ff Mon Sep 17 00:00:00 2001 |
2 | Subject: Remove -Werror from compiler flags | 2 | From: "martin f. krafft" <madduck@debian.org> |
3 | Date: Mon, 3 Jan 2022 19:14:12 +0000 | ||
4 | Subject: [PATCH] Remove -Werror from compiler flags | ||
3 | 5 | ||
4 | -Werror seems like a bad idea on released/packaged code because a toolchain | 6 | -Werror seems like a bad idea on released/packaged code because a toolchain |
5 | update (introducing new warnings) could break the build. We'll let upstream | 7 | update (introducing new warnings) could break the build. We'll let upstream |
@@ -8,15 +10,18 @@ use it to beautify the code, but remove it for out builds. | |||
8 | Signed-off-by: martin f. krafft <madduck@debian.org> | 10 | Signed-off-by: martin f. krafft <madduck@debian.org> |
9 | 11 | ||
10 | Upstream-Status: Pending | 12 | Upstream-Status: Pending |
13 | |||
11 | --- | 14 | --- |
12 | Makefile | 2 +- | 15 | Makefile | 2 +- |
13 | 1 files changed, 1 insertions(+), 1 deletions(-) | 16 | 1 file changed, 1 insertion(+), 1 deletion(-) |
14 | 17 | ||
18 | diff --git a/Makefile b/Makefile | ||
19 | index 716c97c..40354ea 100644 | ||
15 | --- a/Makefile | 20 | --- a/Makefile |
16 | +++ b/Makefile | 21 | +++ b/Makefile |
17 | @@ -48,7 +48,7 @@ endif | 22 | @@ -50,7 +50,7 @@ ifeq ($(origin CC),default) |
18 | 23 | CC := $(CROSS_COMPILE)gcc | |
19 | CC ?= $(CROSS_COMPILE)gcc | 24 | endif |
20 | CXFLAGS ?= -ggdb | 25 | CXFLAGS ?= -ggdb |
21 | -CWFLAGS = -Wall -Werror -Wstrict-prototypes -Wextra -Wno-unused-parameter | 26 | -CWFLAGS = -Wall -Werror -Wstrict-prototypes -Wextra -Wno-unused-parameter |
22 | +CWFLAGS = -Wall -Wstrict-prototypes -Wextra -Wno-unused-parameter | 27 | +CWFLAGS = -Wall -Wstrict-prototypes -Wextra -Wno-unused-parameter |
diff --git a/meta/recipes-extended/mdadm/files/mdadm-fix-ptest-build-errors.patch b/meta/recipes-extended/mdadm/files/mdadm-fix-ptest-build-errors.patch deleted file mode 100644 index 8e2a8a9043..0000000000 --- a/meta/recipes-extended/mdadm/files/mdadm-fix-ptest-build-errors.patch +++ /dev/null | |||
@@ -1,43 +0,0 @@ | |||
1 | From 911f6ea9be0c334885aeff66853b111cbd4066df Mon Sep 17 00:00:00 2001 | ||
2 | From: "Maxin B. John" <maxin.john@intel.com> | ||
3 | Date: Wed, 10 Feb 2016 17:28:05 +0200 | ||
4 | Subject: [PATCH] mdadm-fix-ptest-build-errors | ||
5 | |||
6 | builds fail with ptest enabled: | ||
7 | |||
8 | | restripe.c: In function 'test_stripes': | ||
9 | | restripe.c:845:4: error: ignoring return value of 'read', declared with | ||
10 | | attribute warn_unused_result [-Werror=unused-result] | ||
11 | | read(source[i], stripes[i], chunk_size); | ||
12 | | ^ | ||
13 | | cc1: all warnings being treated as errors | ||
14 | | Makefile:214: recipe for target 'test_stripe' failed | ||
15 | |||
16 | Upstream-Status: Pending | ||
17 | |||
18 | Signed-off-by: Maxin B. John <maxin.john@intel.com> | ||
19 | |||
20 | --- | ||
21 | restripe.c | 6 +++++- | ||
22 | 1 file changed, 5 insertions(+), 1 deletion(-) | ||
23 | |||
24 | diff --git a/restripe.c b/restripe.c | ||
25 | index 31b07e8..592ba5d 100644 | ||
26 | --- a/restripe.c | ||
27 | +++ b/restripe.c | ||
28 | @@ -864,10 +864,14 @@ int test_stripes(int *source, unsigned long long *offsets, | ||
29 | |||
30 | while (length > 0) { | ||
31 | int disk; | ||
32 | + ssize_t ret; | ||
33 | |||
34 | for (i = 0 ; i < raid_disks ; i++) { | ||
35 | lseek64(source[i], offsets[i]+start, 0); | ||
36 | - read(source[i], stripes[i], chunk_size); | ||
37 | + ret = read(source[i], stripes[i], chunk_size); | ||
38 | + if (ret == -1) { | ||
39 | + printf("Read Failed\n"); | ||
40 | + } | ||
41 | } | ||
42 | for (i = 0 ; i < data_disks ; i++) { | ||
43 | int disk = geo_map(i, start/chunk_size, raid_disks, | ||