Hi Li, Alex. On 26/11/18 12:51, Alex Bennée wrote: > > Li Qiang <liq...@gmail.com> writes: > >> Cc: qemu-triv...@nongnu.org >> >> Signed-off-by: Li Qiang <liq...@gmail.com> > > Reviewed-by: Alex Bennée <alex.ben...@linaro.org> > >> --- >> target/i386/hax-all.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/target/i386/hax-all.c b/target/i386/hax-all.c >> index 502ce6f0af..464744a406 100644 >> --- a/target/i386/hax-all.c >> +++ b/target/i386/hax-all.c >> @@ -205,7 +205,7 @@ int hax_vcpu_destroy(CPUState *cpu) >> } >> >> /* >> - * 1. The hax_tunnel is also destroied when vcpu destroy >> + * 1. The hax_tunnel is also destroyed when vcpu destroy
I'm not native english speaker. Isn't it: 'The hax_tunnel is also destroyed at vcpu destroy' or 'The hax_tunnel is also destroyed when vcpu is destroyed'? >> * 2. close fd will cause hax module vcpu be cleaned >> */ >> hax_close_fd(vcpu->fd); > > > -- > Alex Bennée >