* [RFC PATCH] Replaces long number representation by BIT() macro
@ 2019-06-13 18:02 ` Leonardo Bras
0 siblings, 0 replies; 9+ messages in thread
From: Leonardo Bras @ 2019-06-13 18:02 UTC (permalink / raw)
To: Michael Ellerman
Cc: Leonardo Bras, Benjamin Herrenschmidt, Paul Mackerras,
linuxppc-dev, linux-kernel
The main reason of this change is to make these bitmasks more readable.
The macro ASM_CONST() just appends an UL to it's parameter, so it can be
easily replaced by BIT_MASK, that already uses a UL representation.
ASM_CONST() in this file may behave different if __ASSEMBLY__ is defined,
as it is used on .S files, just leaving the parameter as is.
However, I have noticed no difference in the generated binary after this
change.
Signed-off-by: Leonardo Bras <leonardo@linux.ibm.com>
---
arch/powerpc/include/asm/firmware.h | 75 ++++++++++++++---------------
1 file changed, 37 insertions(+), 38 deletions(-)
diff --git a/arch/powerpc/include/asm/firmware.h b/arch/powerpc/include/asm/firmware.h
index 00bc42d95679..7a5b0cc0bc85 100644
--- a/arch/powerpc/include/asm/firmware.h
+++ b/arch/powerpc/include/asm/firmware.h
@@ -14,46 +14,45 @@
#ifdef __KERNEL__
-#include <asm/asm-const.h>
-
+#include <linux/bits.h>
/* firmware feature bitmask values */
-#define FW_FEATURE_PFT ASM_CONST(0x0000000000000001)
-#define FW_FEATURE_TCE ASM_CONST(0x0000000000000002)
-#define FW_FEATURE_SPRG0 ASM_CONST(0x0000000000000004)
-#define FW_FEATURE_DABR ASM_CONST(0x0000000000000008)
-#define FW_FEATURE_COPY ASM_CONST(0x0000000000000010)
-#define FW_FEATURE_ASR ASM_CONST(0x0000000000000020)
-#define FW_FEATURE_DEBUG ASM_CONST(0x0000000000000040)
-#define FW_FEATURE_TERM ASM_CONST(0x0000000000000080)
-#define FW_FEATURE_PERF ASM_CONST(0x0000000000000100)
-#define FW_FEATURE_DUMP ASM_CONST(0x0000000000000200)
-#define FW_FEATURE_INTERRUPT ASM_CONST(0x0000000000000400)
-#define FW_FEATURE_MIGRATE ASM_CONST(0x0000000000000800)
-#define FW_FEATURE_PERFMON ASM_CONST(0x0000000000001000)
-#define FW_FEATURE_CRQ ASM_CONST(0x0000000000002000)
-#define FW_FEATURE_VIO ASM_CONST(0x0000000000004000)
-#define FW_FEATURE_RDMA ASM_CONST(0x0000000000008000)
-#define FW_FEATURE_LLAN ASM_CONST(0x0000000000010000)
-#define FW_FEATURE_BULK_REMOVE ASM_CONST(0x0000000000020000)
-#define FW_FEATURE_XDABR ASM_CONST(0x0000000000040000)
-#define FW_FEATURE_MULTITCE ASM_CONST(0x0000000000080000)
-#define FW_FEATURE_SPLPAR ASM_CONST(0x0000000000100000)
-#define FW_FEATURE_LPAR ASM_CONST(0x0000000000400000)
-#define FW_FEATURE_PS3_LV1 ASM_CONST(0x0000000000800000)
-#define FW_FEATURE_HPT_RESIZE ASM_CONST(0x0000000001000000)
-#define FW_FEATURE_CMO ASM_CONST(0x0000000002000000)
-#define FW_FEATURE_VPHN ASM_CONST(0x0000000004000000)
-#define FW_FEATURE_XCMO ASM_CONST(0x0000000008000000)
-#define FW_FEATURE_OPAL ASM_CONST(0x0000000010000000)
-#define FW_FEATURE_SET_MODE ASM_CONST(0x0000000040000000)
-#define FW_FEATURE_BEST_ENERGY ASM_CONST(0x0000000080000000)
-#define FW_FEATURE_TYPE1_AFFINITY ASM_CONST(0x0000000100000000)
-#define FW_FEATURE_PRRN ASM_CONST(0x0000000200000000)
-#define FW_FEATURE_DRMEM_V2 ASM_CONST(0x0000000400000000)
-#define FW_FEATURE_DRC_INFO ASM_CONST(0x0000000800000000)
-#define FW_FEATURE_BLOCK_REMOVE ASM_CONST(0x0000001000000000)
-#define FW_FEATURE_PAPR_SCM ASM_CONST(0x0000002000000000)
+#define FW_FEATURE_PFT BIT(0)
+#define FW_FEATURE_TCE BIT(1)
+#define FW_FEATURE_SPRG0 BIT(2)
+#define FW_FEATURE_DABR BIT(3)
+#define FW_FEATURE_COPY BIT(4)
+#define FW_FEATURE_ASR BIT(5)
+#define FW_FEATURE_DEBUG BIT(6)
+#define FW_FEATURE_TERM BIT(7)
+#define FW_FEATURE_PERF BIT(8)
+#define FW_FEATURE_DUMP BIT(9)
+#define FW_FEATURE_INTERRUPT BIT(10)
+#define FW_FEATURE_MIGRATE BIT(11)
+#define FW_FEATURE_PERFMON BIT(12)
+#define FW_FEATURE_CRQ BIT(13)
+#define FW_FEATURE_VIO BIT(14)
+#define FW_FEATURE_RDMA BIT(15)
+#define FW_FEATURE_LLAN BIT(16)
+#define FW_FEATURE_BULK_REMOVE BIT(17)
+#define FW_FEATURE_XDABR BIT(18)
+#define FW_FEATURE_MULTITCE BIT(19)
+#define FW_FEATURE_SPLPAR BIT(20)
+#define FW_FEATURE_LPAR BIT(22)
+#define FW_FEATURE_PS3_LV1 BIT(23)
+#define FW_FEATURE_HPT_RESIZE BIT(24)
+#define FW_FEATURE_CMO BIT(25)
+#define FW_FEATURE_VPHN BIT(26)
+#define FW_FEATURE_XCMO BIT(27)
+#define FW_FEATURE_OPAL BIT(28)
+#define FW_FEATURE_SET_MODE BIT(30)
+#define FW_FEATURE_BEST_ENERGY BIT(31)
+#define FW_FEATURE_TYPE1_AFFINITY BIT(32)
+#define FW_FEATURE_PRRN BIT(33)
+#define FW_FEATURE_DRMEM_V2 BIT(34)
+#define FW_FEATURE_DRC_INFO BIT(35)
+#define FW_FEATURE_BLOCK_REMOVE BIT(36)
+#define FW_FEATURE_PAPR_SCM BIT(37)
#ifndef __ASSEMBLY__
--
2.17.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [RFC PATCH] Replaces long number representation by BIT() macro
@ 2019-06-13 18:02 ` Leonardo Bras
0 siblings, 0 replies; 9+ messages in thread
From: Leonardo Bras @ 2019-06-13 18:02 UTC (permalink / raw)
To: Michael Ellerman
Cc: Leonardo Bras, Paul Mackerras, linuxppc-dev, linux-kernel
The main reason of this change is to make these bitmasks more readable.
The macro ASM_CONST() just appends an UL to it's parameter, so it can be
easily replaced by BIT_MASK, that already uses a UL representation.
ASM_CONST() in this file may behave different if __ASSEMBLY__ is defined,
as it is used on .S files, just leaving the parameter as is.
However, I have noticed no difference in the generated binary after this
change.
Signed-off-by: Leonardo Bras <leonardo@linux.ibm.com>
---
arch/powerpc/include/asm/firmware.h | 75 ++++++++++++++---------------
1 file changed, 37 insertions(+), 38 deletions(-)
diff --git a/arch/powerpc/include/asm/firmware.h b/arch/powerpc/include/asm/firmware.h
index 00bc42d95679..7a5b0cc0bc85 100644
--- a/arch/powerpc/include/asm/firmware.h
+++ b/arch/powerpc/include/asm/firmware.h
@@ -14,46 +14,45 @@
#ifdef __KERNEL__
-#include <asm/asm-const.h>
-
+#include <linux/bits.h>
/* firmware feature bitmask values */
-#define FW_FEATURE_PFT ASM_CONST(0x0000000000000001)
-#define FW_FEATURE_TCE ASM_CONST(0x0000000000000002)
-#define FW_FEATURE_SPRG0 ASM_CONST(0x0000000000000004)
-#define FW_FEATURE_DABR ASM_CONST(0x0000000000000008)
-#define FW_FEATURE_COPY ASM_CONST(0x0000000000000010)
-#define FW_FEATURE_ASR ASM_CONST(0x0000000000000020)
-#define FW_FEATURE_DEBUG ASM_CONST(0x0000000000000040)
-#define FW_FEATURE_TERM ASM_CONST(0x0000000000000080)
-#define FW_FEATURE_PERF ASM_CONST(0x0000000000000100)
-#define FW_FEATURE_DUMP ASM_CONST(0x0000000000000200)
-#define FW_FEATURE_INTERRUPT ASM_CONST(0x0000000000000400)
-#define FW_FEATURE_MIGRATE ASM_CONST(0x0000000000000800)
-#define FW_FEATURE_PERFMON ASM_CONST(0x0000000000001000)
-#define FW_FEATURE_CRQ ASM_CONST(0x0000000000002000)
-#define FW_FEATURE_VIO ASM_CONST(0x0000000000004000)
-#define FW_FEATURE_RDMA ASM_CONST(0x0000000000008000)
-#define FW_FEATURE_LLAN ASM_CONST(0x0000000000010000)
-#define FW_FEATURE_BULK_REMOVE ASM_CONST(0x0000000000020000)
-#define FW_FEATURE_XDABR ASM_CONST(0x0000000000040000)
-#define FW_FEATURE_MULTITCE ASM_CONST(0x0000000000080000)
-#define FW_FEATURE_SPLPAR ASM_CONST(0x0000000000100000)
-#define FW_FEATURE_LPAR ASM_CONST(0x0000000000400000)
-#define FW_FEATURE_PS3_LV1 ASM_CONST(0x0000000000800000)
-#define FW_FEATURE_HPT_RESIZE ASM_CONST(0x0000000001000000)
-#define FW_FEATURE_CMO ASM_CONST(0x0000000002000000)
-#define FW_FEATURE_VPHN ASM_CONST(0x0000000004000000)
-#define FW_FEATURE_XCMO ASM_CONST(0x0000000008000000)
-#define FW_FEATURE_OPAL ASM_CONST(0x0000000010000000)
-#define FW_FEATURE_SET_MODE ASM_CONST(0x0000000040000000)
-#define FW_FEATURE_BEST_ENERGY ASM_CONST(0x0000000080000000)
-#define FW_FEATURE_TYPE1_AFFINITY ASM_CONST(0x0000000100000000)
-#define FW_FEATURE_PRRN ASM_CONST(0x0000000200000000)
-#define FW_FEATURE_DRMEM_V2 ASM_CONST(0x0000000400000000)
-#define FW_FEATURE_DRC_INFO ASM_CONST(0x0000000800000000)
-#define FW_FEATURE_BLOCK_REMOVE ASM_CONST(0x0000001000000000)
-#define FW_FEATURE_PAPR_SCM ASM_CONST(0x0000002000000000)
+#define FW_FEATURE_PFT BIT(0)
+#define FW_FEATURE_TCE BIT(1)
+#define FW_FEATURE_SPRG0 BIT(2)
+#define FW_FEATURE_DABR BIT(3)
+#define FW_FEATURE_COPY BIT(4)
+#define FW_FEATURE_ASR BIT(5)
+#define FW_FEATURE_DEBUG BIT(6)
+#define FW_FEATURE_TERM BIT(7)
+#define FW_FEATURE_PERF BIT(8)
+#define FW_FEATURE_DUMP BIT(9)
+#define FW_FEATURE_INTERRUPT BIT(10)
+#define FW_FEATURE_MIGRATE BIT(11)
+#define FW_FEATURE_PERFMON BIT(12)
+#define FW_FEATURE_CRQ BIT(13)
+#define FW_FEATURE_VIO BIT(14)
+#define FW_FEATURE_RDMA BIT(15)
+#define FW_FEATURE_LLAN BIT(16)
+#define FW_FEATURE_BULK_REMOVE BIT(17)
+#define FW_FEATURE_XDABR BIT(18)
+#define FW_FEATURE_MULTITCE BIT(19)
+#define FW_FEATURE_SPLPAR BIT(20)
+#define FW_FEATURE_LPAR BIT(22)
+#define FW_FEATURE_PS3_LV1 BIT(23)
+#define FW_FEATURE_HPT_RESIZE BIT(24)
+#define FW_FEATURE_CMO BIT(25)
+#define FW_FEATURE_VPHN BIT(26)
+#define FW_FEATURE_XCMO BIT(27)
+#define FW_FEATURE_OPAL BIT(28)
+#define FW_FEATURE_SET_MODE BIT(30)
+#define FW_FEATURE_BEST_ENERGY BIT(31)
+#define FW_FEATURE_TYPE1_AFFINITY BIT(32)
+#define FW_FEATURE_PRRN BIT(33)
+#define FW_FEATURE_DRMEM_V2 BIT(34)
+#define FW_FEATURE_DRC_INFO BIT(35)
+#define FW_FEATURE_BLOCK_REMOVE BIT(36)
+#define FW_FEATURE_PAPR_SCM BIT(37)
#ifndef __ASSEMBLY__
--
2.17.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [RFC PATCH] Replaces long number representation by BIT() macro
2019-06-13 18:02 ` Leonardo Bras
@ 2019-06-13 20:24 ` Leonardo Bras
-1 siblings, 0 replies; 9+ messages in thread
From: Leonardo Bras @ 2019-06-13 20:24 UTC (permalink / raw)
To: Michael Ellerman; +Cc: Paul Mackerras, linuxppc-dev, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 4767 bytes --]
Sorry, there is a typo on my commit message.
's/BIT_MASK/BIT/'
On Thu, 2019-06-13 at 15:02 -0300, Leonardo Bras wrote:
> The main reason of this change is to make these bitmasks more readable.
>
> The macro ASM_CONST() just appends an UL to it's parameter, so it can be
> easily replaced by BIT_MASK, that already uses a UL representation.
>
> ASM_CONST() in this file may behave different if __ASSEMBLY__ is defined,
> as it is used on .S files, just leaving the parameter as is.
>
> However, I have noticed no difference in the generated binary after this
> change.
>
> Signed-off-by: Leonardo Bras <leonardo@linux.ibm.com>
> ---
> arch/powerpc/include/asm/firmware.h | 75 ++++++++++++++---------------
> 1 file changed, 37 insertions(+), 38 deletions(-)
>
> diff --git a/arch/powerpc/include/asm/firmware.h b/arch/powerpc/include/asm/firmware.h
> index 00bc42d95679..7a5b0cc0bc85 100644
> --- a/arch/powerpc/include/asm/firmware.h
> +++ b/arch/powerpc/include/asm/firmware.h
> @@ -14,46 +14,45 @@
>
> #ifdef __KERNEL__
>
> -#include <asm/asm-const.h>
> -
> +#include <linux/bits.h>
> /* firmware feature bitmask values */
>
> -#define FW_FEATURE_PFT ASM_CONST(0x0000000000000001)
> -#define FW_FEATURE_TCE ASM_CONST(0x0000000000000002)
> -#define FW_FEATURE_SPRG0 ASM_CONST(0x0000000000000004)
> -#define FW_FEATURE_DABR ASM_CONST(0x0000000000000008)
> -#define FW_FEATURE_COPY ASM_CONST(0x0000000000000010)
> -#define FW_FEATURE_ASR ASM_CONST(0x0000000000000020)
> -#define FW_FEATURE_DEBUG ASM_CONST(0x0000000000000040)
> -#define FW_FEATURE_TERM ASM_CONST(0x0000000000000080)
> -#define FW_FEATURE_PERF ASM_CONST(0x0000000000000100)
> -#define FW_FEATURE_DUMP ASM_CONST(0x0000000000000200)
> -#define FW_FEATURE_INTERRUPT ASM_CONST(0x0000000000000400)
> -#define FW_FEATURE_MIGRATE ASM_CONST(0x0000000000000800)
> -#define FW_FEATURE_PERFMON ASM_CONST(0x0000000000001000)
> -#define FW_FEATURE_CRQ ASM_CONST(0x0000000000002000)
> -#define FW_FEATURE_VIO ASM_CONST(0x0000000000004000)
> -#define FW_FEATURE_RDMA ASM_CONST(0x0000000000008000)
> -#define FW_FEATURE_LLAN ASM_CONST(0x0000000000010000)
> -#define FW_FEATURE_BULK_REMOVE ASM_CONST(0x0000000000020000)
> -#define FW_FEATURE_XDABR ASM_CONST(0x0000000000040000)
> -#define FW_FEATURE_MULTITCE ASM_CONST(0x0000000000080000)
> -#define FW_FEATURE_SPLPAR ASM_CONST(0x0000000000100000)
> -#define FW_FEATURE_LPAR ASM_CONST(0x0000000000400000)
> -#define FW_FEATURE_PS3_LV1 ASM_CONST(0x0000000000800000)
> -#define FW_FEATURE_HPT_RESIZE ASM_CONST(0x0000000001000000)
> -#define FW_FEATURE_CMO ASM_CONST(0x0000000002000000)
> -#define FW_FEATURE_VPHN ASM_CONST(0x0000000004000000)
> -#define FW_FEATURE_XCMO ASM_CONST(0x0000000008000000)
> -#define FW_FEATURE_OPAL ASM_CONST(0x0000000010000000)
> -#define FW_FEATURE_SET_MODE ASM_CONST(0x0000000040000000)
> -#define FW_FEATURE_BEST_ENERGY ASM_CONST(0x0000000080000000)
> -#define FW_FEATURE_TYPE1_AFFINITY ASM_CONST(0x0000000100000000)
> -#define FW_FEATURE_PRRN ASM_CONST(0x0000000200000000)
> -#define FW_FEATURE_DRMEM_V2 ASM_CONST(0x0000000400000000)
> -#define FW_FEATURE_DRC_INFO ASM_CONST(0x0000000800000000)
> -#define FW_FEATURE_BLOCK_REMOVE ASM_CONST(0x0000001000000000)
> -#define FW_FEATURE_PAPR_SCM ASM_CONST(0x0000002000000000)
> +#define FW_FEATURE_PFT BIT(0)
> +#define FW_FEATURE_TCE BIT(1)
> +#define FW_FEATURE_SPRG0 BIT(2)
> +#define FW_FEATURE_DABR BIT(3)
> +#define FW_FEATURE_COPY BIT(4)
> +#define FW_FEATURE_ASR BIT(5)
> +#define FW_FEATURE_DEBUG BIT(6)
> +#define FW_FEATURE_TERM BIT(7)
> +#define FW_FEATURE_PERF BIT(8)
> +#define FW_FEATURE_DUMP BIT(9)
> +#define FW_FEATURE_INTERRUPT BIT(10)
> +#define FW_FEATURE_MIGRATE BIT(11)
> +#define FW_FEATURE_PERFMON BIT(12)
> +#define FW_FEATURE_CRQ BIT(13)
> +#define FW_FEATURE_VIO BIT(14)
> +#define FW_FEATURE_RDMA BIT(15)
> +#define FW_FEATURE_LLAN BIT(16)
> +#define FW_FEATURE_BULK_REMOVE BIT(17)
> +#define FW_FEATURE_XDABR BIT(18)
> +#define FW_FEATURE_MULTITCE BIT(19)
> +#define FW_FEATURE_SPLPAR BIT(20)
> +#define FW_FEATURE_LPAR BIT(22)
> +#define FW_FEATURE_PS3_LV1 BIT(23)
> +#define FW_FEATURE_HPT_RESIZE BIT(24)
> +#define FW_FEATURE_CMO BIT(25)
> +#define FW_FEATURE_VPHN BIT(26)
> +#define FW_FEATURE_XCMO BIT(27)
> +#define FW_FEATURE_OPAL BIT(28)
> +#define FW_FEATURE_SET_MODE BIT(30)
> +#define FW_FEATURE_BEST_ENERGY BIT(31)
> +#define FW_FEATURE_TYPE1_AFFINITY BIT(32)
> +#define FW_FEATURE_PRRN BIT(33)
> +#define FW_FEATURE_DRMEM_V2 BIT(34)
> +#define FW_FEATURE_DRC_INFO BIT(35)
> +#define FW_FEATURE_BLOCK_REMOVE BIT(36)
> +#define FW_FEATURE_PAPR_SCM BIT(37)
>
> #ifndef __ASSEMBLY__
>
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [RFC PATCH] Replaces long number representation by BIT() macro
@ 2019-06-13 20:24 ` Leonardo Bras
0 siblings, 0 replies; 9+ messages in thread
From: Leonardo Bras @ 2019-06-13 20:24 UTC (permalink / raw)
To: Michael Ellerman; +Cc: linuxppc-dev, Paul Mackerras, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 4767 bytes --]
Sorry, there is a typo on my commit message.
's/BIT_MASK/BIT/'
On Thu, 2019-06-13 at 15:02 -0300, Leonardo Bras wrote:
> The main reason of this change is to make these bitmasks more readable.
>
> The macro ASM_CONST() just appends an UL to it's parameter, so it can be
> easily replaced by BIT_MASK, that already uses a UL representation.
>
> ASM_CONST() in this file may behave different if __ASSEMBLY__ is defined,
> as it is used on .S files, just leaving the parameter as is.
>
> However, I have noticed no difference in the generated binary after this
> change.
>
> Signed-off-by: Leonardo Bras <leonardo@linux.ibm.com>
> ---
> arch/powerpc/include/asm/firmware.h | 75 ++++++++++++++---------------
> 1 file changed, 37 insertions(+), 38 deletions(-)
>
> diff --git a/arch/powerpc/include/asm/firmware.h b/arch/powerpc/include/asm/firmware.h
> index 00bc42d95679..7a5b0cc0bc85 100644
> --- a/arch/powerpc/include/asm/firmware.h
> +++ b/arch/powerpc/include/asm/firmware.h
> @@ -14,46 +14,45 @@
>
> #ifdef __KERNEL__
>
> -#include <asm/asm-const.h>
> -
> +#include <linux/bits.h>
> /* firmware feature bitmask values */
>
> -#define FW_FEATURE_PFT ASM_CONST(0x0000000000000001)
> -#define FW_FEATURE_TCE ASM_CONST(0x0000000000000002)
> -#define FW_FEATURE_SPRG0 ASM_CONST(0x0000000000000004)
> -#define FW_FEATURE_DABR ASM_CONST(0x0000000000000008)
> -#define FW_FEATURE_COPY ASM_CONST(0x0000000000000010)
> -#define FW_FEATURE_ASR ASM_CONST(0x0000000000000020)
> -#define FW_FEATURE_DEBUG ASM_CONST(0x0000000000000040)
> -#define FW_FEATURE_TERM ASM_CONST(0x0000000000000080)
> -#define FW_FEATURE_PERF ASM_CONST(0x0000000000000100)
> -#define FW_FEATURE_DUMP ASM_CONST(0x0000000000000200)
> -#define FW_FEATURE_INTERRUPT ASM_CONST(0x0000000000000400)
> -#define FW_FEATURE_MIGRATE ASM_CONST(0x0000000000000800)
> -#define FW_FEATURE_PERFMON ASM_CONST(0x0000000000001000)
> -#define FW_FEATURE_CRQ ASM_CONST(0x0000000000002000)
> -#define FW_FEATURE_VIO ASM_CONST(0x0000000000004000)
> -#define FW_FEATURE_RDMA ASM_CONST(0x0000000000008000)
> -#define FW_FEATURE_LLAN ASM_CONST(0x0000000000010000)
> -#define FW_FEATURE_BULK_REMOVE ASM_CONST(0x0000000000020000)
> -#define FW_FEATURE_XDABR ASM_CONST(0x0000000000040000)
> -#define FW_FEATURE_MULTITCE ASM_CONST(0x0000000000080000)
> -#define FW_FEATURE_SPLPAR ASM_CONST(0x0000000000100000)
> -#define FW_FEATURE_LPAR ASM_CONST(0x0000000000400000)
> -#define FW_FEATURE_PS3_LV1 ASM_CONST(0x0000000000800000)
> -#define FW_FEATURE_HPT_RESIZE ASM_CONST(0x0000000001000000)
> -#define FW_FEATURE_CMO ASM_CONST(0x0000000002000000)
> -#define FW_FEATURE_VPHN ASM_CONST(0x0000000004000000)
> -#define FW_FEATURE_XCMO ASM_CONST(0x0000000008000000)
> -#define FW_FEATURE_OPAL ASM_CONST(0x0000000010000000)
> -#define FW_FEATURE_SET_MODE ASM_CONST(0x0000000040000000)
> -#define FW_FEATURE_BEST_ENERGY ASM_CONST(0x0000000080000000)
> -#define FW_FEATURE_TYPE1_AFFINITY ASM_CONST(0x0000000100000000)
> -#define FW_FEATURE_PRRN ASM_CONST(0x0000000200000000)
> -#define FW_FEATURE_DRMEM_V2 ASM_CONST(0x0000000400000000)
> -#define FW_FEATURE_DRC_INFO ASM_CONST(0x0000000800000000)
> -#define FW_FEATURE_BLOCK_REMOVE ASM_CONST(0x0000001000000000)
> -#define FW_FEATURE_PAPR_SCM ASM_CONST(0x0000002000000000)
> +#define FW_FEATURE_PFT BIT(0)
> +#define FW_FEATURE_TCE BIT(1)
> +#define FW_FEATURE_SPRG0 BIT(2)
> +#define FW_FEATURE_DABR BIT(3)
> +#define FW_FEATURE_COPY BIT(4)
> +#define FW_FEATURE_ASR BIT(5)
> +#define FW_FEATURE_DEBUG BIT(6)
> +#define FW_FEATURE_TERM BIT(7)
> +#define FW_FEATURE_PERF BIT(8)
> +#define FW_FEATURE_DUMP BIT(9)
> +#define FW_FEATURE_INTERRUPT BIT(10)
> +#define FW_FEATURE_MIGRATE BIT(11)
> +#define FW_FEATURE_PERFMON BIT(12)
> +#define FW_FEATURE_CRQ BIT(13)
> +#define FW_FEATURE_VIO BIT(14)
> +#define FW_FEATURE_RDMA BIT(15)
> +#define FW_FEATURE_LLAN BIT(16)
> +#define FW_FEATURE_BULK_REMOVE BIT(17)
> +#define FW_FEATURE_XDABR BIT(18)
> +#define FW_FEATURE_MULTITCE BIT(19)
> +#define FW_FEATURE_SPLPAR BIT(20)
> +#define FW_FEATURE_LPAR BIT(22)
> +#define FW_FEATURE_PS3_LV1 BIT(23)
> +#define FW_FEATURE_HPT_RESIZE BIT(24)
> +#define FW_FEATURE_CMO BIT(25)
> +#define FW_FEATURE_VPHN BIT(26)
> +#define FW_FEATURE_XCMO BIT(27)
> +#define FW_FEATURE_OPAL BIT(28)
> +#define FW_FEATURE_SET_MODE BIT(30)
> +#define FW_FEATURE_BEST_ENERGY BIT(31)
> +#define FW_FEATURE_TYPE1_AFFINITY BIT(32)
> +#define FW_FEATURE_PRRN BIT(33)
> +#define FW_FEATURE_DRMEM_V2 BIT(34)
> +#define FW_FEATURE_DRC_INFO BIT(35)
> +#define FW_FEATURE_BLOCK_REMOVE BIT(36)
> +#define FW_FEATURE_PAPR_SCM BIT(37)
>
> #ifndef __ASSEMBLY__
>
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [RFC PATCH] Replaces long number representation by BIT() macro
2019-06-13 18:02 ` Leonardo Bras
@ 2019-07-02 15:19 ` Michael Ellerman
-1 siblings, 0 replies; 9+ messages in thread
From: Michael Ellerman @ 2019-07-02 15:19 UTC (permalink / raw)
To: Leonardo Bras
Cc: Leonardo Bras, Benjamin Herrenschmidt, Paul Mackerras,
linuxppc-dev, linux-kernel
Hi Leonardo,
Leonardo Bras <leonardo@linux.ibm.com> writes:
> The main reason of this change is to make these bitmasks more readable.
>
> The macro ASM_CONST() just appends an UL to it's parameter, so it can be
> easily replaced by BIT_MASK, that already uses a UL representation.
>
> ASM_CONST() in this file may behave different if __ASSEMBLY__ is defined,
> as it is used on .S files, just leaving the parameter as is.
Thanks for the patch, but I don't consider this an improvement in
readability.
At boot we print the firmware features, eg:
firmware_features = 0x00000001c45ffc5f
And it's much easier to match that up to the full constants, than to the
bit numbers.
Similarly in memory or register dumps.
What we could do is switch to the `UL` macro from include/linux/const.h,
rather than using our own ASM_CONST.
cheers
> diff --git a/arch/powerpc/include/asm/firmware.h b/arch/powerpc/include/asm/firmware.h
> index 00bc42d95679..7a5b0cc0bc85 100644
> --- a/arch/powerpc/include/asm/firmware.h
> +++ b/arch/powerpc/include/asm/firmware.h
> @@ -14,46 +14,45 @@
>
> #ifdef __KERNEL__
>
> -#include <asm/asm-const.h>
> -
> +#include <linux/bits.h>
> /* firmware feature bitmask values */
>
> -#define FW_FEATURE_PFT ASM_CONST(0x0000000000000001)
> -#define FW_FEATURE_TCE ASM_CONST(0x0000000000000002)
> -#define FW_FEATURE_SPRG0 ASM_CONST(0x0000000000000004)
> -#define FW_FEATURE_DABR ASM_CONST(0x0000000000000008)
> -#define FW_FEATURE_COPY ASM_CONST(0x0000000000000010)
> -#define FW_FEATURE_ASR ASM_CONST(0x0000000000000020)
> -#define FW_FEATURE_DEBUG ASM_CONST(0x0000000000000040)
> -#define FW_FEATURE_TERM ASM_CONST(0x0000000000000080)
> -#define FW_FEATURE_PERF ASM_CONST(0x0000000000000100)
> -#define FW_FEATURE_DUMP ASM_CONST(0x0000000000000200)
> -#define FW_FEATURE_INTERRUPT ASM_CONST(0x0000000000000400)
> -#define FW_FEATURE_MIGRATE ASM_CONST(0x0000000000000800)
> -#define FW_FEATURE_PERFMON ASM_CONST(0x0000000000001000)
> -#define FW_FEATURE_CRQ ASM_CONST(0x0000000000002000)
> -#define FW_FEATURE_VIO ASM_CONST(0x0000000000004000)
> -#define FW_FEATURE_RDMA ASM_CONST(0x0000000000008000)
> -#define FW_FEATURE_LLAN ASM_CONST(0x0000000000010000)
> -#define FW_FEATURE_BULK_REMOVE ASM_CONST(0x0000000000020000)
> -#define FW_FEATURE_XDABR ASM_CONST(0x0000000000040000)
> -#define FW_FEATURE_MULTITCE ASM_CONST(0x0000000000080000)
> -#define FW_FEATURE_SPLPAR ASM_CONST(0x0000000000100000)
> -#define FW_FEATURE_LPAR ASM_CONST(0x0000000000400000)
> -#define FW_FEATURE_PS3_LV1 ASM_CONST(0x0000000000800000)
> -#define FW_FEATURE_HPT_RESIZE ASM_CONST(0x0000000001000000)
> -#define FW_FEATURE_CMO ASM_CONST(0x0000000002000000)
> -#define FW_FEATURE_VPHN ASM_CONST(0x0000000004000000)
> -#define FW_FEATURE_XCMO ASM_CONST(0x0000000008000000)
> -#define FW_FEATURE_OPAL ASM_CONST(0x0000000010000000)
> -#define FW_FEATURE_SET_MODE ASM_CONST(0x0000000040000000)
> -#define FW_FEATURE_BEST_ENERGY ASM_CONST(0x0000000080000000)
> -#define FW_FEATURE_TYPE1_AFFINITY ASM_CONST(0x0000000100000000)
> -#define FW_FEATURE_PRRN ASM_CONST(0x0000000200000000)
> -#define FW_FEATURE_DRMEM_V2 ASM_CONST(0x0000000400000000)
> -#define FW_FEATURE_DRC_INFO ASM_CONST(0x0000000800000000)
> -#define FW_FEATURE_BLOCK_REMOVE ASM_CONST(0x0000001000000000)
> -#define FW_FEATURE_PAPR_SCM ASM_CONST(0x0000002000000000)
> +#define FW_FEATURE_PFT BIT(0)
> +#define FW_FEATURE_TCE BIT(1)
> +#define FW_FEATURE_SPRG0 BIT(2)
> +#define FW_FEATURE_DABR BIT(3)
> +#define FW_FEATURE_COPY BIT(4)
> +#define FW_FEATURE_ASR BIT(5)
> +#define FW_FEATURE_DEBUG BIT(6)
> +#define FW_FEATURE_TERM BIT(7)
> +#define FW_FEATURE_PERF BIT(8)
> +#define FW_FEATURE_DUMP BIT(9)
> +#define FW_FEATURE_INTERRUPT BIT(10)
> +#define FW_FEATURE_MIGRATE BIT(11)
> +#define FW_FEATURE_PERFMON BIT(12)
> +#define FW_FEATURE_CRQ BIT(13)
> +#define FW_FEATURE_VIO BIT(14)
> +#define FW_FEATURE_RDMA BIT(15)
> +#define FW_FEATURE_LLAN BIT(16)
> +#define FW_FEATURE_BULK_REMOVE BIT(17)
> +#define FW_FEATURE_XDABR BIT(18)
> +#define FW_FEATURE_MULTITCE BIT(19)
> +#define FW_FEATURE_SPLPAR BIT(20)
> +#define FW_FEATURE_LPAR BIT(22)
> +#define FW_FEATURE_PS3_LV1 BIT(23)
> +#define FW_FEATURE_HPT_RESIZE BIT(24)
> +#define FW_FEATURE_CMO BIT(25)
> +#define FW_FEATURE_VPHN BIT(26)
> +#define FW_FEATURE_XCMO BIT(27)
> +#define FW_FEATURE_OPAL BIT(28)
> +#define FW_FEATURE_SET_MODE BIT(30)
> +#define FW_FEATURE_BEST_ENERGY BIT(31)
> +#define FW_FEATURE_TYPE1_AFFINITY BIT(32)
> +#define FW_FEATURE_PRRN BIT(33)
> +#define FW_FEATURE_DRMEM_V2 BIT(34)
> +#define FW_FEATURE_DRC_INFO BIT(35)
> +#define FW_FEATURE_BLOCK_REMOVE BIT(36)
> +#define FW_FEATURE_PAPR_SCM BIT(37)
>
> #ifndef __ASSEMBLY__
>
> --
> 2.17.1
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [RFC PATCH] Replaces long number representation by BIT() macro
@ 2019-07-02 15:19 ` Michael Ellerman
0 siblings, 0 replies; 9+ messages in thread
From: Michael Ellerman @ 2019-07-02 15:19 UTC (permalink / raw)
To: Leonardo Bras; +Cc: Leonardo Bras, Paul Mackerras, linuxppc-dev, linux-kernel
Hi Leonardo,
Leonardo Bras <leonardo@linux.ibm.com> writes:
> The main reason of this change is to make these bitmasks more readable.
>
> The macro ASM_CONST() just appends an UL to it's parameter, so it can be
> easily replaced by BIT_MASK, that already uses a UL representation.
>
> ASM_CONST() in this file may behave different if __ASSEMBLY__ is defined,
> as it is used on .S files, just leaving the parameter as is.
Thanks for the patch, but I don't consider this an improvement in
readability.
At boot we print the firmware features, eg:
firmware_features = 0x00000001c45ffc5f
And it's much easier to match that up to the full constants, than to the
bit numbers.
Similarly in memory or register dumps.
What we could do is switch to the `UL` macro from include/linux/const.h,
rather than using our own ASM_CONST.
cheers
> diff --git a/arch/powerpc/include/asm/firmware.h b/arch/powerpc/include/asm/firmware.h
> index 00bc42d95679..7a5b0cc0bc85 100644
> --- a/arch/powerpc/include/asm/firmware.h
> +++ b/arch/powerpc/include/asm/firmware.h
> @@ -14,46 +14,45 @@
>
> #ifdef __KERNEL__
>
> -#include <asm/asm-const.h>
> -
> +#include <linux/bits.h>
> /* firmware feature bitmask values */
>
> -#define FW_FEATURE_PFT ASM_CONST(0x0000000000000001)
> -#define FW_FEATURE_TCE ASM_CONST(0x0000000000000002)
> -#define FW_FEATURE_SPRG0 ASM_CONST(0x0000000000000004)
> -#define FW_FEATURE_DABR ASM_CONST(0x0000000000000008)
> -#define FW_FEATURE_COPY ASM_CONST(0x0000000000000010)
> -#define FW_FEATURE_ASR ASM_CONST(0x0000000000000020)
> -#define FW_FEATURE_DEBUG ASM_CONST(0x0000000000000040)
> -#define FW_FEATURE_TERM ASM_CONST(0x0000000000000080)
> -#define FW_FEATURE_PERF ASM_CONST(0x0000000000000100)
> -#define FW_FEATURE_DUMP ASM_CONST(0x0000000000000200)
> -#define FW_FEATURE_INTERRUPT ASM_CONST(0x0000000000000400)
> -#define FW_FEATURE_MIGRATE ASM_CONST(0x0000000000000800)
> -#define FW_FEATURE_PERFMON ASM_CONST(0x0000000000001000)
> -#define FW_FEATURE_CRQ ASM_CONST(0x0000000000002000)
> -#define FW_FEATURE_VIO ASM_CONST(0x0000000000004000)
> -#define FW_FEATURE_RDMA ASM_CONST(0x0000000000008000)
> -#define FW_FEATURE_LLAN ASM_CONST(0x0000000000010000)
> -#define FW_FEATURE_BULK_REMOVE ASM_CONST(0x0000000000020000)
> -#define FW_FEATURE_XDABR ASM_CONST(0x0000000000040000)
> -#define FW_FEATURE_MULTITCE ASM_CONST(0x0000000000080000)
> -#define FW_FEATURE_SPLPAR ASM_CONST(0x0000000000100000)
> -#define FW_FEATURE_LPAR ASM_CONST(0x0000000000400000)
> -#define FW_FEATURE_PS3_LV1 ASM_CONST(0x0000000000800000)
> -#define FW_FEATURE_HPT_RESIZE ASM_CONST(0x0000000001000000)
> -#define FW_FEATURE_CMO ASM_CONST(0x0000000002000000)
> -#define FW_FEATURE_VPHN ASM_CONST(0x0000000004000000)
> -#define FW_FEATURE_XCMO ASM_CONST(0x0000000008000000)
> -#define FW_FEATURE_OPAL ASM_CONST(0x0000000010000000)
> -#define FW_FEATURE_SET_MODE ASM_CONST(0x0000000040000000)
> -#define FW_FEATURE_BEST_ENERGY ASM_CONST(0x0000000080000000)
> -#define FW_FEATURE_TYPE1_AFFINITY ASM_CONST(0x0000000100000000)
> -#define FW_FEATURE_PRRN ASM_CONST(0x0000000200000000)
> -#define FW_FEATURE_DRMEM_V2 ASM_CONST(0x0000000400000000)
> -#define FW_FEATURE_DRC_INFO ASM_CONST(0x0000000800000000)
> -#define FW_FEATURE_BLOCK_REMOVE ASM_CONST(0x0000001000000000)
> -#define FW_FEATURE_PAPR_SCM ASM_CONST(0x0000002000000000)
> +#define FW_FEATURE_PFT BIT(0)
> +#define FW_FEATURE_TCE BIT(1)
> +#define FW_FEATURE_SPRG0 BIT(2)
> +#define FW_FEATURE_DABR BIT(3)
> +#define FW_FEATURE_COPY BIT(4)
> +#define FW_FEATURE_ASR BIT(5)
> +#define FW_FEATURE_DEBUG BIT(6)
> +#define FW_FEATURE_TERM BIT(7)
> +#define FW_FEATURE_PERF BIT(8)
> +#define FW_FEATURE_DUMP BIT(9)
> +#define FW_FEATURE_INTERRUPT BIT(10)
> +#define FW_FEATURE_MIGRATE BIT(11)
> +#define FW_FEATURE_PERFMON BIT(12)
> +#define FW_FEATURE_CRQ BIT(13)
> +#define FW_FEATURE_VIO BIT(14)
> +#define FW_FEATURE_RDMA BIT(15)
> +#define FW_FEATURE_LLAN BIT(16)
> +#define FW_FEATURE_BULK_REMOVE BIT(17)
> +#define FW_FEATURE_XDABR BIT(18)
> +#define FW_FEATURE_MULTITCE BIT(19)
> +#define FW_FEATURE_SPLPAR BIT(20)
> +#define FW_FEATURE_LPAR BIT(22)
> +#define FW_FEATURE_PS3_LV1 BIT(23)
> +#define FW_FEATURE_HPT_RESIZE BIT(24)
> +#define FW_FEATURE_CMO BIT(25)
> +#define FW_FEATURE_VPHN BIT(26)
> +#define FW_FEATURE_XCMO BIT(27)
> +#define FW_FEATURE_OPAL BIT(28)
> +#define FW_FEATURE_SET_MODE BIT(30)
> +#define FW_FEATURE_BEST_ENERGY BIT(31)
> +#define FW_FEATURE_TYPE1_AFFINITY BIT(32)
> +#define FW_FEATURE_PRRN BIT(33)
> +#define FW_FEATURE_DRMEM_V2 BIT(34)
> +#define FW_FEATURE_DRC_INFO BIT(35)
> +#define FW_FEATURE_BLOCK_REMOVE BIT(36)
> +#define FW_FEATURE_PAPR_SCM BIT(37)
>
> #ifndef __ASSEMBLY__
>
> --
> 2.17.1
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [RFC PATCH] Replaces long number representation by BIT() macro
2019-07-02 15:19 ` Michael Ellerman
(?)
@ 2019-07-02 16:16 ` Segher Boessenkool
2019-07-02 16:48 ` Segher Boessenkool
-1 siblings, 1 reply; 9+ messages in thread
From: Segher Boessenkool @ 2019-07-02 16:16 UTC (permalink / raw)
To: Michael Ellerman
Cc: Leonardo Bras, Paul Mackerras, linuxppc-dev, linux-kernel
On Wed, Jul 03, 2019 at 01:19:34AM +1000, Michael Ellerman wrote:
> What we could do is switch to the `UL` macro from include/linux/const.h,
> rather than using our own ASM_CONST.
You need gas 2.28 or later for that though.
https://sourceware.org/git/?p=binutils-gdb.git;a=commitdiff;h=86b80085
https://sourceware.org/git/?p=binutils-gdb.git;a=commitdiff;h=e140100a
What is the minimum required (for powerpc) now?
Segher
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [RFC PATCH] Replaces long number representation by BIT() macro
2019-07-02 16:16 ` Segher Boessenkool
@ 2019-07-02 16:48 ` Segher Boessenkool
2019-07-07 13:15 ` Michael Ellerman
0 siblings, 1 reply; 9+ messages in thread
From: Segher Boessenkool @ 2019-07-02 16:48 UTC (permalink / raw)
To: Michael Ellerman
Cc: Leonardo Bras, Paul Mackerras, linuxppc-dev, linux-kernel
On Tue, Jul 02, 2019 at 11:16:35AM -0500, Segher Boessenkool wrote:
> On Wed, Jul 03, 2019 at 01:19:34AM +1000, Michael Ellerman wrote:
> > What we could do is switch to the `UL` macro from include/linux/const.h,
> > rather than using our own ASM_CONST.
>
> You need gas 2.28 or later for that though.
Oh, but apparently I cannot read. That macro should work fine.
Segher
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [RFC PATCH] Replaces long number representation by BIT() macro
2019-07-02 16:48 ` Segher Boessenkool
@ 2019-07-07 13:15 ` Michael Ellerman
0 siblings, 0 replies; 9+ messages in thread
From: Michael Ellerman @ 2019-07-07 13:15 UTC (permalink / raw)
To: Segher Boessenkool
Cc: Leonardo Bras, Paul Mackerras, linuxppc-dev, linux-kernel
Segher Boessenkool <segher@kernel.crashing.org> writes:
> On Tue, Jul 02, 2019 at 11:16:35AM -0500, Segher Boessenkool wrote:
>> On Wed, Jul 03, 2019 at 01:19:34AM +1000, Michael Ellerman wrote:
>> > What we could do is switch to the `UL` macro from include/linux/const.h,
>> > rather than using our own ASM_CONST.
>>
>> You need gas 2.28 or later for that though.
>
> Oh, but apparently I cannot read. That macro should work fine.
:)
Yeah one day we'll be able to drop them entirely, but not yet.
The official minimum is 2.20:
https://www.kernel.org/doc/html/latest/process/changes.html
But my "old" toolchain is binutils 2.22, so that's effectively the
minimum for anything I test. I'm not sure many people are actually
testing with 2.20.
cheers
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2019-07-07 13:17 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-13 18:02 [RFC PATCH] Replaces long number representation by BIT() macro Leonardo Bras
2019-06-13 18:02 ` Leonardo Bras
2019-06-13 20:24 ` Leonardo Bras
2019-06-13 20:24 ` Leonardo Bras
2019-07-02 15:19 ` Michael Ellerman
2019-07-02 15:19 ` Michael Ellerman
2019-07-02 16:16 ` Segher Boessenkool
2019-07-02 16:48 ` Segher Boessenkool
2019-07-07 13:15 ` Michael Ellerman
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.