[PATCH] xentrace: trace domain destruction

[PATCH] xentrace: trace domain destruction

[George Dunlap]

TRC_SCHED_DOM_REM is defined, but never used.  Use it.

Signed-off-by: George Dunlap <george.dunlap@eu.citrix.com>

Re: [PATCH] xentrace: trace domain destruction

[George Dunlap]

Attatching the patch this time.
 -G

On Mon, Mar 9, 2009 at 2:30 PM, George Dunlap
<George.Dunlap@eu.citrix.com> wrote:
> TRC_SCHED_DOM_REM is defined, but never used.  Use it.
>
> Signed-off-by: George Dunlap <george.dunlap@eu.citrix.com>
>

Re: [PATCH] xentrace: trace domain destruction

[George Dunlap]

[-- Attachment #1: Type: text/plain, Size: 570 bytes --]

Once more, with feeling...

Turns out that if the file is empty (because for example, one forgot
to type "hg qrefresh"), gmail helpfully doesn't attach anything at
all, but doesn't mention the fact.

 -George

On Mon, Mar 9, 2009 at 3:18 PM, George Dunlap
<George.Dunlap@eu.citrix.com> wrote:
> Attatching the patch this time.
>  -G
>
> On Mon, Mar 9, 2009 at 2:30 PM, George Dunlap
> <George.Dunlap@eu.citrix.com> wrote:
>> TRC_SCHED_DOM_REM is defined, but never used.  Use it.
>>
>> Signed-off-by: George Dunlap <george.dunlap@eu.citrix.com>
>>
>

[-- Attachment #2: trace.dom-destroy.diff --]
[-- Type: text/x-diff, Size: 606 bytes --]

diff -r b249f3e979a5 xen/common/domain.c
--- a/xen/common/domain.c	Mon Mar 09 10:32:24 2009 +0000
+++ b/xen/common/domain.c	Mon Mar 09 15:44:07 2009 +0000
@@ -30,6 +30,7 @@
 #include <public/sched.h>
 #include <public/vcpu.h>
 #include <xsm/xsm.h>
+#include <xen/trace.h>
 
 /* Linux config option: propageted to domain0 */
 /* xen_processor_pmbits: xen control Cx, Px, ... */
@@ -595,6 +596,8 @@
         return;
 
     /* Delete from task list and task hashtable. */
+    TRACE_1D(TRC_SCHED_DOM_REM, d->domain_id);
+
     spin_lock(&domlist_update_lock);
     pd = &domain_list;
     while ( *pd != d ) 

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

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