PR/60281 CVS commit: src/sys/opencrypto

["Taylor R Campbell" <riastradh%netbsd.org@localhost>]

The following reply was made to PR kern/60281; it has been noted by GNATS.

From: "Taylor R Campbell" <riastradh%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/60281 CVS commit: src/sys/opencrypto
Date: Tue, 19 May 2026 19:00:01 +0000

 Module Name:	src
 Committed By:	riastradh
 Date:		Tue May 19 19:00:01 UTC 2026
 
 Modified Files:
 	src/sys/opencrypto: cryptodev.c cryptodev_internal.h ocryptodev.c
 
 Log Message:
 crypto(4): Nix spurious mutex_exit; add missing bounds checks.
 
 Consistently use `foo = kmem_alloc(n * sizeof(*foo), ...)' instead of
 `sizeof(struct whatever_foo_is)'.  Makes it easier for a reader to
 notice a discrepancy this way.
 
 Move CRYPTODEV_OPS_MAX to cryptodev_internal.h so it can be used by
 the compat ocryptodev.c shims too.  I think this is waaaaaaaaaaaaay
 too high, by the way.  For example, it looks like qat(4) puts a limit
 of 16384 on the number of sessions.  Other devices like hifn(4) look
 like they're limited to numbers of sessions ranging from 2 to around
 256.
 
 PR kern/60281: crypto(4): bugs in reference counting and test
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.132 -r1.133 src/sys/opencrypto/cryptodev.c
 cvs rdiff -u -r1.4 -r1.5 src/sys/opencrypto/cryptodev_internal.h
 cvs rdiff -u -r1.18 -r1.19 src/sys/opencrypto/ocryptodev.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.