All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Josef Bacik <josef@toxicpanda.com>
Cc: oe-kbuild-all@lists.linux.dev,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Nikolay Borisov <nborisov@suse.com>, Qu Wenruo <wqu@suse.com>,
	Johannes Thumshirn <jthumshirn@suse.de>,
	David Sterba <dsterba@suse.com>,
	Anand Jain <anand.jain@oracle.com>,
	Sasha Levin <sashal@kernel.org>
Subject: [stable:linux-5.4.y 633/9999] include/linux/list.h:63:20: warning: storing the address of local variable 'ticket' in '*&space_info_37(D)->priority_tickets.prev'
Date: Wed, 12 Apr 2023 00:40:46 +0800	[thread overview]
Message-ID: <202304120017.2CRjmyHB-lkp@intel.com> (raw)

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git linux-5.4.y
head:   32bea3bac5ca484c6f7e302c8c96fc686f62e7b4
commit: 39c5eb1482b2461ea54e0d23c4742987c20480fa [633/9999] btrfs: don't pass system_chunk into can_overcommit
config: parisc-randconfig-r013-20230411 (https://download.01.org/0day-ci/archive/20230412/202304120017.2CRjmyHB-lkp@intel.com/config)
compiler: hppa-linux-gcc (GCC) 12.1.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git/commit/?id=39c5eb1482b2461ea54e0d23c4742987c20480fa
        git remote add stable https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git
        git fetch --no-tags stable linux-5.4.y
        git checkout 39c5eb1482b2461ea54e0d23c4742987c20480fa
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=parisc olddefconfig
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=parisc SHELL=/bin/bash fs/btrfs/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Link: https://lore.kernel.org/oe-kbuild-all/202304120017.2CRjmyHB-lkp@intel.com/

All warnings (new ones prefixed by >>):

   In file included from include/linux/mm.h:99,
                    from fs/btrfs/ctree.h:9,
                    from fs/btrfs/space-info.c:4:
   include/asm-generic/pgtable.h: In function 'pte_clear_not_present_full':
   arch/parisc/include/asm/pgtable.h:91:23: warning: variable 'old_pte' set but not used [-Wunused-but-set-variable]
      91 |                 pte_t old_pte;                                  \
         |                       ^~~~~~~
   arch/parisc/include/asm/pgtable.h:316:34: note: in expansion of macro 'set_pte_at'
     316 | #define pte_clear(mm, addr, xp)  set_pte_at(mm, addr, xp, __pte(0))
         |                                  ^~~~~~~~~~
   include/asm-generic/pgtable.h:201:9: note: in expansion of macro 'pte_clear'
     201 |         pte_clear(mm, address, ptep);
         |         ^~~~~~~~~
   include/asm-generic/pgtable.h: In function '__ptep_modify_prot_commit':
   arch/parisc/include/asm/pgtable.h:91:23: warning: variable 'old_pte' set but not used [-Wunused-but-set-variable]
      91 |                 pte_t old_pte;                                  \
         |                       ^~~~~~~
   include/asm-generic/pgtable.h:629:9: note: in expansion of macro 'set_pte_at'
     629 |         set_pte_at(vma->vm_mm, addr, ptep, pte);
         |         ^~~~~~~~~~
   In file included from fs/btrfs/space-info.c:6:
   fs/btrfs/sysfs.h: At top level:
   fs/btrfs/sysfs.h:16:1: warning: type qualifiers ignored on function return type [-Wignored-qualifiers]
      16 | const char * const btrfs_feature_set_name(enum btrfs_feature_set set);
         | ^~~~~
   In file included from include/linux/rculist.h:10,
                    from include/linux/pid.h:5,
                    from include/linux/sched.h:14,
                    from fs/btrfs/misc.h:6,
                    from fs/btrfs/space-info.c:3:
   In function '__list_add',
       inlined from 'list_add_tail' at include/linux/list.h:93:2,
       inlined from '__reserve_metadata_bytes' at fs/btrfs/space-info.c:1079:4:
>> include/linux/list.h:63:20: warning: storing the address of local variable 'ticket' in '*&space_info_37(D)->priority_tickets.prev' [-Wdangling-pointer=]
      63 |         next->prev = new;
         |         ~~~~~~~~~~~^~~~~
   fs/btrfs/space-info.c: In function '__reserve_metadata_bytes':
   fs/btrfs/space-info.c:1028:31: note: 'ticket' declared here
    1028 |         struct reserve_ticket ticket;
         |                               ^~~~~~
   fs/btrfs/space-info.c:1028:31: note: 'space_info' declared here


vim +63 include/linux/list.h

d7c816733d501b Kees Cook        2016-08-17  49  
^1da177e4c3f41 Linus Torvalds   2005-04-16  50  /*
^1da177e4c3f41 Linus Torvalds   2005-04-16  51   * Insert a new entry between two known consecutive entries.
^1da177e4c3f41 Linus Torvalds   2005-04-16  52   *
^1da177e4c3f41 Linus Torvalds   2005-04-16  53   * This is only for internal list manipulation where we know
^1da177e4c3f41 Linus Torvalds   2005-04-16  54   * the prev/next entries already!
^1da177e4c3f41 Linus Torvalds   2005-04-16  55   */
^1da177e4c3f41 Linus Torvalds   2005-04-16  56  static inline void __list_add(struct list_head *new,
^1da177e4c3f41 Linus Torvalds   2005-04-16  57  			      struct list_head *prev,
^1da177e4c3f41 Linus Torvalds   2005-04-16  58  			      struct list_head *next)
^1da177e4c3f41 Linus Torvalds   2005-04-16  59  {
d7c816733d501b Kees Cook        2016-08-17  60  	if (!__list_add_valid(new, prev, next))
d7c816733d501b Kees Cook        2016-08-17  61  		return;
d7c816733d501b Kees Cook        2016-08-17  62  
^1da177e4c3f41 Linus Torvalds   2005-04-16 @63  	next->prev = new;
^1da177e4c3f41 Linus Torvalds   2005-04-16  64  	new->next = next;
^1da177e4c3f41 Linus Torvalds   2005-04-16  65  	new->prev = prev;
1c97be677f72b3 Paul E. McKenney 2015-09-20  66  	WRITE_ONCE(prev->next, new);
^1da177e4c3f41 Linus Torvalds   2005-04-16  67  }
^1da177e4c3f41 Linus Torvalds   2005-04-16  68  

:::::: The code at line 63 was first introduced by commit
:::::: 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 Linux-2.6.12-rc2

:::::: TO: Linus Torvalds <torvalds@ppc970.osdl.org>
:::::: CC: Linus Torvalds <torvalds@ppc970.osdl.org>

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests

                 reply	other threads:[~2023-04-11 16:41 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=202304120017.2CRjmyHB-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=anand.jain@oracle.com \
    --cc=dsterba@suse.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=josef@toxicpanda.com \
    --cc=jthumshirn@suse.de \
    --cc=nborisov@suse.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=sashal@kernel.org \
    --cc=wqu@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.