dri-devel.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: Chris Wilson <chris@chris-wilson.co.uk>
Cc: kbuild-all@01.org, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH] drm: Add deprecation warnings to the old midlayer callbacks
Date: Fri, 2 Jun 2017 07:06:13 +0800	[thread overview]
Message-ID: <201706020645.w3dHy9Fn%fengguang.wu@intel.com> (raw)
In-Reply-To: <20170601130026.15964-1-chris@chris-wilson.co.uk>

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

Hi Chris,

[auto build test WARNING on drm/drm-next]
[also build test WARNING on v4.12-rc3 next-20170601]
[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/Chris-Wilson/drm-Add-deprecation-warnings-to-the-old-midlayer-callbacks/20170602-003053
base:   git://people.freedesktop.org/~airlied/linux.git drm-next
config: ia64-allmodconfig (attached as .config)
compiler: ia64-linux-gcc (GCC) 6.2.0
reproduce:
        wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=ia64 

All warnings (new ones prefixed by >>):

   drivers/gpu/drm/drm_gem.c: In function 'drm_gem_object_free':
>> drivers/gpu/drm/drm_gem.c:803:2: warning: 'gem_free_object' is deprecated [-Wdeprecated-declarations]
     } else if (DRM_DEPRECATED_WARN(dev->driver->gem_free_object)) {
     ^
   In file included from include/drm/drmP.h:76:0,
                    from drivers/gpu/drm/drm_gem.c:39:
   include/drm/drm_drv.h:377:9: note: declared here
     void (*gem_free_object) (struct drm_gem_object *obj);
            ^~~~~~~~~~~~~~~
   drivers/gpu/drm/drm_gem.c:806:3: warning: 'gem_free_object' is deprecated [-Wdeprecated-declarations]
      dev->driver->gem_free_object(obj);
      ^~~
   In file included from include/drm/drmP.h:76:0,
                    from drivers/gpu/drm/drm_gem.c:39:
   include/drm/drm_drv.h:377:9: note: declared here
     void (*gem_free_object) (struct drm_gem_object *obj);
            ^~~~~~~~~~~~~~~
--
   drivers/gpu/drm/drm_drv.c: In function 'drm_dev_register':
>> drivers/gpu/drm/drm_drv.c:781:2: warning: 'load' is deprecated [-Wdeprecated-declarations]
     if (DRM_DEPRECATED_WARN(dev->driver->load)) {
     ^~
   In file included from drivers/gpu/drm/drm_drv.c:36:0:
   include/drm/drm_drv.h:86:8: note: declared here
     int (*load) (struct drm_device *, unsigned long flags);
           ^~~~
   drivers/gpu/drm/drm_drv.c:782:3: warning: 'load' is deprecated [-Wdeprecated-declarations]
      ret = dev->driver->load(dev, flags);
      ^~~
   In file included from drivers/gpu/drm/drm_drv.c:36:0:
   include/drm/drm_drv.h:86:8: note: declared here
     int (*load) (struct drm_device *, unsigned long flags);
           ^~~~
   drivers/gpu/drm/drm_drv.c: In function 'drm_dev_unregister':
>> drivers/gpu/drm/drm_drv.c:833:2: warning: 'unload' is deprecated [-Wdeprecated-declarations]
     if (DRM_DEPRECATED_WARN(dev->driver->unload))
     ^~
   In file included from drivers/gpu/drm/drm_drv.c:36:0:
   include/drm/drm_drv.h:166:9: note: declared here
     void (*unload) (struct drm_device *);
            ^~~~~~
   drivers/gpu/drm/drm_drv.c:834:3: warning: 'unload' is deprecated [-Wdeprecated-declarations]
      dev->driver->unload(dev);
      ^~~
   In file included from drivers/gpu/drm/drm_drv.c:36:0:
   include/drm/drm_drv.h:166:9: note: declared here
     void (*unload) (struct drm_device *);
            ^~~~~~

vim +/gem_free_object +803 drivers/gpu/drm/drm_gem.c

   787	 * @kref: kref of the object to free
   788	 *
   789	 * Called after the last reference to the object has been lost.
   790	 * Must be called holding &drm_device.struct_mutex.
   791	 *
   792	 * Frees the object
   793	 */
   794	void
   795	drm_gem_object_free(struct kref *kref)
   796	{
   797		struct drm_gem_object *obj =
   798			container_of(kref, struct drm_gem_object, refcount);
   799		struct drm_device *dev = obj->dev;
   800	
   801		if (dev->driver->gem_free_object_unlocked) {
   802			dev->driver->gem_free_object_unlocked(obj);
 > 803		} else if (DRM_DEPRECATED_WARN(dev->driver->gem_free_object)) {
   804			WARN_ON(!mutex_is_locked(&dev->struct_mutex));
   805	
   806			dev->driver->gem_free_object(obj);
   807		}
   808	}
   809	EXPORT_SYMBOL(drm_gem_object_free);
   810	
   811	/**

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

[-- Attachment #3: Type: text/plain, Size: 160 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

      parent reply	other threads:[~2017-06-01 23:06 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-01 13:00 [PATCH] drm: Add deprecation warnings to the old midlayer callbacks Chris Wilson
2017-06-01 18:25 ` Emil Velikov
2017-06-20  9:00   ` Daniel Vetter
2017-06-01 23:04 ` kbuild test robot
2017-06-01 23:06 ` kbuild test robot [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201706020645.w3dHy9Fn%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=chris@chris-wilson.co.uk \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=kbuild-all@01.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).