Skip to contentSkip to conversation

mdadm: fix CFLAGS and ptest issues

PATCH In review [email protected] 1 participant 4 messages Nov 9, 2015

lore
Share & tools
lore
#0 lore raw

v2 cover mdadm: fix CFLAGS and ptest issues

From: Wenzong Fan <[email protected]>

V2 change:

* also fix another ptest error while building poky-lsb:

  raid6check.c:352:2: error: ignoring return value of posix_memalign, \
  declared with attribute warn_unused_result [-Werror=unused-result]

The following changes since commit fc45deac89ef63ca1c44e763c38ced7dfd72cbe1:

build-appliance-image: Update to jethro head revision (2015-11-03 14:03:03 +0000)

are available in the git repository at:

git://git.pokylinux.org/poky-contrib wenzong/mdadm
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=wenzong/mdadm

Wenzong Fan (1):
mdadm: fix CFLAGS and ptest issues

 .../mdadm/files/mdadm-fix-ptest-build-errors.patch | 50 ++++++++++++++++++++++
 meta/recipes-extended/mdadm/mdadm_3.3.4.bb         |  3 +-
 2 files changed, 52 insertions(+), 1 deletion(-)
 create mode 100644 meta/recipes-extended/mdadm/files/mdadm-fix-ptest-build-errors.patch
Signature · 1 line
1.9.1
#1 lore raw

PATCH v2 1/1 mdadm: fix CFLAGS and ptest issues

From: Wenzong Fan <[email protected]>

* Pass global CFLAGS to build:

The CFLAGS does not pass to build at all since it was redefined by mdadm Makefile:

  CFLAGS = $(CWFLAGS) $(CXFLAGS) ...

This could be done by setting 'CXFLAGS="${CFLAGS}"'.

* Also fix ptest build errors caused by global CFLAGS:

  raid6check.c:352:2: error: ignoring return value of posix_memalign, \
  declared with attribute warn_unused_result [-Werror=unused-result]

  raid6check.c:315:8: error: 'stripe_buf' may be used uninitialized \
  in this function [-Werror=maybe-uninitialized]
---
 .../mdadm/files/mdadm-fix-ptest-build-errors.patch | 50 ++++++++++++++++++++++
 meta/recipes-extended/mdadm/mdadm_3.3.4.bb         |  3 +-
 2 files changed, 52 insertions(+), 1 deletion(-)
 create mode 100644 meta/recipes-extended/mdadm/files/mdadm-fix-ptest-build-errors.patch
meta/recipes-extended/mdadm/files/mdadm-fix-ptest-build-errors.patch new file +50 -0
new file mode 100644
index 0000000..f7c5514
@@ -0,0 +1,50 @@
+From f3acf8499a4cc400206c5c56f0a6c69192ed55de Mon Sep 17 00:00:00 2001
+From: Wenzong Fan <[email protected]>
+Date: Sat, 7 Nov 2015 04:21:17 -0500
+Subject: [PATCH] mdadm: fix ptest build errors
+
+Check return value for posix_memalign() to fix ptest build error:
+
+  raid6check.c:352:2: error: ignoring return value of posix_memalign, \
+  declared with attribute warn_unused_result [-Werror=unused-result]
+
+Initialize *stripe_buf as NULL to fix ptest build error:
+
+  raid6check.c: In function 'check_stripes':
+  raid6check.c:315:8: error: 'stripe_buf' may be used uninitialized \
+  in this function [-Werror=maybe-uninitialized]
+
+Upstream-Status: Pending
+
+Signed-off-by: Wenzong Fan <[email protected]>
+---
+ raid6check.c | 6 ++++--
+ 1 file changed, 4 insertions(+), 2 deletions(-)
+
+diff --git a/raid6check.c b/raid6check.c
+index cb8522e..9462bcf 100644
+--- a/raid6check.c
++++ b/raid6check.c
+@@ -312,7 +312,7 @@ int check_stripes(struct mdinfo *info, int *source, unsigned long long *offsets,
+ 	/* read the data and p and q blocks, and check we got them right */
+ 	int data_disks = raid_disks - 2;
+ 	int syndrome_disks = data_disks + is_ddf(layout) * 2;
+-	char *stripe_buf;
++	char *stripe_buf = NULL;
+ 
+ 	/* stripes[] is indexed by raid_disk and holds chunks from each device */
+ 	char **stripes = xmalloc(raid_disks * sizeof(char*));
+@@ -349,7 +349,9 @@ int check_stripes(struct mdinfo *info, int *source, unsigned long long *offsets,
+ 	if (!tables_ready)
+ 		make_tables();
+ 
+-	posix_memalign((void**)&stripe_buf, 4096, raid_disks * chunk_size);
++	if (posix_memalign((void**)&stripe_buf, 4096, raid_disks * chunk_size) != 0)
++		goto exitCheck;
++
+ 	block_index_for_slot += 2;
+ 	blocks += 2;
+ 	blocks_page += 2;
+-- 
+1.9.1
+
meta/recipes-extended/mdadm/mdadm_3.3.4.bb +2 -1
index 1198167..0993611 100644
@@ -13,6 +13,7 @@ SRC_URI = "${KERNELORG_MIRROR}/linux/utils/raid/mdadm/${BPN}-${PV}.tar.xz \
            file://gcc-4.9.patch \
            file://mdadm-3.3.2_x32_abi_time_t.patch \
            file://0001-Fix-typo-in-comparision.patch \
+           file://mdadm-fix-ptest-build-errors.patch \
            file://run-ptest \
 	  "
 SRC_URI[md5sum] = "7ca8b114710f98f53f20c5787b674a09"
@@ -22,7 +23,7 @@ CFLAGS += "-fno-strict-aliasing"
 
 inherit autotools-brokensep
 
-EXTRA_OEMAKE = "CHECK_RUN_DIR=0"
+EXTRA_OEMAKE = 'CHECK_RUN_DIR=0 CXFLAGS="${CFLAGS}"'
 # PPC64 and MIPS64 uses long long for u64 in the kernel, but powerpc's asm/types.h
 # prevents 64-bit userland from seeing this definition, instead defaulting
 # to u64 == long in userspace. Define __SANE_USERSPACE_TYPES__ to get
Signature · 1 line
1.9.1
#2 lore raw
In reply to [email protected]

The patch for this thread fails to send, anyways I need to re-send them again.

Please ignore this cover.

Wenzong

Quoted 26 lines · On 11/09/2015 09:16 AM, [email protected] wrote:

From: Wenzong Fan <[email protected]>

V2 change:

* also fix another ptest error while building poky-lsb:

   raid6check.c:352:2: error: ignoring return value of posix_memalign, \
   declared with attribute warn_unused_result [-Werror=unused-result]

The following changes since commit fc45deac89ef63ca1c44e763c38ced7dfd72cbe1:

   build-appliance-image: Update to jethro head revision (2015-11-03 14:03:03 +0000)

are available in the git repository at:

git://git.pokylinux.org/poky-contrib wenzong/mdadm
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=wenzong/mdadm

Wenzong Fan (1):
mdadm: fix CFLAGS and ptest issues

  .../mdadm/files/mdadm-fix-ptest-build-errors.patch | 50 ++++++++++++++++++++++
  meta/recipes-extended/mdadm/mdadm_3.3.4.bb         |  3 +-
  2 files changed, 52 insertions(+), 1 deletion(-)

create mode 100644 meta/recipes-extended/mdadm/files/mdadm-fix-ptest-build-errors.patch

#3 lore raw
In reply to [email protected]

Patch included.

Wenzong

Quoted 26 lines · On 11/09/2015 09:26 AM, [email protected] wrote:

From: Wenzong Fan <[email protected]>

V2 change:

* also fix another ptest error while building poky-lsb:

   raid6check.c:352:2: error: ignoring return value of posix_memalign, \
   declared with attribute warn_unused_result [-Werror=unused-result]

The following changes since commit fc45deac89ef63ca1c44e763c38ced7dfd72cbe1:

   build-appliance-image: Update to jethro head revision (2015-11-03 14:03:03 +0000)

are available in the git repository at:

git://git.pokylinux.org/poky-contrib wenzong/mdadm
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=wenzong/mdadm

Wenzong Fan (1):
mdadm: fix CFLAGS and ptest issues

  .../mdadm/files/mdadm-fix-ptest-build-errors.patch | 50 ++++++++++++++++++++++
  meta/recipes-extended/mdadm/mdadm_3.3.4.bb         |  3 +-
  2 files changed, 52 insertions(+), 1 deletion(-)

create mode 100644 meta/recipes-extended/mdadm/files/mdadm-fix-ptest-build-errors.patch