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] support of lock profiling in Xen

To: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [Patch] support of lock profiling in Xen
From: Juergen Gross <juergen.gross@xxxxxxxxxxxxxx>
Date: Thu, 08 Oct 2009 11:35:47 +0200
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Thu, 08 Oct 2009 02:36:11 -0700
Dkim-signature: v=1; a=rsa-sha256; c=simple/simple; d=ts.fujitsu.com; i=juergen.gross@xxxxxxxxxxxxxx; q=dns/txt; s=s1536b; t=1254994613; x=1286530613; h=from:sender:reply-to:subject:date:message-id:to:cc: mime-version:content-transfer-encoding:content-id: content-description:resent-date:resent-from:resent-sender: resent-to:resent-cc:resent-message-id:in-reply-to: references:list-id:list-help:list-unsubscribe: list-subscribe:list-post:list-owner:list-archive; z=From:=20Juergen=20Gross=20<juergen.gross@xxxxxxxxxxxxxx> |Subject:=20Re:=20[Xen-devel]=20[Patch]=20support=20of=20 lock=20profiling=20in=20Xen|Date:=20Thu,=2008=20Oct=20200 9=2011:35:47=20+0200|Message-ID:=20<4ACDB273.1090909@xxxx ujitsu.com>|To:=20Keir=20Fraser=20<keir.fraser@xxxxxxxxxx com>|CC:=20"xen-devel@xxxxxxxxxxxxxxxxxxx"=20<xen-devel@l ists.xensource.com>|MIME-Version:=201.0 |Content-Transfer-Encoding:=207bit|In-Reply-To:=20<C6F36D 8C.16DCA%keir.fraser@xxxxxxxxxxxxx>|References:=20<C6F36D 8C.16DCA%keir.fraser@xxxxxxxxxxxxx>; bh=xSSlyhfAGeQcFX6fypECu5GTrQDHbtI99XyBlMWTmEM=; b=pYycbzt0y0pWrq4Cz1XVBtQ29cr65xPlEJ4mgZw7ilbW9iYEUmB/0hay qtfYDAQLXwUoQ6PiIzkLRR2XShQbG1bHTOKGXJBB6s+swEmSKdtODxZwf D99VvEhAl5w6pHKCx50kPQGueHPIfpBBxfqtB9Ja3tL/hYamE5gJVjeU2 SC84VnPXAsParI3bZyY6dY3Jff3hg0gKTg6ciPciOE0a4/0RPaW5bBjCr t+BC3qty17iEGscjycr5U6xEeOj3z;
Domainkey-signature: s=s1536a; d=ts.fujitsu.com; c=nofws; q=dns; h=X-SBRSScore:X-IronPort-AV:Received:X-IronPort-AV: Received:Received:Message-ID:Date:From:Organization: User-Agent:MIME-Version:To:CC:Subject:References: In-Reply-To:X-Enigmail-Version:Content-Type: Content-Transfer-Encoding; b=b2CUQ3usSlSXCbbjFFGwXh8dcOO5fZni+jKB1GpEkGM3WYIGXTDCUnsf bCsE0+GNNZVF+/uXTjEDEZoNxbIv9MP8ZOBEcK7f5EDC5eimSGKktxww1 E7TW8vE5YsF5jlqyKQMY+BSNbhIDYTB8Ock1TAGWHjupztSNDJ8YBHR78 dazdZNM28FBd+IVPz/oSyizAlw1AaoVoHOE0dqQG9jnkQ187kGEDU/M9D vlZ5JUmDzKtIlIqmhtIgPdiI4JMHt;
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C6F36D8C.16DCA%keir.fraser@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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Organization: Fujitsu Technology Solutions
References: <C6F36D8C.16DCA%keir.fraser@xxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla-Thunderbird 2.0.0.22 (X11/20090707)
Keir Fraser wrote:
> On 08/10/2009 10:04, "Juergen Gross" <juergen.gross@xxxxxxxxxxxxxx> wrote:
> 
>>>> It would even be possible to define lock arrays if summing up all data for
>>>> e.g. domain specific locks is not desired.
>>> Allowing spinlock_init() to specify not only a name but also a
>>> 'category-specific index' might be okay. For example, domid, in the case of
>>> a domain-specific lock. Hiding the allocation of profiling info structs in
>>> the spinlock subsystem somehow... (e.g., radix tree to make a simple
>>> growable array).
>> I'll try another patch...
> 
> Well, see my email just now. I could perhaps live with perfc-style
> declarations after all... Because this might make sense if locks can be
> broadly categorised as global or as per-domain, and it could avoid you
> having to make your implementation much more complex?
> 
> After all, now I think about it, an 'arbitrary' index specified to
> spinlock_init() is just making our lives more complicated if it will
> basically *always* be a domid. And knowing at a higher level which locks are
> actually per-domain could help your pretty printing?

I thought in this direction, too.
I would, however, try not to limit it to per domain locks, even if the first
implementation would support only global locks and per domain locks.

The perfc style declarations should only be necessary for the dynamically
initialized locks, and those could live near the domain structure definition
(or any other structure where they are needed).

I think the statically defined locks can be handled via an own data section
requiring no centrally defined index.


Juergen

-- 
Juergen Gross                 Principal Developer Operating Systems
TSP ES&S SWE OS6                       Telephone: +49 (0) 89 636 47950
Fujitsu Technolgy Solutions               e-mail: juergen.gross@xxxxxxxxxxxxxx
Otto-Hahn-Ring 6                        Internet: ts.fujitsu.com
D-81739 Muenchen                 Company details: ts.fujitsu.com/imprint.html

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