| Message ID | 4FF6B232.4050502@cn.fujitsu.com |
|---|---|
| State | New |
| Headers | show |
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 > */ >
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 >> */ >> >
> > 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
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
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(-)
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 */
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(-)