Hi Wei: Thanks for review. On 8/10/2021 6:56 PM, Wei Liu wrote:
On Mon, Aug 09, 2021 at 01:56:05PM -0400, Tianyu Lan wrote: [...]static int hv_cpu_init(unsigned int cpu) { union hv_vp_assist_msr_contents msr = { 0 }; @@ -85,6 +111,8 @@ static int hv_cpu_init(unsigned int cpu) } }+ hyperv_init_ghcb();+Why is the return value not checked here? If that's not required, can you leave a comment?
The check is necessary here. Will update in the next version. Thanks.