hyperv/util unify the negotitation for utils

[honzhan_microsoft.com]

Summary

Unify the negotiation for shutdown/timesync/heartbeat/kvp, and add the log for selected version.

On windows 2012 or 2016, the negotiation version for both KVP and timesync does not match the Hyper-V expectation. The command "Get-VMIntegrationService" will output "The protocol version of the component installed in the virtual machine does not match the version expected by the hosting system". We have to upgrade the version of KVP and time sync on VM side. That is future work.

Submitted by: Hongjiang Zhang <honzhan microsoft com>

[sepherosa_gmail.com]

sys/dev/hyperv/utilities/hv_kvp.h

152 ↗

(On Diff #21490)

I don't think these changes are related to the topic here.

sys/dev/hyperv/utilities/hv_util.c

58 ↗

(On Diff #21490)

I don't like this, driver should have this instead of gathering them here.

96 ↗

(On Diff #21490)

These should be in each driver.

156 ↗

(On Diff #21490)

Can you keep the vmbus_ic_negomsg? I don't want to rename this function again.

159 ↗

(On Diff #21490)

I have already defined a new message for negotiate. Please don't use the old one.

169 ↗

(On Diff #21490)

Use the new message, this type of accessing is ugly.

178 ↗

(On Diff #21490)

Same as above comment.