All of lore.kernel.org
 help / color / mirror / Atom feed
* [sashal-stable:pending-5.10 51/60] drivers/tty/hvc/hvc_xen.c:422:57: warning: passing argument 1 of 'bind_interdomain_evtchn_to_irq_lateeoi' makes pointer from integer without a cast
@ 2023-03-26  5:04 kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2023-03-26  5:04 UTC (permalink / raw)
  To: Sasha Levin; +Cc: oe-kbuild-all

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git pending-5.10
head:   86de57a583009244b8dab2847952503da3a08a19
commit: 9cd1b7ae28690f24cc448eb0bf28ec7a237c0224 [51/60] xen/events: link interdomain events to associated xenbus device
config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20230326/202303261247.tZTtQYLO-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-8) 11.3.0
reproduce (this is a W=1 build):
        # https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git/commit/?id=9cd1b7ae28690f24cc448eb0bf28ec7a237c0224
        git remote add sashal-stable https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git
        git fetch --no-tags sashal-stable pending-5.10
        git checkout 9cd1b7ae28690f24cc448eb0bf28ec7a237c0224
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        make W=1 O=build_dir ARCH=x86_64 olddefconfig
        make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/tty/hvc/ drivers/xen/events/

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/202303261247.tZTtQYLO-lkp@intel.com/

All warnings (new ones prefixed by >>):

   drivers/tty/hvc/hvc_xen.c: In function 'xencons_connect_backend':
