Skip to content

Read DDF and IMSM members by their level (1.71) - #95

Merged
abrignoni merged 1 commit into
mainfrom
fix/foreign-raid-members
Oct 17, 2026
Merged

abrignoni merged 1 commit into
mainfrom
fix/foreign-raid-members

Conversation

@abrignoni

Copy link
Copy Markdown
Owner

Reads the RAID level of DDF and Intel RST members so only a healthy mirror member is read alone.

  • RAID1 in its normal state: read on its own, noted as a copy of the mirror.
  • Any other level, state or bad metadata: the disk and everything on it is listed, not walked.
  • Spanned LVM groups are matched by id, so two groups with the same name stay apart.
  • No IMSM image written by a real controller was available; IMSM is tested on metadata the self-test builds.
  • Output matches 1.70 except the DDF fixtures' notes.

🤖 Generated with Claude Code

… id (1.71)

1.70 named any disk with DDF or Intel IMSM (RST) metadata a RAID member and
walked nothing on it. 1.71 reads the level from the metadata, checked as
mdadm-4.5 reads it (DDF CRCs and header choice, IMSM checksum, RAID10 as
DDF and IMSM encode it, virtual and physical disk state, map state):
- a RAID1 in its normal state: the disk is read on its own as a copy of the
  mirror, with a note that its array is not assembled;
- any other level, state, or metadata that does not check out: the disk and
  everything found on it (partitions, LVM, LUKS, BitLocker, qnx6) is listed
  and not walked, with the reason. qnxprobe 1.67 walked a DDF RAID5 member
  and listed a 3,096.3 PiB file.
An IMSM disk read alone is now named, and the RAID section uses "an Intel".

Volume groups spread over several images are matched by VG id, not name, so
two groups sharing a default name are never read through each other's disks.

Fixed after independent review. Output is unchanged from 1.70 except on the
DDF fixtures' notes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@abrignoni
abrignoni merged commit dfe51bd into main Oct 17, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant