Re: [kvm-devel] [PATCH 0/2][KVM] guest time accounting

2007-08-13 Thread Avi Kivity
Laurent Vivier wrote: The aim of these two patches is to measure the CPU time used by a virtual machine. All comments are welcome... I'm not sure it's the good way to do that. [PATCH 1/2] introduce a new field, guest, in cpustat to store the time used by the CPU to run virtual CPU. Modify

Re: [kvm-devel] [PATCH 0/2][KVM] guest time accounting

2007-08-13 Thread Laurent Vivier
Avi Kivity wrote: Laurent Vivier wrote: The aim of these two patches is to measure the CPU time used by a virtual machine. All comments are welcome... I'm not sure it's the good way to do that. [PATCH 1/2] introduce a new field, guest, in cpustat to store the time used by the CPU to run

Re: [kvm-devel] [PATCH 0/2][KVM] guest time accounting

2007-08-13 Thread Avi Kivity
Laurent Vivier wrote: - perhaps the new fields should be guarded by a #ifdef CONFIG_HYPERVISOR (selected by CONFIG_KVM)? that way the (minor) additional overhead is only incurred if it can possibly be used. I imagine that our canine cousin will want to use this as well. There is also a

Re: [kvm-devel] [PATCH 0/2][KVM] guest time accounting

2007-08-13 Thread Laurent Vivier
Avi Kivity wrote: Laurent Vivier wrote: - perhaps the new fields should be guarded by a #ifdef CONFIG_HYPERVISOR (selected by CONFIG_KVM)? that way the (minor) additional overhead is only incurred if it can possibly be used. I imagine that our canine cousin will want to use this as well.

Re: [kvm-devel] [PATCH 0/2][KVM] guest time accounting

2007-08-13 Thread Avi Kivity
Laurent Vivier wrote: Are these options for using the kernel as a guest or host? I'd guess the former. I didn't find CONFIG_HYPERVISOR. I meant, add a new option CONFIG_HYPERVISOR. The good one seems to be CONFIG_VIRTUALIZATION that is used to activate CONFIG_KVM. It's

Re: [kvm-devel] [PATCH 0/2][KVM] guest time accounting

2007-08-13 Thread Christian Borntraeger
Am Montag, 13. August 2007 schrieb Laurent Vivier: [copying Ingo and Rusty] @Avi, seems that sourceforge is mangling the cc list? The patches look good. A couple of comments: - perhaps the new fields should be guarded by a #ifdef CONFIG_HYPERVISOR (selected by CONFIG_KVM)? that

Re: [kvm-devel] [PATCH 0/2][KVM] guest time accounting

2007-08-13 Thread Avi Kivity
Christian Borntraeger wrote: Am Montag, 13. August 2007 schrieb Laurent Vivier: [copying Ingo and Rusty] @Avi, seems that sourceforge is mangling the cc list? It's not configured to do so. Can you be more specific? The patches look good. A couple of comments: - perhaps

Re: [kvm-devel] [PATCH 0/2][KVM] guest time accounting

2007-08-13 Thread Avi Kivity
Christian Borntraeger wrote: Am Montag, 13. August 2007 schrieb Avi Kivity: Christian Borntraeger wrote: Am Montag, 13. August 2007 schrieb Laurent Vivier: [copying Ingo and Rusty] @Avi, seems that sourceforge is mangling the cc list? It's not

Re: [PATCH 0/7] Modify lguest32 to make room for lguest64

2007-08-13 Thread Jes Sorensen
Rusty Russell wrote: Since I can't build a module over two directories, that seems to destroy the idea of an i386/ subdir. Instead I've done a patch which renames the *clearly* i386-specific things to i386_name, which at least works. I've pushed it into the repository

Re: [PATCH] [545/2many] MAINTAINERS - XEN HYPERVISOR INTERFACE

2007-08-13 Thread Chris Wright
* [EMAIL PROTECTED] ([EMAIL PROTECTED]) wrote: Add file pattern to MAINTAINER entry Signed-off-by: Joe Perches [EMAIL PROTECTED] diff --git a/MAINTAINERS b/MAINTAINERS index e4e1cc3..8395aba 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -5153,6 +5153,11 @@ M: [EMAIL PROTECTED] L:

Re: [PATCH] [545/2many] MAINTAINERS - XEN HYPERVISOR INTERFACE

2007-08-13 Thread Randy Dunlap
On Mon, 13 Aug 2007 11:55:36 -0700 Chris Wright wrote: * [EMAIL PROTECTED] ([EMAIL PROTECTED]) wrote: Add file pattern to MAINTAINER entry Signed-off-by: Joe Perches [EMAIL PROTECTED] diff --git a/MAINTAINERS b/MAINTAINERS index e4e1cc3..8395aba 100644 --- a/MAINTAINERS +++

