Page MenuHomeFreeBSD
Differential D9529

Output file name from "preload" type md(4) device
ClosedPublic
Actions

Authored by stevek on Feb 10 2017, 5:22 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 1, 2:48 PM
Unknown Object (File)
Sun, Dec 1, 2:48 PM
Unknown Object (File)
Sun, Dec 1, 2:47 PM
Unknown Object (File)
Sun, Dec 1, 2:32 PM
Unknown Object (File)
Nov 13 2024, 10:45 AM
Unknown Object (File)
Oct 3 2024, 9:27 PM
Unknown Object (File)
Oct 1 2024, 8:16 PM
Unknown Object (File)
Sep 29 2024, 12:54 AM
View All 63 Files
Subscribers
imp

Details

Reviewers
brooks
Commits
rS313701: For MD_PRELOAD type md(4) devices, if there is a file name in the preloaded
Summary

For MD_PRELOAD type md(4) devices, if there is a file name in the preloaded
meta-data, copy it into the softc structure.

When returning md(4) device details to the caller, include the file name in
any MD_PRELOAD type devices if it is set (first character is not NUL.)

In mdconfig, for "preload" type md(4) devices, if there is file config
available, print it in the file column of the output.

Test Plan

Built and tested on x86 and ARM by pre-loading ISO images from the loader and
checking mdconfig output. Also created other md types to ensure output from
mdconfig looks correct.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

stevek updated this revision to Diff 24970.Feb 10 2017, 5:22 PM
stevek retitled this revision from to Output file name from "preload" type md(4) device.
stevek updated this object.
stevek edited the test plan for this revision. (Show Details)
stevek added a reviewer: brooks.
Herald added a subscriber: imp. · View Herald TranscriptFeb 10 2017, 5:22 PM
brooks accepted this revision.Feb 10 2017, 5:34 PM
brooks edited edge metadata.

Looks good to me.

sys/dev/md/md.c
1694 ↗(On Diff #24970)

Not new in your code, but this case is awful. It does does a copyout to the buffer and just hopes it's at least PATH_MAX.

This revision is now accepted and ready to land.Feb 10 2017, 5:34 PM
stevek added inline comments.Feb 13 2017, 5:41 PM
sys/dev/md/md.c
1694 ↗(On Diff #24970)

yes, the problem is the md_ioctl structure does not have a size for the md_file member and it really should.

Closed by commit rS313701: For MD_PRELOAD type md(4) devices, if there is a file name in the preloaded (authored by stevek). · Explain WhyFeb 13 2017, 5:44 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sbin/
mdconfig/
3 lines
sys/
dev/
md/
11 lines

Diff 25102

View Options

head/sbin/mdconfig/mdconfig.c

Loading...
View Options

head/sys/dev/md/md.c

Loading...