[Intel-gfx] [drm-tip:drm-tip 4/8] drivers/gpu/drm/msm/msm_drv.c:458:15: error: too many arguments to function 'drm_aperture_remove_framebuffers'
Thomas Zimmermann
tzimmermann at suse.de
Mon Apr 17 04:54:16 UTC 2023
Should be fixed in drm-tip.
Am 16.04.23 um 18:05 schrieb kernel test robot:
> tree: git://anongit.freedesktop.org/drm/drm-tip drm-tip
> head: c7b438b3ce89d13c67cade787768522b6cf9776b
> commit: 6db3267139706e62b93b39fc37e9d0ca230d9b50 [4/8] Merge remote-tracking branch 'drm-misc/drm-misc-next' into drm-tip
> config: sparc-allyesconfig (https://download.01.org/0day-ci/archive/20230416/202304162325.ylTNxYSY-lkp@intel.com/config)
> compiler: sparc64-linux-gcc (GCC) 12.1.0
> reproduce (this is a W=1 build):
> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> git remote add drm-tip git://anongit.freedesktop.org/drm/drm-tip
> git fetch --no-tags drm-tip drm-tip
> git checkout 6db3267139706e62b93b39fc37e9d0ca230d9b50
> # save the config file
> mkdir build_dir && cp config build_dir/.config
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=sparc olddefconfig
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=sparc SHELL=/bin/bash drivers/gpu/
>
> If you fix the issue, kindly add following tag where applicable
> | Reported-by: kernel test robot <lkp at intel.com>
> | Link: https://lore.kernel.org/oe-kbuild-all/202304162325.ylTNxYSY-lkp@intel.com/
>
> All errors (new ones prefixed by >>):
>
> drivers/gpu/drm/msm/msm_drv.c: In function 'msm_drm_init':
>>> drivers/gpu/drm/msm/msm_drv.c:458:15: error: too many arguments to function 'drm_aperture_remove_framebuffers'
> 458 | ret = drm_aperture_remove_framebuffers(false, drv);
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> In file included from drivers/gpu/drm/msm/msm_drv.c:16:
> include/drm/drm_aperture.h:32:1: note: declared here
> 32 | drm_aperture_remove_framebuffers(const struct drm_driver *req_driver)
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
>
> vim +/drm_aperture_remove_framebuffers +458 drivers/gpu/drm/msm/msm_drv.c
>
> 60d476af960158 Johan Hovold 2023-03-06 404
> 70a59dd82959f8 Daniel Vetter 2020-11-04 405 static int msm_drm_init(struct device *dev, const struct drm_driver *drv)
> 5bf9c0b614542d Rob Clark 2015-03-03 406 {
> ec919e6e7146e0 AngeloGioacchino Del Regno 2021-12-01 407 struct msm_drm_private *priv = dev_get_drvdata(dev);
> 2b669875332fbd Archit Taneja 2016-05-02 408 struct drm_device *ddev;
> 5bf9c0b614542d Rob Clark 2015-03-03 409 struct msm_kms *kms;
> 25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 410 int ret, i;
> 5bf9c0b614542d Rob Clark 2015-03-03 411
> 5d40a4b8d824e3 Javier Martinez Canillas 2021-12-17 412 if (drm_firmware_drivers_only())
> 5d40a4b8d824e3 Javier Martinez Canillas 2021-12-17 413 return -ENODEV;
> 5d40a4b8d824e3 Javier Martinez Canillas 2021-12-17 414
> 2b669875332fbd Archit Taneja 2016-05-02 415 ddev = drm_dev_alloc(drv, dev);
> 0f2886057be322 Tom Gundersen 2016-09-21 416 if (IS_ERR(ddev)) {
> 6a41da17e87dee Mamta Shukla 2018-10-20 417 DRM_DEV_ERROR(dev, "failed to allocate drm_device\n");
> 0f2886057be322 Tom Gundersen 2016-09-21 418 return PTR_ERR(ddev);
> 2b669875332fbd Archit Taneja 2016-05-02 419 }
> 2b669875332fbd Archit Taneja 2016-05-02 420 ddev->dev_private = priv;
> 68209390f11603 Rob Clark 2016-05-17 421 priv->dev = ddev;
> 5bf9c0b614542d Rob Clark 2015-03-03 422
> 5bf9c0b614542d Rob Clark 2015-03-03 423 priv->wq = alloc_ordered_workqueue("msm", 0);
> ca090c837b4307 Johan Hovold 2023-03-06 424 if (!priv->wq) {
> ca090c837b4307 Johan Hovold 2023-03-06 425 ret = -ENOMEM;
> ca090c837b4307 Johan Hovold 2023-03-06 426 goto err_put_dev;
> ca090c837b4307 Johan Hovold 2023-03-06 427 }
> 5bf9c0b614542d Rob Clark 2015-03-03 428
> 6ed0897cd800c3 Rob Clark 2021-03-31 429 INIT_LIST_HEAD(&priv->objects);
> 6ed0897cd800c3 Rob Clark 2021-03-31 430 mutex_init(&priv->obj_lock);
> 6ed0897cd800c3 Rob Clark 2021-03-31 431
> b352ba54a82072 Rob Clark 2022-08-02 432 /*
> b352ba54a82072 Rob Clark 2022-08-02 433 * Initialize the LRUs:
> b352ba54a82072 Rob Clark 2022-08-02 434 */
> b352ba54a82072 Rob Clark 2022-08-02 435 mutex_init(&priv->lru.lock);
> b352ba54a82072 Rob Clark 2022-08-02 436 drm_gem_lru_init(&priv->lru.unbacked, &priv->lru.lock);
> b352ba54a82072 Rob Clark 2022-08-02 437 drm_gem_lru_init(&priv->lru.pinned, &priv->lru.lock);
> b352ba54a82072 Rob Clark 2022-08-02 438 drm_gem_lru_init(&priv->lru.willneed, &priv->lru.lock);
> b352ba54a82072 Rob Clark 2022-08-02 439 drm_gem_lru_init(&priv->lru.dontneed, &priv->lru.lock);
> 48e7f18392c66f Kristian H. Kristensen 2019-03-20 440
> d984457b31c4c5 Rob Clark 2020-10-23 441 /* Teach lockdep about lock ordering wrt. shrinker: */
> d984457b31c4c5 Rob Clark 2020-10-23 442 fs_reclaim_acquire(GFP_KERNEL);
> b352ba54a82072 Rob Clark 2022-08-02 443 might_lock(&priv->lru.lock);
> d984457b31c4c5 Rob Clark 2020-10-23 444 fs_reclaim_release(GFP_KERNEL);
> 5bf9c0b614542d Rob Clark 2015-03-03 445
> 2b669875332fbd Archit Taneja 2016-05-02 446 drm_mode_config_init(ddev);
> 060530f1ea6740 Rob Clark 2014-03-03 447
> d863f0c7b53628 Craig Tatlor 2020-12-30 448 ret = msm_init_vram(ddev);
> 77050c3febb180 Jeykumar Sankaran 2018-06-27 449 if (ret)
> a75b49db6529b2 Johan Hovold 2023-03-06 450 goto err_cleanup_mode_config;
> 060530f1ea6740 Rob Clark 2014-03-03 451
> d863f0c7b53628 Craig Tatlor 2020-12-30 452 /* Bind all our sub-components: */
> d863f0c7b53628 Craig Tatlor 2020-12-30 453 ret = component_bind_all(dev, ddev);
> 13f15565f7887a Rob Clark 2015-05-07 454 if (ret)
> 60d476af960158 Johan Hovold 2023-03-06 455 goto err_deinit_vram;
> 13f15565f7887a Rob Clark 2015-05-07 456
> 3aa4e828be97b9 Thomas Zimmermann 2023-04-03 457 /* the fw fb could be anywhere in memory */
> 3aa4e828be97b9 Thomas Zimmermann 2023-04-03 @458 ret = drm_aperture_remove_framebuffers(false, drv);
> 3aa4e828be97b9 Thomas Zimmermann 2023-04-03 459 if (ret)
> 3aa4e828be97b9 Thomas Zimmermann 2023-04-03 460 goto err_msm_uninit;
> 3aa4e828be97b9 Thomas Zimmermann 2023-04-03 461
> d5653a994a665c Robin Murphy 2020-09-03 462 dma_set_max_seg_size(dev, UINT_MAX);
> db735fc4036bbe Sean Paul 2020-01-21 463
> 68209390f11603 Rob Clark 2016-05-17 464 msm_gem_shrinker_init(ddev);
> 68209390f11603 Rob Clark 2016-05-17 465
> 5d44531be0982d Dmitry Baryshkov 2022-04-19 466 if (priv->kms_init) {
> 5d44531be0982d Dmitry Baryshkov 2022-04-19 467 ret = priv->kms_init(ddev);
> 5d44531be0982d Dmitry Baryshkov 2022-04-19 468 if (ret) {
> 6a41da17e87dee Mamta Shukla 2018-10-20 469 DRM_DEV_ERROR(dev, "failed to load kms\n");
> b2ccfdf1dde588 Jonathan Marek 2018-11-21 470 priv->kms = NULL;
> 77050c3febb180 Jeykumar Sankaran 2018-06-27 471 goto err_msm_uninit;
> c8afe684c95cd1 Rob Clark 2013-06-26 472 }
> 5d44531be0982d Dmitry Baryshkov 2022-04-19 473 kms = priv->kms;
> 5d44531be0982d Dmitry Baryshkov 2022-04-19 474 } else {
> 5d44531be0982d Dmitry Baryshkov 2022-04-19 475 /* valid only for the dummy headless case, where of_node=NULL */
> 5d44531be0982d Dmitry Baryshkov 2022-04-19 476 WARN_ON(dev->of_node);
> 5d44531be0982d Dmitry Baryshkov 2022-04-19 477 kms = NULL;
> 5d44531be0982d Dmitry Baryshkov 2022-04-19 478 }
> c8afe684c95cd1 Rob Clark 2013-06-26 479
> bb676df12b5e81 Jeykumar Sankaran 2018-06-11 480 /* Enable normalization of plane zpos */
> bb676df12b5e81 Jeykumar Sankaran 2018-06-11 481 ddev->mode_config.normalize_zpos = true;
> bb676df12b5e81 Jeykumar Sankaran 2018-06-11 482
> c8afe684c95cd1 Rob Clark 2013-06-26 483 if (kms) {
> 2d99ced787e3d0 Rob Clark 2019-08-29 484 kms->dev = ddev;
> c8afe684c95cd1 Rob Clark 2013-06-26 485 ret = kms->funcs->hw_init(kms);
> c8afe684c95cd1 Rob Clark 2013-06-26 486 if (ret) {
> 6a41da17e87dee Mamta Shukla 2018-10-20 487 DRM_DEV_ERROR(dev, "kms hw init failed: %d\n", ret);
> 77050c3febb180 Jeykumar Sankaran 2018-06-27 488 goto err_msm_uninit;
> c8afe684c95cd1 Rob Clark 2013-06-26 489 }
> c8afe684c95cd1 Rob Clark 2013-06-26 490 }
> c8afe684c95cd1 Rob Clark 2013-06-26 491
> c51720a6d0b402 Kuogee Hsieh 2022-07-06 492 drm_helper_move_panel_connectors_to_head(ddev);
> c51720a6d0b402 Kuogee Hsieh 2022-07-06 493
> 2b669875332fbd Archit Taneja 2016-05-02 494 ddev->mode_config.funcs = &mode_config_funcs;
> d14659f5de7d28 Sean Paul 2018-02-28 495 ddev->mode_config.helper_private = &mode_config_helper_funcs;
> c8afe684c95cd1 Rob Clark 2013-06-26 496
> 25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 497 for (i = 0; i < priv->num_crtcs; i++) {
> 25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 498 /* initialize event thread */
> 25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 499 priv->event_thread[i].crtc_id = priv->crtcs[i]->base.id;
> 25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 500 priv->event_thread[i].dev = ddev;
> 1041dee2178ffd Bernard 2020-07-21 501 priv->event_thread[i].worker = kthread_create_worker(0,
> 25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 502 "crtc_event:%d", priv->event_thread[i].crtc_id);
> 1041dee2178ffd Bernard 2020-07-21 503 if (IS_ERR(priv->event_thread[i].worker)) {
> a1c9b1e3bdd6d8 Zhen Lei 2021-05-08 504 ret = PTR_ERR(priv->event_thread[i].worker);
> 4971f090aa7f6c Linus Torvalds 2018-12-25 505 DRM_DEV_ERROR(dev, "failed to create crtc_event kthread\n");
> c79bb6b92defdc Dmitry Baryshkov 2022-06-18 506 priv->event_thread[i].worker = NULL;
> 7f9743abaa79d3 Jeykumar Sankaran 2018-10-10 507 goto err_msm_uninit;
> 7f9743abaa79d3 Jeykumar Sankaran 2018-10-10 508 }
> 7f9743abaa79d3 Jeykumar Sankaran 2018-10-10 509
> 6d2b84a4e5b954 Linus Torvalds 2020-08-06 510 sched_set_fifo(priv->event_thread[i].worker->task);
> 25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 511 }
> 25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 512
> 2b669875332fbd Archit Taneja 2016-05-02 513 ret = drm_vblank_init(ddev, priv->num_crtcs);
> c8afe684c95cd1 Rob Clark 2013-06-26 514 if (ret < 0) {
> 6a41da17e87dee Mamta Shukla 2018-10-20 515 DRM_DEV_ERROR(dev, "failed to initialize vblank\n");
> 77050c3febb180 Jeykumar Sankaran 2018-06-27 516 goto err_msm_uninit;
> c8afe684c95cd1 Rob Clark 2013-06-26 517 }
> c8afe684c95cd1 Rob Clark 2013-06-26 518
> a2b3a5571f386e Archit Taneja 2016-05-18 519 if (kms) {
> 2b669875332fbd Archit Taneja 2016-05-02 520 pm_runtime_get_sync(dev);
> f026e431cf8611 Thomas Zimmermann 2021-08-03 521 ret = msm_irq_install(ddev, kms->irq);
> 2b669875332fbd Archit Taneja 2016-05-02 522 pm_runtime_put_sync(dev);
> c8afe684c95cd1 Rob Clark 2013-06-26 523 if (ret < 0) {
> 6a41da17e87dee Mamta Shukla 2018-10-20 524 DRM_DEV_ERROR(dev, "failed to install IRQ handler\n");
> 77050c3febb180 Jeykumar Sankaran 2018-06-27 525 goto err_msm_uninit;
> c8afe684c95cd1 Rob Clark 2013-06-26 526 }
> a2b3a5571f386e Archit Taneja 2016-05-18 527 }
> c8afe684c95cd1 Rob Clark 2013-06-26 528
> 2b669875332fbd Archit Taneja 2016-05-02 529 ret = drm_dev_register(ddev, 0);
> 2b669875332fbd Archit Taneja 2016-05-02 530 if (ret)
> 77050c3febb180 Jeykumar Sankaran 2018-06-27 531 goto err_msm_uninit;
> 2b669875332fbd Archit Taneja 2016-05-02 532
> 6a7e0b0e9fb839 Fabio Estevam 2021-09-14 533 if (kms) {
> 98659487b845c0 Abhinav Kumar 2021-04-16 534 ret = msm_disp_snapshot_init(ddev);
> 98659487b845c0 Abhinav Kumar 2021-04-16 535 if (ret)
> 98659487b845c0 Abhinav Kumar 2021-04-16 536 DRM_DEV_ERROR(dev, "msm_disp_snapshot_init failed ret = %d\n", ret);
> 6a7e0b0e9fb839 Fabio Estevam 2021-09-14 537 }
> 2b669875332fbd Archit Taneja 2016-05-02 538 drm_mode_config_reset(ddev);
> cf3a7e4ce08e68 Rob Clark 2014-11-08 539
> 2b669875332fbd Archit Taneja 2016-05-02 540 ret = msm_debugfs_late_init(ddev);
> a7d3c9509b2fec Rob Clark 2014-05-30 541 if (ret)
> 77050c3febb180 Jeykumar Sankaran 2018-06-27 542 goto err_msm_uninit;
> a7d3c9509b2fec Rob Clark 2014-05-30 543
> 2b669875332fbd Archit Taneja 2016-05-02 544 drm_kms_helper_poll_init(ddev);
> c8afe684c95cd1 Rob Clark 2013-06-26 545
> 940b869c2f2fb7 Thomas Zimmermann 2023-04-03 546 if (kms)
> 940b869c2f2fb7 Thomas Zimmermann 2023-04-03 547 msm_fbdev_setup(ddev);
> 940b869c2f2fb7 Thomas Zimmermann 2023-04-03 548
> c8afe684c95cd1 Rob Clark 2013-06-26 549 return 0;
> c8afe684c95cd1 Rob Clark 2013-06-26 550
> 77050c3febb180 Jeykumar Sankaran 2018-06-27 551 err_msm_uninit:
> 2b669875332fbd Archit Taneja 2016-05-02 552 msm_drm_uninit(dev);
> 214b09db619784 Johan Hovold 2023-03-06 553
> 214b09db619784 Johan Hovold 2023-03-06 554 return ret;
> 214b09db619784 Johan Hovold 2023-03-06 555
> 60d476af960158 Johan Hovold 2023-03-06 556 err_deinit_vram:
> 60d476af960158 Johan Hovold 2023-03-06 557 msm_deinit_vram(ddev);
> a75b49db6529b2 Johan Hovold 2023-03-06 558 err_cleanup_mode_config:
> a75b49db6529b2 Johan Hovold 2023-03-06 559 drm_mode_config_cleanup(ddev);
> a75b49db6529b2 Johan Hovold 2023-03-06 560 destroy_workqueue(priv->wq);
> 214b09db619784 Johan Hovold 2023-03-06 561 err_put_dev:
> 214b09db619784 Johan Hovold 2023-03-06 562 drm_dev_put(ddev);
> 214b09db619784 Johan Hovold 2023-03-06 563
> c8afe684c95cd1 Rob Clark 2013-06-26 564 return ret;
> c8afe684c95cd1 Rob Clark 2013-06-26 565 }
> c8afe684c95cd1 Rob Clark 2013-06-26 566
>
> :::::: The code at line 458 was first introduced by commit
> :::::: 3aa4e828be97b9bacad5679201e22796f15e763b drm/msm: Clear aperture ownership outside of fbdev code
>
> :::::: TO: Thomas Zimmermann <tzimmermann at suse.de>
> :::::: CC: Dmitry Baryshkov <dmitry.baryshkov at linaro.org>
>
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Ivo Totev
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 840 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/intel-gfx/attachments/20230417/59201218/attachment.sig>
More information about the Intel-gfx
mailing list