On Wed, Oct 21, 2020 at 11:45 AM Michal Privoznik <mprivozn@xxxxxxxxxx> wrote: > > The more I look into the code the more things to fix I find. Well, here > are some fixes. > > Michal Prívozník (4): > hyperv: Don't overwrite errors from hypervCreateInvokeParamsList() > hyperv: Use hypervRequestStateChange() in hypervDomainSuspend() > hyperv: Use two empty lines between functions > hyperv: Reformat > > src/hyperv/hyperv_driver.c | 186 +++++++++--------------- > src/hyperv/hyperv_util.c | 1 - > src/hyperv/hyperv_wmi.c | 288 +++++++++++++++++++------------------ > src/hyperv/hyperv_wmi.h | 15 +- > 4 files changed, 229 insertions(+), 261 deletions(-) > > -- > 2.26.2 > Series LGTM. Reviewed-by: Neal Gompa <ngompa13@xxxxxxxxx> But... uhh... there's also this series from Matt Coleman, which I just reviewed: https://www.redhat.com/archives/libvir-list/2020-October/msg01282.html Can we put the two together somehow? -- 真実はいつも一つ！/ Always, there's only one truth!
