qemu-devel
[Top][All Lists]
Advanced

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

Re: [PATCH 0/2] Improve vmstate_vmbus_dev handling


From: Thomas Huth
Subject: Re: [PATCH 0/2] Improve vmstate_vmbus_dev handling
Date: Mon, 7 Jun 2021 09:16:00 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.0

On 05/06/2021 18.01, Maciej S. Szmigiero wrote:
Hi Thomas,

On 03.06.2021 12:41, Thomas Huth wrote:
I accidentally came accross vmstate_vmbus_dev and noticed that
it is currently not used at all ... wire it up and make it
static, since it is only used in one file.

Thomas Huth (2):
   hw/hyperv/vmbus: Wire up vmstate_vmbus_dev
   hw/hyperv/vmbus: Make vmstate_vmbus_dev static

  hw/hyperv/vmbus.c         | 29 +++++++++++++++--------------
  include/hw/hyperv/vmbus.h |  3 ---
  2 files changed, 15 insertions(+), 17 deletions(-)


I think the idea is to embed vmstate_vmbus_dev into a child device
VMStateDescription using VMSTATE_STRUCT() - since particular VMBus
devices aren't merged yet there are currently no users of it.

Ok, then never mind. But some comments in the source code would be really helpful - in the current shape, it simply looks like an oversight...

 Thomas




reply via email to

[Prev in Thread] Current Thread [Next in Thread]