[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [PATCH v3] x86/ioapic: Remove use of TRUE/FALSE/1/0


  • To: Kevin Stefanov <kevin.stefanov@xxxxxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
  • Date: Tue, 17 Aug 2021 03:10:52 +0000
  • Accept-language: en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=JLaJl9Q08+NSpwSz81YJ3bNSNr9SyoRWWad5KDs+P6Q=; b=XfUzQYOsj5hiJVMuIN/WFT4fA7vVoIJ6/xM7vZlJ+osbx9NyWGWitE8gyifA1IPa8s4xkpGpJvmwuBo3U2EqgXnEHmC73cwo6pLsWp0c+3LaSqAE0IKNh6jd7G0FWt8x1k43fBacBOQ6mHr+qx4bMROhiFNRSsRibEZQFwgXDMv00377ii58qUXC7LMVR/fkNJapmVUdyxIN9zsFLwF275nC74fyW3mgb/sBdw0hbaCv59R486C0fhy7ExvfNvbegF/CACa89/KXG4tSvhX4w/v/rBdyteb6scH55Q5rihYjiLoy+cfVfVlWE3FOhMl5NMNGoFdh0lwI62C3w9jG1g==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=irz0HUVrYukegCvPhkbDoxPlwOu8fKe8bkMvw/aSH/5QGGua1DTjzG/wGJFUkQn84Ge1rL+UPS0QTa5BCF6D4vxl4KsDoZy1cblxx2BWpxe4vGBKg5ehF0WP+4F7ZEuxOACD6Q9GWBYJ1tFGJLrHfJpstsVCD2tJ9JryaeZgUOF2ZZo852DZ2VqgZN8KVDfltboZh+CY+xRsLEb0DyEtvlpfceU8sddg2/ObWiFvzxoVpyR7ShEysMYepDt7rob/rbObEFcnSfjn5AyuUllRonOikM8830ai+WDUfRu2KUHzjlrkcKAS2com+rZvLWVia4G9UvXYJh6PmJFWvHnWIA==
  • Authentication-results: citrix.com; dkim=none (message not signed) header.d=none;citrix.com; dmarc=none action=none header.from=intel.com;
  • Cc: "Cooper, Andrew" <andrew.cooper3@xxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
  • Delivery-date: Tue, 17 Aug 2021 03:11:04 +0000
  • Dlp-product: dlpe-windows
  • Dlp-reaction: no-action
  • Dlp-version: 11.5.1.3
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHXkpxJdrGR4C4efUC5d8DHQyu3Sat3BYhw
  • Thread-topic: [PATCH v3] x86/ioapic: Remove use of TRUE/FALSE/1/0

> From: Kevin Stefanov <kevin.stefanov@xxxxxxxxxx>
> Sent: Monday, August 16, 2021 8:43 PM
> 
> Also fix stray usage in VT-d.
> 
> Suggested-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> Signed-off-by: Kevin Stefanov <kevin.stefanov@xxxxxxxxxx>

Reviewed-by: Kevin Tian <kevin.tian@xxxxxxxxx>

> ---
> CC: Jan Beulich <jbeulich@xxxxxxxx>
> CC: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> CC: "Roger Pau Monné" <roger.pau@xxxxxxxxxx>
> CC: Wei Liu <wl@xxxxxxx>
> CC: Kevin Tian <kevin.tian@xxxxxxxxx>
> 
> v2:
>  * Also replace literal 1/0
> v3:
>  * Fix 1->false conversion error
> ---
>  xen/arch/x86/io_apic.c                 | 38 +++++++++++++-------------
>  xen/drivers/passthrough/vtd/intremap.c |  6 ++--
>  xen/drivers/passthrough/vtd/utils.c    |  2 +-
>  3 files changed, 23 insertions(+), 23 deletions(-)
> 
> diff --git a/xen/arch/x86/io_apic.c b/xen/arch/x86/io_apic.c
> index 58b26d962c..c3ad9efac8 100644
> --- a/xen/arch/x86/io_apic.c
> +++ b/xen/arch/x86/io_apic.c
> @@ -289,7 +289,7 @@ static void __io_apic_eoi(unsigned int apic, unsigned
> int vector, unsigned int p
>      {
>          /* If vector is unknown, read it from the IO-APIC */
>          if ( vector == IRQ_VECTOR_UNASSIGNED )
> -            vector = __ioapic_read_entry(apic, pin, TRUE).vector;
> +            vector = __ioapic_read_entry(apic, pin, true).vector;
> 
>          *(IO_APIC_BASE(apic)+16) = vector;
>      }
> @@ -300,28 +300,28 @@ static void __io_apic_eoi(unsigned int apic,
> unsigned int vector, unsigned int p
>          struct IO_APIC_route_entry entry;
>          bool need_to_unmask = false;
> 
> -        entry = __ioapic_read_entry(apic, pin, TRUE);
> +        entry = __ioapic_read_entry(apic, pin, true);
> 
>          if ( ! entry.mask )
>          {
>              /* If entry is not currently masked, mask it and make
>               * a note to unmask it later */
>              entry.mask = 1;
> -            __ioapic_write_entry(apic, pin, TRUE, entry);
> +            __ioapic_write_entry(apic, pin, true, entry);
>              need_to_unmask = true;
>          }
> 
>          /* Flip the trigger mode to edge and back */
>          entry.trigger = 0;
> -        __ioapic_write_entry(apic, pin, TRUE, entry);
> +        __ioapic_write_entry(apic, pin, true, entry);
>          entry.trigger = 1;
> -        __ioapic_write_entry(apic, pin, TRUE, entry);
> +        __ioapic_write_entry(apic, pin, true, entry);
> 
>          if ( need_to_unmask )
>          {
>              /* Unmask if neccesary */
>              entry.mask = 0;
> -            __ioapic_write_entry(apic, pin, TRUE, entry);
> +            __ioapic_write_entry(apic, pin, true, entry);
>          }
>      }
>  }
> @@ -344,7 +344,7 @@ int save_IO_APIC_setup(struct IO_APIC_route_entry
> **ioapic_entries)
>              return -ENOMEM;
> 
>          for (pin = 0; pin < nr_ioapic_entries[apic]; pin++)
> -         ioapic_entries[apic][pin] = __ioapic_read_entry(apic, pin, 1);
> +         ioapic_entries[apic][pin] = __ioapic_read_entry(apic, pin, true);
>      }
> 
>      return 0;
> @@ -374,7 +374,7 @@ void mask_IO_APIC_setup(struct
> IO_APIC_route_entry **ioapic_entries)
>              if (!entry.mask) {
>                  entry.mask = 1;
> 
> -                ioapic_write_entry(apic, pin, 1, entry);
> +                ioapic_write_entry(apic, pin, true, entry);
>              }
>          }
>      }
> @@ -1047,7 +1047,7 @@ static void __init setup_IO_APIC_irqs(void)
> 
>              SET_DEST(entry, logical, cpu_mask_to_apicid(TARGET_CPUS));
>              spin_lock_irqsave(&ioapic_lock, flags);
> -            __ioapic_write_entry(apic, pin, 0, entry);
> +            __ioapic_write_entry(apic, pin, false, entry);
>              spin_unlock_irqrestore(&ioapic_lock, flags);
>          }
>      }
> @@ -1091,7 +1091,7 @@ static void __init setup_ExtINT_IRQ0_pin(unsigned
> int apic, unsigned int pin, in
>      /*
>       * Add it to the IO-APIC irq-routing table:
>       */
> -    ioapic_write_entry(apic, pin, 0, entry);
> +    ioapic_write_entry(apic, pin, false, entry);
> 
>      enable_8259A_irq(irq_to_desc(0));
>  }
> @@ -1203,7 +1203,7 @@ static void /*__init*/ __print_IO_APIC(bool boot)
>       for (i = 0; i <= reg_01.bits.entries; i++) {
>              struct IO_APIC_route_entry entry;
> 
> -            entry = ioapic_read_entry(apic, i, 0);
> +            entry = ioapic_read_entry(apic, i, false);
> 
>              if ( x2apic_enabled && iommu_intremap )
>                  printk(KERN_DEBUG " %02x %08x", i, entry.dest.dest32);
> @@ -1290,7 +1290,7 @@ static void __init enable_IO_APIC(void)
>          int pin;
>          /* See if any of the pins is in ExtINT mode */
>          for (pin = 0; pin < nr_ioapic_entries[apic]; pin++) {
> -            struct IO_APIC_route_entry entry = ioapic_read_entry(apic, pin, 
> 0);
> +            struct IO_APIC_route_entry entry = ioapic_read_entry(apic, pin,
> false);
> 
>              /* If the interrupt line is enabled and in ExtInt mode
>               * I have found the pin where the i8259 is connected.
> @@ -1361,7 +1361,7 @@ void disable_IO_APIC(void)
>          /*
>           * Add it to the IO-APIC irq-routing table:
>           */
> -        ioapic_write_entry(ioapic_i8259.apic, ioapic_i8259.pin, 0, entry);
> +        ioapic_write_entry(ioapic_i8259.apic, ioapic_i8259.pin, false, 
> entry);
>      }
>      disconnect_bsp_APIC(ioapic_i8259.pin != -1);
>  }
> @@ -1841,7 +1841,7 @@ static void __init unlock_ExtINT_logic(void)
>      if ( pin == -1 || apic == -1 )
>          return;
> 
> -    entry0 = ioapic_read_entry(apic, pin, 0);
> +    entry0 = ioapic_read_entry(apic, pin, false);
>      clear_IO_APIC_pin(apic, pin);
> 
>      memset(&entry1, 0, sizeof(entry1));
> @@ -1854,7 +1854,7 @@ static void __init unlock_ExtINT_logic(void)
>      entry1.trigger = 0;
>      entry1.vector = 0;
> 
> -    ioapic_write_entry(apic, pin, 0, entry1);
> +    ioapic_write_entry(apic, pin, false, entry1);
> 
>      save_control = CMOS_READ(RTC_CONTROL);
>      save_freq_select = CMOS_READ(RTC_FREQ_SELECT);
> @@ -1873,7 +1873,7 @@ static void __init unlock_ExtINT_logic(void)
>      CMOS_WRITE(save_freq_select, RTC_FREQ_SELECT);
>      clear_IO_APIC_pin(apic, pin);
> 
> -    ioapic_write_entry(apic, pin, 0, entry0);
> +    ioapic_write_entry(apic, pin, false, entry0);
>  }
> 
>  /*
> @@ -2287,7 +2287,7 @@ int io_apic_set_pci_routing (int ioapic, int pin, int
> irq, int edge_level, int a
>          disable_8259A_irq(desc);
> 
>      spin_lock_irqsave(&ioapic_lock, flags);
> -    __ioapic_write_entry(ioapic, pin, 0, entry);
> +    __ioapic_write_entry(ioapic, pin, false, entry);
>      spin_unlock(&ioapic_lock);
> 
>      spin_lock(&desc->lock);
> @@ -2476,7 +2476,7 @@ int ioapic_guest_write(unsigned long physbase,
> unsigned int reg, u32 val)
>          rte.mask = 1;
>      }
> 
> -    __ioapic_write_entry(apic, pin, 0, rte);
> +    __ioapic_write_entry(apic, pin, false, rte);
> 
>      spin_unlock_irqrestore(&ioapic_lock, flags);
> 
> @@ -2529,7 +2529,7 @@ void dump_ioapic_irq_info(void)
> 
>              printk("      Apic 0x%02x, Pin %2d: ", entry->apic, pin);
> 
> -            rte = ioapic_read_entry(entry->apic, pin, 0);
> +            rte = ioapic_read_entry(entry->apic, pin, false);
> 
>              printk("vec=%02x delivery=%-5s dest=%c status=%d "
>                     "polarity=%d irr=%d trig=%c mask=%d dest_id:%0*x\n",
> diff --git a/xen/drivers/passthrough/vtd/intremap.c
> b/xen/drivers/passthrough/vtd/intremap.c
> index d0f70d90eb..01152f2006 100644
> --- a/xen/drivers/passthrough/vtd/intremap.c
> +++ b/xen/drivers/passthrough/vtd/intremap.c
> @@ -427,7 +427,7 @@ unsigned int io_apic_read_remap_rte(
>          ( (index = apic_pin_2_ir_idx[apic][ioapic_pin]) < 0 ) )
>          return __io_apic_read(apic, reg);
> 
> -    old_rte = __ioapic_read_entry(apic, ioapic_pin, 1);
> +    old_rte = __ioapic_read_entry(apic, ioapic_pin, true);
> 
>      if ( remap_entry_to_ioapic_rte(iommu, index, &old_rte) )
>          return __io_apic_read(apic, reg);
> @@ -448,7 +448,7 @@ void io_apic_write_remap_rte(
>      struct vtd_iommu *iommu = ioapic_to_iommu(IO_APIC_ID(apic));
>      int saved_mask;
> 
> -    old_rte = __ioapic_read_entry(apic, ioapic_pin, 1);
> +    old_rte = __ioapic_read_entry(apic, ioapic_pin, true);
> 
>      remap_rte = (struct IO_APIC_route_remap_entry *) &old_rte;
> 
> @@ -468,7 +468,7 @@ void io_apic_write_remap_rte(
>              __io_apic_write(apic, reg & ~1, *(u32 *)&old_rte);
>      }
>      else
> -        __ioapic_write_entry(apic, ioapic_pin, 1, old_rte);
> +        __ioapic_write_entry(apic, ioapic_pin, true, old_rte);
>  }
> 
>  static void set_msi_source_id(struct pci_dev *pdev, struct iremap_entry *ire)
> diff --git a/xen/drivers/passthrough/vtd/utils.c
> b/xen/drivers/passthrough/vtd/utils.c
> index 4febcf506d..70add3cc8e 100644
> --- a/xen/drivers/passthrough/vtd/utils.c
> +++ b/xen/drivers/passthrough/vtd/utils.c
> @@ -281,7 +281,7 @@ void vtd_dump_iommu_info(unsigned char key)
>              for ( i = 0; i <= reg_01.bits.entries; i++ )
>              {
>                  struct IO_APIC_route_entry rte =
> -                    __ioapic_read_entry(apic, i, TRUE);
> +                    __ioapic_read_entry(apic, i, true);
> 
>                  remap = (struct IO_APIC_route_remap_entry *) &rte;
>                  if ( !remap->format )
> --
> 2.25.1


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.