Re: [PATCH RFC V2 4/5] kvm guest : Added configuration support to enable debug information for KVM Guests

2011-10-25 Thread Raghavendra K T

On 10/24/2011 03:45 PM, Avi Kivity wrote:

On 10/23/2011 09:07 PM, Raghavendra K T wrote:

Added configuration support to enable debug information
for KVM Guests in debugfs

Signed-off-by: Srivatsa Vaddagiriva...@linux.vnet.ibm.com
Signed-off-by: Suzuki Poulosesuz...@in.ibm.com
Signed-off-by: Raghavendra K Traghavendra...@linux.vnet.ibm.com
---
diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index 1f03f82..ed34269 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -562,6 +562,15 @@ config KVM_GUEST
  This option enables various optimizations for running under the KVM
  hypervisor.

+config KVM_DEBUG_FS
+   bool Enable debug information for KVM Guests in debugfs
+   depends on KVM_GUEST
+   default n
+   ---help---
+ This option enables collection of various statistics for KVM guest.
+ Statistics are displayed in debugfs filesystem. Enabling this option
+ may incur significant overhead.
+
  source arch/x86/lguest/Kconfig




This might be better implemented through tracepoints, which an be
enabled dynamically.


Interesting option. I 'll explore this one. Thanks for pointing.
--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH RFC V2 4/5] kvm guest : Added configuration support to enable debug information for KVM Guests

2011-10-25 Thread Jeremy Fitzhardinge
On 10/24/2011 03:15 AM, Avi Kivity wrote:
 On 10/23/2011 09:07 PM, Raghavendra K T wrote:
 Added configuration support to enable debug information
 for KVM Guests in debugfs
 
 Signed-off-by: Srivatsa Vaddagiri va...@linux.vnet.ibm.com
 Signed-off-by: Suzuki Poulose suz...@in.ibm.com
 Signed-off-by: Raghavendra K T raghavendra...@linux.vnet.ibm.com
 ---
 diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
 index 1f03f82..ed34269 100644
 --- a/arch/x86/Kconfig
 +++ b/arch/x86/Kconfig
 @@ -562,6 +562,15 @@ config KVM_GUEST
This option enables various optimizations for running under the KVM
hypervisor.
  
 +config KVM_DEBUG_FS
 +bool Enable debug information for KVM Guests in debugfs
 +depends on KVM_GUEST
 +default n
 +---help---
 +  This option enables collection of various statistics for KVM guest.
 +  Statistics are displayed in debugfs filesystem. Enabling this option
 +  may incur significant overhead.
 +
  source arch/x86/lguest/Kconfig
  

 This might be better implemented through tracepoints, which an be
 enabled dynamically.

Tracepoints use spinlocks, so that could get awkward.

J

--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH RFC V2 4/5] kvm guest : Added configuration support to enable debug information for KVM Guests

2011-10-24 Thread Sasha Levin
On Mon, 2011-10-24 at 00:37 +0530, Raghavendra K T wrote:
 Added configuration support to enable debug information
 for KVM Guests in debugfs
 
 Signed-off-by: Srivatsa Vaddagiri va...@linux.vnet.ibm.com
 Signed-off-by: Suzuki Poulose suz...@in.ibm.com
 Signed-off-by: Raghavendra K T raghavendra...@linux.vnet.ibm.com
 ---
 diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
 index 1f03f82..ed34269 100644
 --- a/arch/x86/Kconfig
 +++ b/arch/x86/Kconfig
 @@ -562,6 +562,15 @@ config KVM_GUEST
 This option enables various optimizations for running under the KVM
 hypervisor.
  
 +config KVM_DEBUG_FS
 + bool Enable debug information for KVM Guests in debugfs
 + depends on KVM_GUEST

Shouldn't it depend on DEBUG_FS as well?

 + default n
 + ---help---
 +   This option enables collection of various statistics for KVM guest.
 +   Statistics are displayed in debugfs filesystem. Enabling this option
 +   may incur significant overhead.
 +
  source arch/x86/lguest/Kconfig
  
  config PARAVIRT

-- 

Sasha.

--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH RFC V2 4/5] kvm guest : Added configuration support to enable debug information for KVM Guests

2011-10-24 Thread Avi Kivity
On 10/23/2011 09:07 PM, Raghavendra K T wrote:
 Added configuration support to enable debug information
 for KVM Guests in debugfs
 
 Signed-off-by: Srivatsa Vaddagiri va...@linux.vnet.ibm.com
 Signed-off-by: Suzuki Poulose suz...@in.ibm.com
 Signed-off-by: Raghavendra K T raghavendra...@linux.vnet.ibm.com
 ---
 diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
 index 1f03f82..ed34269 100644
 --- a/arch/x86/Kconfig
 +++ b/arch/x86/Kconfig
 @@ -562,6 +562,15 @@ config KVM_GUEST
 This option enables various optimizations for running under the KVM
 hypervisor.
  
 +config KVM_DEBUG_FS
 + bool Enable debug information for KVM Guests in debugfs
 + depends on KVM_GUEST
 + default n
 + ---help---
 +   This option enables collection of various statistics for KVM guest.
 +   Statistics are displayed in debugfs filesystem. Enabling this option
 +   may incur significant overhead.
 +
  source arch/x86/lguest/Kconfig
  


This might be better implemented through tracepoints, which an be
enabled dynamically.

-- 
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.

--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH RFC V2 4/5] kvm guest : Added configuration support to enable debug information for KVM Guests

2011-10-24 Thread Raghavendra K T

On 10/24/2011 03:31 PM, Sasha Levin wrote:

On Mon, 2011-10-24 at 00:37 +0530, Raghavendra K T wrote:

Added configuration support to enable debug information
for KVM Guests in debugfs
+config KVM_DEBUG_FS
+   bool Enable debug information for KVM Guests in debugfs
+   depends on KVM_GUEST


Shouldn't it depend on DEBUG_FS as well?

Thanks again for pointing. will correct this too.
--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH RFC V2 4/5] kvm guest : Added configuration support to enable debug information for KVM Guests

2011-10-23 Thread Raghavendra K T
Added configuration support to enable debug information
for KVM Guests in debugfs

Signed-off-by: Srivatsa Vaddagiri va...@linux.vnet.ibm.com
Signed-off-by: Suzuki Poulose suz...@in.ibm.com
Signed-off-by: Raghavendra K T raghavendra...@linux.vnet.ibm.com
---
diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index 1f03f82..ed34269 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -562,6 +562,15 @@ config KVM_GUEST
  This option enables various optimizations for running under the KVM
  hypervisor.
 
+config KVM_DEBUG_FS
+   bool Enable debug information for KVM Guests in debugfs
+   depends on KVM_GUEST
+   default n
+   ---help---
+ This option enables collection of various statistics for KVM guest.
+ Statistics are displayed in debugfs filesystem. Enabling this option
+ may incur significant overhead.
+
 source arch/x86/lguest/Kconfig
 
 config PARAVIRT
--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html