All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 2/2] remove inclusion of asm/config.h
@ 2012-01-12 16:40 Jan Beulich
  2012-01-12 16:58 ` Keir Fraser
  0 siblings, 1 reply; 2+ messages in thread
From: Jan Beulich @ 2012-01-12 16:40 UTC (permalink / raw)
  To: Keir Fraser, xen-devel

[-- Attachment #1: Type: text/plain, Size: 3946 bytes --]

This was always bogus (xen/config.h should have been used instead) and
is superfluous now that xen/config.h gets included through the compiler
command line.

Signed-off-by: Jan Beulich <jbeulich@suse.com>

--- a/xen/arch/ia64/vmx/optvfault.S
+++ b/xen/arch/ia64/vmx/optvfault.S
@@ -6,8 +6,6 @@
  * Xuefei Xu (Anthony Xu) <anthony.xu@intel.com>
  */
 
-#include <linux/config.h>
-#include <asm/config.h>
 #include <asm/pgtable.h>
 #include <asm/asmmacro.h>
 #include <asm/kregs.h>
--- a/xen/arch/ia64/xen/cpufreq/cpufreq.c
+++ b/xen/arch/ia64/xen/cpufreq/cpufreq.c
@@ -21,7 +21,6 @@
 #include <xen/xmalloc.h>
 #include <asm/bug.h>
 #include <asm/io.h>
-#include <asm/config.h>
 #include <asm/processor.h>
 #include <asm/percpu.h>
 #include <asm/pal.h>
--- a/xen/arch/ia64/xen/ivt.S
+++ b/xen/arch/ia64/xen/ivt.S
@@ -1,7 +1,6 @@
 #include <asm/debugger.h>
 #include <asm/vhpt.h>
 #include <public/arch-ia64.h>
-#include <asm/config.h>
 /*
  * arch/ia64/kernel/ivt.S
  *
@@ -43,8 +42,6 @@
  * Table is based upon EAS2.6 (Oct 1999)
  */
 
-#include <linux/config.h>
-
 #include <asm/asmmacro.h>
 #include <asm/break.h>
 #include <asm/ia32.h>
--- a/xen/arch/x86/acpi/cpufreq/cpufreq.c
+++ b/xen/arch/x86/acpi/cpufreq/cpufreq.c
@@ -38,7 +38,6 @@
 #include <asm/bug.h>
 #include <asm/msr.h>
 #include <asm/io.h>
-#include <asm/config.h>
 #include <asm/processor.h>
 #include <asm/percpu.h>
 #include <asm/cpufeature.h>
--- a/xen/arch/x86/acpi/cpufreq/powernow.c
+++ b/xen/arch/x86/acpi/cpufreq/powernow.c
@@ -32,7 +32,6 @@
 #include <asm/bug.h>
 #include <asm/msr.h>
 #include <asm/io.h>
-#include <asm/config.h>
 #include <asm/processor.h>
 #include <asm/percpu.h>
 #include <asm/cpufeature.h>
--- a/xen/drivers/cpufreq/cpufreq.c
+++ b/xen/drivers/cpufreq/cpufreq.c
@@ -41,7 +41,6 @@
 #include <xen/cpu.h>
 #include <asm/bug.h>
 #include <asm/io.h>
-#include <asm/config.h>
 #include <asm/processor.h>
 #include <asm/percpu.h>
 #include <acpi/acpi.h>
--- a/xen/drivers/cpufreq/cpufreq_ondemand.c
+++ b/xen/drivers/cpufreq/cpufreq_ondemand.c
@@ -18,7 +18,6 @@
 #include <xen/types.h>
 #include <xen/sched.h>
 #include <xen/timer.h>
-#include <asm/config.h>
 #include <acpi/cpufreq/cpufreq.h>
 
 #define DEF_FREQUENCY_UP_THRESHOLD              (80)
--- a/xen/drivers/cpufreq/utility.c
+++ b/xen/drivers/cpufreq/utility.c
@@ -28,7 +28,6 @@
 #include <xen/sched.h>
 #include <xen/timer.h>
 #include <xen/trace.h>
-#include <asm/config.h>
 #include <acpi/cpufreq/cpufreq.h>
 #include <public/sysctl.h>
 
--- a/xen/include/asm-ia64/linux-xen/asm/perfmon.h
+++ b/xen/include/asm-ia64/linux-xen/asm/perfmon.h
@@ -7,7 +7,6 @@
 #define _ASM_IA64_PERFMON_H
 
 #ifdef XEN
-#include <asm/config.h>
 #ifndef pt_regs
 #define pt_regs cpu_user_regs
 #endif
--- a/xen/include/asm-ia64/privop_stat.h
+++ b/xen/include/asm-ia64/privop_stat.h
@@ -1,6 +1,6 @@
 #ifndef _XEN_IA64_PRIVOP_STAT_H
 #define _XEN_IA64_PRIVOP_STAT_H
-#include <asm/config.h>
+
 #include <xen/types.h>
 #include <public/xen.h>
 
--- a/xen/include/asm-ia64/xensystem.h
+++ b/xen/include/asm-ia64/xensystem.h
@@ -10,7 +10,6 @@
  * 	Kun Tian (Kevin Tian) <kevin.tian@intel.com>
  *
  */
-#include <asm/config.h>
 
 /* Define HV space hierarchy.
    VMM memory space is protected by CPL for paravirtualized domains and
--- a/xen/include/asm-x86/hvm/svm/nestedsvm.h
+++ b/xen/include/asm-x86/hvm/svm/nestedsvm.h
@@ -19,7 +19,6 @@
 #ifndef __ASM_X86_HVM_SVM_NESTEDSVM_H__
 #define __ASM_X86_HVM_SVM_NESTEDSVM_H__
 
-#include <asm/config.h>
 #include <asm/hvm/hvm.h>
 #include <asm/hvm/svm/vmcb.h>
 
--- a/xen/include/asm-x86/hvm/vmx/vmcs.h
+++ b/xen/include/asm-x86/hvm/vmx/vmcs.h
@@ -19,7 +19,6 @@
 #ifndef __ASM_X86_HVM_VMX_VMCS_H__
 #define __ASM_X86_HVM_VMX_VMCS_H__
 
-#include <asm/config.h>
 #include <asm/hvm/io.h>
 #include <asm/hvm/vpmu.h>
 




[-- Attachment #2: no-include-asm-config-h.patch --]
[-- Type: text/plain, Size: 3976 bytes --]

remove inclusion of asm/config.h

This was always bogus (xen/config.h should have been used instead) and
is superfluous now that xen/config.h gets included through the compiler
command line.

Signed-off-by: Jan Beulich <jbeulich@suse.com>

--- a/xen/arch/ia64/vmx/optvfault.S
+++ b/xen/arch/ia64/vmx/optvfault.S
@@ -6,8 +6,6 @@
  * Xuefei Xu (Anthony Xu) <anthony.xu@intel.com>
  */
 
-#include <linux/config.h>
-#include <asm/config.h>
 #include <asm/pgtable.h>
 #include <asm/asmmacro.h>
 #include <asm/kregs.h>
--- a/xen/arch/ia64/xen/cpufreq/cpufreq.c
+++ b/xen/arch/ia64/xen/cpufreq/cpufreq.c
@@ -21,7 +21,6 @@
 #include <xen/xmalloc.h>
 #include <asm/bug.h>
 #include <asm/io.h>
-#include <asm/config.h>
 #include <asm/processor.h>
 #include <asm/percpu.h>
 #include <asm/pal.h>
--- a/xen/arch/ia64/xen/ivt.S
+++ b/xen/arch/ia64/xen/ivt.S
@@ -1,7 +1,6 @@
 #include <asm/debugger.h>
 #include <asm/vhpt.h>
 #include <public/arch-ia64.h>
-#include <asm/config.h>
 /*
  * arch/ia64/kernel/ivt.S
  *
@@ -43,8 +42,6 @@
  * Table is based upon EAS2.6 (Oct 1999)
  */
 
-#include <linux/config.h>
-
 #include <asm/asmmacro.h>
 #include <asm/break.h>
 #include <asm/ia32.h>
--- a/xen/arch/x86/acpi/cpufreq/cpufreq.c
+++ b/xen/arch/x86/acpi/cpufreq/cpufreq.c
@@ -38,7 +38,6 @@
 #include <asm/bug.h>
 #include <asm/msr.h>
 #include <asm/io.h>
-#include <asm/config.h>
 #include <asm/processor.h>
 #include <asm/percpu.h>
 #include <asm/cpufeature.h>
--- a/xen/arch/x86/acpi/cpufreq/powernow.c
+++ b/xen/arch/x86/acpi/cpufreq/powernow.c
@@ -32,7 +32,6 @@
 #include <asm/bug.h>
 #include <asm/msr.h>
 #include <asm/io.h>
-#include <asm/config.h>
 #include <asm/processor.h>
 #include <asm/percpu.h>
 #include <asm/cpufeature.h>
--- a/xen/drivers/cpufreq/cpufreq.c
+++ b/xen/drivers/cpufreq/cpufreq.c
@@ -41,7 +41,6 @@
 #include <xen/cpu.h>
 #include <asm/bug.h>
 #include <asm/io.h>
-#include <asm/config.h>
 #include <asm/processor.h>
 #include <asm/percpu.h>
 #include <acpi/acpi.h>
--- a/xen/drivers/cpufreq/cpufreq_ondemand.c
+++ b/xen/drivers/cpufreq/cpufreq_ondemand.c
@@ -18,7 +18,6 @@
 #include <xen/types.h>
 #include <xen/sched.h>
 #include <xen/timer.h>
-#include <asm/config.h>
 #include <acpi/cpufreq/cpufreq.h>
 
 #define DEF_FREQUENCY_UP_THRESHOLD              (80)
--- a/xen/drivers/cpufreq/utility.c
+++ b/xen/drivers/cpufreq/utility.c
@@ -28,7 +28,6 @@
 #include <xen/sched.h>
 #include <xen/timer.h>
 #include <xen/trace.h>
-#include <asm/config.h>
 #include <acpi/cpufreq/cpufreq.h>
 #include <public/sysctl.h>
 
--- a/xen/include/asm-ia64/linux-xen/asm/perfmon.h
+++ b/xen/include/asm-ia64/linux-xen/asm/perfmon.h
@@ -7,7 +7,6 @@
 #define _ASM_IA64_PERFMON_H
 
 #ifdef XEN
-#include <asm/config.h>
 #ifndef pt_regs
 #define pt_regs cpu_user_regs
 #endif
--- a/xen/include/asm-ia64/privop_stat.h
+++ b/xen/include/asm-ia64/privop_stat.h
@@ -1,6 +1,6 @@
 #ifndef _XEN_IA64_PRIVOP_STAT_H
 #define _XEN_IA64_PRIVOP_STAT_H
-#include <asm/config.h>
+
 #include <xen/types.h>
 #include <public/xen.h>
 
--- a/xen/include/asm-ia64/xensystem.h
+++ b/xen/include/asm-ia64/xensystem.h
@@ -10,7 +10,6 @@
  * 	Kun Tian (Kevin Tian) <kevin.tian@intel.com>
  *
  */
-#include <asm/config.h>
 
 /* Define HV space hierarchy.
    VMM memory space is protected by CPL for paravirtualized domains and
--- a/xen/include/asm-x86/hvm/svm/nestedsvm.h
+++ b/xen/include/asm-x86/hvm/svm/nestedsvm.h
@@ -19,7 +19,6 @@
 #ifndef __ASM_X86_HVM_SVM_NESTEDSVM_H__
 #define __ASM_X86_HVM_SVM_NESTEDSVM_H__
 
-#include <asm/config.h>
 #include <asm/hvm/hvm.h>
 #include <asm/hvm/svm/vmcb.h>
 
--- a/xen/include/asm-x86/hvm/vmx/vmcs.h
+++ b/xen/include/asm-x86/hvm/vmx/vmcs.h
@@ -19,7 +19,6 @@
 #ifndef __ASM_X86_HVM_VMX_VMCS_H__
 #define __ASM_X86_HVM_VMX_VMCS_H__
 
-#include <asm/config.h>
 #include <asm/hvm/io.h>
 #include <asm/hvm/vpmu.h>
 

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH 2/2] remove inclusion of asm/config.h
  2012-01-12 16:40 [PATCH 2/2] remove inclusion of asm/config.h Jan Beulich
@ 2012-01-12 16:58 ` Keir Fraser
  0 siblings, 0 replies; 2+ messages in thread
