[libvirt] [PATCH v3 0/3] nodedev: update caps before quering nodedev infos
eskultet at redhat.com
Thu Jan 25 09:33:01 UTC 2018
On Wed, Jan 10, 2018 at 08:14:48PM +0800, Wu Zongyong wrote:
> Some capabilities of node devices rely on what driver they bound to,
> and therefore, these capabilities may change when the driver change.
> In current implemention, it is not consistent between real status and
> the status we get by invoking nodedev interfaces.
> So, this series of patches try to manually update devices' capabilities
> each time before nodedev driver interfaces invoked.
Hi, so I finally posted the series I mentioned in my previous reviews,
unfortunately, after applying your patches there were a lot of merge conflicts
so I had to write them from scratch, so I at least added you as an honorable
mention in patch equivalents of your patches 2 and 3. Unless someone objects to
the series as a whole, your patch 1 will be unnecessary.
Feel free to have a look at the series  and test whether it works as
expected (it did in my testing, both virsh and python).
> Wu Zongyong (3):
> 1. split a single patch to three part
> 2. fix memory leak and lock problems
> 3. update caps before invoking nodedev driver interfaces (refer to patch 3/3)
> nodedev: add macro guard to node_device_udev header file
> nodedev: update mdev_types caps before dumpxml
> nodedev: update caps before invoking nodedev driver interfaces
> src/node_device/node_device_driver.c | 55 +++++++++++++++++++++++++++++++
> src/node_device/node_device_linux_sysfs.c | 22 +++++++++++++
> src/node_device/node_device_udev.c | 37 +++++++++++++++++----
> src/node_device/node_device_udev.h | 18 +++++++---
> 4 files changed, 121 insertions(+), 11 deletions(-)
More information about the libvir-list