WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

RE: [Xen-devel] [PATCH] Reuse irq number for virq/ipi after vcpu unplug/

To: "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH] Reuse irq number for virq/ipi after vcpu unplug/plug
From: "Tian, Kevin" <kevin.tian@xxxxxxxxx>
Date: Mon, 5 Feb 2007 10:49:01 +0800
Delivery-date: Sun, 04 Feb 2007 18:48:37 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <C1EBA09A.8D7D%Keir.Fraser@xxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcdHTqIuaJIC5LiAQZGiyexI3PPLIQAyY637AA7yNvAABZImIQAZTV3w
Thread-topic: [Xen-devel] [PATCH] Reuse irq number for virq/ipi after vcpu unplug/plug
So, attached patch zeros irq stats at bind_evtchn_to_cpu, since
rate of change is the real concern. By this way, we don't need to
care whether stale stats are on multiple cpus at irq unbinding.

Thanks,
Kevin.
 
>From: Keir Fraser [mailto:Keir.Fraser@xxxxxxxxxxxx] 
>Sent: 2007年2月4日 22:39
>
>On 4/2/07 12:28, "Tian, Kevin" <kevin.tian@xxxxxxxxx> wrote:
>
>> Another point I'm not for sure is the save/restore and migration.
>> From user perspective, he has no knowledge that cpu1...N are
>> unplugged and then plugged along the process. Thus will anyone
>> feel strange about the fact that all cpus except cpu0 gets their
>> per-vcpu interrupt stats empty after restored or migrated? This
>> seems not an explicit lifetime restart...
>
>...and the same is true for his network and block-device interrupts.
>Currently these may come back with different IRQs too, and 
>'fixing' this
>would be somewhat painful (e.g., it would probably need some evtchn
>interface changes). Why do you consider the VIRQs/IPIs so special?
>
>Your original point that having non-zero counts for CPU!=n for 
>an interrupt
>that is specific to CPU==n looks weird and may confuse people 
>is a fair one.
>Zeroing stats is a nice simple solution to that.
>
>As you say /proc/interrupts is used for IRQ load balancing, 
>where rate of
>change matters. I'm not aware of any other application uses of
>/proc/interrupts -- certainly none that are 'stateful' in 
>terms of requiring
>a given interrupt source to stick with its original IRQ, or 
>require accurate
>aggregate counts.
>
> -- Keir
>

Attachment: zero_irqstats_at_unbind.patch
Description: zero_irqstats_at_unbind.patch

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel