On Tue, Jun 22, 2021 at 10:03 AM Boris Fiuczynski <fiuczy@xxxxxxxxxxxxx> 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
