The Linux Kernel Mailing List
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: Parav Pandit <pandit.parav@gmail.com>
Cc: kbuild-all@01.org, cgroups@vger.kernel.org,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-rdma@vger.kernel.org, tj@kernel.org, lizefan@huawei.com,
	hannes@cmpxchg.org, dledford@redhat.com, liranl@mellanox.com,
	sean.hefty@intel.com, jgunthorpe@obsidianresearch.com,
	haggaie@mellanox.com, corbet@lwn.net, james.l.morris@oracle.com,
	serge@hallyn.com, ogerlitz@mellanox.com, matanb@mellanox.com,
	raindel@mellanox.com, akpm@linux-foundation.org,
	linux-security-module@vger.kernel.org, pandit.parav@gmail.com
Subject: Re: [PATCHv4 1/3] rdmacg: Added rdma cgroup controller.
Date: Sun, 31 Jan 2016 08:37:39 +0800	[thread overview]
Message-ID: <201601310851.uY7ukvCn%fengguang.wu@intel.com> (raw)
In-Reply-To: <1454192901-31003-2-git-send-email-pandit.parav@gmail.com>

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

Hi Parav,

[auto build test ERROR on cgroup/for-next]
[also build test ERROR on v4.5-rc1 next-20160129]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/Parav-Pandit/rdmacg-IB-core-rdma-controller-support/20160131-063313
base:   https://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup.git for-next
config: i386-allmodconfig (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All error/warnings (new ones prefixed by >>):

   In file included from arch/x86/include/asm/bug.h:35:0,
                    from include/linux/bug.h:4,
                    from include/linux/mmdebug.h:4,
                    from include/linux/gfp.h:4,
                    from include/linux/slab.h:14,
                    from kernel/cgroup_rdma.c:8:
   kernel/cgroup_rdma.c: In function 'find_cg_rpool_locked':
>> include/linux/lockdep.h:340:51: error: invalid type argument of '->' (have 'spinlock_t {aka struct spinlock}')
    #define lockdep_is_held(lock) lock_is_held(&(lock)->dep_map)
                                                      ^
   include/asm-generic/bug.h:86:25: note: in definition of macro 'WARN_ON'
     int __ret_warn_on = !!(condition);    \
                            ^
>> include/linux/lockdep.h:366:27: note: in expansion of macro 'lockdep_is_held'
      WARN_ON(debug_locks && !lockdep_is_held(l)); \
                              ^
>> kernel/cgroup_rdma.c:179:2: note: in expansion of macro 'lockdep_assert_held'
     lockdep_assert_held(cg->rpool_list_lock);
     ^

coccinelle warnings: (new ones prefixed by >>)

>> kernel/cgroup_rdma.c:912:1-7: preceding lock on line 879
--
>> kernel/cgroup_rdma.c:787:2-3: Unneeded semicolon
   kernel/cgroup_rdma.c:611:2-3: Unneeded semicolon

Please review and possibly fold the followup patch.

vim +340 include/linux/lockdep.h

7531e2f34 Peter Zijlstra  2008-08-11  334  			 int trylock, int read, int check,
7531e2f34 Peter Zijlstra  2008-08-11  335  			 struct lockdep_map *nest_lock, unsigned long ip);
fbb9ce953 Ingo Molnar     2006-07-03  336  
fbb9ce953 Ingo Molnar     2006-07-03  337  extern void lock_release(struct lockdep_map *lock, int nested,
fbb9ce953 Ingo Molnar     2006-07-03  338  			 unsigned long ip);
fbb9ce953 Ingo Molnar     2006-07-03  339  
f607c6685 Peter Zijlstra  2009-07-20 @340  #define lockdep_is_held(lock)	lock_is_held(&(lock)->dep_map)
f607c6685 Peter Zijlstra  2009-07-20  341  
f607c6685 Peter Zijlstra  2009-07-20  342  extern int lock_is_held(struct lockdep_map *lock);
f607c6685 Peter Zijlstra  2009-07-20  343  
00ef9f734 Peter Zijlstra  2008-12-04  344  extern void lock_set_class(struct lockdep_map *lock, const char *name,
00ef9f734 Peter Zijlstra  2008-12-04  345  			   struct lock_class_key *key, unsigned int subclass,
64aa348ed Peter Zijlstra  2008-08-11  346  			   unsigned long ip);
64aa348ed Peter Zijlstra  2008-08-11  347  
00ef9f734 Peter Zijlstra  2008-12-04  348  static inline void lock_set_subclass(struct lockdep_map *lock,
00ef9f734 Peter Zijlstra  2008-12-04  349  		unsigned int subclass, unsigned long ip)
00ef9f734 Peter Zijlstra  2008-12-04  350  {
00ef9f734 Peter Zijlstra  2008-12-04  351  	lock_set_class(lock, lock->name, lock->key, subclass, ip);
00ef9f734 Peter Zijlstra  2008-12-04  352  }
00ef9f734 Peter Zijlstra  2008-12-04  353  
cf40bd16f Nick Piggin     2009-01-21  354  extern void lockdep_set_current_reclaim_state(gfp_t gfp_mask);
cf40bd16f Nick Piggin     2009-01-21  355  extern void lockdep_clear_current_reclaim_state(void);
cf40bd16f Nick Piggin     2009-01-21  356  extern void lockdep_trace_alloc(gfp_t mask);
cf40bd16f Nick Piggin     2009-01-21  357  
a24fc60d6 Peter Zijlstra  2015-06-11  358  extern void lock_pin_lock(struct lockdep_map *lock);
a24fc60d6 Peter Zijlstra  2015-06-11  359  extern void lock_unpin_lock(struct lockdep_map *lock);
a24fc60d6 Peter Zijlstra  2015-06-11  360  
cf40bd16f Nick Piggin     2009-01-21  361  # define INIT_LOCKDEP				.lockdep_recursion = 0, .lockdep_reclaim_gfp = 0,
fbb9ce953 Ingo Molnar     2006-07-03  362  
e3a55fd18 Jarek Poplawski 2007-03-22  363  #define lockdep_depth(tsk)	(debug_locks ? (tsk)->lockdep_depth : 0)
d5abe6691 Peter Zijlstra  2006-12-06  364  
b1ae345d9 Johannes Berg   2013-02-21  365  #define lockdep_assert_held(l)	do {				\
b1ae345d9 Johannes Berg   2013-02-21 @366  		WARN_ON(debug_locks && !lockdep_is_held(l));	\
b1ae345d9 Johannes Berg   2013-02-21  367  	} while (0)
f607c6685 Peter Zijlstra  2009-07-20  368  
9a37110d2 Peter Hurley    2014-09-10  369  #define lockdep_assert_held_once(l)	do {				\