Re: [PATCH] [545/2many] MAINTAINERS - XEN HYPERVISOR INTERFACE

2007-08-13 Thread Joe Perches
On Mon, 2007-08-13 at 11:55 -0700, Chris Wright wrote: I think this data will easily become stale. What is the point again? Write a patch, notify the appropriate maintainers. ___ Virtualization mailing list Virtualization@lists.linux-foundation.org

Re: [PATCH] [545/2many] MAINTAINERS - XEN HYPERVISOR INTERFACE

2007-08-13 Thread Chris Wright
* Randy Dunlap ([EMAIL PROTECTED]) wrote: On Mon, 13 Aug 2007 11:55:36 -0700 Chris Wright wrote: * [EMAIL PROTECTED] ([EMAIL PROTECTED]) wrote: +F: arch/i386/xen/ +F: drivers/*/xen-*front.c +F: drivers/xen/ +F: include/asm-i386/xen/ +F: include/xen/

Re: [PATCH] [545/2many] MAINTAINERS - XEN HYPERVISOR INTERFACE

2007-08-13 Thread Valdis . Kletnieks
On Mon, 13 Aug 2007 12:10:05 PDT, Randy Dunlap said: Agreed. But not everyone wants to or should have to use git, so what are the alternatives? http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=tree and start chasing down 'History' pointers? (Of course, mentioning that in

Re: [PATCH] [545/2many] MAINTAINERS - XEN HYPERVISOR INTERFACE

2007-08-13 Thread Adrian Bunk
On Mon, Aug 13, 2007 at 03:22:59PM -0400, [EMAIL PROTECTED] wrote: On Mon, 13 Aug 2007 12:10:05 PDT, Randy Dunlap said: Agreed. But not everyone wants to or should have to use git, so what are the alternatives? http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=tree

Re: [PATCH] [545/2many] MAINTAINERS - XEN HYPERVISOR INTERFACE

2007-08-13 Thread Adrian Bunk
On Mon, Aug 13, 2007 at 12:19:38PM -0700, Chris Wright wrote: * Randy Dunlap ([EMAIL PROTECTED]) wrote: On Mon, 13 Aug 2007 11:55:36 -0700 Chris Wright wrote: * [EMAIL PROTECTED] ([EMAIL PROTECTED]) wrote: +F: arch/i386/xen/ +F: drivers/*/xen-*front.c +F: drivers/xen/

Re: [kvm-devel] [PATCH 0/2][KVM] guest time accounting

2007-08-13 Thread Heiko Carstens
The only thing remotely relevant in the list config is that 'Filter out duplicate messages to list members (if possible)' is set as a default for new members. Maybe this means that if a cc is also part of the list, that cc is stripped (which seems a wierd implementation; I'd have

Re: [PATCH] [545/2many] MAINTAINERS - XEN HYPERVISOR INTERFACE

2007-08-13 Thread Alan Cox
Agreed. But not everyone wants to or should have to use git, so what are the alternatives? Export the recent git history each release into an XML file at known locations on the kernel web site. Wait for tools to appear ___ Virtualization mailing list

Re: [PATCH] [545/2many] MAINTAINERS - XEN HYPERVISOR INTERFACE

2007-08-13 Thread Adrian Bunk
On Mon, Aug 13, 2007 at 04:32:19PM -0400, [EMAIL PROTECTED] wrote: On Mon, 13 Aug 2007 22:13:24 +0200, Adrian Bunk said: On Mon, Aug 13, 2007 at 03:22:59PM -0400, [EMAIL PROTECTED] wrote: On Mon, 13 Aug 2007 12:10:05 PDT, Randy Dunlap said: Agreed. But not everyone wants to or

Re: [PATCH] [545/2many] MAINTAINERS - XEN HYPERVISOR INTERFACE

2007-08-13 Thread Adrian Bunk
On Mon, Aug 13, 2007 at 09:54:50PM +0100, Alan Cox wrote: Agreed. But not everyone wants to or should have to use git, so what are the alternatives? Export the recent git history each release into an XML file at known locations on the kernel web site. Wait for tools to appear How can the

[PATCH] i386: modified CONFIG_PHYSICAL_ALIGN

2007-08-13 Thread Yoshimi Ichiyanagi
CONFIG_PHYSICAL_ALIGN gives the physical address's alignment where the kernel is loaded. CONFIG_PHYSICAL_ALIGN's default value is different from i386 kernel with the x86_84 kernel. The i386's default CONFIG_PHYSICAL_ALIGN value is 1MB, but the x86_64's is 2MB. The problem is, if you compile