Patchwork [2/7,v6] update linux headers

login
register
mail settings
Submitter Wen Congyang
Date July 6, 2012, 9:38 a.m.
Message ID <4FF6B232.4050502@cn.fujitsu.com>
Download mbox | patch
Permalink /patch/169388/
State New
Headers show

Comments

Wen Congyang - July 6, 2012, 9:38 a.m.
Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
---
 linux-headers/asm-x86/kvm_para.h |    2 ++
 linux-headers/linux/kvm_para.h   |    6 ++++++
 2 files changed, 8 insertions(+), 0 deletions(-)
Jan Kiszka - July 6, 2012, 10:25 a.m.
On 2012-07-06 11:38, Wen Congyang wrote:
> Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>

Which kvm.git hash is this referring? Please state this to avoid that we
are merging support for kernel features that are still under review.

Jan

> ---
>  linux-headers/asm-x86/kvm_para.h |    2 ++
>  linux-headers/linux/kvm_para.h   |    6 ++++++
>  2 files changed, 8 insertions(+), 0 deletions(-)
> 
> diff --git a/linux-headers/asm-x86/kvm_para.h b/linux-headers/asm-x86/kvm_para.h
> index f2ac46a..f9d858f 100644
> --- a/linux-headers/asm-x86/kvm_para.h
> +++ b/linux-headers/asm-x86/kvm_para.h
> @@ -89,5 +89,7 @@ struct kvm_vcpu_pv_apf_data {
>  	__u32 enabled;
>  };
>  
> +#define KVM_PV_PORT	(0x505UL)
> +
>  
>  #endif /* _ASM_X86_KVM_PARA_H */
> diff --git a/linux-headers/linux/kvm_para.h b/linux-headers/linux/kvm_para.h
> index 7bdcf93..4fbd625 100644
> --- a/linux-headers/linux/kvm_para.h
> +++ b/linux-headers/linux/kvm_para.h
> @@ -20,6 +20,12 @@
>  #define KVM_HC_FEATURES			3
>  #define KVM_HC_PPC_MAP_MAGIC_PAGE	4
>  
> +/* The bit of the value read from KVM_PV_PORT */
> +#define KVM_PV_FEATURE_PANICKED	0
> +
> +/* The value writen to KVM_PV_PORT */
> +#define KVM_PV_PANICKED		1
> +
>  /*
>   * hypercalls use architecture specific
>   */
>
Wen Congyang - July 6, 2012, 10:50 a.m.
At 07/06/2012 06:25 PM, Jan Kiszka Wrote:
> On 2012-07-06 11:38, Wen Congyang wrote:
>> Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
> 
> Which kvm.git hash is this referring? Please state this to avoid that we
> are merging support for kernel features that are still under review.

The following kvm.git:
http://git.kernel.org/?p=virt/kvm/kvm.git;a=summary

I apply my kvm's patch, and use ./scripts/update-linux-headers.sh to
generate this patch.

Thanks
Wen Congyang

