On Tue, Mar 29 2011 at 9:42am -0400, Martin K. Petersen <martin.petersen@xxxxxxxxxx> wrote: > >>>>> "Mike" == Mike Snitzer <snitzer@xxxxxxxxxx> writes: > > Mike, > > Mike> But I think we have a related issue that needs discussion, given > Mike> that an integrity profile mismatch will cause MD's assemble to > Mike> fail (rather than warn and continue to assemble without integrity > Mike> support). > > Mike> DM doesn't fail to load a DM device due to a integrity profile > Mike> mismatch; it just emits a warning and continues. > > Mike> In contrast, MD will now disallow adding a normal disk (without > Mike> integrity support) to an array that has historically had a > Mike> symmetric integrity profile across all members. > > You would invalidate all your existing integrity metadata, tagging, > etc. on existing metadevice members. That seems to be a policy decision, > so if we go down that path it would have to be keyed off a force > assembly option passed down from userland tooling. Turning off features > and/or losing metadata really should not be done without the user's > explicit consent. > > Also, let's assume you run an integrity-aware app on a DM device and you > add a non-integrity drive. The DM device is then no longer capable of > carrying integrity metadata out to storage. What happens to the app? > What about outstanding writes with metadata attached? > > Good discussion topic for next week, methinks... Sure, I'm just trying to reconcile the difference in behavior between MD and DM. Seems DM missed out on the integrity profile error propagation treatment that MD just received. Easily resolved (though there are some dragons lying in wait relative to where this inconsistency is detected by DM, hint: must be during DM table load). Current hook, dm_table_set_integrity, disables the integrity support of the DM device well beyond the point of no return (when resuming a DM device that was allowed to have a new table loaded). I welcome discussing this in a bit more detail in the hallway track at LSF ;) Thanks, Mike -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
