|  |  | 
  
    |  |  | 
 
  |   |  | 
  
    |  |  | 
  
    |  |  | 
  
    |   xen-devel
RE: [Xen-devel] [PATCH] Reduce overhead in find_domain_by_id() [0/2] 
| To: | "Keir Fraser" <keir@xxxxxxxxxxxxx>,	"Apparao, Padmashree K" <padmashree.k.apparao@xxxxxxxxx>,	<xen-devel@xxxxxxxxxxxxxxxxxxx> |  
| Subject: | RE: [Xen-devel] [PATCH] Reduce overhead in find_domain_by_id() [0/2] |  
| From: | "Santos, Jose Renato G" <joserenato.santos@xxxxxx> |  
| Date: | Thu, 7 Dec 2006 22:17:54 -0600 |  
| Cc: | "Turner, Yoshio" <yoshio_turner@xxxxxx>,	Jose Renato Santos <jsantos@xxxxxxxxxx>,	G John Janakiraman <john@xxxxxxxxxxxxxxxxxxx> |  
| Delivery-date: | Thu, 07 Dec 2006 20:18:27 -0800 |  
| Envelope-to: | www-data@xxxxxxxxxxxxxxxxxx |  
| In-reply-to: | <C19DB32A.5AAE%keir@xxxxxxxxxxxxx> |  
| 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: | AccY1tScspQxhfmoTr62Wf8QIRwz8wAO8dbrAAZN7LAAAQAGywAfw6ewABIlWWoAIgAvcA== |  
| Thread-topic: | [Xen-devel] [PATCH] Reduce overhead in find_domain_by_id() [0/2] |  
|  
> -----Original Message-----
> From: Keir Fraser [mailto:keir@xxxxxxxxxxxxx] 
> Sent: Thursday, December 07, 2006 4:02 AM
> To: Santos, Jose Renato G; Keir Fraser; Apparao, Padmashree 
> K; xen-devel@xxxxxxxxxxxxxxxxxxx
> Cc: Turner, Yoshio; Jose Renato Santos; G John Janakiraman
> Subject: Re: [Xen-devel] [PATCH] Reduce overhead in 
> find_domain_by_id() [0/2]
> 
> On 7/12/06 03:49, "Santos, Jose Renato G" 
> <joserenato.santos@xxxxxx> wrote:
> 
> > Can't think of anything else. If you want I can remove these and 
> > submit a revised patch.
> > It seems that there is not much more we could remove. Not 
> sure if you 
> > have something else in mind...
> 
> I suppose the patch isn't actually as big as I first 
> imagined. I'll review it and probably apply it pretty much as 
> is. It's just that RCU has always seemed rather over 
> complicated to me (lots of different queues, for example) for 
> what should be a rather simple concept to implement. This is 
> possibly just ignorance on my part. :-)
> 
>
Yes, the multiple queues seems somewhat complicated, but
I think we need them to be able to handle multiple callbacks.
Maybe we can get rid of of of them ("done")...
> As for per-cpu refcounts, I suspect a better scheme would be 
> find_domain_by_id_noref(). Idea being that often we take a 
> reference for a short period of time (in particular, scope of 
> one function) and with delayed destruction we can now safely 
> use a found domain pointer with no refcnt increment.
> 
Good point. This is indeed a much better scheme. I will 
work out a patch for find_domain_by_id_noref().
Thanks
Renato
>  -- Keir
> 
> 
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
 | 
 |  | 
  
    |  |  |