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:
[...]

quoted

  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.