From 1be547932b035d1775c66c5f17b6a67174c5e575 Mon Sep 17 00:00:00 2001 From: pjd Date: Sat, 9 Aug 2008 19:42:37 +0000 Subject: Sessions in-use are at the end of the queue, so use TAILQ_FOREACH_REVERSE() when looking for them. Idea from: Patrick Lamaiziere --- sys/crypto/via/padlock.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'sys/crypto') diff --git a/sys/crypto/via/padlock.c b/sys/crypto/via/padlock.c index 9ac41b2..9818f3e 100644 --- a/sys/crypto/via/padlock.c +++ b/sys/crypto/via/padlock.c @@ -59,7 +59,7 @@ __FBSDID("$FreeBSD$"); struct padlock_softc { int32_t sc_cid; uint32_t sc_sid; - TAILQ_HEAD(, padlock_session) sc_sessions; + TAILQ_HEAD(padlock_sessions_head, padlock_session) sc_sessions; struct rwlock sc_sessions_lock; }; @@ -259,7 +259,8 @@ padlock_freesession(device_t dev, uint64_t tid) uint32_t sid = ((uint32_t)tid) & 0xffffffff; rw_wlock(&sc->sc_sessions_lock); - TAILQ_FOREACH(ses, &sc->sc_sessions, ses_next) { + TAILQ_FOREACH_REVERSE(ses, &sc->sc_sessions, padlock_sessions_head, + ses_next) { if (ses->ses_id == sid) break; } @@ -323,7 +324,8 @@ padlock_process(device_t dev, struct cryptop *crp, int hint __unused) } rw_rlock(&sc->sc_sessions_lock); - TAILQ_FOREACH(ses, &sc->sc_sessions, ses_next) { + TAILQ_FOREACH_REVERSE(ses, &sc->sc_sessions, padlock_sessions_head, + ses_next) { if (ses->ses_id == (crp->crp_sid & 0xffffffff)) break; } -- cgit v1.1