[PATCH] swap : remove unused field of swapper_space

[PATCH] swap : remove unused field of swapper_space

[Huang Shijie]

There is no place to use the i_mmap_nonlinear of swapper_space, so
remove it.

Signed-off-by: Huang Shijie <shijie8@gmail.com>
---
 mm/swap_state.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/mm/swap_state.c b/mm/swap_state.c
index 6d1daeb..be16a6b 100644
--- a/mm/swap_state.c
+++ b/mm/swap_state.c
@@ -43,7 +43,6 @@ struct address_space swapper_space = {
 	.page_tree	= RADIX_TREE_INIT(GFP_ATOMIC|__GFP_NOWARN),
 	.tree_lock	= __SPIN_LOCK_UNLOCKED(swapper_space.tree_lock),
 	.a_ops		= &swap_aops,
-	.i_mmap_nonlinear = LIST_HEAD_INIT(swapper_space.i_mmap_nonlinear),
 	.backing_dev_info = &swap_backing_dev_info,
 };
 
-- 
1.6.0.6

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH] swap : remove unused field of swapper_space

[Hugh Dickins]

On Fri, 25 Sep 2009, Huang Shijie wrote:

> There is no place to use the i_mmap_nonlinear of swapper_space, so
> remove it.
> 
> Signed-off-by: Huang Shijie <shijie8@gmail.com>

I say NAK, unless you're very sure:
see commit comment below when I added that line.

Hugh

> ---
>  mm/swap_state.c |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/mm/swap_state.c b/mm/swap_state.c
> index 6d1daeb..be16a6b 100644
> --- a/mm/swap_state.c
> +++ b/mm/swap_state.c
> @@ -43,7 +43,6 @@ struct address_space swapper_space = {
>  	.page_tree	= RADIX_TREE_INIT(GFP_ATOMIC|__GFP_NOWARN),
>  	.tree_lock	= __SPIN_LOCK_UNLOCKED(swapper_space.tree_lock),
>  	.a_ops		= &swap_aops,
> -	.i_mmap_nonlinear = LIST_HEAD_INIT(swapper_space.i_mmap_nonlinear),
>  	.backing_dev_info = &swap_backing_dev_info,
>  };
>  

commit e11f2cc49856eabafed09cf30e190646f78b7207
Author: Hugh Dickins <hugh@veritas.com>
Date:   Fri Jun 4 20:51:55 2004 -0700

    [PATCH] mm: swapper_space.i_mmap_nonlinear
    
    Initialize swapper_space.i_mmap_nonlinear, so mapping_mapped reports false on
    it (as it used to do).  Update comment on swapper_space, now more fields are
    used than those initialized explicitly.
    
    Signed-off-by: Hugh Dickins <hugh@veritas.com>
    Signed-off-by: Andrew Morton <akpm@osdl.org>
    Signed-off-by: Linus Torvalds <torvalds@osdl.org>

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH] swap : remove unused field of swapper_space

[Huang Shijie]

Hugh Dickins a??e??:
> On Fri, 25 Sep 2009, Huang Shijie wrote:
>
>   
>> There is no place to use the i_mmap_nonlinear of swapper_space, so
>> remove it.
>>
>> Signed-off-by: Huang Shijie <shijie8@gmail.com>
>>     
>
> I say NAK, unless you're very sure:
> see commit comment below when I added that line.
>
> Hugh
>
>   
>> ---
>>  mm/swap_state.c |    1 -
>>  1 files changed, 0 insertions(+), 1 deletions(-)
>>
>> diff --git a/mm/swap_state.c b/mm/swap_state.c
>> index 6d1daeb..be16a6b 100644
>> --- a/mm/swap_state.c
>> +++ b/mm/swap_state.c
>> @@ -43,7 +43,6 @@ struct address_space swapper_space = {
>>  	.page_tree	= RADIX_TREE_INIT(GFP_ATOMIC|__GFP_NOWARN),
>>  	.tree_lock	= __SPIN_LOCK_UNLOCKED(swapper_space.tree_lock),
>>  	.a_ops		= &swap_aops,
>> -	.i_mmap_nonlinear = LIST_HEAD_INIT(swapper_space.i_mmap_nonlinear),
>>  	.backing_dev_info = &swap_backing_dev_info,
>>  };
>>  
>>     
>
> commit e11f2cc49856eabafed09cf30e190646f78b7207
> Author: Hugh Dickins <hugh@veritas.com>
> Date:   Fri Jun 4 20:51:55 2004 -0700
>
>     [PATCH] mm: swapper_space.i_mmap_nonlinear
>     
>     Initialize swapper_space.i_mmap_nonlinear, so mapping_mapped reports false on
>     it (as it used to do).  Update comment on swapper_space, now more fields are
>     used than those initialized explicitly.
>     
>     Signed-off-by: Hugh Dickins <hugh@veritas.com>
>     Signed-off-by: Andrew Morton <akpm@osdl.org>
>     Signed-off-by: Linus Torvalds <torvalds@osdl.org>
>
>   
Yes ,you are right.

I didn't notice that there were other places calling mapping_mapped() in 
non-x86 archs.
The result generated by the default "make cscope" fooled me.

thanks.


--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>