272461 |
03-Oct-2014 |
gjb |
Copy stable/10@r272459 to releng/10.1 as part of the 10.1-RELEASE process.
Approved by: re (implicit) Sponsored by: The FreeBSD Foundation
|
260817 |
17-Jan-2014 |
avg |
MFC r258622: dtrace sdt: remove the ugly sname parameter of SDT_PROBE_DEFINE
|
259980 |
28-Dec-2013 |
peter |
MFC r259109: allow ZFS to co-habitate with crypto / aesni
|
256281 |
10-Oct-2013 |
gjb |
Copy head (r256279) to stable/10 as part of the 10.0-RELEASE cycle.
Approved by: re (implicit) Sponsored by: The FreeBSD Foundation
|
254356 |
15-Aug-2013 |
glebius |
Make sendfile() a method in the struct fileops. Currently only vnode backed file descriptors have this method implemented.
Reviewed by: kib Sponsored by: Nginx, Inc. Sponsored by: Netflix
|
247061 |
20-Feb-2013 |
pjd |
When porting XTS-related code from OpenBSD I forgot to update copyright (only OpenBSD was credited in one of two commits). Fix it.
Reported by: Theo de Raadt <deraadt@cvs.openbsd.org> Reviewed by: Damien Miller <djm@mindrot.org>
|
241394 |
10-Oct-2012 |
kevlo |
Revert previous commit...
Pointyhat to: kevlo (myself)
|
241370 |
09-Oct-2012 |
kevlo |
Prefer NULL over 0 for pointers
|
224914 |
16-Aug-2011 |
kib |
Add the fo_chown and fo_chmod methods to struct fileops and use them to implement fchown(2) and fchmod(2) support for several file types that previously lacked it. Add MAC entries for chown/chmod done on posix shared memory and (old) in-kernel posix semaphores.
Based on the submission by: glebius Reviewed by: rwatson Approved by: re (bz)
|
220245 |
01-Apr-2011 |
kib |
After the r219999 is merged to stable/8, rename fallocf(9) to falloc(9) and remove the falloc() version that lacks flag argument. This is done to reduce the KPI bloat.
Requested by: jhb X-MFC-note: do not
|
219026 |
25-Feb-2011 |
vanhu |
fixed size of AH_ALEN_MAX, which is 64 bytes for SHA-512.
Obtained from: Matthias Drochner <M.Drochner@fz-juelich.de> MFC after: 3d
|
215295 |
14-Nov-2010 |
marius |
Let cryptosoft(4) add its pseudo-device with a specific unit number and its probe method return BUS_PROBE_NOWILDCARD so it doesn't get attached to real devices hanging off of nexus(4) with no specific devclass set. Actually, the more desirable fix for this would be to get rid of the newbus interface of cryptosoft(4) altogether but apparently crypto(9) was written with support for cryptographic hardware in mind so that approach would require some KPI breaking changes which don't seem worth it.
MFC after: 1 week
|
213068 |
23-Sep-2010 |
pjd |
Add support for AES-XTS.
Obtained from: OpenBSD MFC after: 1 week
|
213065 |
23-Sep-2010 |
pjd |
Remove redundant space.
MFC after: 1 week
|
211616 |
22-Aug-2010 |
rpaulo |
Add an extra comment to the SDT probes definition. This allows us to get use '-' in probe names, matching the probe names in Solaris.[1]
Add userland SDT probes definitions to sys/sdt.h.
Sponsored by: The FreeBSD Foundation Discussed with: rwaston [1]
|
211181 |
11-Aug-2010 |
jhb |
Fix typo in comment.
|
210631 |
29-Jul-2010 |
kib |
Add compat32 shims for opencrypto(4).
Reviewed by: bz MFC after: 3 weeks
|
208834 |
05-Jun-2010 |
kib |
Use the fpu_kern_enter() interface to properly separate usermode FPU context from in-kernel execution of padlock instructions and to handle spurious FPUDNA exceptions that sometime are raised when doing padlock calculations.
Globally mark crypto(9) kthread as using FPU.
Reviewed by: pjd Hardware provided by: Sentex Communications Tested by: pho PR: amd64/135014 MFC after: 1 month
|
201898 |
09-Jan-2010 |
bz |
Add comments trying to explain what bad things happen here, i.e. how hashed MD5/SHA are implemented, abusing Final() for padding and sw_octx to transport the key from the beginning to the end.
Enlightened about what was going on here by: cperciva Reviewed by: cperciva MFC After: 3 days X-MFC with: r187826 PR: kern/126468
|
199906 |
29-Nov-2009 |
bz |
In case the compression result is the same size as the orignal version, the compression was useless as well. Make sure to not update the data and return, else we would waste resources when decompressing.
This also avoids the copyback() changing data other consumers like xform_ipcomp.c would have ignored because of no win and sent out without noting that compression was used, resulting in invalid packets at the receiver.
MFC after: 5 days
|
199904 |
29-Nov-2009 |
bz |
Add SDT iter probes forgotten in r199885.
MFC after: 5 days
|
199895 |
28-Nov-2009 |
bz |
Change memory managment from a fixed size array to a list. This is needed to avoid running into out of buffer situations where we cannot alloc a new buffer because we hit the array size limit (ZBUF). Use a combined allocation for the struct and the actual data buffer to not increase the number of malloc calls. [1]
Defer initialization of zbuf until we actually need it.
Make sure the output buffer will be large enough in all cases.
Details discussed with: kib [1] Reviewed by: kib [1] MFC after: 6 days
|
199887 |
28-Nov-2009 |
bz |
Z_PARTIAL_FLUSH is marked deprecated. Z_SYNC_FLUSH is the suggested replacement but only use it for inflate. For deflate use Z_FINISH as Z_SYNC_FLUSH adds a trailing marker in some cases that inflate(), despite the comment in zlib, does npt seem to cope well with, resulting in errors when uncompressing exactly fills the outbut buffer without a Z_STREAM_END and a successive call returns an error.
MFC after: 6 days
|
199885 |
28-Nov-2009 |
bz |
Add SDT probes for opencrypto:deflate:deflate_gobal:*. They are not nice but they were helpful.
MFC after: 6 days
|
199884 |
28-Nov-2009 |
bz |
Define an SDT provider for "opencrypto".
MFC after: 6 days
|
196825 |
04-Sep-2009 |
pjd |
If crypto operation is finished with EAGAIN, don't repeat operation from the return context, but from the original context. Before repeating operation clear DONE flag and error.
Reviewed by: sam Obtained from: Wheel Sp. z o.o. (http://www.wheel.pl)
|
192636 |
23-May-2009 |
raj |
Fix cryptodev UIO creation.
Cryptodev uses UIO structure do get data from userspace and pass it to cryptographic engines. Initially UIO size is equal to size of data passed to engine, but if UIO is prepared for hash calculation an additional small space is created to hold result of operation.
While creating space for the result, UIO I/O vector size is correctly extended, but uio_resid field in UIO structure is not modified.
As bus_dma code uses uio_resid field to determine size of UIO DMA mapping, resulting mapping hasn't correct size. This leads to a crash if all the following conditions are met:
1. Hardware cryptographic accelerator writes result of hash operation using DMA. 2. Size of input data is less or equal than (n * PAGE_SIZE), 3. Size of input data plus size of hash result is grather than (n * PAGE_SIZE, where n is the same as in point 2.
This patch fixes this problem by adding size of the extenstion to uio_resid field in UIO structure.
Submitted by: Piotr Ziecik kosmo ! semihalf dot com Reviewed by: philip Obtained from: Semihalf
|
188154 |
05-Feb-2009 |
imp |
Fix return type for detach routine (should be int) Fix first parameter for identify routine (should be driver_t *)
|
187826 |
28-Jan-2009 |
bz |
While OpenBSD's crypto/ framework has sha1 and md5 implementations that can cope with a result buffer of NULL in the "Final" function, we cannot. Thus pass in a temporary buffer long enough for either md5 or sha1 results so that we do not panic.
PR: bin/126468 MFC after: 1 week
|
184477 |
30-Oct-2008 |
dfr |
Don't hang if encrypting/decrypting using struct iovecs where one of the iovecs ends on a crypto block boundary.
|
184214 |
23-Oct-2008 |
des |
Fix a number of style issues in the MALLOC / FREE commit. I've tried to be careful not to fix anything that was already broken; the NFSv4 code is particularly bad in this respect.
|
184205 |
23-Oct-2008 |
des |
Retire the MALLOC and FREE macros. They are an abomination unto style(9).
MFC after: 3 months
|
175140 |
07-Jan-2008 |
jhb |
Make ftruncate a 'struct file' operation rather than a vnode operation. This makes it possible to support ftruncate() on non-vnode file types in the future. - 'struct fileops' grows a 'fo_truncate' method to handle an ftruncate() on a given file descriptor. - ftruncate() moves to kern/sys_generic.c and now just fetches a file object and invokes fo_truncate(). - The vnode-specific portions of ftruncate() move to vn_truncate() in vfs_vnops.c which implements fo_truncate() for vnode file types. - Non-vnode file types return EINVAL in their fo_truncate() method.
Submitted by: rwatson
|
174988 |
30-Dec-2007 |
jeff |
Remove explicit locking of struct file. - Introduce a finit() which is used to initailize the fields of struct file in such a way that the ops vector is only valid after the data, type, and flags are valid. - Protect f_flag and f_count with atomic operations. - Remove the global list of all files and associated accounting. - Rewrite the unp garbage collection such that it no longer requires the global list of all files and instead uses a list of all unp sockets. - Mark sockets in the accept queue so we don't incorrectly gc them.
Tested by: kris, pho
|
172836 |
20-Oct-2007 |
julian |
Rename the kthread_xxx (e.g. kthread_create()) calls to kproc_xxx as they actually make whole processes. Thos makes way for us to add REAL kthread_create() and friends that actually make theads. it turns out that most of these calls actually end up being moved back to the thread version when it's added. but we need to make this cosmetic change first.
I'd LOVE to do this rename in 7.0 so that we can eventually MFC the new kthread_xxx() calls.
|
172474 |
08-Oct-2007 |
kib |
Deny attempt to malloc unbounded amount of the memory. Convert malloc()/bzero() to malloc(M_ZERO).
Obtained from: OpenBSD MFC after: 3 days Approved by: re (kensmith)
|
171238 |
05-Jul-2007 |
peter |
Quiet warnings. These do not appear to be actually used uninitialized, but gcc's optimizer isn't smart enough to see that. Pre-initializing seems harmless enough.
Approved by: re (rwatson)
|
169425 |
09-May-2007 |
gnn |
Integrate the Camellia Block Cipher. For more information see RFC 4132 and its bibliography.
Submitted by: Tomoyuki Okazaki <okazaki at kick dot gr dot jp> MFC after: 1 month
|
168355 |
04-Apr-2007 |
rwatson |
Replace custom file descriptor array sleep lock constructed using a mutex and flags with an sxlock. This leads to a significant and measurable performance improvement as a result of access to shared locking for frequent lookup operations, reduced general overhead, and reduced overhead in the event of contention. All of these are imported for threaded applications where simultaneous access to a shared file descriptor array occurs frequently. Kris has reported 2x-4x transaction rate improvements on 8-core MySQL benchmarks; smaller improvements can be expected for many workloads as a result of reduced overhead.
- Generally eliminate the distinction between "fast" and regular acquisisition of the filedesc lock; the plan is that they will now all be fast. Change all locking instances to either shared or exclusive locks.
- Correct a bug (pointed out by kib) in fdfree() where previously msleep() was called without the mutex held; sx_sleep() is now always called with the sxlock held exclusively.
- Universally hold the struct file lock over changes to struct file, rather than the filedesc lock or no lock. Always update the f_ops field last. A further memory barrier is required here in the future (discussed with jhb).
- Improve locking and reference management in linux_at(), which fails to properly acquire vnode references before using vnode pointers. Annotate improper use of vn_fullpath(), which will be replaced at a future date.
In fcntl(), we conservatively acquire an exclusive lock, even though in some cases a shared lock may be sufficient, which should be revisited. The dropping of the filedesc lock in fdgrowtable() is no longer required as the sxlock can be held over the sleep operation; we should consider removing that (pointed out by attilio).
Tested by: kris Discussed with: jhb, kris, attilio, jeff
|
167756 |
21-Mar-2007 |
sam |
add missing file from last commit that overhauls crypto/driver api's
|
167755 |
21-Mar-2007 |
sam |
Overhaul driver/subsystem api's: o make all crypto drivers have a device_t; pseudo drivers like the s/w crypto driver synthesize one o change the api between the crypto subsystem and drivers to use kobj; cryptodev_if.m defines this api o use the fact that all crypto drivers now have a device_t to add support for specifying which of several potential devices to use when doing crypto operations o add new ioctls that allow user apps to select a specific crypto device to use (previous ioctls maintained for compatibility) o overhaul crypto subsystem code to eliminate lots of cruft and hide implementation details from drivers o bring in numerous fixes from Michale Richardson/hifn; mostly for 795x parts o add an optional mechanism for mmap'ing the hifn 795x public key h/w to user space for use by openssl (not enabled by default) o update crypto test tools to use new ioctl's and add cmd line options to specify a device to use for tests
These changes will also enable much future work on improving the core crypto subsystem; including proper load balancing and interposing code between the core and drivers to dispatch small operations to the s/w driver as appropriate.
These changes were instigated by the work of Michael Richardson.
Reviewed by: pjd Approved by: re
|
159346 |
06-Jun-2006 |
pjd |
When DIAGNOSTIC is defined, verify if we don't free crypto requests from the crypto queue or from the return queue.
|
159242 |
04-Jun-2006 |
pjd |
Use newly added functions to simplify the code.
|
159241 |
04-Jun-2006 |
pjd |
- Replace COPYDATA() and COPYBACK() macros with crypto_copydata() and crypto_copyback() functions. - Add crypto_apply() function.
This will allow for more code simplification.
|
159240 |
04-Jun-2006 |
pjd |
Prefer hardware crypto over software crypto.
Before the change if a hardware crypto driver was loaded after the software crypto driver, calling crypto_newsession() with hard=0, will always choose software crypto.
|
159235 |
04-Jun-2006 |
pjd |
Use newly added defines instead of magic values.
|
159234 |
04-Jun-2006 |
pjd |
Move COPYDATA() and COPYBACK() macros to cryptodev.h, they will be used in padlock(4) as well.
|
159232 |
04-Jun-2006 |
pjd |
- Remove HMAC_BLOCK_LEN, it serves no purpose. - Use defines of used algorithm instead of HMAC_BLOCK_LEN.
|
159230 |
04-Jun-2006 |
pjd |
- Use define of an algorithm with the biggest block length to describe EALG_MAX_BLOCK_LEN instead of hardcoded value. - Kill an unused define.
|
159229 |
04-Jun-2006 |
pjd |
Rename HMAC_BLOCK_MAXLEN to HMAC_MAX_BLOCK_LEN to be consistent with EALG_MAX_BLOCK_LEN.
|
159228 |
04-Jun-2006 |
pjd |
Rename AALG_MAX_RESULT_LEN to HASH_MAX_LEN to look more constent with other defines.
|
159227 |
04-Jun-2006 |
pjd |
- Add defines with hash length for each hash algorithm. - Add defines with block length for each HMAC algorithm. - Add AES_BLOCK_LEN define which is an alias for RIJNDAEL128_BLOCK_LEN. - Add NULL_BLOCK_LEN define.
|
159223 |
04-Jun-2006 |
pjd |
Kill an unused argument.
|
158832 |
22-May-2006 |
pjd |
Remove (now unused) crp_mac field.
|
158831 |
22-May-2006 |
pjd |
Fix usage of HMAC algorithms via /dev/crypto.
|
158827 |
22-May-2006 |
pjd |
Improve the code responsible for waking up the crypto_proc thread. Checking if the queues are empty is not enough for the crypto_proc thread (it is enough for the crypto_ret_thread), because drivers can be marked as blocked. In a situation where we have operations related to different crypto drivers in the queue, it is possible that one driver is marked as blocked. In this case, the queue will not be empty and we won't wakeup the crypto_proc thread to execute operations for the others drivers.
Simply setting a global variable to 1 when we goes to sleep and setting it back to 0 when we wake up is sufficient. The variable is protected with the queue lock.
|
158826 |
22-May-2006 |
pjd |
Don't wakeup the crypto_ret_proc thread if it is running already. Before the change if the thread was working on symmetric operation, we would send unnecessary wakeup after adding asymmetric operation (when asym queue was empty) and vice versa.
|
158825 |
22-May-2006 |
pjd |
Don't set cc_kqblocked twice and don't increment cryptostats.cs_kblocks twice if we call crypto_kinvoke() from crypto_proc thread. This change also removes unprotected access to cc_kqblocked field (CRYPTO_Q_LOCK() should be used for protection).
|
158824 |
22-May-2006 |
pjd |
Document how we synchronize access to the fields in the cryptocap structure.
|
158823 |
22-May-2006 |
pjd |
We must synchronize access to cc_qblocked, because there could be a race where crypto_invoke() returns ERESTART and before we set cc_qblocked to 1, crypto_unblock() is called and sets it to 0. This way we mark device as blocked forever.
Fix it by not setting cc_qblocked in the fast path and by protecting crypto_invoke() in the crypto_proc thread with CRYPTO_Q_LOCK(). This won't slow things down, because there is no contention - we have only one crypto thread. Actually it can be slightly faster, because we save two atomic ops per crypto request. The fast code path remains lock-less.
|
158716 |
18-May-2006 |
pjd |
Silent Coverity Prevent report by asserting that cap != NULL.
Coverity ID: 1414
|
158703 |
17-May-2006 |
pjd |
- Fix a very old bug in HMAC/SHA{384,512}. When HMAC is using SHA384 or SHA512, the blocksize is 128 bytes, not 64 bytes as anywhere else. The bug also exists in NetBSD, OpenBSD and various other independed implementations I look at. - We cannot decide which hash function to use for HMAC based on the key length, because any HMAC function can use any key length. To fix it split CRYPTO_SHA2_HMAC into three algorithm: CRYPTO_SHA2_256_HMAC, CRYPTO_SHA2_384_HMAC and CRYPTO_SHA2_512_HMAC. Those names are consistent with OpenBSD's naming. - Remove authsize field from auth_hash structure. - Allow consumer to define size of hash he wants to receive. This allows to use HMAC not only for IPsec, where 96 bits MAC is requested. The size of requested MAC is defined at newsession time in the cri_mlen field - when 0, entire MAC will be returned. - Add swcr_authprepare() function which prepares authentication key. - Allow to provide key for every authentication operation, not only at newsession time by honoring CRD_F_KEY_EXPLICIT flag. - Make giving key at newsession time optional - don't try to operate on it if its NULL. - Extend COPYBACK()/COPYDATA() macros to handle CRYPTO_BUF_CONTIG buffer type as well. - Accept CRYPTO_BUF_IOV buffer type in swcr_authcompute() as we have cuio_apply() now. - 16 bits for key length (SW_klen) is more than enough.
Reviewed by: sam
|
158702 |
17-May-2006 |
pjd |
- Make opencrypto more SMP friendly by dropping the queue lock around crypto_invoke(). This allows to serve multiple crypto requests in parallel and not bached requests are served lock-less. Drivers should not depend on the queue lock beeing held around crypto_invoke() and if they do, that's an error in the driver - it should do its own synchronization. - Don't forget to wakeup the crypto thread when new requests is queued and only if both symmetric and asymmetric queues are empty. - Symmetric requests use sessions and there is no way driver can disappear when there is an active session, so we don't need to check this, but assert this. This is also safe to not use the driver lock in this case. - Assymetric requests don't use sessions, so don't check the driver in crypto_kinvoke(). - Protect assymetric operation with the driver lock, because if there is no symmetric session, driver can disappear. - Don't send assymetric request to the driver if it is marked as blocked. - Add an XXX comment, because I don't think migration to another driver is safe when there are pending requests using freed session. - Remove 'hint' argument from crypto_kinvoke(), as it serves no purpose. - Don't hold the driver lock around kprocess method call, instead use cc_koperations to track number of in-progress requests. - Cleanup register/unregister code a bit. - Other small simplifications and cleanups.
Reviewed by: sam
|
158701 |
17-May-2006 |
pjd |
Remove cri_rnd. It is not used.
Reviewed by: sam
|
158700 |
17-May-2006 |
pjd |
If kern.cryptodevallowsoft is TRUE allow also for symmetric software crypto in kernel. Useful for testing.
Reviewed by: sam
|
158699 |
17-May-2006 |
pjd |
Forgot about adding cuio_apply() here.
Reviewed by: sam
|
158698 |
17-May-2006 |
pjd |
- Implement cuio_apply(), an equivalent to m_apply(9). - Implement CUIO_SKIP() macro which is only responsible for skipping the given number of bytes from iovec list. This allows to avoid duplicating the same code in three functions.
Reviewed by: sam
|
157665 |
11-Apr-2006 |
pjd |
Be sure to wakeup the crypto thread when new request was queued. This should fix a hang when starting cryptokeytest (and more).
MFC after: 1 month
|
157637 |
10-Apr-2006 |
pjd |
- Simplify the code by using arc4rand(9) instead of arc4random(9) in a loop. - Correct a comment.
MFC after: 2 weeks
|
157205 |
28-Mar-2006 |
pjd |
Fix memory leak which occurs when crypto.ko module is unloaded.
Discussed with: sam MFC after 3 days
|
156281 |
04-Mar-2006 |
wkoszek |
crypto.ko depends on zlib.
Submitted by: Ben Kelly <bkelly at vadev.org> Approved by: rwatson Point hat to: me MFC after: 1 day
|
156072 |
27-Feb-2006 |
wkoszek |
This patch fixes a problem, which exists if you have IPSEC in your kernel and want to have crypto support loaded as KLD. By moving zlib to separate module and adding MODULE_DEPEND directives, it is possible to use such configuration without complication. Otherwise, since IPSEC is linked with zlib (just like crypto.ko) you'll get following error:
interface zlib.1 already present in the KLD 'kernel'!
Approved by: cognet (mentor)
|
149228 |
18-Aug-2005 |
pjd |
Fix bogus check. It was possible to panic the kernel by giving 0 length. This is actually a local DoS, as every user can use /dev/crypto if there is crypto hardware in the system and cryptodev.ko is loaded (or compiled into the kernel).
Reported by: Mike Tancsa <mike@sentex.net> MFC after: 1 day
|
149143 |
16-Aug-2005 |
pjd |
Check key size for rijndael, as invalid key size can lead to kernel panic. It checked other algorithms against this bug and it seems they aren't affected.
Reported by: Mike Tancsa <mike@sentex.net> PR: i386/84860 Reviewed by: phk, cperciva(x2)
|
146797 |
30-May-2005 |
scottl |
malloc.h relies on param.h for a definition of MAXCPU. I guess that there is other header pollution that makes this work right now, but it falls over when doing a RELENG_5 -> HEAD upgrade.
|
143423 |
11-Mar-2005 |
ume |
just use crypto/rijndael, and nuke opencrypto/rindael.[ch]. the two became almost identical since latest KAME merge.
Discussed with: sam
|
143408 |
11-Mar-2005 |
ume |
- use 1/2 space for rijndael context in ipsec - rijndael_set_key() always sets up full context - rijndaelKeySetupDec() gets back original protoype
Reviewed by: sam Obtained from: OpenBSD
|
143406 |
11-Mar-2005 |
ume |
refer opencrypto/cast.h directly.
|
142715 |
27-Feb-2005 |
phk |
Use dynamic major number allocation.
|
139825 |
07-Jan-2005 |
imp |
/* -> /*- for license, minor formatting changes
|
137806 |
17-Nov-2004 |
phk |
Push Giant down through ioctl.
Don't grab Giant in the upper syscall/wrapper code
NET_LOCK_GIANT in the socket code (sockets/fifos).
mtx_lock(&Giant) in the vnode code.
mtx_lock(&Giant) in the opencrypto code. (This may actually not be needed, but better safe than sorry).
Devfs grabs Giant if the driver is marked as needing Giant.
|
133423 |
10-Aug-2004 |
rwatson |
Don't acquire Giant in cryptof_close(), as the code is intended to be able to run MPsafe (and appears to be MPsafe).
Discussed with (some time ago): sam
|
132549 |
22-Jul-2004 |
rwatson |
Push acquisition of Giant from fdrop_closed() into fo_close() so that individual file object implementations can optionally acquire Giant if they require it:
- soo_close(): depends on debug.mpsafenet - pipe_close(): Giant not acquired - kqueue_close(): Giant required - vn_close(): Giant required - cryptof_close(): Giant required (conservative)
Notes:
Giant is still acquired in close() even when closing MPSAFE objects due to kqueue requiring Giant in the calling closef() code. Microbenchmarks indicate that this removal of Giant cuts 3%-3% off of pipe create/destroy pairs from user space with SMP compiled into the kernel.
The cryptodev and opencrypto code appears MPSAFE, but I'm unable to test it extensively and so have left Giant over fo_close(). It can probably be removed given some testing and review.
|
130585 |
16-Jun-2004 |
phk |
Do the dreaded s/dev_t/struct cdev */ Bump __FreeBSD_version accordingly.
|
129880 |
30-May-2004 |
phk |
add missing #include <sys/module.h>
|
126674 |
05-Mar-2004 |
jhb |
kthread_exit() no longer requires Giant, so don't force callers to acquire Giant just to call kthread_exit().
Requested by: many
|
126080 |
21-Feb-2004 |
phk |
Device megapatch 4/6:
Introduce d_version field in struct cdevsw, this must always be initialized to D_VERSION.
Flip sense of D_NOGIANT flag to D_NEEDGIANT, this involves removing four D_NOGIANT flags and adding 145 D_NEEDGIANT flags.
|
125424 |
04-Feb-2004 |
phk |
Do not aggressively unroll the AES implementation, in non-benchmarking use it is same speed on small cache cpus and slower on largecache cpus.
Approved by: sam@
|
125330 |
02-Feb-2004 |
phk |
Add CRD_F_KEY_EXPLICIT which allows the key to be changed per operation, just like it was possible to change the IV.
Currently supported on Hifn and software engines only.
Approved by: sam@
|
123564 |
16-Dec-2003 |
bms |
style(9) pass and type fixups.
Submitted by: bde
|
123558 |
15-Dec-2003 |
bms |
Purge crmbuf.c as the routines are now in uipc_mbuf.c.
Reviewed by: sam Sponsored by: spc.org
|
123557 |
15-Dec-2003 |
bms |
Push m_apply() and m_getptr() up into the colleciton of standard mbuf routines, and purge them from opencrypto.
Reviewed by: sam Obtained from: NetBSD Sponsored by: spc.org
|
122908 |
19-Nov-2003 |
sam |
eliminate an unnecessary 8Kbyte bzero that was being done for each submitted operation
Submitted by: Thor Lancelot Simon Reviewed by: jhb Approved by: re (jhb)
|
121256 |
19-Oct-2003 |
dwmalone |
falloc allocates a file structure and adds it to the file descriptor table, acquiring the necessary locks as it works. It usually returns two references to the new descriptor: one in the descriptor table and one via a pointer argument.
As falloc releases the FILEDESC lock before returning, there is a potential for a process to close the reference in the file descriptor table before falloc's caller gets to use the file. I don't think this can happen in practice at the moment, because Giant indirectly protects closes.
To stop the file being completly closed in this situation, this change makes falloc set the refcount to two when both references are returned. This makes life easier for several of falloc's callers, because the first thing they previously did was grab an extra reference on the file.
Reviewed by: iedowse Idea run past: jhb
|
120506 |
27-Sep-2003 |
phk |
The present defaults for the open and close for device drivers which provide no methods does not make any sense, and is not used by any driver.
It is a pretty hard to come up with even a theoretical concept of a device driver which would always fail open and close with ENODEV.
Change the defaults to be nullopen() and nullclose() which simply does nothing.
Remove explicit initializations to these from the drivers which already used them.
|
117058 |
30-Jun-2003 |
sam |
consolidate callback optimization check in one location by adding a flag for crypto operations that indicates the crypto code should do the check in crypto_done
MFC after: 1 day
|
116924 |
27-Jun-2003 |
sam |
Add support to eliminate a context switch per crypto op when using the software crypto device:
o record crypto device capabilities in each session id o add a capability that indicates if the crypto driver operates synchronously o tag the software crypto driver as operating synchronously
This commit also introduces crypto session id macros that cleanup their construction and querying.
|
116546 |
18-Jun-2003 |
phk |
Initialize struct fileops with C99 sparse initialization.
|
116191 |
11-Jun-2003 |
obrien |
Use __FBSDID().
|
115746 |
02-Jun-2003 |
sam |
Flush my local cache of cryto subsystem fixes:
o add a ``done'' flag for crypto operations; this is set when the operation completes and is intended for callers to check operations that may complete ``prematurely'' because of direct callbacks o close a race for operations where the crypto driver returns ERESTART: we need to hold the q lock to insure the blocked state for the driver and any driver-private state is consistent; otherwise drivers may take an interrupt and notify the crypto subsystem that it can unblock the driver but operations will be left queued and never be processed o close a race in /dev/crypto where operations can complete before the caller can sleep waiting for the callback: use a per-session mutex and the new done flag to handle this o correct crypto_dispatch's handling of operations where the driver returns ERESTART: the return value must be zero and not ERESTART, otherwise the caller may free the crypto request despite it being queued for later handling (this typically results in a later panic) o change crypto mutex ``names'' so witness printouts and the like are more meaningful
|
111815 |
03-Mar-2003 |
phk |
Gigacommit to improve device-driver source compatibility between branches:
Initialize struct cdevsw using C99 sparse initializtion and remove all initializations to default values.
This patch is automatically generated and has been tested by compiling LINT with all the fields in struct cdevsw in reverse order on alpha, sparc64 and i386.
Approved by: re(scottl)
|
111297 |
23-Feb-2003 |
sam |
o add a CRYPTO_F_CBIMM flag to symmetric ops to indicate the callback should be done in crypto_done rather than in the callback thread o use this flag to mark operations from /dev/crypto since the callback routine just does a wakeup; this eliminates the last unneeded ctx switch o change CRYPTO_F_NODELAY to CRYPTO_F_BATCH with an inverted meaning so "0" becomes the default/desired setting (needed for user-mode compatibility with openbsd) o change crypto_dispatch to honor CRYPTO_F_BATCH instead of always dispatching immediately o remove uses of CRYPTO_F_NODELAY o define COP_F_BATCH for ops submitted through /dev/crypto and pass this on to the op that is submitted
Similar changes and more eventually coming for asymmetric ops.
MFC if re gives approval.
|
111119 |
19-Feb-2003 |
imp |
Back out M_* changes, per decision of the TRB.
Approved by: trb
|
109623 |
21-Jan-2003 |
alfred |
Remove M_TRYWAIT/M_WAITOK/M_WAIT. Callers should use 0. Merge M_NOWAIT/M_DONTWAIT into a single flag M_NOWAIT.
|
109153 |
13-Jan-2003 |
dillon |
Bow to the whining masses and change a union back into void *. Retain removal of unnecessary casts and throw in some minor cleanups to see if anyone complains, just for the hell of it.
|
109123 |
12-Jan-2003 |
dillon |
Change struct file f_data to un_data, a union of the correct struct pointer types, and remove a huge number of casts from code using it.
Change struct xfile xf_data to xun_data (ABI is still compatible).
If we need to add a #define for f_data and xf_data we can, but I don't think it will be necessary. There are no operational changes in this commit.
|
108990 |
09-Jan-2003 |
sam |
Invoke the driver directly when a request is submitted via crypto_dispatch or crypto_kdispatch unless the driver is currently blocked. This eliminates the context switch to the dispatch thread for virtually all requests.
Note that this change means that for software crypto drivers the caller will now block until the request is completed and the callback is dispatched to the callback thread (h/w drivers will typically just dispatch the op to the device and return quickly). If this is an issue we can either implement a non-blocking interface in the s/w crypto driver or use either the "no delay" flag in the crypto request or the "software driver" capability flag to control what to do.
Sponsored by: Vernier Networks
|
108813 |
06-Jan-2003 |
sam |
correct flags passed when allocation crypto request structures so we don't block and the returned data is zero'd
|
108588 |
03-Jan-2003 |
sam |
manage kernel threads properly; especially shutting them down on module unload
Reviewed by: jhb
|
108587 |
03-Jan-2003 |
sam |
MFS: crypto timing support; purge usercrypto sysctl (just don't config cryptodev or kldunload cryptodev module); crypto statistcs; remove unused alloctype field from crypto op to offset addition of the performance time stamp
Supported by: Vernier Networks
|
106677 |
08-Nov-2002 |
sam |
correct minor # in make_dev call
Submitted by: Doug Ambrisko" <ambrisko@verniernetworks.com>
|
106676 |
08-Nov-2002 |
sam |
change load order so module is present before crypto drivers
Submitted by: Doug Ambrisko" <ambrisko@verniernetworks.com>
|
105251 |
16-Oct-2002 |
markm |
Module-ize the 'core' crypto stuff. This may still need to be compiled into the kernel by default (if required), but other modules can now depend() on this.
Fix inter-module dependancy.
Earlier version OK'ed by: sam
|
104908 |
11-Oct-2002 |
mike |
Change iov_base's type from `char *' to the standard `void *'. All uses of iov_base which assume its type is `char *' (in order to do pointer arithmetic) have been updated to cast iov_base to `char *'.
|
104628 |
07-Oct-2002 |
sam |
o split crypto_proc into two threads: one for processing requests and one for processing callbacks. This closes race conditions caused by locking too many things with a single mutex. o reclaim crypto requests under certain (impossible) failure conditions
|
104627 |
07-Oct-2002 |
sam |
remove CIOGSSESSION (get software session); it was added only for testing
|
104476 |
04-Oct-2002 |
sam |
In-kernel crypto framework derived from openbsd. This facility provides a consistent interface to h/w and s/w crypto algorithms for use by the kernel and (for h/w at least) by user-mode apps. Access for user-level code is through a /dev/crypto device that'll eventually be used by openssl to (potentially) accelerate many applications. Coming soon is an IPsec that makes use of this service to accelerate ESP, AH, and IPCOMP protocols.
Included here is the "core" crypto support, /dev/crypto driver, various crypto algorithms that are not already present in the KAME crypto area, and support routines used by crypto device drivers.
Obtained from: openbsd
|