Page MenuHomeFreeBSD

Make sys/geom/eli more easily reusable in userspace
ClosedPublic

Authored by allanjude on Dec 24 2015, 7:48 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 26 2024, 8:38 AM
Unknown Object (File)
Feb 25 2024, 1:58 PM
Unknown Object (File)
Jan 27 2024, 3:40 PM
Unknown Object (File)
Jan 6 2024, 1:39 PM
Unknown Object (File)
Dec 20 2023, 12:17 AM
Unknown Object (File)
Dec 5 2023, 3:40 AM
Unknown Object (File)
Dec 1 2023, 2:08 AM
Unknown Object (File)
Nov 29 2023, 9:34 AM
Subscribers

Details

Summary
  • Break the HMAC parts of geli out into a separate file, so they can be used without requiring openssl or the entire crypto framework
  • some ifdef _KERNEL's to make remaining stuff work in userland
  • Need a better resolution for struct g_eli_softc

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1814
Build 1821: arc lint + arc unit

Event Timeline

allanjude retitled this revision from to Make sys/geom/eli more easily reusable in userspace.
allanjude updated this object.
allanjude edited the test plan for this revision. (Show Details)

Add change to sys/conf/files required for when GEOM_ELI is compiled into the kernel (found with LINT kernels during universe build)

delphij requested changes to this revision.Dec 27 2015, 7:59 AM
delphij added a reviewer: delphij.
delphij added a subscriber: delphij.

Thanks very much for working on this! It looks good overall except the softc definition (I think it would be better if we just ifdef the software crypto portion out, this way we don't have two places to define the same structure, making it easier for future maintenance).

sys/geom/eli/g_eli.h
211–239

Usually ! _KERNEL for consistency.

213

Can the software portion be #ifdef'ed instead?

This revision now requires changes to proceed.Dec 27 2015, 7:59 AM
allanjude added inline comments.
sys/geom/eli/g_eli.h
213

not sure why I didn't do it this way originally. At first I was cutting too much stuff out, like struct mtx, and RB_HEAD, to avoid those includes.

allanjude edited edge metadata.
allanjude marked an inline comment as done.

do struct g_eli_softc properly

bapt added a reviewer: bapt.
delphij edited edge metadata.

Looks good to me now, thanks!

This revision is now accepted and ready to land.Dec 27 2015, 5:16 PM

It looks good overall, but please be sure you run GELI regression tests. Thanks!

sys/geom/eli/g_eli.h
178

Maybe we could just use 'bool' now?

allanjude edited edge metadata.

Add missing reference to g_eli_hmac.c to /lib/geom/geom_eli.so (sbin/geom/class/eli)

This revision now requires review to proceed.Dec 31 2015, 8:06 PM

With that fix, it now passed the geom_eli regression suite

Files=20 Tests=14996
Result: PASS

allanjude edited edge metadata.

Instead of typedef'ing boolean_t in the !_KERNEL case, just use 'int' in that case

This revision was automatically updated to reflect the committed changes.