From: Zhang Yanfei <zhangyanfei@cn.fujitsu.com>
To: linux-kernel@vger.kernel.org
Cc: akpm@linux-foundation.org, mm-commits@vger.kernel.org,
zhangyanfei@cn.fujitsu.con, liwanp@linux.vnet.ibm.com,
hannes@cmpxchg.org, iamjoonsoo.kim@lge.com
Subject: Re: + mm-vmalloc-remove-useless-variable-in-vmap_block.patch added to -mm tree
Date: Thu, 08 Aug 2013 08:54:04 +0800 [thread overview]
Message-ID: <5202EC2C.60602@cn.fujitsu.com> (raw)
In-Reply-To: <5202b698.rirANKTNRZ/yHc1T%akpm@linux-foundation.org>
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
parent reply other threads:[~2013-08-08 0:54 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <5202b698.rirANKTNRZ/yHc1T%akpm@linux-foundation.org>]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5202EC2C.60602@cn.fujitsu.com \
--to=zhangyanfei@cn.fujitsu.com \
--cc=akpm@linux-foundation.org \
--cc=hannes@cmpxchg.org \
--cc=iamjoonsoo.kim@lge.com \
--cc=linux-kernel@vger.kernel.org \
--cc=liwanp@linux.vnet.ibm.com \
--cc=mm-commits@vger.kernel.org \
--cc=zhangyanfei@cn.fujitsu.con \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox