[libvirt] [PATCH 3/3] nodedev: update caps before invoking nodedev driver interfaces

Erik Skultety eskultet at redhat.com
Thu Jan 11 10:15:30 UTC 2018


On Wed, Jan 10, 2018 at 08:14:51PM +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.
> So, it is necessary to manually update devices' capabilities each time
> before nodedev driver interfaces invoked.
>
> Signed-off-by: Wu Zongyong <cordius.wu at huawei.com>
> ---

Thank you for posting the patch, since I hadn't noticed the problem with other
APIs until I read it.
It was a sad realization that a driver change is not reflected by a udev/kernel
CHANGE event, that would make things much much simpler. I have an idea to either
make this patch shorter or not needed at all though, we'll see when I finish my
patch (it's a long-needed refactor). Despite I haven't found any major flaws in
this patch, let's just put it on hold for a while until I finish my
investigation/work on my patch and see we can really do better here.

Thanks,
Erik




More information about the libvir-list mailing list