From owner-freebsd-bugs@FreeBSD.ORG Tue Sep 23 13:26:11 2014 Return-Path: Delivered-To: freebsd-bugs@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 68010FD1 for ; Tue, 23 Sep 2014 13:26:11 +0000 (UTC) Received: from kenobi.freebsd.org (kenobi.freebsd.org [IPv6:2001:1900:2254:206a::16:76]) (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 4E32ACF4 for ; Tue, 23 Sep 2014 13:26:11 +0000 (UTC) Received: from bugs.freebsd.org ([127.0.1.118]) by kenobi.freebsd.org (8.14.9/8.14.9) with ESMTP id s8NDQBEr000671 for ; Tue, 23 Sep 2014 13:26:11 GMT (envelope-from bugzilla-noreply@freebsd.org) From: bugzilla-noreply@freebsd.org To: freebsd-bugs@FreeBSD.org Subject: [Bug 193873] New: [PATCH] Unify dumpsys() under generic kern_dump.c. Date: Tue, 23 Sep 2014 13:26:11 +0000 X-Bugzilla-Reason: AssignedTo X-Bugzilla-Type: new X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: Base System X-Bugzilla-Component: kern X-Bugzilla-Version: 11.0-CURRENT X-Bugzilla-Keywords: X-Bugzilla-Severity: Affects Only Me X-Bugzilla-Who: conrad.meyer@isilon.com X-Bugzilla-Status: Needs Triage X-Bugzilla-Priority: --- X-Bugzilla-Assigned-To: freebsd-bugs@FreeBSD.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: bug_id short_desc product version rep_platform op_sys bug_status bug_severity priority component assigned_to reporter attachments.created Message-ID: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Bugzilla-URL: https://bugs.freebsd.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: freebsd-bugs@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 23 Sep 2014 13:26:11 -0000 https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=193873 Bug ID: 193873 Summary: [PATCH] Unify dumpsys() under generic kern_dump.c. Product: Base System Version: 11.0-CURRENT Hardware: Any OS: Any Status: Needs Triage Severity: Affects Only Me Priority: --- Component: kern Assignee: freebsd-bugs@FreeBSD.org Reporter: conrad.meyer@isilon.com Created attachment 147599 --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=147599&action=edit (Applies with patch -p1 to CURRENT as of r272027) (To varying degrees by arch.) x86, ARM, and MIPS are all relatively similar and straightforward. Some MD-specific methods are left in dump_machdep.c in each arch to provide mach-dependent implementations. (Map a region temporarily for dumping, unmap a region, iterate physical memory segments, flush WB caches.) Sparc and PowerPC are weirder. PowerPC had a merged dump/minidump path that used a different md_pa structure, pmap_md, plumbed through its MMU interface. So, that was ripped out and replaced with the standard path. Sparc uses its own non-ELF dumping header and that makes its dumpsys different enough that unification wasn't an improvement. However, some logic shared with other archs (blk_dump == cb_dumpdata) was refactored away. Patch build-tested against: - ARMv6 / CHROMEBOOK - AMD64 / GENERIC - i386 / GENERIC - MIPS / WZR-300HP - MIPS64 / SWARM64_SMP - PPC / MPC85XX (cpu=booke) - PPC / GENERIC (cpu=aim) - PPC64 / GENERIC64 (cpu=aim64) - Sparc64 / GENERIC Sponsored by: EMC / Isilon storage division Notes: * This patch is the first step towards some other dump/minidump improvements. I didn't want to apply them independently to 7-8 different architectures' forks of dumpsys. * Patch applies cleanly against a git tree; some hunks include SVN $FreeBSD$ macros which may require some coddling to apply to an SVN tree. * powerpc/dump_machdep.c is entirely deleted (something patch(1) doesn't really do well) * Net 875 lines removed: 21 files changed, 820 insertions(+), 1697 deletions(-) sys/amd64/include/md_var.h | 5 + sys/arm/arm/dump_machdep.c | 325 +---------------------------- sys/arm/include/md_var.h | 7 + sys/conf/files | 1 + sys/conf/files.powerpc | 1 - sys/i386/include/md_var.h | 5 + sys/kern/kern_dump.c | 402 ++++++++++++++++++++++++++++++++++++ sys/mips/include/md_var.h | 8 + sys/mips/mips/dump_machdep.c | 318 +--------------------------- sys/powerpc/aim/mmu_oea.c | 167 +++++++-------- sys/powerpc/aim/mmu_oea64.c | 164 +++++++-------- sys/powerpc/booke/pmap.c | 228 ++++++++++---------- sys/powerpc/include/md_var.h | 10 + sys/powerpc/include/pmap.h | 13 +- sys/powerpc/powerpc/dump_machdep.c | 315 ---------------------------- sys/powerpc/powerpc/mmu_if.m | 41 ++-- sys/powerpc/powerpc/pmap_dispatch.c | 28 ++- sys/sparc64/include/md_var.h | 5 + sys/sparc64/sparc64/dump_machdep.c | 115 +++-------- sys/sys/conf.h | 12 ++ sys/x86/x86/dump_machdep.c | 347 +------------------------------ -- You are receiving this mail because: You are the assignee for the bug.