All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>,
	intel-gfx@lists.freedesktop.org,
	Chris Wilson <chris.p.wilson@intel.com>,
	Lionel G Landwerlin <lionel.g.landwerlin@intel.com>
Cc: kbuild-all@lists.01.org
Subject: Re: [Intel-gfx] [PATCH 2/6] drm/i915/gt: Lock intel_engine_apply_whitelist with uncore->lock
Date: Sat, 3 Oct 2020 10:52:19 +0800	[thread overview]
Message-ID: <202010031031.ephHSGp1-lkp@intel.com> (raw)
In-Reply-To: <20201002232801.51936-3-umesh.nerlige.ramappa@intel.com>

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

Hi Umesh,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on drm-tip/drm-tip linus/master v5.9-rc7 next-20201002]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Umesh-Nerlige-Ramappa/Allow-privileged-user-to-map-the-OA-buffer/20201003-073003
base:   git://anongit.freedesktop.org/drm-intel for-linux-next
config: x86_64-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce (this is a W=1 build):
        # https://github.com/0day-ci/linux/commit/d8a4c73d3c843dd6cb5bb248a90e5b4e0b28953c
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Umesh-Nerlige-Ramappa/Allow-privileged-user-to-map-the-OA-buffer/20201003-073003
        git checkout d8a4c73d3c843dd6cb5bb248a90e5b4e0b28953c
        # save the attached .config to linux build tree
        make W=1 ARCH=x86_64 

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

All warnings (new ones prefixed by >>):

>> drivers/gpu/drm/i915/gt/intel_workarounds.c:1648:6: warning: no previous prototype for 'intel_engine_apply_whitelist_locked' [-Wmissing-prototypes]
    1648 | void intel_engine_apply_whitelist_locked(struct intel_engine_cs *engine)
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

vim +/intel_engine_apply_whitelist_locked +1648 drivers/gpu/drm/i915/gt/intel_workarounds.c

  1647	
> 1648	void intel_engine_apply_whitelist_locked(struct intel_engine_cs *engine)
  1649	{
  1650		const struct i915_wa_list *wal = &engine->whitelist;
  1651		struct intel_uncore *uncore = engine->uncore;
  1652		const u32 base = engine->mmio_base;
  1653		struct i915_wa *wa;
  1654		unsigned int i;
  1655		enum forcewake_domains fw;
  1656	
  1657		lockdep_assert_held(&uncore->lock);
  1658	
  1659		fw = wal_get_fw(uncore, wal, FW_REG_WRITE);
  1660		intel_uncore_forcewake_get__locked(uncore, fw);
  1661	
  1662		for (i = 0, wa = wal->list; i < wal->count; i++, wa++)
  1663			intel_uncore_write_fw(uncore,
  1664					      RING_FORCE_TO_NONPRIV(base, i),
  1665					      i915_mmio_reg_offset(wa->reg));
  1666	
  1667		/* And clear the rest just in case of garbage */
  1668		for (; i < RING_MAX_NONPRIV_SLOTS; i++)
  1669			intel_uncore_write_fw(uncore,
  1670					      RING_FORCE_TO_NONPRIV(base, i),
  1671					      i915_mmio_reg_offset(RING_NOPID(base)));
  1672	
  1673		intel_uncore_forcewake_put__locked(uncore, fw);
  1674	}
  1675	

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

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 75985 bytes --]

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

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [Intel-gfx] [PATCH 2/6] drm/i915/gt: Lock intel_engine_apply_whitelist with uncore->lock
Date: Sat, 03 Oct 2020 10:52:19 +0800	[thread overview]
Message-ID: <202010031031.ephHSGp1-lkp@intel.com> (raw)
In-Reply-To: <20201002232801.51936-3-umesh.nerlige.ramappa@intel.com>

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

Hi Umesh,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on drm-tip/drm-tip linus/master v5.9-rc7 next-20201002]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Umesh-Nerlige-Ramappa/Allow-privileged-user-to-map-the-OA-buffer/20201003-073003
base:   git://anongit.freedesktop.org/drm-intel for-linux-next
config: x86_64-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce (this is a W=1 build):
        # https://github.com/0day-ci/linux/commit/d8a4c73d3c843dd6cb5bb248a90e5b4e0b28953c
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Umesh-Nerlige-Ramappa/Allow-privileged-user-to-map-the-OA-buffer/20201003-073003
        git checkout d8a4c73d3c843dd6cb5bb248a90e5b4e0b28953c
        # save the attached .config to linux build tree
        make W=1 ARCH=x86_64 

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

