(in-package "SB!THREAD")
+(def!type thread-name ()
+ 'simple-string)
+
+(def!struct (thread (:constructor %make-thread))
+ #!+sb-doc
+ "Thread type. Do not rely on threads being structs as it may change
+in future versions."
+ (name nil :type (or thread-name null))
+ (%alive-p nil :type boolean)
+ (os-thread nil :type (or integer null))
+ (interruptions nil :type list)
+ (result nil :type list)
+ (interruptions-lock
+ (make-mutex :name "thread interruptions lock")
+ :type mutex)
+ (result-lock
+ (make-mutex :name "thread result lock")
+ :type mutex)
+ waiting-for)
+
(def!struct mutex
#!+sb-doc
"Mutex type."
- (name nil :type (or null simple-string))
+ (name nil :type (or null thread-name))
(%owner nil :type (or null thread))
#!+(and (not sb-lutex) sb-thread)
- (state 0 :type fixnum)
+ (state 0 :type fixnum)
#!+(and sb-lutex sb-thread)
(lutex (make-lutex)))
-;;; FIXME: We probably want to rename the accessor MUTEX-OWNER.
(defun mutex-value (mutex)
- "Current owner of the mutex, NIL if the mutex is free."
+ "Current owner of the mutex, NIL if the mutex is free. May return a
+stale value, use MUTEX-OWNER instead."
(mutex-%owner mutex))
+(defun holding-mutex-p (mutex)
+ "Test whether the current thread is holding MUTEX."
+ ;; This is about the only use for which a stale value of owner is
+ ;; sufficient.
+ (eq sb!thread:*current-thread* (mutex-%owner mutex)))
+
(defsetf mutex-value set-mutex-value)
(declaim (inline set-mutex-value))
(def!struct spinlock
#!+sb-doc
"Spinlock type."
- (name nil :type (or null simple-string))
+ (name nil :type (or null thread-name))
(value nil))
+(sb!xc:defmacro without-thread-waiting-for ((&key already-without-interrupts) &body body)
+ (with-unique-names (thread prev)
+ (let ((without (if already-without-interrupts
+ 'progn
+ 'without-interrupts))
+ (with (if already-without-interrupts
+ 'progn
+ 'with-local-interrupts)))
+ `(let* ((,thread *current-thread*)
+ (,prev (thread-waiting-for ,thread)))
+ (flet ((exec () ,@body))
+ (if ,prev
+ (,without
+ (unwind-protect
+ (progn
+ (setf (thread-waiting-for ,thread) nil)
+ (,with (exec)))
+ (setf (thread-waiting-for ,thread) ,prev)))
+ (exec)))))))
+
(sb!xc:defmacro with-mutex ((mutex &key (value '*current-thread*) (wait-p t))
&body body)
#!+sb-doc
- "Acquire MUTEX for the dynamic scope of BODY, setting it to
-NEW-VALUE or some suitable default value if NIL. If WAIT-P is non-NIL
-and the mutex is in use, sleep until it is available"
+ "Acquire MUTEX for the dynamic scope of BODY, setting it to VALUE or
+some suitable default value if NIL. If WAIT-P is non-NIL and the mutex
+is in use, sleep until it is available"
`(dx-flet ((with-mutex-thunk () ,@body))
(call-with-mutex
#'with-mutex-thunk
,value
,wait-p)))
-(sb!xc:defmacro with-system-mutex ((mutex &key without-gcing allow-with-interrupts) &body body)
+(sb!xc:defmacro with-system-mutex ((mutex
+ &key without-gcing allow-with-interrupts)
+ &body body)
`(dx-flet ((with-system-mutex-thunk () ,@body))
(,(cond (without-gcing
'call-with-system-mutex/without-gcing)
#'with-spinlock-thunk
,spinlock)))
-;;; KLUDGE: this separate implementation for (NOT SB-THREAD) is not
-;;; strictly necessary; GET-MUTEX and RELEASE-MUTEX are implemented.
-;;; However, there would be a (possibly slight) performance hit in
-;;; using them.
+(macrolet ((def (name &optional variant)
+ `(defun ,(if variant (symbolicate name "/" variant) name)
+ (function mutex)
+ (declare (function function))
+ (flet ((%call-with-system-mutex ()
+ (dx-let (got-it)
+ (unwind-protect
+ (when (setf got-it (get-mutex mutex))
+ (funcall function))
+ (when got-it
+ (release-mutex mutex))))))
+ (declare (inline %call-with-system-mutex))
+ ,(ecase variant
+ (:without-gcing
+ `(without-gcing (%call-with-system-mutex)))
+ (:allow-with-interrupts
+ `(without-interrupts
+ (allow-with-interrupts (%call-with-system-mutex))))
+ ((nil)
+ `(without-interrupts (%call-with-system-mutex))))))))
+ (def call-with-system-mutex)
+ (def call-with-system-mutex :without-gcing)
+ (def call-with-system-mutex :allow-with-interrupts))
+
#!-sb-thread
(progn
(macrolet ((def (name &optional variant)
- `(defun ,(if variant (symbolicate name "/" variant) name) (function lock)
+ `(defun ,(if variant (symbolicate name "/" variant) name)
+ (function lock)
(declare (ignore lock) (function function))
,(ecase variant
(:without-gcing
`(without-gcing (funcall function)))
(:allow-with-interrupts
- `(without-interrupts (allow-with-interrupts (funcall function))))
+ `(without-interrupts
+ (allow-with-interrupts (funcall function))))
((nil)
`(without-interrupts (funcall function)))))))
- (def call-with-system-mutex)
- (def call-with-system-mutex :without-gcing)
- (def call-with-system-mutex :allow-with-interrupts)
(def call-with-system-spinlock)
(def call-with-recursive-system-spinlock)
(def call-with-recursive-system-spinlock :without-gcing))
;;; closes over GOT-IT causes a value-cell to be allocated for it --
;;; and we prefer that to go on the stack since it can.
(progn
- (macrolet ((def (name &optional variant)
- `(defun ,(if variant (symbolicate name "/" variant) name) (function mutex)
- (declare (function function))
- (flet ((%call-with-system-mutex ()
- (dx-let (got-it)
- (unwind-protect
- (when (setf got-it (get-mutex mutex))
- (funcall function))
- (when got-it
- (release-mutex mutex))))))
- (declare (inline %call-with-system-mutex))
- ,(ecase variant
- (:without-gcing
- `(without-gcing (%call-with-system-mutex)))
- (:allow-with-interrupts
- `(without-interrupts (allow-with-interrupts (%call-with-system-mutex))))
- ((nil)
- `(without-interrupts (%call-with-system-mutex))))))))
- (def call-with-system-mutex)
- (def call-with-system-mutex :without-gcing)
- (def call-with-system-mutex :allow-with-interrupts))
-
(defun call-with-system-spinlock (function spinlock)
(declare (function function))
(without-interrupts
(release-spinlock spinlock))))))
(macrolet ((def (name &optional variant)
- `(defun ,(if variant (symbolicate name "/" variant) name) (function spinlock)
+ `(defun ,(if variant (symbolicate name "/" variant) name)
+ (function spinlock)
(declare (function function))
(flet ((%call-with-system-spinlock ()
- (dx-let ((inner-lock-p (eq *current-thread* (spinlock-value spinlock)))
+ (dx-let ((inner-lock-p
+ (eq *current-thread*
+ (spinlock-value spinlock)))
(got-it nil))
(unwind-protect
- (when (or inner-lock-p (setf got-it (get-spinlock spinlock)))
+ (when (or inner-lock-p
+ (setf got-it
+ (get-spinlock spinlock)))
(funcall function))
(when got-it
(release-spinlock spinlock))))))
(when got-it
(release-mutex mutex))))))
-
-
(defun call-with-recursive-spinlock (function spinlock)
(declare (function function))
(dx-let ((inner-lock-p (eq (spinlock-value spinlock) *current-thread*))