Hi Maciej,
On 1/24/23 10:56 AM, Maciej S. Szmigiero wrote:
On 18.01.2023 10:21, Gavin Shan wrote:
There are two VMs created in prepare_vm(), which isn't necessary.
To remove the second created and unnecessary VM.
Signed-off-by: Gavin Shan <gshan@xxxxxxxxxx>
It's weird that we ended with two __vm_create_with_one_vcpu() calls,
it looks like the second one was accidentally (re-)introduced during
'kvmarm-6.2' merge, so maybe?:
Fixes: eb5618911af0 ("Merge tag 'kvmarm-6.2' of https://git.kernel.org/pub/scm/linux/kernel/git/kvmarm/kvmarm into HEAD")
Anyway, thanks for spotting this:
Reviewed-by: Maciej S. Szmigiero <maciej.szmigiero@xxxxxxxxxx>
Thanks for your review. Right, it'd better to have the fix tag as your suggested.
I think I probably needn't respin. I guess Paolo, Marc or Oliver may help
to add the fix tag when it's merged.
Thanks,
Gavin
