All of lore.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: kbuild@lists.01.org
Subject: Re: [Intel-gfx] [PATCH 4/4] drm/selftests: selftest for timeline semaphore
Date: Sat, 11 Apr 2020 17:36:36 +0800	[thread overview]
Message-ID: <202004111758.khTQ26ya%lkp@intel.com> (raw)

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

CC: kbuild-all(a)lists.01.org
In-Reply-To: <20200410165158.29546-4-venkata.s.dhanalakota@intel.com>
References: <20200410165158.29546-4-venkata.s.dhanalakota@intel.com>
TO: Venkata Sandeep Dhanalakota <venkata.s.dhanalakota@intel.com>
CC: intel-gfx(a)lists.freedesktop.org
CC: chris.p.wilson(a)intel.com

Hi Venkata,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on drm-tip/drm-tip]
[cannot apply to drm-intel/for-linux-next linus/master v5.6 next-20200410]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Venkata-Sandeep-Dhanalakota/drm-i915-introduce-a-mechanism-to-extend-execbuf2/20200411-031057
base:   git://anongit.freedesktop.org/drm/drm-tip drm-tip
reproduce:
        # apt-get install sparse
        # sparse version: 
        make ARCH=x86_64 allmodconfig
        make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
:::::: branch date: 14 hours ago
:::::: commit date: 14 hours ago

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


sparse warnings: (new ones prefixed by >>)

   drivers/gpu/drm/selftests/test-drm_timeline_semaphore.c:349:53: sparse: warning: incorrect type in argument 1 (different address spaces)
>> drivers/gpu/drm/selftests/test-drm_timeline_semaphore.c:349:53: sparse:    expected struct dma_fence *fence
>> drivers/gpu/drm/selftests/test-drm_timeline_semaphore.c:349:53: sparse:    got struct dma_fence [noderef] <asn:4> *fence
   drivers/gpu/drm/selftests/test-drm_timeline_semaphore.c:366:44: sparse: warning: incorrect type in argument 1 (different address spaces)
   drivers/gpu/drm/selftests/test-drm_timeline_semaphore.c:366:44: sparse:    expected struct dma_fence *fence
   drivers/gpu/drm/selftests/test-drm_timeline_semaphore.c:366:44: sparse:    got struct dma_fence [noderef] <asn:4> *fence
   drivers/gpu/drm/selftests/test-drm_timeline_semaphore.c:471:53: sparse: warning: incorrect type in argument 1 (different address spaces)
   drivers/gpu/drm/selftests/test-drm_timeline_semaphore.c:471:53: sparse:    expected struct dma_fence *fence
   drivers/gpu/drm/selftests/test-drm_timeline_semaphore.c:471:53: sparse:    got struct dma_fence [noderef] <asn:4> *fence

# https://github.com/0day-ci/linux/commit/362cf777546c74e0be786cf6ae2f42031a6a03c8
git remote add linux-review https://github.com/0day-ci/linux
git remote update linux-review
git checkout 362cf777546c74e0be786cf6ae2f42031a6a03c8
vim +349 drivers/gpu/drm/selftests/test-drm_timeline_semaphore.c

362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  329  
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  330  static int igt_waitseqno(void *ignored)
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  331  {
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  332  	struct fence_chain *signal;
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  333  	struct mock_timeline *t[6];
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  334  	struct chain_info ci;
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  335  	struct dma_fence *f;
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  336  	int i, count = 5;
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  337  	int err = 0;
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  338  
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  339  	allocate_chains(&ci, 1, 0);
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  340  	if (IS_ERR_OR_NULL(ci.chains))
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  341  		return -ENOMEM;
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  342  
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  343  	signal = ci.chains[0];
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  344  	atomic_set(&signal->signalers, count);
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  345  
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  346  	/* wait for submit */
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  347  	for (i = 0;i < count; i++) {
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  348  		t[i] = timeline(i, 0x0);
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10 @349  		dma_fence_add_callback(t[i]->syncobj->fence,
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  350  				       &signal->cb,
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  351  				       fence_callback);
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  352  	}
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  353  
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  354  	for (i = 0;i < count; i++) {
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  355  		if(dma_fence_is_signaled(&signal->chain.base))
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  356  			 err = -1;
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  357  
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  358  		f = drm_syncobj_fence_get(t[i]->syncobj);
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  359  		dma_fence_signal(f);
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  360  	}
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  361  
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  362  	if(!dma_fence_is_signaled(&signal->chain.base))
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  363  		 err = -1;
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  364  
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  365  	for (i = 0;i < count; i++) {
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  366  		dma_fence_put(t[i]->syncobj->fence);
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  367  		kmem_cache_free(slab_timeline, t[i]);
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  368  	}
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  369  	delete_chains(&ci);
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  370  	return err;
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  371  }
362cf777546c74 Venkata Sandeep Dhanalakota 2020-04-10  372  

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

             reply	other threads:[~2020-04-11  9:36 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-11  9:36 kbuild test robot [this message]
  -- strict thread matches above, loose matches on Subject: below --
2020-04-11  5:52 [Intel-gfx] [PATCH 4/4] drm/selftests: selftest for timeline semaphore kbuild test robot
2020-04-10 16:51 [Intel-gfx] [PATCH 1/4] drm/i915: introduce a mechanism to extend execbuf2 Venkata Sandeep Dhanalakota
2020-04-10 16:51 ` [Intel-gfx] [PATCH 4/4] drm/selftests: selftest for timeline semaphore Venkata Sandeep Dhanalakota
2020-04-10 21:11   ` kbuild test robot
2020-04-10 21:11     ` kbuild test robot
2020-04-14 13:19   ` Lionel Landwerlin

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=202004111758.khTQ26ya%lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild@lists.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 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.