>> drivers/tty/hvc/hvc_xen.c:422:57: warning: passing argument 1 of 'bind_interdomain_evtchn_to_irq_lateeoi' makes pointer from integer without a cast [-Wint-conversion]
     422 |         irq = bind_interdomain_evtchn_to_irq_lateeoi(dev->otherend_id, evtchn);
         |                                                      ~~~^~~~~~~~~~~~~
         |                                                         |
         |                                                         int
   In file included from drivers/tty/hvc/hvc_xen.c:25:
   include/xen/events.h:39:66: note: expected 'struct xenbus_device *' but argument is of type 'int'
      39 | int bind_interdomain_evtchn_to_irq_lateeoi(struct xenbus_device *dev,
         |                                            ~~~~~~~~~~~~~~~~~~~~~~^~~
   drivers/tty/hvc/hvc_xen.c: In function 'xen_hvc_init':
   drivers/tty/hvc/hvc_xen.c:587:29: error: implicit declaration of function 'bind_evtchn_to_irq_lateeoi'; did you mean 'bind_evtchn_to_irqhandler_lateeoi'? [-Werror=implicit-function-declaration]
     587 |                 info->irq = bind_evtchn_to_irq_lateeoi(info->evtchn);
         |                             ^~~~~~~~~~~~~~~~~~~~~~~~~~
         |                             bind_evtchn_to_irqhandler_lateeoi
   cc1: some warnings being treated as errors
--
>> drivers/xen/events/events_base.c:1199:5: warning: no previous prototype for 'bind_evtchn_to_irq_lateeoi' [-Wmissing-prototypes]
    1199 | int bind_evtchn_to_irq_lateeoi(evtchn_port_t evtchn)
         |     ^~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/xen/events/events_base.c: In function 'bind_evtchn_to_irq_lateeoi':
   drivers/xen/events/events_base.c:1201:16: error: too few arguments to function 'bind_evtchn_to_irq_chip'
    1201 |         return bind_evtchn_to_irq_chip(evtchn, &xen_lateeoi_chip);
         |                ^~~~~~~~~~~~~~~~~~~~~~~
   drivers/xen/events/events_base.c:1153:12: note: declared here
    1153 | static int bind_evtchn_to_irq_chip(evtchn_port_t evtchn, struct irq_chip *chip,
         |            ^~~~~~~~~~~~~~~~~~~~~~~
   drivers/xen/events/events_base.c:1202:1: error: control reaches end of non-void function [-Werror=return-type]
    1202 | }
         | ^
   cc1: some warnings being treated as errors


vim +/bind_interdomain_evtchn_to_irq_lateeoi +422 drivers/tty/hvc/hvc_xen.c

02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  410  
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  411  static int xencons_connect_backend(struct xenbus_device *dev,
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  412  				  struct xencons_info *info)
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  413  {
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  414  	int ret, evtchn, devid, ref, irq;
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  415  	struct xenbus_transaction xbt;
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  416  	grant_ref_t gref_head;
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  417  
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  418  	ret = xenbus_alloc_evtchn(dev, &evtchn);
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  419  	if (ret)
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  420  		return ret;
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  421  	info->evtchn = evtchn;
8fa3a370cc2af8 drivers/tty/hvc/hvc_xen.c Juergen Gross       2021-12-16 @422  	irq = bind_interdomain_evtchn_to_irq_lateeoi(dev->otherend_id, evtchn);
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  423  	if (irq < 0)
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  424  		return irq;
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  425  	info->irq = irq;
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  426  	devid = dev->nodename[strlen(dev->nodename) - 1] - '0';
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  427  	info->hvc = hvc_alloc(xenbus_devid_to_vtermno(devid),
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  428  			irq, &domU_hvc_ops, 256);
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  429  	if (IS_ERR(info->hvc))
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  430  		return PTR_ERR(info->hvc);
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  431  	ret = gnttab_alloc_grant_references(1, &gref_head);
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  432  	if (ret < 0)
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  433  		return ret;
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  434  	info->gntref = gref_head;
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  435  	ref = gnttab_claim_grant_reference(&gref_head);
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  436  	if (ref < 0)
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  437  		return ref;
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  438  	gnttab_grant_foreign_access_ref(ref, info->xbdev->otherend_id,
859e3267c515d0 drivers/tty/hvc/hvc_xen.c Julien Grall        2015-08-07  439  					virt_to_gfn(info->intf), 0);
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  440  
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  441   again:
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  442  	ret = xenbus_transaction_start(&xbt);
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  443  	if (ret) {
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  444  		xenbus_dev_fatal(dev, ret, "starting transaction");
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  445  		return ret;
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  446  	}
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  447  	ret = xenbus_printf(xbt, dev->nodename, "ring-ref", "%d", ref);
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  448  	if (ret)
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  449  		goto error_xenbus;
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  450  	ret = xenbus_printf(xbt, dev->nodename, "port", "%u",
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  451  			    evtchn);
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  452  	if (ret)
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  453  		goto error_xenbus;
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  454  	ret = xenbus_transaction_end(xbt, 0);
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  455  	if (ret) {
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  456  		if (ret == -EAGAIN)
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  457  			goto again;
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  458  		xenbus_dev_fatal(dev, ret, "completing transaction");
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  459  		return ret;
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  460  	}
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  461  
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  462  	xenbus_switch_state(dev, XenbusStateInitialised);
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  463  	return 0;
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  464  
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  465   error_xenbus:
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  466  	xenbus_transaction_end(xbt, 1);
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  467  	xenbus_dev_fatal(dev, ret, "writing xenstore");
02e19f9c7cacfb drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-30  468  	return ret;
eb5ef07151ba3c drivers/tty/hvc/hvc_xen.c Stefano Stabellini  2012-01-27  469  }
6b9b732d0e396a drivers/char/hvc_xen.c    Jeremy Fitzhardinge 2008-05-26  470  

:::::: The code at line 422 was first introduced by commit
:::::: 8fa3a370cc2af858a9ba662ca4f2bd0917550563 xen/console: harden hvc_xen against event channel storms

:::::: TO: Juergen Gross <jgross@suse.com>
:::::: CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

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

^ permalink raw reply	[flat|nested] only message in thread
only message in thread, other threads:[~2023-03-26  5:04 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-26  5:04 [sashal-stable:pending-5.10 51/60] drivers/tty/hvc/hvc_xen.c:422:57: warning: passing argument 1 of 'bind_interdomain_evtchn_to_irq_lateeoi' makes pointer from integer without a cast kernel test robot

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.