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

	F103425182: D23400.id67423.diff
	Sun, Nov 24, 8:53 PM

	Unknown Object (File)
	Mon, Nov 11, 12:57 PM

	Unknown Object (File)
	Sat, Nov 2, 4:55 AM

	Unknown Object (File)
	Sep 30 2024, 5:28 PM

	Unknown Object (File)
	Sep 16 2024, 4:05 PM

	Unknown Object (File)
	Sep 7 2024, 6:19 PM

	Unknown Object (File)
	Sep 7 2024, 7:14 AM

	Unknown Object (File)
	Aug 31 2024, 4:35 AM

View All 38 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 Not Applicable

Unit

Tests Not Applicable

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 ↗	(On Diff #67423)	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

head/

tests/

sys/

geom/

class/

eli/

31 lines

Diff 67428

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

Loading...