Messages in this thread Patch in this message | | | From | Arnd Bergmann <> | Subject | Re: [PATCH] s390 (6/6): oprofile for s390. | Date | Thu, 29 Apr 2004 14:39:56 +0200 |
| |
Martin Schwidefsky wrote: > > > +#if defined(CONFIG_OPROFILE) || defined(CONFIG_OPROFILE_MODULE) > > > > this must depend on CONFIG_PROFILING ? > > Currently oprofile is the only profiling method implemented for s390. > So this doesn't really change much. But in principle you are right, > as soon as a second profiling method gets added this would have to > be changed to CONFIG_PROFILING.
No, it should already be changed now, because the code is also needed for readprofile. CONFIG_PROFILING should enable the generic profiling code here, even if CONFIG_OPROFILE is not set. Note that the identical code on i386 is always compiled in, regardless of CONFIG_PROFILING and CONFIG_OPROFILE.
Arnd <><
=== Enable basic profiling code on s390 depending on CONFIG_PROFILING, not CONFIG_OPROFILE. ===== arch/s390/kernel/time.c 1.25 vs edited ===== --- 1.25/arch/s390/kernel/time.c Wed Apr 28 19:51:41 2004 +++ edited/arch/s390/kernel/time.c Thu Apr 29 11:13:36 2004 @@ -179,7 +179,7 @@ #endif /* CONFIG_ARCH_S390X */ -#if defined(CONFIG_OPROFILE) || defined(CONFIG_OPROFILE_MODULE) +#ifdef CONFIG_PROFILING extern char _stext, _etext; /* @@ -223,7 +223,7 @@ } #else #define s390_do_profile(regs) do { ; } while(0) -#endif /* CONFIG_OPROFILE */ +#endif /* CONFIG_PROFILING */ /* ===== arch/s390/oprofile/Kconfig 1.1 vs edited ===== --- 1.1/arch/s390/oprofile/Kconfig Wed Apr 28 19:51:41 2004 +++ edited/arch/s390/oprofile/Kconfig Thu Apr 29 11:16:21 2004 @@ -5,8 +5,8 @@ config PROFILING bool "Profiling support (EXPERIMENTAL)" help - Say Y here to enable the extended profiling support mechanisms used - by profilers such as OProfile. + Say Y here to enable profiling support mechanisms used by + profilers such as readprofile or OProfile. config OPROFILE - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |