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. >>> >>> >>

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 "

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 0

[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 uploa

[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 mo

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

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

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 t

[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 cp

[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 use

[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]> -- - [EM

[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//stat to display these new fields. Signed-off-by: Laurent Vivier <[EMAIL PROTECTED]> --

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

2007-08-20 Thread Avi Kivity
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 > +++ kvm/drivers/kvm/kvm.h 20

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 L

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 >> +++ kvm

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 pa

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

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

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 pattern

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 w

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 13:04:0

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 certainl

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 === >>

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//state this value by process > - allow KVM to use these 3 previous functionnalities > So, currently time spe

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

2007-08-20 Thread laurent . vivier
> 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//state this value by process > > - allow KVM to use these 3 previous functionnalities > > > > So, curre

[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. Downlo

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//state, at system wide level > >

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,

[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 sti

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; +

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) { > >

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 th

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

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//state this value by process > > - all

[kvm-devel] [PATCH][Rebased LAPIC5] Enable TPR shadow on CR8 access

2007-08-20 Thread Yang, Sheng
This patch enabled TPR shadow of VMX on CR8 access. 64bit Windows using CR8 access TPR frequently. The TPR shadow can improve the performance of access TPR by not causing vmexit. Signed-off-by: Sheng Yang <[EMAIL PROTECTED]> Notice: This patch based on the rebased lapic5, for the current lapic5 i

[kvm-devel] [PATCH][Rebased LAPIC5] Reading PPR directly from function rather than apic page

2007-08-20 Thread Yang, Sheng
After introducing TPR shadow, many TPR change won't cause vmexit, so the present method of updating PPR can't catch it. Though we can also update PPR everytime when we want to read PPR, it's somehow ugly. Because there are only two places need reading PPR, and PPR is read-only, we do it in more cl