| Message ID | 20220812103009.8362-1-kunyu@nfschina.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/5] kvm/kvm_main: Modify the offset type to size_t, which is consistent with the calling function | expand |
Please use "KVM: " for the shortlog scope, and write an actual shortlog. E.g. KVM: Drop unnecessary initialization of "npages" in hva_to_pfn_slow() On Fri, Aug 12, 2022, Li kunyu wrote: > The npages variable does not need to be initialized and assigned, it is > first assigned before it is used. > > Signed-off-by: Li kunyu <kunyu@nfschina.com> > --- With a proper shortlog, Reviewed-by: Sean Christopherson <seanjc@google.com>
thanks, re-send these two patches after modification.
diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c index 82f2b90718a9..f047799bf19b 100644 --- a/virt/kvm/kvm_main.c +++ b/virt/kvm/kvm_main.c @@ -2516,7 +2516,7 @@ static int hva_to_pfn_slow(unsigned long addr, bool *async, bool write_fault, { unsigned int flags = FOLL_HWPOISON; struct page *page; - int npages = 0; + int npages; might_sleep();
The npages variable does not need to be initialized and assigned, it is first assigned before it is used. Signed-off-by: Li kunyu <kunyu@nfschina.com> --- virt/kvm/kvm_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)