Hi Neil, these patches break the unit test 10ddf-geometry. I saw the regression with both patches applied. The problem occurs when subarrays are deleted. With these patches in place, sync_metadata() will not overwrite deleted conf records on disk, but lseek() over them instead. When the meta data is read back, this will cause errors. I would like to ask you to play safe here and revert these patches. It might be possible to fix the --kill-subarray problem, but there are other possible scenarios where the number of valid conf records on a disk decreases - I don't think we have a reliable way to check whether it is safe to skip over empty entries. We must also be prepared for other DDF implementations to read our meta data, so we must refrain from putting any writing anything that might be confusing. Regards Martin -- To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
