Re: [RFC PATCH 1/9] misc: Add Surface Aggregator subsystem

[kernel test robot <lkp@intel.com>]

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

Hi Maximilian,

[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on kbuild/for-next]
[also build test WARNING on linux/master linus/master v5.9-rc6]
[cannot apply to char-misc/char-misc-testing next-20200923]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Maximilian-Luz/Add-support-for-Microsoft-Surface-System-Aggregator-Module/20200923-231704
base:   https://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.git for-next
config: x86_64-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce (this is a W=1 build):
        # save the attached .config to linux build tree
        make W=1 ARCH=x86_64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   drivers/misc/surface_aggregator/ssh_packet_layer.c: In function '__ssh_ptl_queue_push':
>> drivers/misc/surface_aggregator/ssh_packet_layer.c:426:20: warning: variable 'head' set but not used [-Wunused-but-set-variable]
     426 |  struct list_head *head;
         |                    ^~~~

# https://github.com/0day-ci/linux/commit/47d88a9d590f18684cd52fb7bf56e7aa93e27aac
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Maximilian-Luz/Add-support-for-Microsoft-Surface-System-Aggregator-Module/20200923-231704
git checkout 47d88a9d590f18684cd52fb7bf56e7aa93e27aac
vim +/head +426 drivers/misc/surface_aggregator/ssh_packet_layer.c

   421	
   422	/* must be called with queue lock held */
   423	static int __ssh_ptl_queue_push(struct ssh_packet *packet)
   424	{
   425		struct ssh_ptl *ptl = packet->ptl;
 > 426		struct list_head *head;
   427	
   428	
   429		if (test_bit(SSH_PTL_SF_SHUTDOWN_BIT, &ptl->state))
   430			return -ESHUTDOWN;
   431	
   432		// avoid further transitions when cancelling/completing
   433		if (test_bit(SSH_PACKET_SF_LOCKED_BIT, &packet->state))
   434			return -EINVAL;
   435	
   436		// if this packet has already been queued, do not add it
   437		if (test_and_set_bit(SSH_PACKET_SF_QUEUED_BIT, &packet->state))
   438			return -EALREADY;
   439	
   440		head = __ssh_ptl_queue_find_entrypoint(packet);
   441	
   442		list_add_tail(&ssh_packet_get(packet)->queue_node, &ptl->queue.head);
   443		return 0;
   444	}
   445	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

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