Skip to content

Commit

Permalink
highmem: remove unneeded #ifdef CONFIG_TRACE_IRQFLAGS_SUPPORT for deb…
Browse files Browse the repository at this point in the history
…ug_kmap_atomic()

In f4112de ("mm: introduce
debug_kmap_atomic") I said that debug_kmap_atomic() needs
CONFIG_TRACE_IRQFLAGS_SUPPORT.

It was wrong.  (I thought irqs_disabled() is only available when the
architecture has CONFIG_TRACE_IRQFLAGS_SUPPORT)

Remove the #ifdef CONFIG_TRACE_IRQFLAGS_SUPPORT check to enable
kmap_atomic() debugging for the architectures which do not have
CONFIG_TRACE_IRQFLAGS_SUPPORT.

Reported-by: Andrew Morton <[email protected]>
Signed-off-by: Akinobu Mita <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
  • Loading branch information
mita authored and torvalds committed May 25, 2010
1 parent fd23855 commit ff3d58c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion include/linux/highmem.h
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ static inline void invalidate_kernel_vmap_range(void *vaddr, int size)

#include <asm/kmap_types.h>

#if defined(CONFIG_DEBUG_HIGHMEM) && defined(CONFIG_TRACE_IRQFLAGS_SUPPORT)
#ifdef CONFIG_DEBUG_HIGHMEM

void debug_kmap_atomic(enum km_type type);

Expand Down
2 changes: 1 addition & 1 deletion mm/highmem.c
Original file line number Diff line number Diff line change
Expand Up @@ -422,7 +422,7 @@ void __init page_address_init(void)

#endif /* defined(CONFIG_HIGHMEM) && !defined(WANT_PAGE_VIRTUAL) */

#if defined(CONFIG_DEBUG_HIGHMEM) && defined(CONFIG_TRACE_IRQFLAGS_SUPPORT)
#ifdef CONFIG_DEBUG_HIGHMEM

void debug_kmap_atomic(enum km_type type)
{
Expand Down

0 comments on commit ff3d58c

Please sign in to comment.