lkml.org 
[lkml]   [2004]   [Apr]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectRe: [PATCH] s390 (6/6): oprofile for s390.
Date
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/
\
 
 \ /
  Last update: 2005-03-22 14:02    [W:0.094 / U:0.224 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site