From: Keir Fraser @ 2012-01-12 16:58 UTC (permalink / raw)
  To: Jan Beulich, xen-devel

On 12/01/2012 16:40, "Jan Beulich" <JBeulich@suse.com> wrote:

> This was always bogus (xen/config.h should have been used instead) and
> is superfluous now that xen/config.h gets included through the compiler
> command line.
> 
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

Acked-by: Keir Fraser <keir@xen.org>

> --- a/xen/arch/ia64/vmx/optvfault.S
> +++ b/xen/arch/ia64/vmx/optvfault.S
> @@ -6,8 +6,6 @@
>   * Xuefei Xu (Anthony Xu) <anthony.xu@intel.com>
>   */
>  
> -#include <linux/config.h>
> -#include <asm/config.h>
>  #include <asm/pgtable.h>
>  #include <asm/asmmacro.h>
>  #include <asm/kregs.h>
> --- a/xen/arch/ia64/xen/cpufreq/cpufreq.c
> +++ b/xen/arch/ia64/xen/cpufreq/cpufreq.c
> @@ -21,7 +21,6 @@
>  #include <xen/xmalloc.h>
>  #include <asm/bug.h>
>  #include <asm/io.h>
> -#include <asm/config.h>
>  #include <asm/processor.h>
>  #include <asm/percpu.h>
>  #include <asm/pal.h>
> --- a/xen/arch/ia64/xen/ivt.S
> +++ b/xen/arch/ia64/xen/ivt.S
> @@ -1,7 +1,6 @@
>  #include <asm/debugger.h>
>  #include <asm/vhpt.h>
>  #include <public/arch-ia64.h>
> -#include <asm/config.h>
>  /*
>   * arch/ia64/kernel/ivt.S
>   *
> @@ -43,8 +42,6 @@
>   * Table is based upon EAS2.6 (Oct 1999)
>   */
>  
> -#include <linux/config.h>
> -
>  #include <asm/asmmacro.h>
>  #include <asm/break.h>
>  #include <asm/ia32.h>
> --- a/xen/arch/x86/acpi/cpufreq/cpufreq.c
> +++ b/xen/arch/x86/acpi/cpufreq/cpufreq.c
> @@ -38,7 +38,6 @@
>  #include <asm/bug.h>
>  #include <asm/msr.h>
>  #include <asm/io.h>
> -#include <asm/config.h>
>  #include <asm/processor.h>
>  #include <asm/percpu.h>
>  #include <asm/cpufeature.h>
> --- a/xen/arch/x86/acpi/cpufreq/powernow.c
> +++ b/xen/arch/x86/acpi/cpufreq/powernow.c
> @@ -32,7 +32,6 @@
>  #include <asm/bug.h>
>  #include <asm/msr.h>
>  #include <asm/io.h>
> -#include <asm/config.h>
>  #include <asm/processor.h>
>  #include <asm/percpu.h>
>  #include <asm/cpufeature.h>
> --- a/xen/drivers/cpufreq/cpufreq.c
> +++ b/xen/drivers/cpufreq/cpufreq.c
> @@ -41,7 +41,6 @@
>  #include <xen/cpu.h>
>  #include <asm/bug.h>
>  #include <asm/io.h>
> -#include <asm/config.h>
>  #include <asm/processor.h>
>  #include <asm/percpu.h>
>  #include <acpi/acpi.h>
> --- a/xen/drivers/cpufreq/cpufreq_ondemand.c
> +++ b/xen/drivers/cpufreq/cpufreq_ondemand.c
> @@ -18,7 +18,6 @@
>  #include <xen/types.h>
>  #include <xen/sched.h>
>  #include <xen/timer.h>
> -#include <asm/config.h>
>  #include <acpi/cpufreq/cpufreq.h>
>  
>  #define DEF_FREQUENCY_UP_THRESHOLD              (80)
> --- a/xen/drivers/cpufreq/utility.c
> +++ b/xen/drivers/cpufreq/utility.c
> @@ -28,7 +28,6 @@
>  #include <xen/sched.h>
>  #include <xen/timer.h>
>  #include <xen/trace.h>
> -#include <asm/config.h>
>  #include <acpi/cpufreq/cpufreq.h>
>  #include <public/sysctl.h>
>  
> --- a/xen/include/asm-ia64/linux-xen/asm/perfmon.h
> +++ b/xen/include/asm-ia64/linux-xen/asm/perfmon.h
> @@ -7,7 +7,6 @@
>  #define _ASM_IA64_PERFMON_H
>  
>  #ifdef XEN
> -#include <asm/config.h>
>  #ifndef pt_regs
>  #define pt_regs cpu_user_regs
>  #endif
> --- a/xen/include/asm-ia64/privop_stat.h
> +++ b/xen/include/asm-ia64/privop_stat.h
> @@ -1,6 +1,6 @@
>  #ifndef _XEN_IA64_PRIVOP_STAT_H
>  #define _XEN_IA64_PRIVOP_STAT_H
> -#include <asm/config.h>
> +
>  #include <xen/types.h>
>  #include <public/xen.h>
>  
> --- a/xen/include/asm-ia64/xensystem.h
> +++ b/xen/include/asm-ia64/xensystem.h
> @@ -10,7 +10,6 @@
>   *  Kun Tian (Kevin Tian) <kevin.tian@intel.com>
>   *
>   */
> -#include <asm/config.h>
>  
>  /* Define HV space hierarchy.
>     VMM memory space is protected by CPL for paravirtualized domains and
> --- a/xen/include/asm-x86/hvm/svm/nestedsvm.h
> +++ b/xen/include/asm-x86/hvm/svm/nestedsvm.h
> @@ -19,7 +19,6 @@
>  #ifndef __ASM_X86_HVM_SVM_NESTEDSVM_H__
>  #define __ASM_X86_HVM_SVM_NESTEDSVM_H__
>  
> -#include <asm/config.h>
>  #include <asm/hvm/hvm.h>
>  #include <asm/hvm/svm/vmcb.h>
>  
> --- a/xen/include/asm-x86/hvm/vmx/vmcs.h
> +++ b/xen/include/asm-x86/hvm/vmx/vmcs.h
> @@ -19,7 +19,6 @@
>  #ifndef __ASM_X86_HVM_VMX_VMCS_H__
>  #define __ASM_X86_HVM_VMX_VMCS_H__
>  
> -#include <asm/config.h>
>  #include <asm/hvm/io.h>
>  #include <asm/hvm/vpmu.h>
>  
> 
> 
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-01-12 16:58 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-01-12 16:40 [PATCH 2/2] remove inclusion of asm/config.h Jan Beulich
2012-01-12 16:58 ` Keir Fraser

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.