Re: [PATCH 02/15] hyperv: uapi-fy synic event flags definitions

From: kbuild test robot <lkp@intel.com>
To: Roman Kagan <rkagan@virtuozzo.com>
Cc: kbuild-all@01.org, "Paolo Bonzini" <pbonzini@redhat.com>,
	"Radim Krčmář" <rkrcmar@redhat.com>,
	"K. Y. Srinivasan" <kys@microsoft.com>,
	"Vitaly Kuznetsov" <vkuznets@redhat.com>,
	kvm@vger.kernel.org, "Denis V . Lunev" <den@openvz.org>,
	"Haiyang Zhang" <haiyangz@microsoft.com>,
	x86@kernel.org, linux-kernel@vger.kernel.org,
	"Ingo Molnar" <mingo@redhat.com>,
	"Roman Kagan" <rkagan@virtuozzo.com>,
	"H. Peter Anvin" <hpa@zytor.com>,
	devel@linuxdriverproject.org,
	"Thomas Gleixner" <tglx@linutronix.de>
Subject: Re: [PATCH 02/15] hyperv: uapi-fy synic event flags definitions
Date: Wed, 21 Dec 2016 18:58:09 +0800	[thread overview]
Message-ID: <201612211823.IRqgzVXH%fengguang.wu@intel.com> (raw)
In-Reply-To: <20161220155602.6298-3-rkagan@virtuozzo.com>

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

Hi Roman,

[auto build test ERROR on linus/master]
[also build test ERROR on v4.9 next-20161221]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Roman-Kagan/hyperv-more-stuff-to-uapi-cleanup/20161221-130415
config: x86_64-rhel (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

Note: the linux-review/Roman-Kagan/hyperv-more-stuff-to-uapi-cleanup/20161221-130415 HEAD 35a54cd6795c4aa6f101c58ab076c6bdd63b3779 builds fine.
      It only hurts bisectibility.

All errors (new ones prefixed by >>):

   drivers/hv/connection.c: In function 'vmbus_set_event':
>> drivers/hv/connection.c:473:3: error: implicit declaration of function 'sync_set_bit' [-Werror=implicit-function-declaration]
      sync_set_bit(child_relid & 31,
      ^~~~~~~~~~~~
   cc1: some warnings being treated as errors
--
   drivers/hv/channel.c: In function 'vmbus_setevent':
>> drivers/hv/channel.c:53:3: error: implicit declaration of function 'sync_set_bit' [-Werror=implicit-function-declaration]
      sync_set_bit(channel->offermsg.child_relid & 31,
      ^~~~~~~~~~~~
   cc1: some warnings being treated as errors

vim +/sync_set_bit +473 drivers/hv/connection.c

1b807e101 drivers/hv/connection.c         K. Y. Srinivasan 2015-12-21  467  void vmbus_set_event(struct vmbus_channel *channel)
3e7ee4902 drivers/staging/hv/Connection.c Hank Janssen     2009-07-13  468  {
21c3bef5d drivers/hv/connection.c         K. Y. Srinivasan 2012-12-01  469  	u32 child_relid = channel->offermsg.child_relid;
3be777740 drivers/hv/connection.c         K. Y. Srinivasan 2012-12-01  470  
3be777740 drivers/hv/connection.c         K. Y. Srinivasan 2012-12-01  471  	if (!channel->is_dedicated_interrupt) {
454f18a96 drivers/staging/hv/Connection.c Bill Pemberton   2009-07-27  472  		/* Each u32 represents 32 channels */
223565857 drivers/staging/hv/connection.c Olaf Hering      2011-03-21 @473  		sync_set_bit(child_relid & 31,
da9fcb726 drivers/staging/hv/connection.c Haiyang Zhang    2011-01-26  474  			(unsigned long *)vmbus_connection.send_int_page +
15b2f6479 drivers/staging/hv/connection.c Haiyang Zhang    2011-01-26  475  			(child_relid >> 5));
3be777740 drivers/hv/connection.c         K. Y. Srinivasan 2012-12-01  476  	}

:::::: The code at line 473 was first introduced by commit
:::::: 22356585712d1ff08fbfed152edd8b386873b238 staging: hv: use sync_bitops when interacting with the hypervisor

:::::: TO: Olaf Hering <olaf@aepfle.de>
:::::: CC: Greg Kroah-Hartman <gregkh@suse.de>

---
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: 38270 bytes --]