Already done by clear_page() in alloc_vcpu_struct()
Signed-off-by: Baodong Chen <chenbaodong@mxnavi.com>
---
xen/arch/arm/domain.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/xen/arch/arm/domain.c b/xen/arch/arm/domain.c
index ff330b3..ad1b106 100644
--- a/xen/arch/arm/domain.c
+++ b/xen/arch/arm/domain.c
@@ -557,7 +557,6 @@ int arch_vcpu_create(struct vcpu *v)
- sizeof(struct cpu_info));
memset(v->arch.cpu_info, 0, sizeof(*v->arch.cpu_info));
- memset(&v->arch.saved_context, 0, sizeof(v->arch.saved_context));
v->arch.saved_context.sp = (register_t)v->arch.cpu_info;
v->arch.saved_context.pc = (register_t)continue_new_vcpu;
--
2.7.4
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
Hi, Thank you for the patch. The title should be at max 80 characters. So how about the following title? "xen/arm: domain: Remove redundant memset for v->arch.saved_context" On 6/10/19 6:15 AM, Baodong Chen wrote: > Already done by clear_page() in alloc_vcpu_struct() Please try to make sentence in the commit message. For here I would suggest: "v->arch.saved_context is already zeroed in alloc_vcpu_struct() by clear_page(). So there are no need to memset it again in arch_vcpu_create()." If you are happy with the two changes, I can do them on commit. Cheers, > > Signed-off-by: Baodong Chen <chenbaodong@mxnavi.com> > --- > xen/arch/arm/domain.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/xen/arch/arm/domain.c b/xen/arch/arm/domain.c > index ff330b3..ad1b106 100644 > --- a/xen/arch/arm/domain.c > +++ b/xen/arch/arm/domain.c > @@ -557,7 +557,6 @@ int arch_vcpu_create(struct vcpu *v) > - sizeof(struct cpu_info)); > memset(v->arch.cpu_info, 0, sizeof(*v->arch.cpu_info)); > > - memset(&v->arch.saved_context, 0, sizeof(v->arch.saved_context)); > v->arch.saved_context.sp = (register_t)v->arch.cpu_info; > v->arch.saved_context.pc = (register_t)continue_new_vcpu; > > -- Julien Grall _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
On 6/11/19 04:11, Julien Grall wrote: > Hi, > > Thank you for the patch. The title should be at max 80 characters. So > how about the following title? > > "xen/arm: domain: Remove redundant memset for v->arch.saved_context" Max 80 characters, roger that. > > On 6/10/19 6:15 AM, Baodong Chen wrote: >> Already done by clear_page() in alloc_vcpu_struct() > > Please try to make sentence in the commit message. For here I would > suggest: > > "v->arch.saved_context is already zeroed in alloc_vcpu_struct() by > clear_page(). So there are no need to memset it again in > arch_vcpu_create()." > > If you are happy with the two changes, I can do them on commit. Thanks, please. > > Cheers, > >> >> Signed-off-by: Baodong Chen <chenbaodong@mxnavi.com> >> --- >> xen/arch/arm/domain.c | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/xen/arch/arm/domain.c b/xen/arch/arm/domain.c >> index ff330b3..ad1b106 100644 >> --- a/xen/arch/arm/domain.c >> +++ b/xen/arch/arm/domain.c >> @@ -557,7 +557,6 @@ int arch_vcpu_create(struct vcpu *v) >> - sizeof(struct cpu_info)); >> memset(v->arch.cpu_info, 0, sizeof(*v->arch.cpu_info)); >> - memset(&v->arch.saved_context, 0, sizeof(v->arch.saved_context)); >> v->arch.saved_context.sp = (register_t)v->arch.cpu_info; >> v->arch.saved_context.pc = (register_t)continue_new_vcpu; >> > _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
On 11/06/2019 01:09, chenbaodong wrote: > > On 6/11/19 04:11, Julien Grall wrote: >> Hi, >> >> Thank you for the patch. The title should be at max 80 characters. So how >> about the following title? >> >> "xen/arm: domain: Remove redundant memset for v->arch.saved_context" > Max 80 characters, roger that. >> >> On 6/10/19 6:15 AM, Baodong Chen wrote: >>> Already done by clear_page() in alloc_vcpu_struct() >> >> Please try to make sentence in the commit message. For here I would suggest: >> >> "v->arch.saved_context is already zeroed in alloc_vcpu_struct() by >> clear_page(). So there are no need to memset it again in arch_vcpu_create()." >> >> If you are happy with the two changes, I can do them on commit. > Thanks, please. Committed, thank you! Cheers, >> >> Cheers, >> >>> >>> Signed-off-by: Baodong Chen <chenbaodong@mxnavi.com> >>> --- >>> xen/arch/arm/domain.c | 1 - >>> 1 file changed, 1 deletion(-) >>> >>> diff --git a/xen/arch/arm/domain.c b/xen/arch/arm/domain.c >>> index ff330b3..ad1b106 100644 >>> --- a/xen/arch/arm/domain.c >>> +++ b/xen/arch/arm/domain.c >>> @@ -557,7 +557,6 @@ int arch_vcpu_create(struct vcpu *v) >>> - sizeof(struct cpu_info)); >>> memset(v->arch.cpu_info, 0, sizeof(*v->arch.cpu_info)); >>> - memset(&v->arch.saved_context, 0, sizeof(v->arch.saved_context)); >>> v->arch.saved_context.sp = (register_t)v->arch.cpu_info; >>> v->arch.saved_context.pc = (register_t)continue_new_vcpu; >>> >> -- Julien Grall _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
© 2016 - 2024 Red Hat, Inc.