diff options
author | Mateusz Guzik <mjg@FreeBSD.org> | 2022-11-15 10:56:13 +0000 |
---|---|---|
committer | Mateusz Guzik <mjg@FreeBSD.org> | 2022-11-15 14:17:10 +0000 |
commit | c3f1a13902c15f350d59335e99c8945e515a915c (patch) | |
tree | a6b2eba030af53b3226cf9cc26452d3159bef123 | |
parent | a72edfea571909070761d2d143df7d818b4f51b2 (diff) | |
download | src-c3f1a13902c15f350d59335e99c8945e515a915c.tar.gz src-c3f1a13902c15f350d59335e99c8945e515a915c.zip |
Retire broken GPROF support from the kernel
The option is not even recognized and with that patched it does not
compile. Even if it did work, it would be prohibitively expensive to
use.
Interested parties can use pmcstat or dtrace instead.
-rw-r--r-- | sys/amd64/amd64/mp_machdep.c | 3 | ||||
-rw-r--r-- | sys/arm/include/asm.h | 15 | ||||
-rw-r--r-- | sys/i386/i386/mp_machdep.c | 3 | ||||
-rw-r--r-- | sys/kern/kern_clock.c | 22 | ||||
-rw-r--r-- | sys/kern/link_elf.c | 11 | ||||
-rw-r--r-- | sys/x86/x86/mp_x86.c | 3 |
6 files changed, 3 insertions, 54 deletions
diff --git a/sys/amd64/amd64/mp_machdep.c b/sys/amd64/amd64/mp_machdep.c index 2ed3a57866a2..f41e8dafcc86 100644 --- a/sys/amd64/amd64/mp_machdep.c +++ b/sys/amd64/amd64/mp_machdep.c @@ -41,9 +41,6 @@ __FBSDID("$FreeBSD$"); #include <sys/bus.h> #include <sys/cpuset.h> #include <sys/domainset.h> -#ifdef GPROF -#include <sys/gmon.h> -#endif #include <sys/kdb.h> #include <sys/kernel.h> #include <sys/ktr.h> diff --git a/sys/arm/include/asm.h b/sys/arm/include/asm.h index b346f50ca210..72bf244c449a 100644 --- a/sys/arm/include/asm.h +++ b/sys/arm/include/asm.h @@ -69,15 +69,6 @@ #define _ASM_TYPE_FUNCTION #function #define _ASM_TYPE_OBJECT #object -/* XXX Is this still the right prologue for profiling? */ -#ifdef GPROF -#define _PROF_PROLOGUE \ - mov ip, lr; \ - bl __mcount -#else -#define _PROF_PROLOGUE -#endif - /* * EENTRY()/EEND() mark "extra" entry/exit points from a function. * LEENTRY()/LEEND() are the same for local symbols. @@ -106,14 +97,14 @@ #define _ENTRY(x) .text; _ALIGN_TEXT; _EENTRY(x); _FNSTART #define _END(x) _LEND(x) -#define ENTRY(y) _ENTRY(_C_LABEL(y)); _PROF_PROLOGUE +#define ENTRY(y) _ENTRY(_C_LABEL(y)); #define EENTRY(y) _EENTRY(_C_LABEL(y)); #define ENTRY_NP(y) _ENTRY(_C_LABEL(y)) #define EENTRY_NP(y) _EENTRY(_C_LABEL(y)) #define END(y) _END(_C_LABEL(y)) #define EEND(y) _EEND(_C_LABEL(y)) -#define ASENTRY(y) _ENTRY(_ASM_LABEL(y)); _PROF_PROLOGUE -#define ASLENTRY(y) _LENTRY(_ASM_LABEL(y)); _PROF_PROLOGUE +#define ASENTRY(y) _ENTRY(_ASM_LABEL(y)); +#define ASLENTRY(y) _LENTRY(_ASM_LABEL(y)); #define ASEENTRY(y) _EENTRY(_ASM_LABEL(y)); #define ASLEENTRY(y) _LEENTRY(_ASM_LABEL(y)); #define ASENTRY_NP(y) _ENTRY(_ASM_LABEL(y)) diff --git a/sys/i386/i386/mp_machdep.c b/sys/i386/i386/mp_machdep.c index 98d716027060..9a8e82c5fc52 100644 --- a/sys/i386/i386/mp_machdep.c +++ b/sys/i386/i386/mp_machdep.c @@ -51,9 +51,6 @@ __FBSDID("$FreeBSD$"); #include <sys/bus.h> #include <sys/cons.h> /* cngetc() */ #include <sys/cpuset.h> -#ifdef GPROF -#include <sys/gmon.h> -#endif #include <sys/kdb.h> #include <sys/kernel.h> #include <sys/ktr.h> diff --git a/sys/kern/kern_clock.c b/sys/kern/kern_clock.c index f155d2da7e04..b8122b7d6468 100644 --- a/sys/kern/kern_clock.c +++ b/sys/kern/kern_clock.c @@ -74,10 +74,6 @@ __FBSDID("$FreeBSD$"); #include <sys/limits.h> #include <sys/timetc.h> -#ifdef GPROF -#include <sys/gmon.h> -#endif - #ifdef HWPMC_HOOKS #include <sys/pmckern.h> PMC_SOFT_DEFINE( , , clock, hard); @@ -775,10 +771,6 @@ void profclock(int cnt, int usermode, uintfptr_t pc) { struct thread *td; -#ifdef GPROF - struct gmonparam *g; - uintfptr_t i; -#endif td = curthread; if (usermode) { @@ -791,20 +783,6 @@ profclock(int cnt, int usermode, uintfptr_t pc) if (td->td_proc->p_flag & P_PROFIL) addupc_intr(td, pc, cnt); } -#ifdef GPROF - else { - /* - * Kernel statistics are just like addupc_intr, only easier. - */ - g = &_gmonparam; - if (g->state == GMON_PROF_ON && pc >= g->lowpc) { - i = PC_TO_I(g, pc); - if (i < g->textsize) { - KCOUNT(g, i) += cnt; - } - } - } -#endif #ifdef HWPMC_HOOKS if (td->td_intr_frame != NULL) PMC_SOFT_CALL_TF( , , clock, prof, td->td_intr_frame); diff --git a/sys/kern/link_elf.c b/sys/kern/link_elf.c index 2c86c99452f8..5f0649d7540c 100644 --- a/sys/kern/link_elf.c +++ b/sys/kern/link_elf.c @@ -34,9 +34,6 @@ __FBSDID("$FreeBSD$"); #include <sys/param.h> #include <sys/systm.h> -#ifdef GPROF -#include <sys/gmon.h> -#endif #include <sys/kernel.h> #include <sys/lock.h> #include <sys/malloc.h> @@ -1180,14 +1177,6 @@ link_elf_load_file(linker_class_t cls, const char* filename, segs[i]->p_memsz - segs[i]->p_filesz); } -#ifdef GPROF - /* Update profiling information with the new text segment. */ - mtx_lock(&Giant); - kmupetext((uintfptr_t)(mapbase + segs[0]->p_vaddr - base_vaddr + - segs[0]->p_memsz)); - mtx_unlock(&Giant); -#endif - ef->dynamic = (Elf_Dyn *) (mapbase + phdyn->p_vaddr - base_vaddr); lf->address = ef->address; diff --git a/sys/x86/x86/mp_x86.c b/sys/x86/x86/mp_x86.c index 61b08dd1ee97..da5c7f52f955 100644 --- a/sys/x86/x86/mp_x86.c +++ b/sys/x86/x86/mp_x86.c @@ -47,9 +47,6 @@ __FBSDID("$FreeBSD$"); #include <sys/cons.h> /* cngetc() */ #include <sys/cpuset.h> #include <sys/csan.h> -#ifdef GPROF -#include <sys/gmon.h> -#endif #include <sys/interrupt.h> #include <sys/kdb.h> #include <sys/kernel.h> |