Re: d451564 breakage

From: Ingo Molnar
Date: Fri Nov 13 2009 - 17:48:40 EST



* Russell King <rmk@xxxxxxxxxxxxxxxx> wrote:

> Change:
>
> highmem: Fix debug_kmap_atomic() to also handle KM_IRQ_PTE, KM_NMI, and KM_NMI_PTE
>
> Appears to break ARM:
>
> mm/highmem.c: In function ???debug_kmap_atomic???:
> mm/highmem.c:436: error: ???KM_NMI??? undeclared (first use in this function)

indeed - sorry.

Note that debug_kmap_atomic() will be removed in v2.6.33 so i'd suggest
to just do the easy solution and add #ifndef dummy definitions to
mm/highmem.c to cover ARM - we'll remove the whole cruft for good.

Btw., testing sidenote: i test the ARM defconfig and it didnt break
there. Perhaps highmem is off in the ARM defconfig? It would be helpful
if the ARM defconfig enabled highmem.

Thanks,

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/