From owner-svn-src-all@freebsd.org Sun Aug 14 13:23:03 2016 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 61A65BB9C6D; Sun, 14 Aug 2016 13:23:03 +0000 (UTC) (envelope-from bde@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 2DD52118D; Sun, 14 Aug 2016 13:23:03 +0000 (UTC) (envelope-from bde@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id u7EDN2V1020105; Sun, 14 Aug 2016 13:23:02 GMT (envelope-from bde@FreeBSD.org) Received: (from bde@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id u7EDN2DI020104; Sun, 14 Aug 2016 13:23:02 GMT (envelope-from bde@FreeBSD.org) Message-Id: <201608141323.u7EDN2DI020104@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: bde set sender to bde@FreeBSD.org using -f From: Bruce Evans Date: Sun, 14 Aug 2016 13:23:02 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r304078 - head/sys/ddb X-SVN-Group: head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 14 Aug 2016 13:23:03 -0000 Author: bde Date: Sun Aug 14 13:23:02 2016 New Revision: 304078 URL: https://svnweb.freebsd.org/changeset/base/304078 Log: Don't print an extra newline after the instruction when printing for single stepping of multiple instructions (e.g., s/p, and n/p). db_print_loc_and_inst() already prints a newline on all arches although it probably shouldn't. Especially on SMP systems, single stepping tends to deadlock or panic too quickly to be useful for anything except finding bugs in itself, but with printing "itself" includes console drivers so it is useful for generating stress tests for console drivers. Modified: head/sys/ddb/db_run.c Modified: head/sys/ddb/db_run.c ============================================================================== --- head/sys/ddb/db_run.c Sun Aug 14 13:17:59 2016 (r304077) +++ head/sys/ddb/db_run.c Sun Aug 14 13:23:02 2016 (r304078) @@ -145,7 +145,6 @@ db_stop_at_pc(bool *is_breakpoint) if (db_sstep_print) { db_printf("\t\t"); db_print_loc_and_inst(pc); - db_printf("\n"); } return (false); /* continue */ } @@ -165,7 +164,6 @@ db_stop_at_pc(bool *is_breakpoint) for (i = db_call_depth; --i > 0; ) db_printf(" "); db_print_loc_and_inst(pc); - db_printf("\n"); } } if (inst_call(ins))