Re: [PATCH 0/4] hyperv: Deduplicate and reformat

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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!





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux