Re: [kvm-devel] [kvm-ppc-devel] [PATCH 1/5]Add some trace markers and exposeinterfaces in kernel for tracing

2008-04-21 Thread Liu, Eric E
Hollis Blanchard wrote: > On Sunday 20 April 2008 00:38:32 Liu, Eric E wrote: >> Christian Ehrhardt wrote: >>> Liu, Eric E wrote: >>>> Hollis Blanchard wrote: >>>>> On Wednesday 16 April 2008 01:45:34 Liu, Eric E wrote: [...] >>>>> Actua

Re: [kvm-devel] [kvm-ppc-devel] [PATCH 1/5]Add some trace markers and exposeinterfaces in kernel for tracing

2008-04-19 Thread Liu, Eric E
Christian Ehrhardt wrote: > Liu, Eric E wrote: >> Hollis Blanchard wrote: >>> On Wednesday 16 April 2008 01:45:34 Liu, Eric E wrote: [...] >>> Actually... we could have kvmtrace itself insert the metadata, so >>> there would be no chance of it being overwr

Re: [kvm-devel] kvm-trace help

2008-04-18 Thread Liu, Eric E
David S. Ahern wrote: > inline. > > Liu, Eric E wrote: >> David S. Ahern wrote: >>> I am trying to add a trace marker and the data is coming out all >>> 0's. e.g., >>> >>> 0 (+ 0) PTE_WRITE vcpu = 0x0001 pid = 0x240d [

Re: [kvm-devel] kvm-trace help

2008-04-18 Thread Liu, Eric E
David S. Ahern wrote: > I am trying to add a trace marker and the data is coming out all 0's. > e.g., > > 0 (+ 0) PTE_WRITE vcpu = 0x0001 pid = 0x240d [ > gpa = 0x gpte = 0x ] > > Patch is attached. I know the data is non-zero as I added an i

Re: [kvm-devel] [kvm-ppc-devel] [PATCH 1/5]Add some trace markers and exposeinterfaces in kernel for tracing

2008-04-17 Thread Liu, Eric E
Hollis Blanchard wrote: > On Wednesday 16 April 2008 01:45:34 Liu, Eric E wrote: >> >>> Hmm, while we're on the subject, I'm not sure what the best way to >>> automatically byteswap will be. It probably isn't worth it to >>> convert all

Re: [kvm-devel] [PATCH 1/5]Add some trace markers and exposeinterfaces in kernel for tracing

2008-04-15 Thread Liu, Eric E
Hollis Blanchard wrote: > On Tuesday 15 April 2008 22:13:28 Liu, Eric E wrote: >> Hollis Blanchard wrote: >>> On Wednesday 09 April 2008 05:01:36 Liu, Eric E wrote: >>>> +/* This structure represents a single trace buffer record. */ >>>> +str

Re: [kvm-devel] [PATCH 1/5]Add some trace markers and exposeinterfaces in kernel for tracing

2008-04-15 Thread Liu, Eric E
Hollis Blanchard wrote: > On Wednesday 09 April 2008 05:01:36 Liu, Eric E wrote: >> +/* This structure represents a single trace buffer record. */ >> +struct kvm_trace_rec { +   __u32 event:28; >> +   __u32 extra_u32:3; >> +   __u32 cycle_in:1; >> + 

[kvm-devel] [PATCH]kvmtrace:fix a redundant assignment.

2008-04-15 Thread Liu, Eric E
>From 193536b1cd96f646b66bea52bc46fdebcdcfc1cf Mon Sep 17 00:00:00 2001 From: Feng (Eric) Liu <[EMAIL PROTECTED]> Date: Wed, 16 Apr 2008 07:05:22 -0400 Subject: [PATCH] KVM: trace: fix a redundant assignment. Romove the redundant assignment, since the variable "extra" will not be used again. Sign

[kvm-devel] [PATCH 2/2]kvmtrace: add event mask support(userspace part)

2008-04-15 Thread Liu, Eric E
>From 9314f8b249fe6c32f006e6a1cec5d3868b66e6ff Mon Sep 17 00:00:00 2001 From: Feng (Eric) Liu <[EMAIL PROTECTED]> Date: Wed, 16 Apr 2008 05:45:40 -0400 Subject: [PATCH] kvm: user: Add event mask support in kvmtrace. Signed-off-by: Feng (Eric) Liu <[EMAIL PROTECTED]> --- user/kvmtrace.c | 56 +++

[kvm-devel] [PATCH 1/2]kvmtrace: add event mask support (kernel part)

2008-04-15 Thread Liu, Eric E
>From a1b062cfd4d1a91c447b680ac9a2250fe55119ec Mon Sep 17 00:00:00 2001 From: Feng (Eric) Liu <[EMAIL PROTECTED]> Date: Wed, 16 Apr 2008 05:29:37 -0400 Subject: [PATCH] KVM: trace: Add event mask support. Allow user space application to specify one or more filter masks to limit the events being ca

[kvm-devel] [PATCH 0/2]kvmtrace: add event mask to limit tracing data.

2008-04-15 Thread Liu, Eric E
Hi, The following two patches add event mask support in kvmtrace and the corresponding kernel code, so that we can specify filter masks to limit the events being captured. --Eric (Liu, Feng) - This SF.net email is sponsored

[kvm-devel] [PATCH 5/5] The tool for parsing binary trace data and the formats file.

2008-04-09 Thread Liu, Eric E
>From c22da0fa5b09ea7b9e6045a660eb1782afc5200a Mon Sep 17 00:00:00 2001 From: Feng(Eric) Liu <[EMAIL PROTECTED]> Date: Thu, 10 Apr 2008 09:34:20 -0400 Subject: [PATCH] kvm: "kvmtrace_format" parses the binary trace data from kvmtrace, and reformats it according to the rules in the file "formats" of

[kvm-devel] [PATCH 4/5]Add a tool in userspace to collect trace data.

2008-04-09 Thread Liu, Eric E
>From 3338918ad3d15550103047d18e24fcb1d8a4c0b3 Mon Sep 17 00:00:00 2001 From: Feng(Eric) Liu <[EMAIL PROTECTED]> Date: Thu, 10 Apr 2008 09:00:44 -0400 Subject: [PATCH] kvm: Add a tool kvmtrace for collecting binary data from relayfs. It is not an analysis tool. The code is based on blktrace. Signe

[kvm-devel] [PATCH 3/5]Modify userspace Kbuild and kvm_stat for kvm trace

2008-04-09 Thread Liu, Eric E
>From c950129e3876ac89b15af8f668976aa03142e802 Mon Sep 17 00:00:00 2001 From: Feng(Eric) Liu <[EMAIL PROTECTED]> Date: Thu, 10 Apr 2008 09:15:53 -0400 Subject: [PATCH] kvm: Modify "Kbuild" for kvm trace and ensure that "kvm_stat" can work when kvm trace is enabling by userspace app. Signed-off-by:

[kvm-devel] [PATCH 2/5]Add kvm trace kernel support.

2008-04-09 Thread Liu, Eric E
>From 1059e1b1c5bb8881bd82136ef8fb6a352f6bbf53 Mon Sep 17 00:00:00 2001 From: Feng(Eric) Liu <[EMAIL PROTECTED]> Date: Thu, 10 Apr 2008 08:47:53 -0400 Subject: [PATCH] KVM: Add kvm trace kernel support. When config KVM_TRACE, it allows user space app to read the trace of kvm related events through

[kvm-devel] [PATCH 0/5]Add kvm trace support v3

2008-04-09 Thread Liu, Eric E
Hi, Avi and all According to your comments from V2, I work out the new patchsets of kvm trace. Please help to review, Thanks. In this version, fix the issues you pointed out and add a new ioctl PAUSE to make user space app better to control trace stream. Hope it is useful for performance a

[kvm-devel] [PATCH 1/5]Add some trace markers and expose interfaces in kernel for tracing

2008-04-09 Thread Liu, Eric E
>From 49bb016dc7a231409859b553cce52faf07bf2b8b Mon Sep 17 00:00:00 2001 From: Feng (Eric) Liu <[EMAIL PROTECTED]> Date: Thu, 10 Apr 2008 15:31:10 -0400 Subject: [PATCH] KVM: Add some trace markers and expose some interfaces for user space app to control and use tracing data. Signed-off-by: Feng (E

Re: [kvm-devel] [PATCH RFC 2/5] Create relay channels and add trace data

2008-04-07 Thread Liu, Eric E
Avi Kivity wrote: > Liu, Eric E wrote: >>> From 41d65b55580d3f07f9f1c50e89e3d64c5d10fbaf Mon Sep 17 00:00:00 >>> 2001 >> From: Feng (Eric) Liu <[EMAIL PROTECTED]> >> Date: Tue, 1 Apr 2008 07:26:14 -0400 >> Subject: [PATCH] KVM: Add kvm trace support.

Re: [kvm-devel] [PATCH RFC 1/5]Add some trace enties and define interface for tracing

2008-04-07 Thread Liu, Eric E
Avi Kivity wrote: > Liu, Eric E wrote: >>> From d56731ffc6d5742a88a157dfe0e4344d35f7db58 Mon Sep 17 00:00:00 >>> 2001 >> From: Feng(Eric) Liu <[EMAIL PROTECTED]> >> Date: Mon, 31 Mar 2008 10:08:55 -0400 >> Subject: [PATCH] KVM: Add some trace entries

Re: [kvm-devel] Ubuntu Gutsy host / XP guest / -smp 2

2008-04-02 Thread Liu, Eric E
David Abrahams wrote: > on Wed Apr 02 2008, Avi Kivity wrote: > >> David Abrahams wrote: >>> With the title combination, the guest takes nearly 100% of my real >>> CPU time and still only sees one CPU. Is this a known problem, and >>> does it have a known solution? >>> >>> >> >> Can you send

[kvm-devel] [PATCH RFC 2/5] Create relay channels and add trace data

2008-03-30 Thread Liu, Eric E
>From 41d65b55580d3f07f9f1c50e89e3d64c5d10fbaf Mon Sep 17 00:00:00 2001 From: Feng (Eric) Liu <[EMAIL PROTECTED]> Date: Tue, 1 Apr 2008 07:26:14 -0400 Subject: [PATCH] KVM: Add kvm trace support. when config KVM_TRACE, it allows userspace app to read the trace of kvm_related events through the rela

[kvm-devel] [PATCH RFC 3/5] Add a userspace tool for collect tracing data based on blktrace

2008-03-30 Thread Liu, Eric E
>From e0d0f33ce1a536d4f2c3a1763f23a89f0a726cd6 Mon Sep 17 00:00:00 2001 From: Feng (Eric) Liu <[EMAIL PROTECTED]> Date: Mon, 31 Mar 2008 10:57:16 -0400 Subject: [PATCH] kvm: Add a tool kvmtrace for collecting binary data from relayfs. The code is based on blktrace. Signed-off-by: Feng (Eric) Liu <

[kvm-devel] [PATCH RFC 1/5]Add some trace enties and define interface for tracing

2008-03-30 Thread Liu, Eric E
>From d56731ffc6d5742a88a157dfe0e4344d35f7db58 Mon Sep 17 00:00:00 2001 From: Feng(Eric) Liu <[EMAIL PROTECTED]> Date: Mon, 31 Mar 2008 10:08:55 -0400 Subject: [PATCH] KVM: Add some trace entries in current code and define some interfaces for userspace app to contrl and use tracing data. Signed-of

[kvm-devel] [PATCH RFC 5/5] Modify userspace Kbuild and kvm_stat for kvm_trace

2008-03-30 Thread Liu, Eric E
>From a18df6cb088a07ce34fb1981ca3077f5e6925ae2 Mon Sep 17 00:00:00 2001 From: Feng (Eric) Liu <[EMAIL PROTECTED]> Date: Mon, 31 Mar 2008 10:35:59 -0400 Subject: [PATCH] kvm: Modify "Kbuild" for kvm trace and ensure "kvm_stat" work when kvm trace is enabled by userspace app. Signed-off-by: Feng (E

[kvm-devel] [PATCH RFC 4/5] The tool for parsing trace data

2008-03-30 Thread Liu, Eric E
>From 182054d6c1e6a6213f3fa56aab1ca56c39f9ba80 Mon Sep 17 00:00:00 2001 From: Feng(Eric) Liu <[EMAIL PROTECTED]> Date: Mon, 31 Mar 2008 10:52:57 -0400 Subject: [PATCH] kvm: "kvmtrace_format" parses the binary trace data outputted by kvmtrace, and reformats it according to the rules in the file "for

[kvm-devel] [PATCH RFC 0/5]Add kvm trace support V2

2008-03-30 Thread Liu, Eric E
Hi, The following patches add trace support in kvm using relay_fs and trace_mark according to last comments. if we want to use it, we should config kvm_trace firstly which depends on MARKERS and SYSFS, at this case it will add a tiny time penalty to check whether we provide a hook to call a

Re: [kvm-devel] [PATCH RFC 0/4]Porting Xentrace to kvm

2008-03-17 Thread Liu, Eric E
[EMAIL PROTECTED] wrote: > Avi Kivity wrote: >> Jan Kiszka wrote: >>> Avi Kivity wrote: >>> >>>> Liu, Eric E wrote: >>>> >>>>> >>>>> Since we already have had debugfs_entries and some other kernel >>&

[kvm-devel] [PATCH RFC 2/4]Porting Xentrace to kvm

2008-03-13 Thread Liu, Eric E
>From 1fd1215cc56a52109d6df1a2302b391511d3d91b Mon Sep 17 00:00:00 2001 From: Feng(Eric) Liu <[EMAIL PROTECTED]> Date: Sat, 15 Mar 2008 08:09:24 -0400 Subject: [PATCH] KVM: Porting xentrace to kvm. The trace data is outputted into the trace buffer managed as ring buffer for per cpu, and the usersp

[kvm-devel] [PATCH RFC 1/4]Porting Xentrace to kvm

2008-03-13 Thread Liu, Eric E
>From 0d7f1ee470fe907e00ac6246bfa11e5322bc64fb Mon Sep 17 00:00:00 2001 From: Feng (Eric) Liu <[EMAIL PROTECTED]> Date: Sat, 15 Mar 2008 06:07:33 -0400 Subject: [PATCH] KVM: Add some trace entries in current code, when the KVM_TRACE compilation option is enabled, it outputs the data info thrace bu

[kvm-devel] [PATCH RFC 3/4]Porting Xentrace to kvm

2008-03-13 Thread Liu, Eric E
>From df6e45936dd43be14441f5e9ec19a130325d4341 Mon Sep 17 00:00:00 2001 From: Feng (Eric) Liu <[EMAIL PROTECTED]> Date: Sat, 15 Mar 2008 04:53:53 -0400 Subject: [PATCH] kvm: Add a tool kvmtrace for collecting binary data from trace buffer. It can get the buffer virtual address and the size in page

[kvm-devel] [PATCH RFC 4/4]Porting Xentrace to kvm

2008-03-13 Thread Liu, Eric E
>From 2e16085ad8b4ecfe7e053a627a86dc24bbe04315 Mon Sep 17 00:00:00 2001 From: Feng(Eric) Liu <[EMAIL PROTECTED]> Date: Fri, 14 Mar 2008 18:19:46 -0400 Subject: [PATCH] kvm: "kvmtrace_format" prarses the trace data outputted by kvmtrace in binary format, and reformats it according to the rules in th

[kvm-devel] [PATCH RFC 0/4]Porting Xentrace to kvm

2008-03-13 Thread Liu, Eric E
Hi, The following patches port xentrace to kvm which is useful for performance tuning and debugging. It is designed to allow debugging traces of kvm to be generated on Up/Smp machines. Each trace entry is outputted in a trace ring buffer for per cpu which is mapped to userspace, and the userspace

Re: [kvm-devel] [PATCH]fix the building error of preempt.c

2008-02-14 Thread Liu, Eric E
Liu, Eric E wrote: >> From e4fae3b29b694a489638b00214fc41530092c3d1 Mon Sep 17 00:00:00 >> 2001 > From: Feng (Eric) Liu <[EMAIL PROTECTED]> > Date: Sat, 16 Feb 2008 03:41:17 -0500 > Subject: [PATCH] kvm: Fix building error of preempt.c > > current kernel code h

[kvm-devel] [PATCH]fix the building error of preempt.c

2008-02-14 Thread Liu, Eric E
>From e4fae3b29b694a489638b00214fc41530092c3d1 Mon Sep 17 00:00:00 2001 From: Feng (Eric) Liu <[EMAIL PROTECTED]> Date: Sat, 16 Feb 2008 03:41:17 -0500 Subject: [PATCH] kvm: Fix building error of preempt.c current kernel code has merged the file processor_32.h and processor_64.h to processor.h, so

[kvm-devel] fix a potential issue

2008-01-21 Thread Liu, Eric E
>From ab229d437b59317a322ca0efd2b0d239b74e Mon Sep 17 00:00:00 2001 From: Feng(Eric) Liu <[EMAIL PROTECTED]> Date: Tue, 22 Jan 2008 17:01:29 -0500 Subject: [PATCH] kvm: mmu: fix a potential issue Since the type of gpa is u64, so extend pte_size to u64. Otherwise, a potential issue may happen.

[kvm-devel] [PATCH] remove SIGUSR2 from io_sigset to avoid breaking AIO

2008-01-14 Thread Liu, Eric E
If we use Qcow img to boot SMP guest, windows guest may hang , since the vcpu thread which does AIO will wait for the signal SIGUSR2 which may be eated by vcpu 0 's thread. So this patch removes SIGUSR2 from io_sigset and masks it in vcpu 0's thread, it can let the guest boot successfully. >From

[kvm-devel] The reason of using Qcow images to boot smp Windows fail ?

2008-01-08 Thread Liu, Eric E
Hi Avi For current commit Windows SMP guests will fail to boot if using Qcow images based on the original backend file on a remote network file system. I found out that one Ap thread is hang at the place of bdrv_read_em() -> qemu_aio_wait() -> sigwait(), which is waiting for the signal SIG