linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: kbuild test robot <fengguang.wu@intel.com>
To: "Jérôme Glisse" <jglisse@redhat.com>
Cc: kbuild-all@01.org, linux-mm@kvack.org,
	Michal Hocko <mhocko@suse.com>,
	Andrew Morton <akpm@linux-foundation.org>
Subject: [memcg:since-4.12 523/540] mm/shmem.c:660:2: note: in expansion of macro 'free_swap_and_cache'
Date: Fri, 18 Aug 2017 23:21:54 +0800	[thread overview]
Message-ID: <201708182350.gmrt6KV9%fengguang.wu@intel.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 3294 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/mhocko/mm.git since-4.12
head:   ba5e8c23db5729ebdbafad983b07434c829cf5b6
commit: 51b33649873e78b3a7665a339df522ad4d9d2dd8 [523/540] mm/ZONE_DEVICE: new type of ZONE_DEVICE for unaddressable memory
config: cris-etrax-100lx_v2_defconfig (attached as .config)
compiler: cris-linux-gcc (GCC) 6.2.0
reproduce:
        wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout 51b33649873e78b3a7665a339df522ad4d9d2dd8
        # save the attached .config to linux build tree
        make.cross ARCH=cris 

All warnings (new ones prefixed by >>):

   In file included from mm/shmem.c:34:0:
   mm/shmem.c: In function 'shmem_free_swap':
   include/linux/swap.h:490:55: warning: value computed is not used [-Wunused-value]
    #define free_swap_and_cache(e) (is_migration_entry(e) || is_device_private_entry(e))
                                                          ^
>> mm/shmem.c:660:2: note: in expansion of macro 'free_swap_and_cache'
     free_swap_and_cache(radix_to_swp_entry(radswap));
     ^~~~~~~~~~~~~~~~~~~
--
   In file included from mm/madvise.c:22:0:
   mm/madvise.c: In function 'madvise_free_pte_range':
   include/linux/swap.h:490:55: warning: value computed is not used [-Wunused-value]
    #define free_swap_and_cache(e) (is_migration_entry(e) || is_device_private_entry(e))
                                                          ^
>> mm/madvise.c:353:4: note: in expansion of macro 'free_swap_and_cache'
       free_swap_and_cache(entry);
       ^~~~~~~~~~~~~~~~~~~

vim +/free_swap_and_cache +660 mm/shmem.c

6922c0c7 Hugh Dickins    2011-08-03  646  
6922c0c7 Hugh Dickins    2011-08-03  647  /*
7a5d0fbb Hugh Dickins    2011-08-03  648   * Remove swap entry from radix tree, free the swap and its page cache.
7a5d0fbb Hugh Dickins    2011-08-03  649   */
7a5d0fbb Hugh Dickins    2011-08-03  650  static int shmem_free_swap(struct address_space *mapping,
7a5d0fbb Hugh Dickins    2011-08-03  651  			   pgoff_t index, void *radswap)
7a5d0fbb Hugh Dickins    2011-08-03  652  {
6dbaf22c Johannes Weiner 2014-04-03  653  	void *old;
7a5d0fbb Hugh Dickins    2011-08-03  654  
7a5d0fbb Hugh Dickins    2011-08-03  655  	spin_lock_irq(&mapping->tree_lock);
6dbaf22c Johannes Weiner 2014-04-03  656  	old = radix_tree_delete_item(&mapping->page_tree, index, radswap);
7a5d0fbb Hugh Dickins    2011-08-03  657  	spin_unlock_irq(&mapping->tree_lock);
6dbaf22c Johannes Weiner 2014-04-03  658  	if (old != radswap)
6dbaf22c Johannes Weiner 2014-04-03  659  		return -ENOENT;
7a5d0fbb Hugh Dickins    2011-08-03 @660  	free_swap_and_cache(radix_to_swp_entry(radswap));
6dbaf22c Johannes Weiner 2014-04-03  661  	return 0;
7a5d0fbb Hugh Dickins    2011-08-03  662  }
7a5d0fbb Hugh Dickins    2011-08-03  663  

:::::: The code at line 660 was first introduced by commit
:::::: 7a5d0fbb29936fad7f17b1cb001b0c33a5f13328 tmpfs: convert shmem_truncate_range to radix-swap

:::::: TO: Hugh Dickins <hughd@google.com>
:::::: CC: Linus Torvalds <torvalds@linux-foundation.org>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 8707 bytes --]

                 reply	other threads:[~2017-08-18 15:22 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=201708182350.gmrt6KV9%fengguang.wu@intel.com \
    --to=fengguang.wu@intel.com \
    --cc=akpm@linux-foundation.org \
    --cc=jglisse@redhat.com \
    --cc=kbuild-all@01.org \
    --cc=linux-mm@kvack.org \
    --cc=mhocko@suse.com \
    /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