Differential D23400

geli: add a test case for attaching multiple providers with 1 command
ClosedPublic
Actions

Authored by asomers on Jan 28 2020, 5:06 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Mar 11, 12:33 PM

	Unknown Object (File)
	Sat, Mar 7, 6:53 PM

	Unknown Object (File)
	Sat, Mar 7, 6:29 PM

	Unknown Object (File)
	Sat, Mar 7, 10:38 AM

	Unknown Object (File)
	Sat, Mar 7, 12:55 AM

	Unknown Object (File)
	Feb 8 2026, 3:04 PM

	Unknown Object (File)
	Feb 8 2026, 9:31 AM

	Unknown Object (File)
	Jan 31 2026, 2:09 PM

View All 100 Files

Subscribers

Details

Reviewers

Commits

rS357880: MFC r357211:
rS357211: geli: add a test case for attaching multiple providers with 1 command

Summary

geli: add a test case for attaching multiple providers with 1 command

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 28991
Build 26965: arc lint + arc unit

Event Timeline

asomers created this revision.Jan 28 2020, 5:06 PM

Herald added a subscriber: imp. · View Herald TranscriptJan 28 2020, 5:06 PM

Harbormaster completed remote builds in B28991: Diff 67423.Jan 28 2020, 5:06 PM

LGTM. 8kB is overkill for the key, but mostly harmless.

tests/sys/geom/class/eli/attach_test.sh
85	I don't know what geli does with it, but 32 bytes (256 bits) is completely sufficient as a key. I checked and geli doesn't reject 256 bit keys. Unfortunately, it does accept keys smaller than 256 bits. It also accepts keys smaller than 128 bits :-(. This is probably a bug we should fix (orthogonal to this test). I filed PR 243676.

This revision is now accepted and ready to land.Jan 28 2020, 5:25 PM

Closed by commit rS357211: geli: add a test case for attaching multiple providers with 1 command (authored by asomers). · Explain WhyJan 28 2020, 6:32 PM

This revision was automatically updated to reflect the committed changes.

asomers added a commit: rS357211: geli: add a test case for attaching multiple providers with 1 command.

asomers added a commit: rS357880: MFC r357211:.Feb 13 2020, 8:46 PM

Revision Contents
Changeset List

Path

Size

tests/

sys/

geom/

class/

eli/

31 lines

Diff 67423

tests/sys/geom/class/eli/attach_test.sh

Loading...