Re: [libvirt PATCH 6/7] nodedev: Implement virNodeDeviceIsPersistent()/IsActive()
On Tue, Jun 22, 2021 at 10:03 AM Boris Fiuczynski <fiuczy(a)linux.ibm.com> wrote:
On 6/22/21 4:33 PM, Jonathon Jongsma wrote:
> So it appears that there is a bug where an mdev is still marked as
> autostart even after it's undefined. Was there anything else you were
> trying to demonstrate?
>
> Jonathon
Don't you need to resync with mdevctl on nodedev-info?
If you would resync and fail to find the definition for the mdev your
bugfix would be to set the autostart to "no". The shortcut would be to
set autostart to "no" when undefine is called but that would leave out
direct usage of mdectl.
Yes, it appears that there's a bug in my patch that needs to be fixed.
But your email said
"The default "yes" seems to be not really a good
choice even for mdevs.
That makes it sound like you think there's something more
fundamentally wrong than just a simple bug where a value didn't get
updated properly.
Jonathon