All warnings (new ones prefixed by >>):

>> drivers/gpu/drm/i915/gt/intel_workarounds.c:1648:6: warning: no previous prototype for 'intel_engine_apply_whitelist_locked' [-Wmissing-prototypes]
    1648 | void intel_engine_apply_whitelist_locked(struct intel_engine_cs *engine)
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

vim +/intel_engine_apply_whitelist_locked +1648 drivers/gpu/drm/i915/gt/intel_workarounds.c

  1647	
> 1648	void intel_engine_apply_whitelist_locked(struct intel_engine_cs *engine)
  1649	{
  1650		const struct i915_wa_list *wal = &engine->whitelist;
  1651		struct intel_uncore *uncore = engine->uncore;
  1652		const u32 base = engine->mmio_base;
  1653		struct i915_wa *wa;
  1654		unsigned int i;
  1655		enum forcewake_domains fw;
  1656	
  1657		lockdep_assert_held(&uncore->lock);
  1658	
  1659		fw = wal_get_fw(uncore, wal, FW_REG_WRITE);
  1660		intel_uncore_forcewake_get__locked(uncore, fw);
  1661	
  1662		for (i = 0, wa = wal->list; i < wal->count; i++, wa++)
  1663			intel_uncore_write_fw(uncore,
  1664					      RING_FORCE_TO_NONPRIV(base, i),
  1665					      i915_mmio_reg_offset(wa->reg));
  1666	
  1667		/* And clear the rest just in case of garbage */
  1668		for (; i < RING_MAX_NONPRIV_SLOTS; i++)
  1669			intel_uncore_write_fw(uncore,
  1670					      RING_FORCE_TO_NONPRIV(base, i),
  1671					      i915_mmio_reg_offset(RING_NOPID(base)));
  1672	
  1673		intel_uncore_forcewake_put__locked(uncore, fw);
  1674	}
  1675	

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

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 75985 bytes --]

  reply	other threads:[~2020-10-03  2:52 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-02 23:27 [Intel-gfx] [PATCH 0/6] Allow privileged user to map the OA buffer Umesh Nerlige Ramappa
2020-10-02 23:27 ` [Intel-gfx] [PATCH 1/6] drm/i915/perf: Ensure observation logic is not clock gated Umesh Nerlige Ramappa
2020-10-02 23:27 ` [Intel-gfx] [PATCH 2/6] drm/i915/gt: Lock intel_engine_apply_whitelist with uncore->lock Umesh Nerlige Ramappa
2020-10-03  2:52   ` kernel test robot [this message]
2020-10-03  2:52     ` kernel test robot
2020-10-02 23:27 ` [Intel-gfx] [PATCH 3/6] drm/i915/perf: Whitelist OA report trigger registers Umesh Nerlige Ramappa
2020-10-02 23:27 ` [Intel-gfx] [PATCH 4/6] drm/i915/gt: Refactor _wa_add to reuse wa_index and wa_list_grow Umesh Nerlige Ramappa
2020-10-02 23:28 ` [Intel-gfx] [PATCH 5/6] drm/i915/perf: Whitelist OA counter and buffer registers Umesh Nerlige Ramappa
2020-10-02 23:28 ` [Intel-gfx] [PATCH 6/6] drm/i915/perf: Map OA buffer to user space for gen12 performance query Umesh Nerlige Ramappa
2020-10-03  0:13 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Allow privileged user to map the OA buffer Patchwork
2020-10-03  0:15 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2020-10-03  0:35 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2020-10-03  3:04 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
  -- strict thread matches above, loose matches on Subject: below --
2020-10-10  0:20 [Intel-gfx] [PATCH 0/6] " Umesh Nerlige Ramappa
2020-10-10  0:21 ` [Intel-gfx] [PATCH 2/6] drm/i915/gt: Lock intel_engine_apply_whitelist with uncore->lock Umesh Nerlige Ramappa
2020-11-12 19:37   ` Chris Wilson

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=202010031031.ephHSGp1-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=chris.p.wilson@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=kbuild-all@lists.01.org \
    --cc=lionel.g.landwerlin@intel.com \
    --cc=umesh.nerlige.ramappa@intel.com \
    /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.