From mboxrd@z Thu Jan  1 00:00:00 1970
From: Keir Fraser <keir.xen@gmail.com>
Subject: Re: [PATCH] x86: remove open-coded IO-APIC RTE
 reads/writes
Date: Tue, 18 Sep 2012 18:00:31 +0100
Message-ID: <CC7E673F.3F141%keir.xen@gmail.com>
References: <5058B695020000780009C090@nat28.tlf.novell.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <5058B695020000780009C090@nat28.tlf.novell.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Jan Beulich <JBeulich@suse.com>, xen-devel <xen-devel@lists.xen.org>
List-Id: xen-devel@lists.xenproject.org

On 18/09/2012 16:59, "Jan Beulich" <JBeulich@suse.com> wrote:

> This improves readability, not the least through doing away with a
> couple of ugly casts.
> 
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

Acked-by: Keir Fraser <keir@xen.org>

> --- a/xen/arch/x86/io_apic.c
> +++ b/xen/arch/x86/io_apic.c
> @@ -225,9 +225,8 @@ void __ioapic_write_entry(
>  {
>      void (*write)(unsigned int, unsigned int, unsigned int)
>          = raw ? __io_apic_write : io_apic_write;
> -    union entry_union eu = {{0, 0}};
> +    union entry_union eu = { .entry = e };
>  
> -    eu.entry = e;
>      (*write)(apic, 0x11 + 2*pin, eu.w2);
>      (*write)(apic, 0x10 + 2*pin, eu.w1);
>  }
> @@ -2384,8 +2383,7 @@ int ioapic_guest_write(unsigned long phy
>      SET_DEST(rte.dest.dest32, rte.dest.logical.logical_dest,
>               cpu_mask_to_apicid(desc->arch.cpu_mask));
>  
> -    io_apic_write(apic, 0x10 + 2 * pin, *(((int *)&rte) + 0));
> -    io_apic_write(apic, 0x11 + 2 * pin, *(((int *)&rte) + 1));
> +    __ioapic_write_entry(apic, pin, 0, rte);
>      
>      spin_unlock_irqrestore(&ioapic_lock, flags);
>  
> @@ -2414,7 +2412,6 @@ void dump_ioapic_irq_info(void)
>      struct irq_pin_list *entry;
>      struct IO_APIC_route_entry rte;
>      unsigned int irq, pin, printed = 0;
> -    unsigned long flags;
>  
>      if ( !irq_2_pin )
>          return;
> @@ -2436,10 +2433,7 @@ void dump_ioapic_irq_info(void)
>  
>              printk("      Apic 0x%02x, Pin %2d: ", entry->apic, pin);
>  
> -            spin_lock_irqsave(&ioapic_lock, flags);
> -            *(((int *)&rte) + 0) = io_apic_read(entry->apic, 0x10 + 2 * pin);
> -            *(((int *)&rte) + 1) = io_apic_read(entry->apic, 0x11 + 2 * pin);
> -            spin_unlock_irqrestore(&ioapic_lock, flags);
> +            rte = ioapic_read_entry(entry->apic, pin, 0);
>  
>              printk("vec=%02x delivery=%-5s dest=%c status=%d "
>                     "polarity=%d irr=%d trig=%c mask=%d dest_id:%d\n",
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel