(in-package "SB!IMPL")
-(defvar *finalizer-store* nil)
+(defglobal **finalizer-store** nil)
-(defvar *finalizer-store-lock*
+(defglobal **finalizer-store-lock**
(sb!thread:make-mutex :name "Finalizer store lock."))
(defmacro with-finalizer-store-lock (&body body)
- `(sb!thread::call-with-system-mutex (lambda () ,@body)
- *finalizer-store-lock*
- t))
+ `(sb!thread::with-system-mutex (**finalizer-store-lock** :without-gcing t)
+ ,@body))
(defun finalize (object function &key dont-save)
#!+sb-doc
Examples:
- ;;; good (assumes RELEASE-HANDLE is re-entrant)
+ ;;; GOOD, assuming RELEASE-HANDLE is re-entrant.
(let* ((handle (get-handle))
(object (make-object handle)))
(finalize object (lambda () (release-handle handle)))
object)
- ;;; bad, finalizer refers to object being finalized, causing
- ;;; it to be retained indefinitely
+ ;;; BAD, finalizer refers to object being finalized, causing
+ ;;; it to be retained indefinitely!
(let* ((handle (get-handle))
(object (make-object handle)))
- (finalize object (lambda () (release-handle (object-handle object)))))
+ (finalize object
+ (lambda ()
+ (release-handle (object-handle object)))))
- ;;; bad, not re-entrant
+ ;;; BAD, not re-entrant!
(defvar *rec* nil)
(defun oops ()
(progn
(finalize \"oops\" #'oops)
- (oops)) ; causes GC and re-entry to #'oops due to the finalizer
+ (oops)) ; GC causes re-entry to #'oops due to the finalizer
; -> ERROR, caught, WARNING signalled"
(unless object
(error "Cannot finalize NIL."))
(with-finalizer-store-lock
(push (list (make-weak-pointer object) function dont-save)
- *finalizer-store*))
+ **finalizer-store**))
object)
(defun deinit-finalizers ()
;; remove :dont-save finalizers
(with-finalizer-store-lock
- (setf *finalizer-store* (delete-if #'third *finalizer-store*)))
+ (setf **finalizer-store** (delete-if #'third **finalizer-store**)))
nil)
(defun cancel-finalization (object)
;; run.
(when object
(with-finalizer-store-lock
- (setf *finalizer-store*
- (delete object *finalizer-store*
+ (setf **finalizer-store**
+ (delete object **finalizer-store**
:key (lambda (list)
(weak-pointer-value (car list))))))
object))
(defun run-pending-finalizers ()
(let (pending)
(with-finalizer-store-lock
- (setf *finalizer-store*
+ (setf **finalizer-store**
(delete-if (lambda (list)
(when (null (weak-pointer-value (car list)))
(push (second list) pending)
t))
- *finalizer-store*)))
+ **finalizer-store**)))
;; We want to run the finalizer bodies outside the lock in case
;; finalization of X causes finalization to be added for Y.
(dolist (fun pending)