[Qemu-devel] [PATCH] target/ppc: introduce the PPC_BIT() macro

Cédric Le Goater posted 1 patch 6 years, 3 months ago
target/ppc/cpu.h | 105 +++++++++++++++++++++++++++++--------------------------
1 file changed, 56 insertions(+), 49 deletions(-)
[Qemu-devel] [PATCH] target/ppc: introduce the PPC_BIT() macro
Posted by Cédric Le Goater 6 years, 3 months ago
and use them in a couple of obvious places. Other macros will be used
in the model of the XIVE interrupt controller.

Signed-off-by: Cédric Le Goater <clg@kaod.org>
---
 target/ppc/cpu.h | 105 +++++++++++++++++++++++++++++--------------------------
 1 file changed, 56 insertions(+), 49 deletions(-)

diff --git a/target/ppc/cpu.h b/target/ppc/cpu.h
index 989761b79569..370b05e76ede 100644
--- a/target/ppc/cpu.h
+++ b/target/ppc/cpu.h
@@ -87,6 +87,13 @@
 #define PPC_ELF_MACHINE     EM_PPC
 #endif
 
+#define PPC_BIT(bit)            (0x8000000000000000UL >> (bit))
+#define PPC_BIT32(bit)          (0x80000000UL >> (bit))
+#define PPC_BIT8(bit)           (0x80UL >> (bit))
+#define PPC_BITMASK(bs, be)     ((PPC_BIT(bs) - PPC_BIT(be)) | PPC_BIT(bs))
+#define PPC_BITMASK32(bs, be)   ((PPC_BIT32(bs) - PPC_BIT32(be)) | \
+                                 PPC_BIT32(bs))
+
 /*****************************************************************************/
 /* Exception vectors definitions                                             */
 enum {
@@ -371,10 +378,10 @@ struct ppc_slb_t {
 #define MSR_LE   0  /* Little-endian mode                           1 hflags */
 
 /* LPCR bits */
-#define LPCR_VPM0         (1ull << (63 - 0))
-#define LPCR_VPM1         (1ull << (63 - 1))
-#define LPCR_ISL          (1ull << (63 - 2))
-#define LPCR_KBV          (1ull << (63 - 3))
+#define LPCR_VPM0         PPC_BIT(0)
+#define LPCR_VPM1         PPC_BIT(1)
+#define LPCR_ISL          PPC_BIT(2)
+#define LPCR_KBV          PPC_BIT(3)
 #define LPCR_DPFD_SHIFT   (63 - 11)
 #define LPCR_DPFD         (0x7ull << LPCR_DPFD_SHIFT)
 #define LPCR_VRMASD_SHIFT (63 - 16)
@@ -382,41 +389,41 @@ struct ppc_slb_t {
 /* P9: Power-saving mode Exit Cause Enable (Upper Section) Mask */
 #define LPCR_PECE_U_SHIFT (63 - 19)
 #define LPCR_PECE_U_MASK  (0x7ull << LPCR_PECE_U_SHIFT)
-#define LPCR_HVEE         (1ull << (63 - 17)) /* Hypervisor Virt Exit Enable */
+#define LPCR_HVEE         PPC_BIT(17) /* Hypervisor Virt Exit Enable */
 #define LPCR_RMLS_SHIFT   (63 - 37)
 #define LPCR_RMLS         (0xfull << LPCR_RMLS_SHIFT)
-#define LPCR_ILE          (1ull << (63 - 38))
+#define LPCR_ILE          PPC_BIT(38)
 #define LPCR_AIL_SHIFT    (63 - 40)      /* Alternate interrupt location */
 #define LPCR_AIL          (3ull << LPCR_AIL_SHIFT)
-#define LPCR_UPRT         (1ull << (63 - 41)) /* Use Process Table */
-#define LPCR_EVIRT        (1ull << (63 - 42)) /* Enhanced Virtualisation */
-#define LPCR_ONL          (1ull << (63 - 45))
-#define LPCR_LD           (1ull << (63 - 46)) /* Large Decrementer */
-#define LPCR_P7_PECE0     (1ull << (63 - 49))
-#define LPCR_P7_PECE1     (1ull << (63 - 50))
-#define LPCR_P7_PECE2     (1ull << (63 - 51))
-#define LPCR_P8_PECE0     (1ull << (63 - 47))
-#define LPCR_P8_PECE1     (1ull << (63 - 48))
-#define LPCR_P8_PECE2     (1ull << (63 - 49))
-#define LPCR_P8_PECE3     (1ull << (63 - 50))
-#define LPCR_P8_PECE4     (1ull << (63 - 51))
+#define LPCR_UPRT         PPC_BIT(41) /* Use Process Table */
+#define LPCR_EVIRT        PPC_BIT(42) /* Enhanced Virtualisation */
+#define LPCR_ONL          PPC_BIT(45)
+#define LPCR_LD           PPC_BIT(46) /* Large Decrementer */
+#define LPCR_P7_PECE0     PPC_BIT(49)
+#define LPCR_P7_PECE1     PPC_BIT(50)
+#define LPCR_P7_PECE2     PPC_BIT(51)
+#define LPCR_P8_PECE0     PPC_BIT(47)
+#define LPCR_P8_PECE1     PPC_BIT(48)
+#define LPCR_P8_PECE2     PPC_BIT(49)
+#define LPCR_P8_PECE3     PPC_BIT(50)
+#define LPCR_P8_PECE4     PPC_BIT(51)
 /* P9: Power-saving mode Exit Cause Enable (Lower Section) Mask */
 #define LPCR_PECE_L_SHIFT (63 - 51)
 #define LPCR_PECE_L_MASK  (0x1full << LPCR_PECE_L_SHIFT)
-#define LPCR_PDEE         (1ull << (63 - 47)) /* Privileged Doorbell Exit EN */
-#define LPCR_HDEE         (1ull << (63 - 48)) /* Hyperv Doorbell Exit Enable */
-#define LPCR_EEE          (1ull << (63 - 49)) /* External Exit Enable        */
-#define LPCR_DEE          (1ull << (63 - 50)) /* Decrementer Exit Enable     */
-#define LPCR_OEE          (1ull << (63 - 51)) /* Other Exit Enable           */
-#define LPCR_MER          (1ull << (63 - 52))
-#define LPCR_GTSE         (1ull << (63 - 53)) /* Guest Translation Shootdown */
-#define LPCR_TC           (1ull << (63 - 54))
-#define LPCR_HEIC         (1ull << (63 - 59)) /* HV Extern Interrupt Control */
-#define LPCR_LPES0        (1ull << (63 - 60))
-#define LPCR_LPES1        (1ull << (63 - 61))
-#define LPCR_RMI          (1ull << (63 - 62))
-#define LPCR_HVICE        (1ull << (63 - 62)) /* HV Virtualisation Int Enable */
-#define LPCR_HDICE        (1ull << (63 - 63))
+#define LPCR_PDEE         PPC_BIT(47) /* Privileged Doorbell Exit EN */
+#define LPCR_HDEE         PPC_BIT(48) /* Hyperv Doorbell Exit Enable */
+#define LPCR_EEE          PPC_BIT(49) /* External Exit Enable        */
+#define LPCR_DEE          PPC_BIT(50) /* Decrementer Exit Enable     */
+#define LPCR_OEE          PPC_BIT(51) /* Other Exit Enable           */
+#define LPCR_MER          PPC_BIT(52)
+#define LPCR_GTSE         PPC_BIT(53) /* Guest Translation Shootdown */
+#define LPCR_TC           PPC_BIT(54)
+#define LPCR_HEIC         PPC_BIT(59) /* HV Extern Interrupt Control */
+#define LPCR_LPES0        PPC_BIT(60)
+#define LPCR_LPES1        PPC_BIT(61)
+#define LPCR_RMI          PPC_BIT(62)
+#define LPCR_HVICE        PPC_BIT(62) /* HV Virtualisation Int Enable */
+#define LPCR_HDICE        PPC_BIT(63)
 
 #define msr_sf   ((env->msr >> MSR_SF)   & 1)
 #define msr_isf  ((env->msr >> MSR_ISF)  & 1)
@@ -507,22 +514,22 @@ struct ppc_slb_t {
 #define FSCR_IC_TAR         8
 
 /* Exception state register bits definition                                  */
-#define ESR_PIL   (1 << (63 - 36)) /* Illegal Instruction                    */
-#define ESR_PPR   (1 << (63 - 37)) /* Privileged Instruction                 */
-#define ESR_PTR   (1 << (63 - 38)) /* Trap                                   */
-#define ESR_FP    (1 << (63 - 39)) /* Floating-Point Operation               */
-#define ESR_ST    (1 << (63 - 40)) /* Store Operation                        */
-#define ESR_AP    (1 << (63 - 44)) /* Auxiliary Processor Operation          */
-#define ESR_PUO   (1 << (63 - 45)) /* Unimplemented Operation                */
-#define ESR_BO    (1 << (63 - 46)) /* Byte Ordering                          */
-#define ESR_PIE   (1 << (63 - 47)) /* Imprecise exception                    */
-#define ESR_DATA  (1 << (63 - 53)) /* Data Access (Embedded page table)      */
-#define ESR_TLBI  (1 << (63 - 54)) /* TLB Ineligible (Embedded page table)   */
-#define ESR_PT    (1 << (63 - 55)) /* Page Table (Embedded page table)       */
-#define ESR_SPV   (1 << (63 - 56)) /* SPE/VMX operation                      */
-#define ESR_EPID  (1 << (63 - 57)) /* External Process ID operation          */
-#define ESR_VLEMI (1 << (63 - 58)) /* VLE operation                          */
-#define ESR_MIF   (1 << (63 - 62)) /* Misaligned instruction (VLE)           */
+#define ESR_PIL   PPC_BIT(36) /* Illegal Instruction                    */
+#define ESR_PPR   PPC_BIT(37) /* Privileged Instruction                 */
+#define ESR_PTR   PPC_BIT(38) /* Trap                                   */
+#define ESR_FP    PPC_BIT(39) /* Floating-Point Operation               */
+#define ESR_ST    PPC_BIT(40) /* Store Operation                        */
+#define ESR_AP    PPC_BIT(44) /* Auxiliary Processor Operation          */
+#define ESR_PUO   PPC_BIT(45) /* Unimplemented Operation                */
+#define ESR_BO    PPC_BIT(46) /* Byte Ordering                          */
+#define ESR_PIE   PPC_BIT(47) /* Imprecise exception                    */
+#define ESR_DATA  PPC_BIT(53) /* Data Access (Embedded page table)      */
+#define ESR_TLBI  PPC_BIT(54) /* TLB Ineligible (Embedded page table)   */
+#define ESR_PT    PPC_BIT(55) /* Page Table (Embedded page table)       */
+#define ESR_SPV   PPC_BIT(56) /* SPE/VMX operation                      */
+#define ESR_EPID  PPC_BIT(57) /* External Process ID operation          */
+#define ESR_VLEMI PPC_BIT(58) /* VLE operation                          */
+#define ESR_MIF   PPC_BIT(62) /* Misaligned instruction (VLE)           */
 
 /* Transaction EXception And Summary Register bits                           */
 #define TEXASR_FAILURE_PERSISTENT                (63 - 7)
@@ -1991,7 +1998,7 @@ void ppc_compat_add_property(Object *obj, const char *name,
 #define HID0_DEEPNAP        (1 << 24)           /* pre-2.06 */
 #define HID0_DOZE           (1 << 23)           /* pre-2.06 */
 #define HID0_NAP            (1 << 22)           /* pre-2.06 */
-#define HID0_HILE           (1ull << (63 - 19)) /* POWER8 */
+#define HID0_HILE           PPC_BIT(19) /* POWER8 */
 
 /*****************************************************************************/
 /* PowerPC Instructions types definitions                                    */
-- 
2.13.6


Re: [Qemu-devel] [PATCH] target/ppc: introduce the PPC_BIT() macro
Posted by David Gibson 6 years, 3 months ago
On Wed, Dec 06, 2017 at 10:41:50AM +0100, Cédric Le Goater wrote:
> and use them in a couple of obvious places. Other macros will be used
> in the model of the XIVE interrupt controller.
> 
> Signed-off-by: Cédric Le Goater <clg@kaod.org>

Applied to ppc-for-2.12.

> ---
>  target/ppc/cpu.h | 105 +++++++++++++++++++++++++++++--------------------------
>  1 file changed, 56 insertions(+), 49 deletions(-)
> 
> diff --git a/target/ppc/cpu.h b/target/ppc/cpu.h
> index 989761b79569..370b05e76ede 100644
> --- a/target/ppc/cpu.h
> +++ b/target/ppc/cpu.h
> @@ -87,6 +87,13 @@
>  #define PPC_ELF_MACHINE     EM_PPC
>  #endif
>  
> +#define PPC_BIT(bit)            (0x8000000000000000UL >> (bit))
> +#define PPC_BIT32(bit)          (0x80000000UL >> (bit))
> +#define PPC_BIT8(bit)           (0x80UL >> (bit))
> +#define PPC_BITMASK(bs, be)     ((PPC_BIT(bs) - PPC_BIT(be)) | PPC_BIT(bs))
> +#define PPC_BITMASK32(bs, be)   ((PPC_BIT32(bs) - PPC_BIT32(be)) | \
> +                                 PPC_BIT32(bs))
> +
>  /*****************************************************************************/
>  /* Exception vectors definitions                                             */
>  enum {
> @@ -371,10 +378,10 @@ struct ppc_slb_t {
>  #define MSR_LE   0  /* Little-endian mode                           1 hflags */
>  
>  /* LPCR bits */
> -#define LPCR_VPM0         (1ull << (63 - 0))
> -#define LPCR_VPM1         (1ull << (63 - 1))
> -#define LPCR_ISL          (1ull << (63 - 2))
> -#define LPCR_KBV          (1ull << (63 - 3))
> +#define LPCR_VPM0         PPC_BIT(0)
> +#define LPCR_VPM1         PPC_BIT(1)
> +#define LPCR_ISL          PPC_BIT(2)
> +#define LPCR_KBV          PPC_BIT(3)
>  #define LPCR_DPFD_SHIFT   (63 - 11)
>  #define LPCR_DPFD         (0x7ull << LPCR_DPFD_SHIFT)
>  #define LPCR_VRMASD_SHIFT (63 - 16)
> @@ -382,41 +389,41 @@ struct ppc_slb_t {
>  /* P9: Power-saving mode Exit Cause Enable (Upper Section) Mask */
>  #define LPCR_PECE_U_SHIFT (63 - 19)
>  #define LPCR_PECE_U_MASK  (0x7ull << LPCR_PECE_U_SHIFT)
> -#define LPCR_HVEE         (1ull << (63 - 17)) /* Hypervisor Virt Exit Enable */
> +#define LPCR_HVEE         PPC_BIT(17) /* Hypervisor Virt Exit Enable */
>  #define LPCR_RMLS_SHIFT   (63 - 37)
>  #define LPCR_RMLS         (0xfull << LPCR_RMLS_SHIFT)
> -#define LPCR_ILE          (1ull << (63 - 38))
> +#define LPCR_ILE          PPC_BIT(38)
>  #define LPCR_AIL_SHIFT    (63 - 40)      /* Alternate interrupt location */
>  #define LPCR_AIL          (3ull << LPCR_AIL_SHIFT)
> -#define LPCR_UPRT         (1ull << (63 - 41)) /* Use Process Table */
> -#define LPCR_EVIRT        (1ull << (63 - 42)) /* Enhanced Virtualisation */
> -#define LPCR_ONL          (1ull << (63 - 45))
> -#define LPCR_LD           (1ull << (63 - 46)) /* Large Decrementer */
> -#define LPCR_P7_PECE0     (1ull << (63 - 49))
> -#define LPCR_P7_PECE1     (1ull << (63 - 50))
> -#define LPCR_P7_PECE2     (1ull << (63 - 51))
> -#define LPCR_P8_PECE0     (1ull << (63 - 47))
> -#define LPCR_P8_PECE1     (1ull << (63 - 48))
> -#define LPCR_P8_PECE2     (1ull << (63 - 49))
> -#define LPCR_P8_PECE3     (1ull << (63 - 50))
> -#define LPCR_P8_PECE4     (1ull << (63 - 51))
> +#define LPCR_UPRT         PPC_BIT(41) /* Use Process Table */
> +#define LPCR_EVIRT        PPC_BIT(42) /* Enhanced Virtualisation */
> +#define LPCR_ONL          PPC_BIT(45)
> +#define LPCR_LD           PPC_BIT(46) /* Large Decrementer */
> +#define LPCR_P7_PECE0     PPC_BIT(49)
> +#define LPCR_P7_PECE1     PPC_BIT(50)
> +#define LPCR_P7_PECE2     PPC_BIT(51)
> +#define LPCR_P8_PECE0     PPC_BIT(47)
> +#define LPCR_P8_PECE1     PPC_BIT(48)
> +#define LPCR_P8_PECE2     PPC_BIT(49)
> +#define LPCR_P8_PECE3     PPC_BIT(50)
> +#define LPCR_P8_PECE4     PPC_BIT(51)
>  /* P9: Power-saving mode Exit Cause Enable (Lower Section) Mask */
>  #define LPCR_PECE_L_SHIFT (63 - 51)
>  #define LPCR_PECE_L_MASK  (0x1full << LPCR_PECE_L_SHIFT)
> -#define LPCR_PDEE         (1ull << (63 - 47)) /* Privileged Doorbell Exit EN */
> -#define LPCR_HDEE         (1ull << (63 - 48)) /* Hyperv Doorbell Exit Enable */
> -#define LPCR_EEE          (1ull << (63 - 49)) /* External Exit Enable        */
> -#define LPCR_DEE          (1ull << (63 - 50)) /* Decrementer Exit Enable     */
> -#define LPCR_OEE          (1ull << (63 - 51)) /* Other Exit Enable           */
> -#define LPCR_MER          (1ull << (63 - 52))
> -#define LPCR_GTSE         (1ull << (63 - 53)) /* Guest Translation Shootdown */
> -#define LPCR_TC           (1ull << (63 - 54))
> -#define LPCR_HEIC         (1ull << (63 - 59)) /* HV Extern Interrupt Control */
> -#define LPCR_LPES0        (1ull << (63 - 60))
> -#define LPCR_LPES1        (1ull << (63 - 61))
> -#define LPCR_RMI          (1ull << (63 - 62))
> -#define LPCR_HVICE        (1ull << (63 - 62)) /* HV Virtualisation Int Enable */
> -#define LPCR_HDICE        (1ull << (63 - 63))
> +#define LPCR_PDEE         PPC_BIT(47) /* Privileged Doorbell Exit EN */
> +#define LPCR_HDEE         PPC_BIT(48) /* Hyperv Doorbell Exit Enable */
> +#define LPCR_EEE          PPC_BIT(49) /* External Exit Enable        */
> +#define LPCR_DEE          PPC_BIT(50) /* Decrementer Exit Enable     */
> +#define LPCR_OEE          PPC_BIT(51) /* Other Exit Enable           */
> +#define LPCR_MER          PPC_BIT(52)
> +#define LPCR_GTSE         PPC_BIT(53) /* Guest Translation Shootdown */
> +#define LPCR_TC           PPC_BIT(54)
> +#define LPCR_HEIC         PPC_BIT(59) /* HV Extern Interrupt Control */
> +#define LPCR_LPES0        PPC_BIT(60)
> +#define LPCR_LPES1        PPC_BIT(61)
> +#define LPCR_RMI          PPC_BIT(62)
> +#define LPCR_HVICE        PPC_BIT(62) /* HV Virtualisation Int Enable */
> +#define LPCR_HDICE        PPC_BIT(63)
>  
>  #define msr_sf   ((env->msr >> MSR_SF)   & 1)
>  #define msr_isf  ((env->msr >> MSR_ISF)  & 1)
> @@ -507,22 +514,22 @@ struct ppc_slb_t {
>  #define FSCR_IC_TAR         8
>  
>  /* Exception state register bits definition                                  */
> -#define ESR_PIL   (1 << (63 - 36)) /* Illegal Instruction                    */
> -#define ESR_PPR   (1 << (63 - 37)) /* Privileged Instruction                 */
> -#define ESR_PTR   (1 << (63 - 38)) /* Trap                                   */
> -#define ESR_FP    (1 << (63 - 39)) /* Floating-Point Operation               */
> -#define ESR_ST    (1 << (63 - 40)) /* Store Operation                        */
> -#define ESR_AP    (1 << (63 - 44)) /* Auxiliary Processor Operation          */
> -#define ESR_PUO   (1 << (63 - 45)) /* Unimplemented Operation                */
> -#define ESR_BO    (1 << (63 - 46)) /* Byte Ordering                          */
> -#define ESR_PIE   (1 << (63 - 47)) /* Imprecise exception                    */
> -#define ESR_DATA  (1 << (63 - 53)) /* Data Access (Embedded page table)      */
> -#define ESR_TLBI  (1 << (63 - 54)) /* TLB Ineligible (Embedded page table)   */
> -#define ESR_PT    (1 << (63 - 55)) /* Page Table (Embedded page table)       */
> -#define ESR_SPV   (1 << (63 - 56)) /* SPE/VMX operation                      */
> -#define ESR_EPID  (1 << (63 - 57)) /* External Process ID operation          */
> -#define ESR_VLEMI (1 << (63 - 58)) /* VLE operation                          */
> -#define ESR_MIF   (1 << (63 - 62)) /* Misaligned instruction (VLE)           */
> +#define ESR_PIL   PPC_BIT(36) /* Illegal Instruction                    */
> +#define ESR_PPR   PPC_BIT(37) /* Privileged Instruction                 */
> +#define ESR_PTR   PPC_BIT(38) /* Trap                                   */
> +#define ESR_FP    PPC_BIT(39) /* Floating-Point Operation               */
> +#define ESR_ST    PPC_BIT(40) /* Store Operation                        */
> +#define ESR_AP    PPC_BIT(44) /* Auxiliary Processor Operation          */
> +#define ESR_PUO   PPC_BIT(45) /* Unimplemented Operation                */
> +#define ESR_BO    PPC_BIT(46) /* Byte Ordering                          */
> +#define ESR_PIE   PPC_BIT(47) /* Imprecise exception                    */
> +#define ESR_DATA  PPC_BIT(53) /* Data Access (Embedded page table)      */
> +#define ESR_TLBI  PPC_BIT(54) /* TLB Ineligible (Embedded page table)   */
> +#define ESR_PT    PPC_BIT(55) /* Page Table (Embedded page table)       */
> +#define ESR_SPV   PPC_BIT(56) /* SPE/VMX operation                      */
> +#define ESR_EPID  PPC_BIT(57) /* External Process ID operation          */
> +#define ESR_VLEMI PPC_BIT(58) /* VLE operation                          */
> +#define ESR_MIF   PPC_BIT(62) /* Misaligned instruction (VLE)           */
>  
>  /* Transaction EXception And Summary Register bits                           */
>  #define TEXASR_FAILURE_PERSISTENT                (63 - 7)
> @@ -1991,7 +1998,7 @@ void ppc_compat_add_property(Object *obj, const char *name,
>  #define HID0_DEEPNAP        (1 << 24)           /* pre-2.06 */
>  #define HID0_DOZE           (1 << 23)           /* pre-2.06 */
>  #define HID0_NAP            (1 << 22)           /* pre-2.06 */
> -#define HID0_HILE           (1ull << (63 - 19)) /* POWER8 */
> +#define HID0_HILE           PPC_BIT(19) /* POWER8 */
>  
>  /*****************************************************************************/
>  /* PowerPC Instructions types definitions                                    */

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson