Re: [kvm-devel] [PATCH/RFC 3/4]Introduce account modifiers mechanism

2007-08-20 Thread Laurent Vivier
Avi Kivity wrote: Laurent Vivier wrote: Avi Kivity wrote: [...] The normal user/system accounting has the same issue, no? Whereever we happen to land (kernel or user) gets the whole tick. So I think it is okay to have the same limitation for guest time. So this is how it looks

Re: [kvm-devel] [PATCH/RFC 3/4]Introduce account modifiers mechanism

2007-08-20 Thread Avi Kivity
Laurent Vivier wrote: Where did CONFIG_GUEST_ACCOUNTING go? Lost in the sea ... Actually, I thought this modification is not enough expensive (in time and space) to justify a CONFIG_*. But if you think so I can add this in init/Kconfig. The difference between convince

Re: [kvm-devel] [BUG][PATCH?] kvm: unhandled wrmsr: 0xc0000083

2007-08-20 Thread Avi Kivity
Luca Tettamanti wrote: In order to reactivate the CPU after the reset. When the guest reboots I get an unhandled vm exit: unhandled vm exit: 0x8021 rax rbx rcx rdx 0600 rsi rdi rsp

[kvm-devel] KVM Test result, kernel a843332b.. , userspace 356c9ff5..

2007-08-20 Thread Zhao, Yunfeng
Hi, all, This is today's KVM test result against kvm.git a843332b0445c9d60e4c9bda965b10cbe632a088 and kvm-userspace.git 356c9ff52b05e2d0f2b2eee9f861ad7c3ab3babd. Host crash issue still exists on latest KVM, even boot one linux guest will cause host to crash, so most our tests failed. I have

[kvm-devel] new medical marketing data

2007-08-20 Thread Houser P Samuel
Aug 20 - Aug 24: With every purchase of our MD List comes completely free! a Directory for Dentists, Nursing Homes and Hospitals Licensed MDs in the USA 788,933 in total – 17,400 emails MDs in many different specialties Over a dozen sortable fields Lowered Price - $358 *** BONUS: Get

Re: [kvm-devel] KVM Test result, kernel a843332b.. , userspace 356c9ff5..

2007-08-20 Thread Avi Kivity
Zhao, Yunfeng wrote: Hi, all, This is today's KVM test result against kvm.git a843332b0445c9d60e4c9bda965b10cbe632a088 and kvm-userspace.git 356c9ff52b05e2d0f2b2eee9f861ad7c3ab3babd. Host crash issue still exists on latest KVM, even boot one linux guest will cause host to crash, so most our

Re: [kvm-devel] KVM Test result, kernel a843332b.. , userspace 356c9ff5..

2007-08-20 Thread Avi Kivity
Avi Kivity wrote: Zhao, Yunfeng wrote: Hi, all, This is today's KVM test result against kvm.git a843332b0445c9d60e4c9bda965b10cbe632a088 and kvm-userspace.git 356c9ff52b05e2d0f2b2eee9f861ad7c3ab3babd. Host crash issue still exists on latest KVM, even boot one linux guest will cause host to

Re: [kvm-devel] KVM Test result, kernel a843332b.. , userspace 356c9ff5..

2007-08-20 Thread Zhao, Yunfeng
Sorry, I forgot about it since all works well here. Will try to reproduce. Now fixed and pushed (200b412d5f84907f2de0e27b95e70493d0741fa3 in kvm-userspace.git). I didn't see the issue because I usually use --with-patched-kernel and run kvm.git directly. [Yunfeng] Thank you very much for the

Re: [kvm-devel] [Qemu-devel] Re: [PATCH 0/4] Rework alarm timer infrastrucure - take2

2007-08-20 Thread Avi Kivity
Avi Kivity wrote: Paul Brook wrote: Yes, good thinking, but this should only be done if it actually impacts something. Reducing overhead from 0.1% to 0.05% is not worthwhile if it introduces extra complexity. If the overhead is that small, why are we touching this code in the

[kvm-devel] [PATCH 0/4] Virtual Machine Time Accounting

2007-08-20 Thread Laurent Vivier
The aim of these four patches is to introduce Virtual Machine time accounting. _Ingo_, as these patches modify files of the scheduler, could you have a look to them, please ? [PATCH 1/4] as recent CPUs introduce a third running state, after user and system, we need a new field, guest, in cpustat

[kvm-devel] [PATCH 3/4] modify account_system_time() to update guest time in cpustat and task_struct

2007-08-20 Thread Laurent Vivier
[PATCH 3/4] modify account_system_time() to add cputime to cpustat-guest if we are running a VCPU. We add this cputime to cpustat-user instead of cpustat-system because this part of KVM code is in fact user code although it is executed in the kernel. We duplicate VCPU time between guest and user

[kvm-devel] [PATCH 1/4] Introduce a new field guest in cpustat

2007-08-20 Thread Laurent Vivier
[PATCH 1/4] as recent CPUs introduce a third running state, after user and system, we need a new field, guest, in cpustat to store the time used by the CPU to run virtual CPU. Modify /proc/stat to display this new field. Signed-off-by: Laurent Vivier [EMAIL PROTECTED] -- - [EMAIL

[kvm-devel] [PATCH 4/4] Modify KVM to update guest time accounting.

2007-08-20 Thread Laurent Vivier
[PATCH 4/4] Modify KVM to update guest time accounting. Signed-off-by: Laurent Vivier [EMAIL PROTECTED] -- - [EMAIL PROTECTED] -- Software is hard - Donald Knuth Index: kvm/drivers/kvm/kvm.h ===

[kvm-devel] [PATCH 2/4] Introduce a new fields gtime and cgtime in task_struct and signal_struct

2007-08-20 Thread Laurent Vivier
[PATCH 2/4] like for cpustat, introduce the gtime (guest time of the task) and cgtime (guest time of the task children) fields for the tasks. Modify signal_struct and task_struct. Modify /proc/pid/stat to display these new fields. Signed-off-by: Laurent Vivier [EMAIL PROTECTED] -- -

Re: [kvm-devel] [PATCH 0/4] Virtual Machine Time Accounting

2007-08-20 Thread Avi Kivity
Laurent Vivier wrote: The aim of these four patches is to introduce Virtual Machine time accounting. 1-3 look good to me. 4 needs minor edits... -- error compiling committee.c: too many arguments to function - This

Re: [kvm-devel] [PATCH 0/4] Virtual Machine Time Accounting

2007-08-20 Thread John Stoffel
Laurent The aim of these four patches is to introduce Virtual Machine Laurent time accounting. So what does this buy us? What increased functionality? Laurent [PATCH 1/4] as recent CPUs introduce a third running state, Laurent after user and system, we need a new field, guest, in Laurent

Re: [kvm-devel] [PATCH 4/4] Modify KVM to update guest time accounting.

2007-08-20 Thread Laurent Vivier
Avi Kivity wrote: Laurent Vivier wrote: [PATCH 4/4] Modify KVM to update guest time accounting. Index: kvm/drivers/kvm/kvm.h === --- kvm.orig/drivers/kvm/kvm.h2007-08-20 13:23:53.0 +0200 +++

Re: [kvm-devel] [PATCH 00/10] PV-IO v3

2007-08-20 Thread Gregory Haskins
On Sun, 2007-08-19 at 12:24 +0300, Avi Kivity wrote: Rusty Russell wrote: 2) We either need huge descriptors or some chaining mechanism to handle scatter-gather. Or, my preference, have a small sglist in the descriptor; Define small ;) There a certainly patterns

Re: [kvm-devel] [PATCH 2/4] Introduce a new fields gtime and cgtime in task_struct and signal_struct

2007-08-20 Thread Christian Borntraeger
Am Montag, 20. August 2007 schrieb Laurent Vivier: Index: kvm/fs/proc/array.c === --- kvm.orig/fs/proc/array.c2007-08-20 11:11:30.0 +0200 +++ kvm/fs/proc/array.c 2007-08-20 13:04:03.0 +0200 Just a heads up,

Re: [kvm-devel] [PATCH 00/10] PV-IO v3

2007-08-20 Thread Dor Laor
2) We either need huge descriptors or some chaining mechanism to handle scatter-gather. Or, my preference, have a small sglist in the descriptor; Define small ;) There a certainly patterns that cannot/will-not take advantage of SG (for instance, your typical network rx

Re: [kvm-devel] [PATCH 4/4] Modify KVM to update guest time accounting.

2007-08-20 Thread Avi Kivity
Laurent Vivier wrote: Avi Kivity wrote: Laurent Vivier wrote: [PATCH 4/4] Modify KVM to update guest time accounting. Index: kvm/drivers/kvm/kvm.h === --- kvm.orig/drivers/kvm/kvm.h2007-08-20

Re: [kvm-devel] [PATCH 00/10] PV-IO v3

2007-08-20 Thread Avi Kivity
Dor Laor wrote: 2) We either need huge descriptors or some chaining mechanism to handle scatter-gather. Or, my preference, have a small sglist in the descriptor; Define small ;) There a certainly patterns that cannot/will-not take advantage

Re: [kvm-devel] [PATCH 00/10] PV-IO v3

2007-08-20 Thread Avi Kivity
Gregory Haskins wrote: Or, my preference, have a small sglist in the descriptor; Define small ;) 4. There a certainly patterns that cannot/will-not take advantage of SG (for instance, your typical network rx path), and therefore the sg entries are wasted in some

Re: [kvm-devel] [PATCH 2/4] Introduce a new fields gtime and cgtime in task_struct and signal_struct

2007-08-20 Thread Laurent Vivier
Christian Borntraeger wrote: Am Montag, 20. August 2007 schrieb Laurent Vivier: Index: kvm/fs/proc/array.c === --- kvm.orig/fs/proc/array.c2007-08-20 11:11:30.0 +0200 +++ kvm/fs/proc/array.c 2007-08-20

Re: [kvm-devel] [PATCH 00/10] PV-IO v3

2007-08-20 Thread Gregory Haskins
On Mon, 2007-08-20 at 07:03 -0700, Dor Laor wrote: 2) We either need huge descriptors or some chaining mechanism to handle scatter-gather. Or, my preference, have a small sglist in the descriptor; Define small ;) There a certainly patterns that cannot/will-not

Re: [kvm-devel] [PATCH 4/4] Modify KVM to update guest time accounting.

2007-08-20 Thread Laurent Vivier
Avi Kivity wrote: Laurent Vivier wrote: Avi Kivity wrote: Laurent Vivier wrote: [PATCH 4/4] Modify KVM to update guest time accounting. Index: kvm/drivers/kvm/kvm.h === --- kvm.orig/drivers/kvm/kvm.h

Re: [kvm-devel] [PATCH 0/4] Virtual Machine Time Accounting

2007-08-20 Thread Laurent Vivier
Hi John, John Stoffel wrote: Laurent The aim of these four patches is to introduce Virtual Machine Laurent time accounting. So what does this buy us? What increased functionality? functionnalities: - allow to measure time spent by a CPU in a virtual CPU. - allow to display in

Re: [kvm-devel] [2/2] 2.6.23-rc3: known regressions with patches v2

2007-08-20 Thread Michal Piotrowski
Hi all, Here is a list of some known regressions in 2.6.23-rc3 with patches available. Feel free to add new regressions/remove fixed etc. http://kernelnewbies.org/known_regressions List of Aces NameRegressions fixed since 21-Jun-2007 Adrian Bunk9

Re: [kvm-devel] [PATCH 0/4] Virtual Machine Time Accounting

2007-08-20 Thread Jeremy Fitzhardinge
Laurent Vivier wrote: functionnalities: - allow to measure time spent by a CPU in a virtual CPU. - allow to display in /proc/state this value by CPU - allow to display in /proc/pid/state this value by process - allow KVM to use these 3 previous functionnalities So, currently time spent

[kvm-devel] (no subject)

2007-08-20 Thread Gmagm sssssssssssa
http://finance.yahoo.com/q?s=EXMT.PK - This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser.

Re: [kvm-devel] Réf. : Re: [PATCH 0/4] Vir tual Machine Time Accounting

2007-08-20 Thread Glauber de Oliveira Costa
by doing this at kernel level, we can: - measure exactly the guest time, - move this part of system time to user time (as you think it should be user time), - have consistency between system, user and guest time, - report values in /proc/state and /proc/pid/state, at system wide level I'm

Re: [kvm-devel] [PATCH 0/4] Rework alarm timer infrastrucure - take2

2007-08-20 Thread Luca Tettamanti
Il Sun, Aug 19, 2007 at 10:31:26PM +0300, Avi Kivity ha scritto: Luca wrote: On 8/19/07, Luca Tettamanti [EMAIL PROTECTED] wrote: +static uint64_t qemu_next_deadline(void) { +uint64_t nearest_delta_us = ULLONG_MAX; +uint64_t vmdelta_us; Hum, I introduced a bug

[kvm-devel] [PATCH] Add support for passing extra LDFLAGS to qemu's configure

2007-08-20 Thread Jeremy Katz
There are cases[1] where you want to be able to pass more ldflags to qemu's configure. This lets you set LDFLAGS to accomplish that Signed-off-by: Jeremy Katz [EMAIL PROTECTED] Jeremy [1] Such as with the new build-id support in binutils so that you can pass --build-id to the linker while

Re: [kvm-devel] [Qemu-devel] Re: [PATCH 0/4] Rework alarm timer infrastrucure - take2

2007-08-20 Thread malc
On Mon, 20 Aug 2007, Luca Tettamanti wrote: Il Sun, Aug 19, 2007 at 10:31:26PM +0300, Avi Kivity ha scritto: Luca wrote: On 8/19/07, Luca Tettamanti [EMAIL PROTECTED] wrote: +static uint64_t qemu_next_deadline(void) { +uint64_t nearest_delta_us = ULLONG_MAX; +uint64_t vmdelta_us;

Re: [kvm-devel] [Qemu-devel] Re: [PATCH 0/4] Rework alarm timer infrastrucure - take2

2007-08-20 Thread Luca
On 8/20/07, malc [EMAIL PROTECTED] wrote: On Mon, 20 Aug 2007, Luca Tettamanti wrote: Il Sun, Aug 19, 2007 at 10:31:26PM +0300, Avi Kivity ha scritto: Luca wrote: On 8/19/07, Luca Tettamanti [EMAIL PROTECTED] wrote: +static uint64_t qemu_next_deadline(void) { +uint64_t

Re: [kvm-devel] [PATCH 00/10] PV-IO v3

2007-08-20 Thread Rusty Russell
On Mon, 2007-08-20 at 17:12 +0300, Avi Kivity wrote: Dor Laor wrote: Using Rusty's code there is no waste. Each descriptor has a flag (head|next). Next flag stands for pointer to the next descriptor with u32 next index. So the waste is 4 bytes. Sg descriptors are chained on the same

Re: [kvm-devel] [PATCH 00/10] PV-IO v3

2007-08-20 Thread Rusty Russell
On Fri, 2007-08-17 at 09:50 -0400, Gregory Haskins wrote: On Fri, 2007-08-17 at 17:43 +1000, Rusty Russell wrote: Well, for cache reasons you should really try to avoid having both sides write to the same data. Hence two separate cache-aligned regions is better than one region and a flip

Re: [kvm-devel] [PATCH 0/4] Virtual Machine Time Accounting

2007-08-20 Thread Rusty Russell
On Mon, 2007-08-20 at 09:13 -0700, Jeremy Fitzhardinge wrote: Laurent Vivier wrote: functionnalities: - allow to measure time spent by a CPU in a virtual CPU. - allow to display in /proc/state this value by CPU - allow to display in /proc/pid/state this value by process - allow KVM to