X-Git-Url: http://repo.macrolet.net/gitweb/?a=blobdiff_plain;f=tests%2Fdebug.impure.lisp;h=4d02466256fbb550aa0f5714c7fdae5489afcf0f;hb=7be8d1462a207bda809cd7553c5d76c6ebc4dda2;hp=d54308cf9c268c96ef61e5cc67a1623055a06607;hpb=6a5462ff3cc3006e9b29a59c8424e81a279320c9;p=sbcl.git diff --git a/tests/debug.impure.lisp b/tests/debug.impure.lisp index d54308c..4d02466 100644 --- a/tests/debug.impure.lisp +++ b/tests/debug.impure.lisp @@ -22,6 +22,8 @@ ;;;; 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) @@ -112,7 +115,7 @@ (and (equal (car spec) (car frame)) (args-equal (cdr spec) (cdr frame)))) - (print (list :mismatch spec frame)) + (print (list :wanted spec :got frame)) (setf result nil))) frame-specs backtrace) @@ -121,19 +124,17 @@ ;; any way. (Depends on running in the main ;; thread.) FIXME: On Windows we get two ;; extra foreign frames below regular frames. - (let ((end (last backtrace #-win32 2 #+win32 4))) - (unless (equal (caar end) - 'sb-impl::toplevel-init) - (print (list :backtrace-stunted (caar end))) - (setf result nil))) + (unless (find '(sb-impl::toplevel-init) backtrace + :test #'equal) + (print (list :backtrace-stunted backtrace)) + (setf result nil)) (return-from outer-handler))))) (funcall test-function))) result))) (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 @@ -150,27 +151,44 @@ (funcall fun))) (with-test (:name (:undefined-function :bug-346) - :fails-on '(or :alpha :ppc :sparc :mips - (and :x86-64 (or :freebsd :darwin)))) + ;; 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 on Darwin, 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) :darwin)) + (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, @@ -194,16 +212,16 @@ (declare (optimize (speed 1) (debug 2))) ; no tail call elimination (funcall fun))) (with-test (:name (:divide-by-zero :bug-346) - :fails-on '(or :alpha (and :x86-64 :darwin))) ; bug 346 + :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 '(or :alpha (and :x86-64 :darwin))) ; 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 @@ -215,6 +233,19 @@ (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 () @@ -265,17 +296,15 @@ (assert (verify-backtrace (lambda () (bug-354 354)) '((bug-354 354))))) ;;; FIXME: This test really should be broken into smaller pieces -(with-test (:name (:backtrace :misc) - :fails-on '(or (and :x86 (or :sunos)) (and :x86-64 :darwin))) - (write-line "//tl-xep") +(with-test (:name (:backtrace :tl-xep)) (with-details t (assert (verify-backtrace #'namestring '(((sb-c::tl-xep namestring) 0 ?))))) (with-details nil (assert (verify-backtrace #'namestring - '((namestring))))) + '((namestring)))))) - ;; &MORE-PROCESSOR +(with-test (:name (:backtrace :more-processor)) (with-details t (assert (verify-backtrace (lambda () (bt.1.1 :key)) '(((sb-c::&more-processor bt.1.1) &rest)))) @@ -289,10 +318,9 @@ (assert (verify-backtrace (lambda () (bt.1.2 :key)) '((bt.1.2 &rest)))) (assert (verify-backtrace (lambda () (bt.1.3 :key)) - '((bt.1.3 &rest))))) + '((bt.1.3 &rest)))))) - ;; XEP - (write-line "//xep") +(with-test (:name (:backtrace :xep)) (with-details t (assert (verify-backtrace #'bt.2.1 '(((sb-c::xep bt.2.1) 0 ?)))) @@ -306,10 +334,9 @@ (assert (verify-backtrace #'bt.2.2 '((bt.2.2 &rest)))) (assert (verify-backtrace #'bt.2.3 - '((bt.2.3 &rest))))) + '((bt.2.3 &rest)))))) - ;; VARARGS-ENTRY - (write-line "//varargs-entry") +(with-test (:name (:backtrace :varargs-entry)) (with-details t (assert (verify-backtrace #'bt.3.1 '(((sb-c::varargs-entry bt.3.1) :key nil)))) @@ -323,10 +350,9 @@ (assert (verify-backtrace #'bt.3.2 '((bt.3.2 :key ?)))) (assert (verify-backtrace #'bt.3.3 - '((bt.3.3 &rest))))) + '((bt.3.3 &rest)))))) - ;; HAIRY-ARG-PROCESSOR - (write-line "//hairy-args-processor") +(with-test (:name (:backtrace :hairy-args-processor)) (with-details t (assert (verify-backtrace #'bt.4.1 '(((sb-c::hairy-arg-processor bt.4.1) ?)))) @@ -340,10 +366,10 @@ (assert (verify-backtrace #'bt.4.2 '((bt.4.2 ?)))) (assert (verify-backtrace #'bt.4.3 - '((bt.4.3 &rest))))) + '((bt.4.3 &rest)))))) + - ;; &OPTIONAL-PROCESSOR - (write-line "//optional-processor") +(with-test (:name (:backtrace :optional-processor)) (with-details t (assert (verify-backtrace #'bt.5.1 '(((sb-c::&optional-processor bt.5.1))))) @@ -362,9 +388,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 @@ -374,7 +401,7 @@ (defclass clos-typecheck-test () ((slot :type fixnum))) (setf (slot-value (make-instance 'clos-typecheck-test) 'slot) t)))) - '(((sb-pcl::slot-typecheck clos-typecheck-test slot) t))))) + '(((sb-pcl::slot-typecheck fixnum) t))))) (with-test (:name :clos-emf-named) (assert @@ -386,6 +413,33 @@ (clos-emf-named-test nil)))) '(((sb-pcl::emf clos-emf-named-test) ? ? nil))))) +(with-test (:name :bug-310173) + (flet ((make-fun (n) + (let* ((names '(a b)) + (req (loop repeat n collect (pop names)))) + (compile nil + `(lambda (,@req &rest rest) + (let ((* *)) ; no tail-call + (apply '/ ,@req rest))))))) + (assert + (verify-backtrace (lambda () + (funcall (make-fun 0) 10 11 0)) + `((sb-kernel:two-arg-/ 10/11 0) + (/ 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) + (/ 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) + (/ 10 11 0) + ((lambda (a b &rest rest) :in ,*p*) 10 11 0)))))) + ;;;; test TRACE (defun trace-this () @@ -396,20 +450,21 @@ 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 ;;; suspicions that the breakpoint trace might corrupt the whole image ;;; on that platform. -#-(and (or ppc x86 x86-64) darwin) (with-test (:name (trace :encapsulate nil) - :fails-on '(or (and :ppc (not :linux)) :sparc :mips)) + :fails-on '(or (and :ppc (not :linux)) :sparc :mips) + :broken-on '(or :darwin :sunos)) (let ((out (with-output-to-string (*trace-output*) (trace trace-this :encapsulate nil) (assert (eq 'ok (trace-this))) @@ -417,9 +472,9 @@ (assert (search "TRACE-THIS" out)) (assert (search "returned OK" out)))) -#-(and (or ppc x86 x86-64) darwin) (with-test (:name (trace-recursive :encapsulate nil) - :fails-on '(or (and :ppc (not :linux)) :sparc :mips)) + :fails-on '(or (and :ppc (not :linux)) :sparc :mips :sunos) + :broken-on '(or :darwin (and :x86 :sunos))) (let ((out (with-output-to-string (*trace-output*) (trace trace-fact :encapsulate nil) (assert (= 120 (trace-fact 5))) @@ -428,11 +483,54 @@ (assert (search "returned 1" out)) (assert (search "returned 120" out)))) +(defun trace-and-fmakunbound-this (x) + x) + +(with-test (:name :bug-667657) + (trace trace-and-fmakunbound-this) + (fmakunbound 'trace-and-fmakunbound-this) + (untrace) + (assert (not (trace)))) + (with-test (:name :bug-414) (handler-bind ((warning #'error)) (load (compile-file "bug-414.lisp")) (disassemble 'bug-414))) +(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))) + (prog1 (sb-debug:backtrace-as-list 10) + (assert (sb-debug::stack-allocated-p x))))) + (declare (notinline dx-arg-backtrace)) + (assert (member-if (lambda (frame) + (and (consp 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) @@ -466,14 +564,51 @@ :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) + (assert (not valid-p))))) (write-line "/debug.impure.lisp done")