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

1 files changed, 0 insertions, 7 deletions

diff --git a/sys/opencrypto/cryptodev.c b/sys/opencrypto/cryptodev.c
index 26b2447..9b0da7c 100644
--- a/sys/opencrypto/cryptodev.c
+++ b/sys/opencrypto/cryptodev.c
@@ -623,19 +623,12 @@ cryptof_close(struct file *fp, struct thread *td)
 	struct fcrypt *fcr = fp->f_data;
 	struct csession *cse;
 
-	/*
-	 * XXXRW: The cryptodev and called code all appears to be
-	 * MPSAFE, but I'm not set up to test it.  Acquire Giant
-	 * for now.
-	 */
-	mtx_lock(&Giant);
 	while ((cse = TAILQ_FIRST(&fcr->csessions))) {
 		TAILQ_REMOVE(&fcr->csessions, cse, next);
 		(void)csefree(cse);
 	}
 	FREE(fcr, M_XDATA);
 	fp->f_data = NULL;
-	mtx_unlock(&Giant);
 	return 0;
 }