* [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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).