Re: + mm-vmalloc-remove-useless-variable-in-vmap_block.patch added to -mm tree

Re: + mm-vmalloc-remove-useless-variable-in-vmap_block.patch added to -mm tree

[Zhang Yanfei]

Hi Andrew,

于 2013/8/8 5:05, akpm@linux-foundation.org 写道:
> Subject: + mm-vmalloc-remove-useless-variable-in-vmap_block.patch added to -mm tree
> To: iamjoonsoo.kim@lge.com,hannes@cmpxchg.org,liwanp@linux.vnet.ibm.com,zhangyanfei@cn.fujitsu.con
> From: akpm@linux-foundation.org
> Date: Wed, 07 Aug 2013 14:05:28 -0700
> 
> 
> The patch titled
>      Subject: mm, vmalloc: remove useless variable in vmap_block
> has been added to the -mm tree.  Its filename is
>      mm-vmalloc-remove-useless-variable-in-vmap_block.patch
> 
> This patch should soon appear at
>     http://ozlabs.org/~akpm/mmots/broken-out/mm-vmalloc-remove-useless-variable-in-vmap_block.patch
> and later at
>     http://ozlabs.org/~akpm/mmotm/broken-out/mm-vmalloc-remove-useless-variable-in-vmap_block.patch
> 
> Before you just go and hit "reply", please:
>    a) Consider who else should be cc'ed
>    b) Prefer to cc a suitable mailing list as well
>    c) Ideally: find the original patch on the mailing list and do a
>       reply-to-all to that, adding suitable additional cc's
> 
> *** Remember to use Documentation/SubmitChecklist when testing your code ***
> 
> The -mm tree is included into linux-next and is updated
> there every 3-4 working days
> 
> ------------------------------------------------------
> From: Joonsoo Kim <iamjoonsoo.kim@lge.com>
> Subject: mm, vmalloc: remove useless variable in vmap_block
> 
> vbq in vmap_block isn't used. So remove it.
> 
> Signed-off-by: Joonsoo Kim <iamjoonsoo.kim@lge.com>
> Reviewed-by: Wanpeng Li <liwanp@linux.vnet.ibm.com>
> Acked-by: Johannes Weiner <hannes@cmpxchg.org>
> Acked-by: Zhang Yanfei <zhangyanfei@cn.fujitsu.con>

s/zhangyanfei@cn.fujitsu.con/zhangyanfei@cn.fujitsu.com/g

> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> ---
> 
>  mm/vmalloc.c |    2 --
>  1 file changed, 2 deletions(-)
> 
> diff -puN mm/vmalloc.c~mm-vmalloc-remove-useless-variable-in-vmap_block mm/vmalloc.c
> --- a/mm/vmalloc.c~mm-vmalloc-remove-useless-variable-in-vmap_block
> +++ a/mm/vmalloc.c
> @@ -752,7 +752,6 @@ struct vmap_block_queue {
>  struct vmap_block {
>  	spinlock_t lock;
>  	struct vmap_area *va;
> -	struct vmap_block_queue *vbq;
>  	unsigned long free, dirty;
>  	DECLARE_BITMAP(dirty_map, VMAP_BBMAP_BITS);
>  	struct list_head free_list;
> @@ -830,7 +829,6 @@ static struct vmap_block *new_vmap_block
>  	radix_tree_preload_end();
>  
>  	vbq = &get_cpu_var(vmap_block_queue);
> -	vb->vbq = vbq;
>  	spin_lock(&vbq->lock);
>  	list_add_rcu(&vb->free_list, &vbq->free);
>  	spin_unlock(&vbq->lock);
> _
> 
> Patches currently in -mm which might be from iamjoonsoo.kim@lge.com are
> 
> mm-hugetlb-move-up-the-code-which-check-availability-of-free-huge-page.patch
> mm-hugetlb-trivial-commenting-fix.patch
> mm-hugetlb-clean-up-alloc_huge_page.patch
> mm-hugetlb-fix-and-clean-up-node-iteration-code-to-alloc-or-free.patch
> mm-hugetlb-remove-redundant-list_empty-check-in-gather_surplus_pages.patch
> mm-hugetlb-do-not-use-a-page-in-page-cache-for-cow-optimization.patch
> mm-hugetlb-add-vm_noreserve-check-in-vma_has_reserves.patch
> mm-hugetlb-remove-decrement_hugepage_resv_vma.patch
> mm-hugetlb-decrement-reserve-count-if-vm_noreserve-alloc-page-cache.patch
> mm-hugetlb-decrement-reserve-count-if-vm_noreserve-alloc-page-cache-fix.patch
> mm-page_alloc-add-unlikely-macro-to-help-compiler-optimization.patch
> mm-move-pgtable-related-functions-to-right-place.patch
> swap-clean-up-ifdef-in-page_mapping.patch
> mm-vmalloc-remove-useless-variable-in-vmap_block.patch
> mm-vmalloc-use-well-defined-find_last_bit-func.patch
> 
> --
> To unsubscribe from this list: send the line "unsubscribe mm-commits" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 


-- 
Thanks.
Zhang Yanfei