> 
> Jan
> 
>> ---
>>  linux-headers/asm-x86/kvm_para.h |    2 ++
>>  linux-headers/linux/kvm_para.h   |    6 ++++++
>>  2 files changed, 8 insertions(+), 0 deletions(-)
>>
>> diff --git a/linux-headers/asm-x86/kvm_para.h b/linux-headers/asm-x86/kvm_para.h
>> index f2ac46a..f9d858f 100644
>> --- a/linux-headers/asm-x86/kvm_para.h
>> +++ b/linux-headers/asm-x86/kvm_para.h
>> @@ -89,5 +89,7 @@ struct kvm_vcpu_pv_apf_data {
>>  	__u32 enabled;
>>  };
>>  
>> +#define KVM_PV_PORT	(0x505UL)
>> +
>>  
>>  #endif /* _ASM_X86_KVM_PARA_H */
>> diff --git a/linux-headers/linux/kvm_para.h b/linux-headers/linux/kvm_para.h
>> index 7bdcf93..4fbd625 100644
>> --- a/linux-headers/linux/kvm_para.h
>> +++ b/linux-headers/linux/kvm_para.h
>> @@ -20,6 +20,12 @@
>>  #define KVM_HC_FEATURES			3
>>  #define KVM_HC_PPC_MAP_MAGIC_PAGE	4
>>  
>> +/* The bit of the value read from KVM_PV_PORT */
>> +#define KVM_PV_FEATURE_PANICKED	0
>> +
>> +/* The value writen to KVM_PV_PORT */
>> +#define KVM_PV_PANICKED		1
>> +
>>  /*
>>   * hypercalls use architecture specific
>>   */
>>
>
陳韋任 - July 6, 2012, 10:55 a.m.
> > Which kvm.git hash is this referring? Please state this to avoid that we
> > are merging support for kernel features that are still under review.
> 
> The following kvm.git:
> http://git.kernel.org/?p=virt/kvm/kvm.git;a=summary

  What Jia ask for is git hash, should be something like,

    8aca521512a14c439624191bd0a891c52f91b401

Regards,
chenwj
Jan Kiszka - July 6, 2012, 11:16 a.m.
On 2012-07-06 12:50, Wen Congyang wrote:
> At 07/06/2012 06:25 PM, Jan Kiszka Wrote:
>> On 2012-07-06 11:38, Wen Congyang wrote:
>>> Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
>>
>> Which kvm.git hash is this referring? Please state this to avoid that we
>> are merging support for kernel features that are still under review.
> 
> The following kvm.git:
> http://git.kernel.org/?p=virt/kvm/kvm.git;a=summary
> 
> I apply my kvm's patch, and use ./scripts/update-linux-headers.sh to
> generate this patch.

OK, means the feature is not yet upstream. So this series is for
reference and early review until the kernel interface is accepted. I
personally push my kernel bits first before suggesting the userspace
parts. Avoids duplicate work if the former changes again during review.

Jan
Paul Gortmaker - July 7, 2012, 2:35 p.m.
On Fri, Jul 6, 2012 at 5:38 AM, Wen Congyang <wency@cn.fujitsu.com> wrote:
> Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>

Minor nit, but adding text "for kvm" to the end of your subject can be
useful for people who are browsing short log of commits without
the context of the full patch series.   Since there were questions
about the what/why of the patch, perhaps a non-null long log would
have been a good addition too.

Thanks,
Paul.
--

> ---
>  linux-headers/asm-x86/kvm_para.h |    2 ++
>  linux-headers/linux/kvm_para.h   |    6 ++++++
>  2 files changed, 8 insertions(+), 0 deletions(-)

Patch

diff --git a/linux-headers/asm-x86/kvm_para.h b/linux-headers/asm-x86/kvm_para.h
index f2ac46a..f9d858f 100644
--- a/linux-headers/asm-x86/kvm_para.h
+++ b/linux-headers/asm-x86/kvm_para.h
@@ -89,5 +89,7 @@  struct kvm_vcpu_pv_apf_data {
 	__u32 enabled;
 };
 
+#define KVM_PV_PORT	(0x505UL)
+
 
 #endif /* _ASM_X86_KVM_PARA_H */
diff --git a/linux-headers/linux/kvm_para.h b/linux-headers/linux/kvm_para.h
index 7bdcf93..4fbd625 100644
--- a/linux-headers/linux/kvm_para.h
+++ b/linux-headers/linux/kvm_para.h
@@ -20,6 +20,12 @@ 
 #define KVM_HC_FEATURES			3
 #define KVM_HC_PPC_MAP_MAGIC_PAGE	4
 
+/* The bit of the value read from KVM_PV_PORT */
+#define KVM_PV_FEATURE_PANICKED	0
+
+/* The value writen to KVM_PV_PORT */
+#define KVM_PV_PANICKED		1
+
 /*
  * hypercalls use architecture specific
  */