Page MenuHomeFreeBSD

Add support for AES modes to IPSec. This patch is as yet incomplete but this is the place where we're starting the discussion.
ClosedPublic

Authored by gnn on Jun 28 2015, 10:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 26, 6:38 AM
Unknown Object (File)
Fri, Apr 26, 6:23 AM
Unknown Object (File)
Feb 24 2024, 6:25 AM
Unknown Object (File)
Jan 6 2024, 8:49 AM
Unknown Object (File)
Dec 20 2023, 1:33 AM
Unknown Object (File)
Nov 25 2023, 10:43 AM
Unknown Object (File)
Nov 25 2023, 8:35 AM
Unknown Object (File)
Nov 23 2023, 2:36 PM
Subscribers
None

Diff Detail

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

Event Timeline

gnn retitled this revision from to Add support for AES modes to IPSec. This patch is as yet incomplete but this is the place where we're starting the discussion..
gnn added a reviewer: jmg.

The only issue i have with jmg@ locking patch is that it provides consistency for userland threads but i still do not see how it solves the panic due to FPU on a migrated thread!

So i would split this patch further more into the constants changes done to crypto code for introducing the new variable and the ipsec changes without the aesni hooks.

This would allow the changes to be even more review-able/acceptable by others, if needed.

In D2936#57341, @eri wrote:

The only issue i have with jmg@ locking patch is that it provides consistency for userland threads but i still do not see how it solves the panic due to FPU on a migrated thread!

Did my reply to you the other day answer your questions about this? That even if the thread migrates to another CPU, it still holds the original lock for the original CPU's fpu context, and if another thread runs on the original CPU, it cannot lock the mutex, and therefor will not have an issue w/ fpu reuse.

gnn, I need to rev my locking patch after more comments from kib. Once I get my git repo organized, I'll post a formal review for the locking patch.

So i would split this patch further more into the constants changes done to crypto code for introducing the new variable and the ipsec changes without the aesni hooks.

I agree w/ this. Also, some of the userland ipsec stuff can (and should) be committed w/o the kernel stuff.

This would allow the changes to be even more review-able/acceptable by others, if needed.

I'll commit the user space stuff now (setkey and associated files) and wait on your locking patch for the rest.