* [PATCH] IMSM: do not use comma expression for simple assignments
@ 2014-08-06 8:19 Jürg Billeter
2014-12-11 9:06 ` Artur Paszkiewicz
0 siblings, 1 reply; 2+ messages in thread
From: Jürg Billeter @ 2014-08-06 8:19 UTC (permalink / raw)
To: NeilBrown; +Cc: linux-raid, Jürg Billeter
Fixes build with gcc 4.9.
error: right-hand operand of comma expression has no effect
snprintf((char *) spare->sig, MAX_SIGNATURE_LENGTH,
^
Signed-off-by: Jürg Billeter <j@bitron.ch>
---
super-intel.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/super-intel.c b/super-intel.c
index e28ac7d..f24d101 100644
--- a/super-intel.c
+++ b/super-intel.c
@@ -5069,13 +5069,13 @@ static int write_super_imsm_spares(struct intel_super *super, int doclose)
__u32 sum;
struct dl *d;
- spare->mpb_size = __cpu_to_le32(sizeof(struct imsm_super)),
- spare->generation_num = __cpu_to_le32(1UL),
+ spare->mpb_size = __cpu_to_le32(sizeof(struct imsm_super));
+ spare->generation_num = __cpu_to_le32(1UL);
spare->attributes = MPB_ATTRIB_CHECKSUM_VERIFY;
- spare->num_disks = 1,
- spare->num_raid_devs = 0,
- spare->cache_size = mpb->cache_size,
- spare->pwr_cycle_count = __cpu_to_le32(1),
+ spare->num_disks = 1;
+ spare->num_raid_devs = 0;
+ spare->cache_size = mpb->cache_size;
+ spare->pwr_cycle_count = __cpu_to_le32(1);
snprintf((char *) spare->sig, MAX_SIGNATURE_LENGTH,
MPB_SIGNATURE MPB_VERSION_RAID0);
--
2.0.4
--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] IMSM: do not use comma expression for simple assignments
2014-08-06 8:19 [PATCH] IMSM: do not use comma expression for simple assignments Jürg Billeter
@ 2014-12-11 9:06 ` Artur Paszkiewicz
0 siblings, 0 replies; 2+ messages in thread
From: Artur Paszkiewicz @ 2014-12-11 9:06 UTC (permalink / raw)
To: Jürg Billeter, NeilBrown; +Cc: linux-raid, Baldysiak, Pawel
On 08/06/2014 10:19 AM, Jürg Billeter wrote:
> Fixes build with gcc 4.9.
>
> error: right-hand operand of comma expression has no effect
> snprintf((char *) spare->sig, MAX_SIGNATURE_LENGTH,
> ^
>
> Signed-off-by: Jürg Billeter <j@bitron.ch>
> ---
> super-intel.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/super-intel.c b/super-intel.c
> index e28ac7d..f24d101 100644
> --- a/super-intel.c
> +++ b/super-intel.c
> @@ -5069,13 +5069,13 @@ static int write_super_imsm_spares(struct intel_super *super, int doclose)
> __u32 sum;
> struct dl *d;
>
> - spare->mpb_size = __cpu_to_le32(sizeof(struct imsm_super)),
> - spare->generation_num = __cpu_to_le32(1UL),
> + spare->mpb_size = __cpu_to_le32(sizeof(struct imsm_super));
> + spare->generation_num = __cpu_to_le32(1UL);
> spare->attributes = MPB_ATTRIB_CHECKSUM_VERIFY;
> - spare->num_disks = 1,
> - spare->num_raid_devs = 0,
> - spare->cache_size = mpb->cache_size,
> - spare->pwr_cycle_count = __cpu_to_le32(1),
> + spare->num_disks = 1;
> + spare->num_raid_devs = 0;
> + spare->cache_size = mpb->cache_size;
> + spare->pwr_cycle_count = __cpu_to_le32(1);
>
> snprintf((char *) spare->sig, MAX_SIGNATURE_LENGTH,
> MPB_SIGNATURE MPB_VERSION_RAID0);
>
Hi Neil,
We verified that this patch indeed is needed to build mdadm with gcc
4.9. Can you apply it?
Thanks,
Artur
--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-12-11 9:06 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-06 8:19 [PATCH] IMSM: do not use comma expression for simple assignments Jürg Billeter
2014-12-11 9:06 ` Artur Paszkiewicz
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.