Subject: [PATCH] crypto/scatterwalk.c to skip unnecessary flush_dcache_page() calls which may cause kernel panic

I am developing the ipsec tools on ARM platform. When I turn the
CONFIG_DEBUG_VM on, the ipsec traffic triggers the kernel panic
which is caused by the VM_DEBUG_ON(PageSlab(page)) of
page_mapping() in the call of flush_dcache_page().

The root cause is that ipsec/esp intends to encrypt the original
data and dynamically allocate cache memory in socket buffer for
this purpose. However, all the cache memory pages dynamically
allocated in socket buffer will be flagged as PG_slab by
__SetPageSlab(). That's the reason it asserts on
VM_BUG_ON(PageSlab(page)) in flush_dcache_page(). Since the
flush_dcache_page() is to ensure the cache coherency between
kernel mapping and userspace mapping. The kernel cache memory
created in socket buffer for encrypting ipsec traffic does not
need to call this function at all since it is all kernel space
data.

Below is my patch for fixing this bug for CONFIG_DEBUG_VM=y.

Chung-yih


diff --git a/crypto/scatterwalk.c b/crypto/scatterwalk.c
index 9aeeb52..3de89a4 100644
--- a/crypto/scatterwalk.c
+++ b/crypto/scatterwalk.c
@@ -54,7 +54,8 @@ static void scatterwalk_pagedone(struct scatter_walk
*walk, int out,
struct page *page;

page = sg_page(walk->sg) + ((walk->offset - 1) >> PAGE_SHIFT);
- flush_dcache_page(page);
+ if (!PageSlab(page))
+ flush_dcache_page(page);
}

if (more) {


2009-03-20 08:21:05

by David Miller

[permalink] [raw]
Subject: Re: [PATCH] crypto/scatterwalk.c to skip unnecessary flush_dcache_page() calls which may cause kernel panic

From: Chung-Yih Wang (???R?t) <[email protected]>
Date: Fri, 20 Mar 2009 16:15:16 +0800

> Below is my patch for fixing this bug for CONFIG_DEBUG_VM=y.

This exact change is already in the tree, and it's been there
for at least a month.

????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?

Subject: Re: [PATCH] crypto/scatterwalk.c to skip unnecessary flush_dcache_page() calls which may cause kernel panic

That's great. Thanks for the info.

2009/3/20 David Miller <[email protected]>:
> From: Chung-Yih Wang (???R?t) <[email protected]>
> Date: Fri, 20 Mar 2009 16:15:16 +0800
>
>> Below is my patch for fixing this bug for CONFIG_DEBUG_VM=y.
>
> This exact change is already in the tree, and it's been there
> for at least a month.
>
>