:::::: The code at line 340 was first introduced by commit
:::::: f607c6685774811b8112e124f10a053d77015485 lockdep: Introduce lockdep_assert_held()

:::::: TO: Peter Zijlstra <a.p.zijlstra@chello.nl>
:::::: CC: Ingo Molnar <mingo@elte.hu>

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

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 53435 bytes --]

  parent reply	other threads:[~2016-01-31  0:39 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-30 22:28 [PATCHv4 0/3] rdmacg: IB/core: rdma controller support Parav Pandit
2016-01-30 22:28 ` [PATCHv4 1/3] rdmacg: Added rdma cgroup controller Parav Pandit
2016-01-31  0:19   ` kbuild test robot
2016-01-31  0:37   ` kbuild test robot [this message]
2016-01-31  0:37   ` [PATCH] rdmacg: fix semicolon.cocci warnings kbuild test robot
2016-01-31  1:30   ` [PATCHv4 1/3] rdmacg: Added rdma cgroup controller kbuild test robot
2016-01-30 22:28 ` [PATCHv4 2/3] IB/core: added support to use " Parav Pandit
2016-01-31  1:17   ` kbuild test robot
2016-01-30 22:28 ` [PATCHv4 3/3] rdmacg: Added documentation for rdma controller Parav Pandit

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=201601310851.uY7ukvCn%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=akpm@linux-foundation.org \
    --cc=cgroups@vger.kernel.org \
    --cc=corbet@lwn.net \
    --cc=dledford@redhat.com \
    --cc=haggaie@mellanox.com \
    --cc=hannes@cmpxchg.org \
    --cc=james.l.morris@oracle.com \
    --cc=jgunthorpe@obsidianresearch.com \
    --cc=kbuild-all@01.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-rdma@vger.kernel.org \
    --cc=linux-security-module@vger.kernel.org \
    --cc=liranl@mellanox.com \
    --cc=lizefan@huawei.com \
    --cc=matanb@mellanox.com \
    --cc=ogerlitz@mellanox.com \
    --cc=pandit.parav@gmail.com \
    --cc=raindel@mellanox.com \
    --cc=sean.hefty@intel.com \
    --cc=serge@hallyn.com \
    --cc=tj@kernel.org \
    /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