Bart,
>> + if (mode == SD_LBP_DEFAULT && !sdkp->provisioning_override) {
>
> Hmm ... is provisioning_override ever true for the SD_LBP_DEFAULT
> mode? If not, can "&& !sdkp->provisioning_override" be left out?
The two *_override variables are used to prevent subsequent revalidates
from clobbering the mode configured by the user.
I experimented with various approaches for this, including having a
separate SD_LBP_ mode for revalidate, using first_scan, etc. In the end
I felt that the boolean was the best approach to capturing the fact that
the currently active mode was explicitly configured by the user.
Open to suggestions.
--
Martin K. Petersen Oracle Linux Engineering
