From owner-svn-src-all@freebsd.org Mon Dec 16 13:17:40 2019 Return-Path: Delivered-To: svn-src-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 7C0871E3D70; Mon, 16 Dec 2019 13:17:40 +0000 (UTC) (envelope-from luporl@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) server-signature RSA-PSS (4096 bits) client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 47c1z82J2Jz3JM9; Mon, 16 Dec 2019 13:17:40 +0000 (UTC) (envelope-from luporl@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 mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 3159C25C91; Mon, 16 Dec 2019 13:17:40 +0000 (UTC) (envelope-from luporl@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id xBGDHdEC099866; Mon, 16 Dec 2019 13:17:39 GMT (envelope-from luporl@FreeBSD.org) Received: (from luporl@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id xBGDHdhc099864; Mon, 16 Dec 2019 13:17:39 GMT (envelope-from luporl@FreeBSD.org) Message-Id: <201912161317.xBGDHdhc099864@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: luporl set sender to luporl@FreeBSD.org using -f From: Leandro Lupori Date: Mon, 16 Dec 2019 13:17:39 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r355801 - in head/sys: gdb powerpc/include powerpc/powerpc X-SVN-Group: head X-SVN-Commit-Author: luporl X-SVN-Commit-Paths: in head/sys: gdb powerpc/include powerpc/powerpc X-SVN-Commit-Revision: 355801 X-SVN-Commit-Repository: base 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.29 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: Mon, 16 Dec 2019 13:17:40 -0000 Author: luporl Date: Mon Dec 16 13:17:39 2019 New Revision: 355801 URL: https://svnweb.freebsd.org/changeset/base/355801 Log: [PPC] Handle qOffsets packet On PowerPC, this is needed in order for the debugger to find out the memory offset where the kernel image was loaded on the remote target. This fixes symbol resolution when remote debugging a PowerPC kernel. Reviewed by: cem Differential Revision: https://reviews.freebsd.org/D22767 Modified: head/sys/gdb/gdb_main.c head/sys/powerpc/include/gdb_machdep.h head/sys/powerpc/powerpc/gdb_machdep.c Modified: head/sys/gdb/gdb_main.c ============================================================================== --- head/sys/gdb/gdb_main.c Mon Dec 16 09:11:38 2019 (r355800) +++ head/sys/gdb/gdb_main.c Mon Dec 16 13:17:39 2019 (r355801) @@ -769,6 +769,10 @@ gdb_trap(int type, int code) do_qXfer(); } else if (gdb_rx_equal("Search:memory:")) { gdb_do_mem_search(); +#ifdef __powerpc__ + } else if (gdb_rx_equal("Offsets")) { + gdb_cpu_do_offsets(); +#endif } else if (!gdb_cpu_query()) gdb_tx_empty(); break; Modified: head/sys/powerpc/include/gdb_machdep.h ============================================================================== --- head/sys/powerpc/include/gdb_machdep.h Mon Dec 16 09:11:38 2019 (r355800) +++ head/sys/powerpc/include/gdb_machdep.h Mon Dec 16 13:17:39 2019 (r355801) @@ -131,5 +131,6 @@ gdb_end_write(void *arg __unused) void *gdb_cpu_getreg(int, size_t *); void gdb_cpu_setreg(int, void *); int gdb_cpu_signal(int, int); +void gdb_cpu_do_offsets(void); #endif /* !_MACHINE_GDB_MACHDEP_H_ */ Modified: head/sys/powerpc/powerpc/gdb_machdep.c ============================================================================== --- head/sys/powerpc/powerpc/gdb_machdep.c Mon Dec 16 09:11:38 2019 (r355800) +++ head/sys/powerpc/powerpc/gdb_machdep.c Mon Dec 16 13:17:39 2019 (r355801) @@ -48,6 +48,8 @@ __FBSDID("$FreeBSD$"); #include #include +extern vm_offset_t __startkernel; + void * gdb_cpu_getreg(int regnum, size_t *regsz) { @@ -100,4 +102,29 @@ gdb_cpu_signal(int vector, int dummy __unused) return (vector); else return (SIGEMT); +} + +void +gdb_cpu_do_offsets(void) +{ + /* + * On PowerPC, .text starts at KERNBASE + SIZEOF_HEADERS and + * text segment at KERNBASE - SIZEOF_HEADERS. + * On PowerPC64, .text starts at KERNBASE and text segment at + * KERNBASE - 0x100. + * In both cases, the text segment offset is aligned to 64KB. + * + * The __startkernel variable holds the relocated KERNBASE offset. + * Thus, as long as SIZEOF_HEADERS doesn't get bigger than 0x100 + * (which would lead to other issues), aligning __startkernel to + * 64KB gives the text segment offset. + * + * TODO: Add DataSeg to response. On PowerPC64 all sections reside + * in a single LOAD segment, but on PowerPC modifiable data reside + * in a separate segment, that GDB should also relocate. + */ + gdb_tx_begin(0); + gdb_tx_str("TextSeg="); + gdb_tx_varhex(__startkernel & ~0xffff); + gdb_tx_end(); }