X-Git-Url: http://repo.macrolet.net/gitweb/?a=blobdiff_plain;f=tests%2Fdebug.impure.lisp;h=e64e664ce8f29f26fd776073fe61742384a6acdc;hb=6b8604ad86a30578a776d706d01b2f3ab3fac8f2;hp=308d3d66a3a8a4760cdc94ff9dfe147e55cd59e6;hpb=69fe69971242dba6905e9c55f8ce6a9a93c9e403;p=sbcl.git diff --git a/tests/debug.impure.lisp b/tests/debug.impure.lisp index 308d3d6..e64e664 100644 --- a/tests/debug.impure.lisp +++ b/tests/debug.impure.lisp @@ -17,11 +17,13 @@ ;;; The debugger doesn't have any native knowledge of the interpreter (when (eq sb-ext:*evaluator-mode* :interpret) - (sb-ext:quit :unix-status 104)) + (sb-ext:exit :code 104)) ;;;; Check that we get debug arglists right. +(defvar *p* (namestring *load-truename*)) + ;;; FIXME: This should use some get-argslist like functionality that ;;; we actually export. ;;; @@ -61,16 +63,18 @@ ;;; argument of PRINT might be SB-IMPL::OBJECT or SB-KERNEL::OBJ or ;;; whatever. But we do know the general structure that a correct ;;; answer should have, so we can safely do a lot of checks.) -(destructuring-bind (object-sym &optional-sym stream-sym) (get-arglist #'print) - (assert (symbolp object-sym)) - (assert (eql &optional-sym '&optional)) - (assert (symbolp stream-sym))) -(destructuring-bind (dest-sym control-sym &rest-sym format-args-sym) - (get-arglist #'format) - (assert (symbolp dest-sym)) - (assert (symbolp control-sym)) - (assert (eql &rest-sym '&rest)) - (assert (symbolp format-args-sym))) +(with-test (:name :predefined-functions-1) + (destructuring-bind (object-sym &optional-sym stream-sym) (get-arglist #'print) + (assert (symbolp object-sym)) + (assert (eql &optional-sym '&optional)) + (assert (symbolp stream-sym)))) +(with-test (:name :predefined-functions-2) + (destructuring-bind (dest-sym control-sym &rest-sym format-args-sym) + (get-arglist #'format) + (assert (symbolp dest-sym)) + (assert (symbolp control-sym)) + (assert (eql &rest-sym '&rest)) + (assert (symbolp format-args-sym)))) ;;; Check for backtraces generally being correct. Ensure that the ;;; actual backtrace finishes (doesn't signal any errors on its own), @@ -104,7 +108,6 @@ (caar frame-specs) full-backtrace) (setf result nil)) - ;; check that we have all the frames we wanted (mapcar (lambda (spec frame) @@ -131,8 +134,7 @@ (defvar *undefined-function-frame* ;; bug 353 - '(#+(or x86 x86-64) "bogus stack frame" - #-(or x86 x86-64) "undefined function")) + '("undefined function")) ;;; Test for "undefined function" (undefined_tramp) working properly. ;;; Try it with and without tail call elimination, since they can have @@ -149,27 +151,44 @@ (funcall fun))) (with-test (:name (:undefined-function :bug-346) - :fails-on '(or :alpha :ppc :sparc :mips + ;; Failures on ALPHA, SPARC, MIPS, and probably + ;; HPPA are due to not having a full and valid + ;; stack frame for the undefined function frame. + ;; See PPC undefined_tramp for details. + :fails-on '(or :alpha :sparc :mips (and :x86-64 :freebsd))) (assert (verify-backtrace (lambda () (test #'optimized)) (list *undefined-function-frame* - (list '(flet test) #'optimized))))) + (list `(flet test :in ,*p*) #'optimized))))) ;; bug 353: This test fails at least most of the time for x86/linux ;; ca. 0.8.20.16. -- WHN - (with-test (:name (:undefined-function :bug-353) - ;; This used to have fewer :fails-on features pre-0.9.16.38, - ;; but it turns out that the bug was just being masked by - ;; the presence of the IR1 stepper instrumentation (and - ;; is thus again failing now that the instrumentation is - ;; no more). - :fails-on '(or :alpha :mips :ppc)) + (with-test (:name (:undefined-function :bug-353)) (assert (verify-backtrace (lambda () (test #'not-optimized)) (list *undefined-function-frame* - (list '(flet not-optimized)) - (list '(flet test) #'not-optimized)))))) + (list `(flet not-optimized :in ,*p*)) + (list `(flet test :in ,*p*) #'not-optimized)))))) + +(with-test (:name :backtrace-interrupted-condition-wait + :skipped-on '(not :sb-thread) + ;; For some unfathomable reason the backtrace becomes + ;; stunted, ending at _sigtramp, when we add :TIMEOUT NIL to + ;; the frame we expect. If we leave it out, the backtrace is + ;; fine -- but the test fails. I can only boggle right now. + :fails-on '(or (and :x86 :linux) + (and :win32 :sb-thread))) + (let ((m (sb-thread:make-mutex)) + (q (sb-thread:make-waitqueue))) + (assert (verify-backtrace + (lambda () + (sb-thread:with-mutex (m) + (handler-bind ((timeout (lambda (c) + (error "foo")))) + (with-timeout 0.1 + (sb-thread:condition-wait q m))))) + `((sb-thread:condition-wait ,q ,m :timeout nil)))))) ;;; Division by zero was a common error on PPC. It depended on the ;;; return function either being before INTEGER-/-INTEGER in memory, @@ -196,13 +215,13 @@ :fails-on :alpha) ; bug 346 (assert (verify-backtrace (lambda () (test #'optimized)) (list '(/ 42 &rest) - (list '(flet test) #'optimized))))) + (list `(flet test :in ,*p*) #'optimized))))) (with-test (:name (:divide-by-zero :bug-356) :fails-on :alpha) ; bug 356 (assert (verify-backtrace (lambda () (test #'not-optimized)) (list '(/ 42 &rest) - '((flet not-optimized)) - (list '(flet test) #'not-optimized)))))) + `((flet not-optimized :in ,*p*)) + (list `(flet test :in ,*p*) #'not-optimized)))))) (with-test (:name (:throw :no-such-tag) :fails-on '(or @@ -214,24 +233,48 @@ (throw 'no-such-tag t)) (assert (verify-backtrace #'throw-test '((throw-test)))))) +(defun bug-308926 (x) + (let ((v "foo")) + (flet ((bar (z) + (oops v z) + (oops z v))) + (bar x) + (bar v)))) + +(with-test (:name :bug-308926) + (assert (verify-backtrace (lambda () (bug-308926 13)) + '(((flet bar :in bug-308926) 13) + (bug-308926 &rest t))))) + ;;; test entry point handling in backtraces (defun oops () (error "oops")) +(with-test (:name :xep-too-many-arguments) + (assert (verify-backtrace (lambda () (oops 1 2 3 4 5 6)) + '((oops ? ? ? ? ? ?))))) + (defmacro defbt (n ll &body body) - `(progn - ;; normal debug info - (defun ,(intern (format nil "BT.~A.1" n)) ,ll - ,@body) - ;; no arguments saved - (defun ,(intern (format nil "BT.~A.2" n)) ,ll - (declare (optimize (debug 1) (speed 3))) - ,@body) - ;; no lambda-list saved - (defun ,(intern (format nil "BT.~A.3" n)) ,ll - (declare (optimize (debug 0))) - ,@body))) + ;; WTF is this? This is a way to make these tests not depend so much on the + ;; details of LOAD/EVAL. Around 1.0.57 we changed %SIMPLE-EVAL to be + ;; slightly smarter, which meant that things which used to have xeps + ;; suddently had tl-xeps, etc. This takes care of that. + `(funcall + (compile nil + '(lambda () + (progn + ;; normal debug info + (defun ,(intern (format nil "BT.~A.1" n)) ,ll + ,@body) + ;; no arguments saved + (defun ,(intern (format nil "BT.~A.2" n)) ,ll + (declare (optimize (debug 1) (speed 3))) + ,@body) + ;; no lambda-list saved + (defun ,(intern (format nil "BT.~A.3" n)) ,ll + (declare (optimize (debug 0))) + ,@body)))))) (defbt 1 (&key key) (list key)) @@ -304,14 +347,17 @@ (assert (verify-backtrace #'bt.2.3 '((bt.2.3 &rest)))))) +;;; This test is somewhat deceptively named. Due to confusion in debug naming +;;; these functions used to have sb-c::varargs-entry debug names for their +;;; main lambda. (with-test (:name (:backtrace :varargs-entry)) (with-details t (assert (verify-backtrace #'bt.3.1 - '(((sb-c::varargs-entry bt.3.1) :key nil)))) + '((bt.3.1 :key nil)))) (assert (verify-backtrace #'bt.3.2 - '(((sb-c::varargs-entry bt.3.2) :key ?)))) + '((bt.3.2 :key ?)))) (assert (verify-backtrace #'bt.3.3 - '(((sb-c::varargs-entry bt.3.3) &rest))))) + '((bt.3.3 &rest))))) (with-details nil (assert (verify-backtrace #'bt.3.1 '((bt.3.1 :key nil)))) @@ -320,14 +366,17 @@ (assert (verify-backtrace #'bt.3.3 '((bt.3.3 &rest)))))) +;;; This test is somewhat deceptively named. Due to confusion in debug naming +;;; these functions used to have sb-c::hairy-args-processor debug names for +;;; their main lambda. (with-test (:name (:backtrace :hairy-args-processor)) (with-details t (assert (verify-backtrace #'bt.4.1 - '(((sb-c::hairy-arg-processor bt.4.1) ?)))) + '((bt.4.1 ?)))) (assert (verify-backtrace #'bt.4.2 - '(((sb-c::hairy-arg-processor bt.4.2) ?)))) + '((bt.4.2 ?)))) (assert (verify-backtrace #'bt.4.3 - '(((sb-c::hairy-arg-processor bt.4.3) &rest))))) + '((bt.4.3 &rest))))) (with-details nil (assert (verify-backtrace #'bt.4.1 '((bt.4.1 ?)))) @@ -356,9 +405,10 @@ (write-line "//compile nil") (defvar *compile-nil-error* (compile nil '(lambda (x) (cons (when x (error "oops")) nil)))) (defvar *compile-nil-non-tc* (compile nil '(lambda (y) (cons (funcall *compile-nil-error* y) nil)))) -(assert (verify-backtrace (lambda () (funcall *compile-nil-non-tc* 13)) - '(((lambda (x)) 13) - ((lambda (y)) 13)))) +(with-test (:name (:compile nil)) + (assert (verify-backtrace (lambda () (funcall *compile-nil-non-tc* 13)) + `(((lambda (x) :in ,*p*) 13) + ((lambda (y) :in ,*p*) 13))))) (with-test (:name :clos-slot-typecheckfun-named) (assert @@ -391,21 +441,21 @@ (assert (verify-backtrace (lambda () (funcall (make-fun 0) 10 11 0)) - '((sb-kernel:two-arg-/ 10/11 0) + `((sb-kernel:two-arg-/ 10/11 0) (/ 10 11 0) - ((lambda (&rest rest)) 10 11 0)))) + ((lambda (&rest rest) :in ,*p*) 10 11 0)))) (assert (verify-backtrace (lambda () (funcall (make-fun 1) 10 11 0)) - '((sb-kernel:two-arg-/ 10/11 0) + `((sb-kernel:two-arg-/ 10/11 0) (/ 10 11 0) - ((lambda (a &rest rest)) 10 11 0)))) + ((lambda (a &rest rest) :in ,*p*) 10 11 0)))) (assert (verify-backtrace (lambda () (funcall (make-fun 2) 10 11 0)) - '((sb-kernel:two-arg-/ 10/11 0) + `((sb-kernel:two-arg-/ 10/11 0) (/ 10 11 0) - ((lambda (a b &rest rest)) 10 11 0)))))) + ((lambda (a b &rest rest) :in ,*p*) 10 11 0)))))) ;;;; test TRACE @@ -417,12 +467,13 @@ 1 (* n (trace-fact (1- n))))) -(let ((out (with-output-to-string (*trace-output*) - (trace trace-this) - (assert (eq 'ok (trace-this))) - (untrace)))) - (assert (search "TRACE-THIS" out)) - (assert (search "returned OK" out))) +(with-test (:name (trace :simple)) + (let ((out (with-output-to-string (*trace-output*) + (trace trace-this) + (assert (eq 'ok (trace-this))) + (untrace)))) + (assert (search "TRACE-THIS" out)) + (assert (search "returned OK" out)))) ;;; bug 379 ;;; This is not a WITH-TEST :FAILS-ON PPC DARWIN since there are @@ -463,8 +514,12 @@ (load (compile-file "bug-414.lisp")) (disassemble 'bug-414))) -(with-test (:name :bug-310175) - (let ((dx-arg (cons t t))) +(with-test (:name :bug-310175 :fails-on '(not :stack-allocatable-lists)) + ;; KLUDGE: Not all DX-enabled platforms DX CONS, and the compiler + ;; transforms two-arg-LIST* (and one-arg-LIST) to CONS. Therefore, + ;; use two-arg-LIST, which should get through to VOP LIST, and thus + ;; stack-allocate on a predictable set of machines. + (let ((dx-arg (list t t))) (declare (dynamic-extent dx-arg)) (flet ((dx-arg-backtrace (x) (declare (optimize (debug 2))) @@ -473,10 +528,26 @@ (declare (notinline dx-arg-backtrace)) (assert (member-if (lambda (frame) (and (consp frame) - (equal '(flet dx-arg-backtrace) (car frame)) + (consp (car frame)) + (equal '(flet dx-arg-backtrace :in) (butlast (car frame))) (notany #'sb-debug::stack-allocated-p (cdr frame)))) (dx-arg-backtrace dx-arg)))))) +(with-test (:name :bug-795245) + (assert + (eq :ok + (catch 'done + (handler-bind + ((error (lambda (e) + (declare (ignore e)) + (handler-case + (sb-debug:backtrace 100 (make-broadcast-stream)) + (error () + (throw 'done :error)) + (:no-error () + (throw 'done :ok)))))) + (apply '/= nil 1 2 nil)))))) + ;;;; test infinite error protection (defmacro nest-errors (n-levels error-form) @@ -510,14 +581,174 @@ :normal-exit))))))) (write-line "--END OF H-B-A-B--")) -(enable-debugger) +(with-test (:name infinite-error-protection) + (enable-debugger) + (test-inifinite-error-protection)) -(test-inifinite-error-protection) - -#+sb-thread -(let ((thread (sb-thread:make-thread #'test-inifinite-error-protection))) - (loop while (sb-thread:thread-alive-p thread))) +(with-test (:name (infinite-error-protection :thread) + :skipped-on '(not :sb-thread)) + (enable-debugger) + (let ((thread (sb-thread:make-thread #'test-inifinite-error-protection))) + (loop while (sb-thread:thread-alive-p thread)))) +;; unconditional, in case either previous left it enabled (disable-debugger) + +;;;; test some limitations of MAKE-LISP-OBJ + +;;; Older GENCGC systems had a bug in the pointer validation used by +;;; MAKE-LISP-OBJ that made SIMPLE-FUN objects always fail to +;;; validate. +(with-test (:name (make-lisp-obj :simple-funs)) + (sb-sys:without-gcing + (assert (eq #'identity + (sb-kernel:make-lisp-obj + (sb-kernel:get-lisp-obj-address + #'identity)))))) + +;;; Older CHENEYGC systems didn't perform any real pointer validity +;;; checks beyond "is this pointer to somewhere in heap space". +(with-test (:name (make-lisp-obj :pointer-validation)) + ;; Fun and games: We need to test MAKE-LISP-OBJ with a known-bogus + ;; address, but we also need the GC to not pitch a fit if it sees an + ;; object with said bogus address. Thus, construct our known-bogus + ;; object within an area of unboxed storage (a vector) in static + ;; space. We'll make it a simple object, (CONS 0 0), which has an + ;; in-memory representation of two consecutive zero words. We + ;; allocate a three-word vector so that we can guarantee a + ;; double-word aligned double-word of zeros no matter what happens + ;; with the vector-data-offset (currently double-word aligned). + (let* ((memory (sb-int:make-static-vector 3 :element-type `(unsigned-byte ,sb-vm:n-word-bits) + :initial-element 0)) + (vector-data-address (sb-sys:sap-int (sb-kernel::vector-sap memory))) + (object-base-address (logandc2 (+ vector-data-address sb-vm:lowtag-mask) sb-vm:lowtag-mask)) + (object-tagged-address (+ object-base-address sb-vm:list-pointer-lowtag))) + (multiple-value-bind (object valid-p) + (sb-kernel:make-lisp-obj object-tagged-address nil) + (declare (ignore object)) + (assert (not valid-p))))) + +(defun test-debugger (control form &rest targets) + (let ((out (make-string-output-stream)) + (oops t)) + (unwind-protect + (progn + (with-simple-restart (debugger-test-done! "Debugger Test Done!") + (let* ((*debug-io* (make-two-way-stream + (make-string-input-stream control) + (make-broadcast-stream out #+nil *standard-output*))) + ;; Initial announcement goes to *ERROR-OUTPUT* + (*error-output* *debug-io*) + (*invoke-debugger-hook* nil)) + (handler-bind ((error #'invoke-debugger)) + (eval form)))) + (setf oops nil)) + (when oops + (error "Uncontrolled unwind from debugger test."))) + ;; For sanity's sake this is outside the *debug-io* rebinding -- otherwise + ;; it could swallow our asserts! + (with-input-from-string (s (get-output-stream-string out)) + (loop for line = (read-line s nil) + while line + do (assert targets) + #+nil + (format *error-output* "Got: ~A~%" line) + (let ((match (pop targets))) + (if (eq '* match) + ;; Whatever, till the next line matches. + (let ((text (pop targets))) + #+nil + (format *error-output* "Looking for: ~A~%" text) + (unless (search text line) + (push text targets) + (push match targets))) + (unless (search match line) + (format *error-output* "~&Wanted: ~S~% Got: ~S~%" match line) + (setf oops t)))))) + ;; Check that we saw everything we wanted + (when targets + (error "Missed: ~S" targets)) + (assert (not oops)))) + +(with-test (:name (:debugger :source 1)) + (test-debugger + "d + source 0 + debugger-test-done!" + `(progn + (defun this-will-break (x) + (declare (optimize debug)) + (let* ((y (- x x)) + (z (/ x y))) + (+ x z))) + (this-will-break 1)) + '* + "debugger invoked" + '* + "DIVISION-BY-ZERO" + "operands (1 0)" + '* + "INTEGER-/-INTEGER" + "(THIS-WILL-BREAK 1)" + "1]" + "(/ X Y)" + "1]")) + +(with-test (:name (:debugger :source 2)) + (test-debugger + "d + source 0 + debugger-test-done!" + `(locally (declare (optimize (speed 0) (safety 3) (debug 3))) + (let ((f #'(lambda (x cont) + (print x (make-broadcast-stream)) + (if (zerop x) + (error "~%foo") + (funcall cont (1- x) cont))))) + (funcall f 10 f))) + '* + "debugger" + '* + "foo" + '* + "source: (ERROR \"~%foo\")" + '* + "(LAMBDA (X CONT)" + '* + "(FUNCALL CONT (1- X) CONT)" + "1]")) + +(with-test (:name (disassemble :high-debug-eval)) + (eval `(defun this-will-be-disassembled (x) + (declare (optimize debug)) + (+ x x))) + (let* ((oopses (make-string-output-stream)) + (disassembly + (let ((*error-output* oopses)) + (with-output-to-string (*standard-output*) + (disassemble 'this-will-be-disassembled))))) + (with-input-from-string (s disassembly) + (assert (search "; disassembly for THIS-WILL-BE-DISASSEMBLED" + (read-line s)))) + (let ((problems (get-output-stream-string oopses))) + (unless (zerop (length problems)) + (error problems))))) + +(defun this-too-will-be-disasssembled (x) + (declare (optimize debug)) + (+ x x)) + +(with-test (:name (disassemble :high-debug-load)) + (let* ((oopses (make-string-output-stream)) + (disassembly + (let ((*error-output* oopses)) + (with-output-to-string (*standard-output*) + (disassemble 'this-too-will-be-disasssembled))))) + (with-input-from-string (s disassembly) + (assert (equal "; disassembly for THIS-TOO-WILL-BE-DISASSSEMBLED" + (read-line s)))) + (let ((problems (get-output-stream-string oopses))) + (unless (zerop (length problems)) + (error problems))))) (write-line "/debug.impure.lisp done")