Skip to content
  • Mathieu Desnoyers's avatar
    Add HAVE_OPROFILE · 42d4b839
    Mathieu Desnoyers authored
    
    
    Linus:
    On the per-architecture side, I do think it would be better to *not* have
    internal architecture knowledge in a generic file, and as such a line like
    
            depends on X86_32 || IA64 || PPC || S390 || SPARC64 || X86_64 || AVR32
    
    really shouldn't exist in a file like kernel/Kconfig.instrumentation.
    
    It would be much better to do
    
            depends on ARCH_SUPPORTS_KPROBES
    
    in that generic file, and then architectures that do support it would just
    have a
    
            bool ARCH_SUPPORTS_KPROBES
                    default y
    
    in *their* architecture files. That would seem to be much more logical,
    and is readable both for arch maintainers *and* for people who have no
    clue - and don't care - about which architecture is supposed to support
    which interface...
    
    Changelog:
    
    Actually, I know I gave this as the magic incantation, but now that I see
    it, I realize that I should have told you to just use
    
            config ARCH_SUPPORTS_KPROBES
                    def_bool y
    
    instead, which is a bit denser.
    
    We seem to use both kinds of syntax for these things, but this is really
    what "def_bool" is there for...
    
    Changelog :
    
    - Moving to HAVE_*.
    - Add AVR32 oprofile.
    
    Signed-off-by: default avatarMathieu Desnoyers <mathieu.desnoyers@polymtl.ca>
    Cc: Andrew Morton <akpm@linux-foundation.org>
    Cc: Haavard Skinnemoen <hskinnemoen@atmel.com>
    Cc: David Howells <dhowells@redhat.com>
    Cc: Jeff Dike <jdike@addtoit.com>
    Cc: Ananth N Mavinakayanahalli <ananth@in.ibm.com>
    Signed-off-by: default avatarSam Ravnborg <sam@ravnborg.org>
    42d4b839