| Message ID | 20240513181928.720979-1-michael.roth@amd.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | KVM: SEV: Fix unused variable in guest request handling | expand |
On 5/13/24 13:19, Michael Roth wrote: > The variable 'sev' is assigned, but never used. Remove it. > > Fixes: 449ead2d1edb ("KVM: SEV: Provide support for SNP_GUEST_REQUEST NAE event") > Signed-off-by: Michael Roth <michael.roth@amd.com> Reviewed-by: Carlos Bilbao <carlos.bilbao.osdev@gmail.com> > --- > arch/x86/kvm/svm/sev.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/arch/x86/kvm/svm/sev.c b/arch/x86/kvm/svm/sev.c > index 59c0d89a4d52..6cf665c410b2 100644 > --- a/arch/x86/kvm/svm/sev.c > +++ b/arch/x86/kvm/svm/sev.c > @@ -3965,14 +3965,11 @@ static int __snp_handle_guest_req(struct kvm *kvm, gpa_t req_gpa, gpa_t resp_gpa > sev_ret_code *fw_err) > { > struct sev_data_snp_guest_request data = {0}; > - struct kvm_sev_info *sev; > int ret; > > if (!sev_snp_guest(kvm)) > return -EINVAL; > > - sev = &to_kvm_svm(kvm)->sev_info; > - > ret = snp_setup_guest_buf(kvm, &data, req_gpa, resp_gpa); > if (ret) > return ret;
> The variable 'sev' is assigned, but never used. Remove it.
Would it be a bit nicer to use the word “Omit” instead of “Fix”
in the summary phrase?
Regards,
Markus
Hey Markus, On 5/19/24 12:50 AM, Markus Elfring wrote: >> The variable 'sev' is assigned, but never used. Remove it. > Would it be a bit nicer to use the word “Omit” instead of “Fix” > in the summary phrase? I can find many instances of "Fix unused variable" in the history of the kernel: ubsan: fix unused variable warning in test module x86/resctrl: Fix unused variable warning in cache_alloc_hsw_probe() octeontx2-pf: Fix unused variable build error etc... but not a single "Omit unused variable" commit. > > Regards, > Markus Thanks, Carlos
>>> The variable 'sev' is assigned, but never used. Remove it. >> Would it be a bit nicer to use the word “Omit” instead of “Fix” >> in the summary phrase? > > > I can find many instances of "Fix unused variable" in the history of the > kernel: … > but not a single "Omit unused variable" commit. Some implementation details were fixed somehow because of a warning or error message. You would probably like to point the desire out in your summary phrase to get rid of another bit of redundant source code. Were any analysis tools involved in the discovery of corresponding change possibilities? Regards, Markus
diff --git a/arch/x86/kvm/svm/sev.c b/arch/x86/kvm/svm/sev.c index 59c0d89a4d52..6cf665c410b2 100644 --- a/arch/x86/kvm/svm/sev.c +++ b/arch/x86/kvm/svm/sev.c @@ -3965,14 +3965,11 @@ static int __snp_handle_guest_req(struct kvm *kvm, gpa_t req_gpa, gpa_t resp_gpa sev_ret_code *fw_err) { struct sev_data_snp_guest_request data = {0}; - struct kvm_sev_info *sev; int ret; if (!sev_snp_guest(kvm)) return -EINVAL; - sev = &to_kvm_svm(kvm)->sev_info; - ret = snp_setup_guest_buf(kvm, &data, req_gpa, resp_gpa); if (ret) return ret;
The variable 'sev' is assigned, but never used. Remove it. Fixes: 449ead2d1edb ("KVM: SEV: Provide support for SNP_GUEST_REQUEST NAE event") Signed-off-by: Michael Roth <michael.roth@amd.com> --- arch/x86/kvm/svm/sev.c | 3 --- 1 file changed, 3 deletions(-)