* [PATCH 1/2] lib/raid6: remove duplicated CFLAGS_REMOVE_altivec8.o
@ 2019-06-16 17:48 Masahiro Yamada
2019-06-16 17:48 ` [PATCH 2/2] lib/raid6: refactor unroll rules with pattern rules Masahiro Yamada
2019-06-20 15:55 ` [PATCH 1/2] lib/raid6: remove duplicated CFLAGS_REMOVE_altivec8.o Masahiro Yamada
0 siblings, 2 replies; 4+ messages in thread
From: Masahiro Yamada @ 2019-06-16 17:48 UTC (permalink / raw)
To: linux-kbuild
Cc: Masahiro Yamada, Michael Ellerman, linux-kernel, Russell King,
Stefan Agner, Joel Stanley, Nick Desaulniers, Nathan Chancellor
No intended change in behavior.
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
---
lib/raid6/Makefile | 1 -
1 file changed, 1 deletion(-)
diff --git a/lib/raid6/Makefile b/lib/raid6/Makefile
index e723eacf7868..74004037033f 100644
--- a/lib/raid6/Makefile
+++ b/lib/raid6/Makefile
@@ -26,7 +26,6 @@ CFLAGS_REMOVE_altivec1.o += -msoft-float
CFLAGS_REMOVE_altivec2.o += -msoft-float
CFLAGS_REMOVE_altivec4.o += -msoft-float
CFLAGS_REMOVE_altivec8.o += -msoft-float
-CFLAGS_REMOVE_altivec8.o += -msoft-float
CFLAGS_REMOVE_vpermxor1.o += -msoft-float
CFLAGS_REMOVE_vpermxor2.o += -msoft-float
CFLAGS_REMOVE_vpermxor4.o += -msoft-float
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 2/2] lib/raid6: refactor unroll rules with pattern rules
2019-06-16 17:48 [PATCH 1/2] lib/raid6: remove duplicated CFLAGS_REMOVE_altivec8.o Masahiro Yamada
@ 2019-06-16 17:48 ` Masahiro Yamada
2019-06-20 15:55 ` Masahiro Yamada
2019-06-20 15:55 ` [PATCH 1/2] lib/raid6: remove duplicated CFLAGS_REMOVE_altivec8.o Masahiro Yamada
1 sibling, 1 reply; 4+ messages in thread
From: Masahiro Yamada @ 2019-06-16 17:48 UTC (permalink / raw)
To: linux-kbuild
Cc: Masahiro Yamada, Michael Ellerman, linux-kernel, Joel Stanley,
Nick Desaulniers, Nathan Chancellor
This Makefile repeats very similar rules.
Let's use pattern rules. $(UNROLL) can be replaced with $*.
No intended change in behavior.
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
---
lib/raid6/Makefile | 97 ++++++----------------------------------------
1 file changed, 11 insertions(+), 86 deletions(-)
diff --git a/lib/raid6/Makefile b/lib/raid6/Makefile
index 74004037033f..42695bc8d451 100644
--- a/lib/raid6/Makefile
+++ b/lib/raid6/Makefile
@@ -12,9 +12,6 @@ raid6_pq-$(CONFIG_S390) += s390vx8.o recov_s390xc.o
hostprogs-y += mktables
-quiet_cmd_unroll = UNROLL $@
- cmd_unroll = $(AWK) -f$(srctree)/$(src)/unroll.awk -vN=$(UNROLL) < $< > $@
-
ifeq ($(CONFIG_ALTIVEC),y)
altivec_flags := -maltivec $(call cc-option,-mabi=altivec)
@@ -50,111 +47,39 @@ CFLAGS_REMOVE_neon8.o += -mgeneral-regs-only
endif
endif
-targets += int1.c
-$(obj)/int1.c: UNROLL := 1
-$(obj)/int1.c: $(src)/int.uc $(src)/unroll.awk FORCE
- $(call if_changed,unroll)
-
-targets += int2.c
-$(obj)/int2.c: UNROLL := 2
-$(obj)/int2.c: $(src)/int.uc $(src)/unroll.awk FORCE
- $(call if_changed,unroll)
-
-targets += int4.c
-$(obj)/int4.c: UNROLL := 4
-$(obj)/int4.c: $(src)/int.uc $(src)/unroll.awk FORCE
- $(call if_changed,unroll)
-
-targets += int8.c
-$(obj)/int8.c: UNROLL := 8
-$(obj)/int8.c: $(src)/int.uc $(src)/unroll.awk FORCE
- $(call if_changed,unroll)
-
-targets += int16.c
-$(obj)/int16.c: UNROLL := 16
-$(obj)/int16.c: $(src)/int.uc $(src)/unroll.awk FORCE
- $(call if_changed,unroll)
+quiet_cmd_unroll = UNROLL $@
+ cmd_unroll = $(AWK) -f$(srctree)/$(src)/unroll.awk -vN=$* < $< > $@
-targets += int32.c
-$(obj)/int32.c: UNROLL := 32
-$(obj)/int32.c: $(src)/int.uc $(src)/unroll.awk FORCE
+targets += int1.c int2.c int4.c int8.c int16.c int32.c
+$(obj)/int%.c: $(src)/int.uc $(src)/unroll.awk FORCE
$(call if_changed,unroll)
CFLAGS_altivec1.o += $(altivec_flags)
-targets += altivec1.c
-$(obj)/altivec1.c: UNROLL := 1
-$(obj)/altivec1.c: $(src)/altivec.uc $(src)/unroll.awk FORCE
- $(call if_changed,unroll)
-
CFLAGS_altivec2.o += $(altivec_flags)
-targets += altivec2.c
-$(obj)/altivec2.c: UNROLL := 2
-$(obj)/altivec2.c: $(src)/altivec.uc $(src)/unroll.awk FORCE
- $(call if_changed,unroll)
-
CFLAGS_altivec4.o += $(altivec_flags)
-targets += altivec4.c
-$(obj)/altivec4.c: UNROLL := 4
-$(obj)/altivec4.c: $(src)/altivec.uc $(src)/unroll.awk FORCE
- $(call if_changed,unroll)
-
CFLAGS_altivec8.o += $(altivec_flags)
-targets += altivec8.c
-$(obj)/altivec8.c: UNROLL := 8
-$(obj)/altivec8.c: $(src)/altivec.uc $(src)/unroll.awk FORCE
+targets += altivec1.c altivec2.c altivec4.c altivec8.c
+$(obj)/altivec%.c: $(src)/altivec.uc $(src)/unroll.awk FORCE
$(call if_changed,unroll)
CFLAGS_vpermxor1.o += $(altivec_flags)
-targets += vpermxor1.c
-$(obj)/vpermxor1.c: UNROLL := 1
-$(obj)/vpermxor1.c: $(src)/vpermxor.uc $(src)/unroll.awk FORCE
- $(call if_changed,unroll)
-
CFLAGS_vpermxor2.o += $(altivec_flags)
-targets += vpermxor2.c
-$(obj)/vpermxor2.c: UNROLL := 2
-$(obj)/vpermxor2.c: $(src)/vpermxor.uc $(src)/unroll.awk FORCE
- $(call if_changed,unroll)
-
CFLAGS_vpermxor4.o += $(altivec_flags)
-targets += vpermxor4.c
-$(obj)/vpermxor4.c: UNROLL := 4
-$(obj)/vpermxor4.c: $(src)/vpermxor.uc $(src)/unroll.awk FORCE
- $(call if_changed,unroll)
-
CFLAGS_vpermxor8.o += $(altivec_flags)
-targets += vpermxor8.c
-$(obj)/vpermxor8.c: UNROLL := 8
-$(obj)/vpermxor8.c: $(src)/vpermxor.uc $(src)/unroll.awk FORCE
+targets += vpermxor1.o vpermxor2.o vpermxor4.o vpermxor8.o
+$(obj)/vpermxor%.c: $(src)/vpermxor.uc $(src)/unroll.awk FORCE
$(call if_changed,unroll)
CFLAGS_neon1.o += $(NEON_FLAGS)
-targets += neon1.c
-$(obj)/neon1.c: UNROLL := 1
-$(obj)/neon1.c: $(src)/neon.uc $(src)/unroll.awk FORCE
- $(call if_changed,unroll)
-
CFLAGS_neon2.o += $(NEON_FLAGS)
-targets += neon2.c
-$(obj)/neon2.c: UNROLL := 2
-$(obj)/neon2.c: $(src)/neon.uc $(src)/unroll.awk FORCE
- $(call if_changed,unroll)
-
CFLAGS_neon4.o += $(NEON_FLAGS)
-targets += neon4.c
-$(obj)/neon4.c: UNROLL := 4
-$(obj)/neon4.c: $(src)/neon.uc $(src)/unroll.awk FORCE
- $(call if_changed,unroll)
-
CFLAGS_neon8.o += $(NEON_FLAGS)
-targets += neon8.c
-$(obj)/neon8.c: UNROLL := 8
-$(obj)/neon8.c: $(src)/neon.uc $(src)/unroll.awk FORCE
+targets += neon1.c neon2.c neon4.c neon8.c
+$(obj)/neon%.c: $(src)/neon.uc $(src)/unroll.awk FORCE
$(call if_changed,unroll)
targets += s390vx8.c
-$(obj)/s390vx8.c: UNROLL := 8
-$(obj)/s390vx8.c: $(src)/s390vx.uc $(src)/unroll.awk FORCE
+$(obj)/s390vx%.c: $(src)/s390vx.uc $(src)/unroll.awk FORCE
$(call if_changed,unroll)
quiet_cmd_mktable = TABLE $@
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 1/2] lib/raid6: remove duplicated CFLAGS_REMOVE_altivec8.o
2019-06-16 17:48 [PATCH 1/2] lib/raid6: remove duplicated CFLAGS_REMOVE_altivec8.o Masahiro Yamada
2019-06-16 17:48 ` [PATCH 2/2] lib/raid6: refactor unroll rules with pattern rules Masahiro Yamada
@ 2019-06-20 15:55 ` Masahiro Yamada
1 sibling, 0 replies; 4+ messages in thread
From: Masahiro Yamada @ 2019-06-20 15:55 UTC (permalink / raw)
To: Linux Kbuild mailing list
Cc: Michael Ellerman, Linux Kernel Mailing List, Russell King,
Stefan Agner, Joel Stanley, Nick Desaulniers, Nathan Chancellor
On Mon, Jun 17, 2019 at 2:50 AM Masahiro Yamada
<yamada.masahiro@socionext.com> wrote:
>
> No intended change in behavior.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> ---
Applied to linux-kbuild.
> lib/raid6/Makefile | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/lib/raid6/Makefile b/lib/raid6/Makefile
> index e723eacf7868..74004037033f 100644
> --- a/lib/raid6/Makefile
> +++ b/lib/raid6/Makefile
> @@ -26,7 +26,6 @@ CFLAGS_REMOVE_altivec1.o += -msoft-float
> CFLAGS_REMOVE_altivec2.o += -msoft-float
> CFLAGS_REMOVE_altivec4.o += -msoft-float
> CFLAGS_REMOVE_altivec8.o += -msoft-float
> -CFLAGS_REMOVE_altivec8.o += -msoft-float
> CFLAGS_REMOVE_vpermxor1.o += -msoft-float
> CFLAGS_REMOVE_vpermxor2.o += -msoft-float
> CFLAGS_REMOVE_vpermxor4.o += -msoft-float
> --
> 2.17.1
>
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 2/2] lib/raid6: refactor unroll rules with pattern rules
2019-06-16 17:48 ` [PATCH 2/2] lib/raid6: refactor unroll rules with pattern rules Masahiro Yamada
@ 2019-06-20 15:55 ` Masahiro Yamada
0 siblings, 0 replies; 4+ messages in thread
From: Masahiro Yamada @ 2019-06-20 15:55 UTC (permalink / raw)
To: Linux Kbuild mailing list
Cc: Michael Ellerman, Linux Kernel Mailing List, Joel Stanley,
Nick Desaulniers, Nathan Chancellor
On Mon, Jun 17, 2019 at 2:49 AM Masahiro Yamada
<yamada.masahiro@socionext.com> wrote:
>
> This Makefile repeats very similar rules.
>
> Let's use pattern rules. $(UNROLL) can be replaced with $*.
>
> No intended change in behavior.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> ---
Applied to linux-kbuild.
>
> lib/raid6/Makefile | 97 ++++++----------------------------------------
> 1 file changed, 11 insertions(+), 86 deletions(-)
>
> diff --git a/lib/raid6/Makefile b/lib/raid6/Makefile
> index 74004037033f..42695bc8d451 100644
> --- a/lib/raid6/Makefile
> +++ b/lib/raid6/Makefile
> @@ -12,9 +12,6 @@ raid6_pq-$(CONFIG_S390) += s390vx8.o recov_s390xc.o
>
> hostprogs-y += mktables
>
> -quiet_cmd_unroll = UNROLL $@
> - cmd_unroll = $(AWK) -f$(srctree)/$(src)/unroll.awk -vN=$(UNROLL) < $< > $@
> -
> ifeq ($(CONFIG_ALTIVEC),y)
> altivec_flags := -maltivec $(call cc-option,-mabi=altivec)
>
> @@ -50,111 +47,39 @@ CFLAGS_REMOVE_neon8.o += -mgeneral-regs-only
> endif
> endif
>
> -targets += int1.c
> -$(obj)/int1.c: UNROLL := 1
> -$(obj)/int1.c: $(src)/int.uc $(src)/unroll.awk FORCE
> - $(call if_changed,unroll)
> -
> -targets += int2.c
> -$(obj)/int2.c: UNROLL := 2
> -$(obj)/int2.c: $(src)/int.uc $(src)/unroll.awk FORCE
> - $(call if_changed,unroll)
> -
> -targets += int4.c
> -$(obj)/int4.c: UNROLL := 4
> -$(obj)/int4.c: $(src)/int.uc $(src)/unroll.awk FORCE
> - $(call if_changed,unroll)
> -
> -targets += int8.c
> -$(obj)/int8.c: UNROLL := 8
> -$(obj)/int8.c: $(src)/int.uc $(src)/unroll.awk FORCE
> - $(call if_changed,unroll)
> -
> -targets += int16.c
> -$(obj)/int16.c: UNROLL := 16
> -$(obj)/int16.c: $(src)/int.uc $(src)/unroll.awk FORCE
> - $(call if_changed,unroll)
> +quiet_cmd_unroll = UNROLL $@
> + cmd_unroll = $(AWK) -f$(srctree)/$(src)/unroll.awk -vN=$* < $< > $@
>
> -targets += int32.c
> -$(obj)/int32.c: UNROLL := 32
> -$(obj)/int32.c: $(src)/int.uc $(src)/unroll.awk FORCE
> +targets += int1.c int2.c int4.c int8.c int16.c int32.c
> +$(obj)/int%.c: $(src)/int.uc $(src)/unroll.awk FORCE
> $(call if_changed,unroll)
>
> CFLAGS_altivec1.o += $(altivec_flags)
> -targets += altivec1.c
> -$(obj)/altivec1.c: UNROLL := 1
> -$(obj)/altivec1.c: $(src)/altivec.uc $(src)/unroll.awk FORCE
> - $(call if_changed,unroll)
> -
> CFLAGS_altivec2.o += $(altivec_flags)
> -targets += altivec2.c
> -$(obj)/altivec2.c: UNROLL := 2
> -$(obj)/altivec2.c: $(src)/altivec.uc $(src)/unroll.awk FORCE
> - $(call if_changed,unroll)
> -
> CFLAGS_altivec4.o += $(altivec_flags)
> -targets += altivec4.c
> -$(obj)/altivec4.c: UNROLL := 4
> -$(obj)/altivec4.c: $(src)/altivec.uc $(src)/unroll.awk FORCE
> - $(call if_changed,unroll)
> -
> CFLAGS_altivec8.o += $(altivec_flags)
> -targets += altivec8.c
> -$(obj)/altivec8.c: UNROLL := 8
> -$(obj)/altivec8.c: $(src)/altivec.uc $(src)/unroll.awk FORCE
> +targets += altivec1.c altivec2.c altivec4.c altivec8.c
> +$(obj)/altivec%.c: $(src)/altivec.uc $(src)/unroll.awk FORCE
> $(call if_changed,unroll)
>
> CFLAGS_vpermxor1.o += $(altivec_flags)
> -targets += vpermxor1.c
> -$(obj)/vpermxor1.c: UNROLL := 1
> -$(obj)/vpermxor1.c: $(src)/vpermxor.uc $(src)/unroll.awk FORCE
> - $(call if_changed,unroll)
> -
> CFLAGS_vpermxor2.o += $(altivec_flags)
> -targets += vpermxor2.c
> -$(obj)/vpermxor2.c: UNROLL := 2
> -$(obj)/vpermxor2.c: $(src)/vpermxor.uc $(src)/unroll.awk FORCE
> - $(call if_changed,unroll)
> -
> CFLAGS_vpermxor4.o += $(altivec_flags)
> -targets += vpermxor4.c
> -$(obj)/vpermxor4.c: UNROLL := 4
> -$(obj)/vpermxor4.c: $(src)/vpermxor.uc $(src)/unroll.awk FORCE
> - $(call if_changed,unroll)
> -
> CFLAGS_vpermxor8.o += $(altivec_flags)
> -targets += vpermxor8.c
> -$(obj)/vpermxor8.c: UNROLL := 8
> -$(obj)/vpermxor8.c: $(src)/vpermxor.uc $(src)/unroll.awk FORCE
> +targets += vpermxor1.o vpermxor2.o vpermxor4.o vpermxor8.o
> +$(obj)/vpermxor%.c: $(src)/vpermxor.uc $(src)/unroll.awk FORCE
> $(call if_changed,unroll)
>
> CFLAGS_neon1.o += $(NEON_FLAGS)
> -targets += neon1.c
> -$(obj)/neon1.c: UNROLL := 1
> -$(obj)/neon1.c: $(src)/neon.uc $(src)/unroll.awk FORCE
> - $(call if_changed,unroll)
> -
> CFLAGS_neon2.o += $(NEON_FLAGS)
> -targets += neon2.c
> -$(obj)/neon2.c: UNROLL := 2
> -$(obj)/neon2.c: $(src)/neon.uc $(src)/unroll.awk FORCE
> - $(call if_changed,unroll)
> -
> CFLAGS_neon4.o += $(NEON_FLAGS)
> -targets += neon4.c
> -$(obj)/neon4.c: UNROLL := 4
> -$(obj)/neon4.c: $(src)/neon.uc $(src)/unroll.awk FORCE
> - $(call if_changed,unroll)
> -
> CFLAGS_neon8.o += $(NEON_FLAGS)
> -targets += neon8.c
> -$(obj)/neon8.c: UNROLL := 8
> -$(obj)/neon8.c: $(src)/neon.uc $(src)/unroll.awk FORCE
> +targets += neon1.c neon2.c neon4.c neon8.c
> +$(obj)/neon%.c: $(src)/neon.uc $(src)/unroll.awk FORCE
> $(call if_changed,unroll)
>
> targets += s390vx8.c
> -$(obj)/s390vx8.c: UNROLL := 8
> -$(obj)/s390vx8.c: $(src)/s390vx.uc $(src)/unroll.awk FORCE
> +$(obj)/s390vx%.c: $(src)/s390vx.uc $(src)/unroll.awk FORCE
> $(call if_changed,unroll)
>
> quiet_cmd_mktable = TABLE $@
> --
> 2.17.1
>
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-06-20 15:56 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-16 17:48 [PATCH 1/2] lib/raid6: remove duplicated CFLAGS_REMOVE_altivec8.o Masahiro Yamada
2019-06-16 17:48 ` [PATCH 2/2] lib/raid6: refactor unroll rules with pattern rules Masahiro Yamada
2019-06-20 15:55 ` Masahiro Yamada
2019-06-20 15:55 ` [PATCH 1/2] lib/raid6: remove duplicated CFLAGS_REMOVE_altivec8.o Masahiro Yamada
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.