dri-devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Vetter <daniel@ffwll.ch>
To: Stephen Rothwell <sfr@canb.auug.org.au>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>,
	Daniel Vetter <daniel.vetter@ffwll.ch>,
	Intel Graphics <intel-gfx@lists.freedesktop.org>,
	Akash Goel <akash.goel@intel.com>,
	DRI <dri-devel@lists.freedesktop.org>,
	Zhi Wang <zhi.a.wang@intel.com>
Subject: Re: linux-next: build failure after merge of the drm-intel tree
Date: Tue, 18 Oct 2016 08:26:01 +0200	[thread overview]
Message-ID: <20161018062600.GS20761@phenom.ffwll.local> (raw)
In-Reply-To: <20161018105920.5fcc62e9@canb.auug.org.au>

On Tue, Oct 18, 2016 at 10:59:20AM +1100, Stephen Rothwell wrote:
> Hi all,
> 
> After merging the drm-intel tree, today's linux-next build (x86_64
> allmodconfig) failed like this:
> 
> drivers/gpu/drm/i915/gvt/handlers.c: In function 'render_mmio_to_ring_id':
> drivers/gpu/drm/i915/gvt/handlers.c:137:31: error: request for member 'mmio_base' in something not a structure or union
>    if (gvt->dev_priv->engine[i].mmio_base == reg)
>                                ^
> drivers/gpu/drm/i915/gvt/scheduler.c: In function 'populate_shadow_context':
> drivers/gpu/drm/i915/gvt/scheduler.c:71:4: error: passing argument 1 of 'intel_lr_context_size' from incompatible pointer type [-Werror=incompatible-pointer-types]
>     &gvt->dev_priv->engine[ring_id]);
>     ^
> In file included from drivers/gpu/drm/i915/i915_drv.h:58:0,
>                  from drivers/gpu/drm/i915/gvt/scheduler.c:36:
> drivers/gpu/drm/i915/intel_lrc.h:84:10: note: expected 'struct intel_engine_cs *' but argument is of type 'struct intel_engine_cs **'
>  uint32_t intel_lr_context_size(struct intel_engine_cs *engine);
>           ^
> drivers/gpu/drm/i915/gvt/scheduler.c: In function 'dispatch_workload':
> drivers/gpu/drm/i915/gvt/scheduler.c:174:41: error: passing argument 1 of 'i915_gem_request_alloc' from incompatible pointer type [-Werror=incompatible-pointer-types]
>   workload->req = i915_gem_request_alloc(&dev_priv->engine[ring_id],
>                                          ^
> In file included from drivers/gpu/drm/i915/intel_ringbuffer.h:6:0,
>                  from drivers/gpu/drm/i915/intel_guc.h:29,
>                  from drivers/gpu/drm/i915/i915_drv.h:57,
>                  from drivers/gpu/drm/i915/gvt/scheduler.c:36:
> drivers/gpu/drm/i915/i915_gem_request.h:156:1: note: expected 'struct intel_engine_cs *' but argument is of type 'struct intel_engine_cs **'
>  i915_gem_request_alloc(struct intel_engine_cs *engine,
>  ^
> drivers/gpu/drm/i915/gvt/scheduler.c: In function 'update_guest_context':
> drivers/gpu/drm/i915/gvt/scheduler.c:301:4: error: passing argument 1 of 'intel_lr_context_size' from incompatible pointer type [-Werror=incompatible-pointer-types]
>     &gvt->dev_priv->engine[ring_id]);
>     ^
> In file included from drivers/gpu/drm/i915/i915_drv.h:58:0,
>                  from drivers/gpu/drm/i915/gvt/scheduler.c:36:
> drivers/gpu/drm/i915/intel_lrc.h:84:10: note: expected 'struct intel_engine_cs *' but argument is of type 'struct intel_engine_cs **'
>  uint32_t intel_lr_context_size(struct intel_engine_cs *engine);
>           ^
> drivers/gpu/drm/i915/gvt/execlist.c: In function 'emulate_execlist_status':
> drivers/gpu/drm/i915/gvt/execlist.c:42:33: error: request for member 'mmio_base' in something not a structure or union
>   (gvt->dev_priv->engine[ring_id].mmio_base + (offset))
>                                  ^  
> drivers/gpu/drm/i915/gvt/execlist.c:97:19: note: in expansion of macro 'execlist_ring_mmio'
>   u32 status_reg = execlist_ring_mmio(vgpu->gvt,
>                    ^
> drivers/gpu/drm/i915/gvt/execlist.c: In function 'emulate_csb_update':
> drivers/gpu/drm/i915/gvt/execlist.c:42:33: error: request for member 'mmio_base' in something not a structure or union
>   (gvt->dev_priv->engine[ring_id].mmio_base + (offset))
>                                  ^  
> drivers/gpu/drm/i915/gvt/execlist.c:136:23: note: in expansion of macro 'execlist_ring_mmio'
>   ctx_status_ptr_reg = execlist_ring_mmio(vgpu->gvt, ring_id,
>                        ^
> drivers/gpu/drm/i915/gvt/execlist.c:42:33: error: request for member 'mmio_base' in something not a structure or union
>   (gvt->dev_priv->engine[ring_id].mmio_base + (offset))
>                                  ^  
> drivers/gpu/drm/i915/gvt/execlist.c:138:23: note: in expansion of macro 'execlist_ring_mmio'
>   ctx_status_buf_reg = execlist_ring_mmio(vgpu->gvt, ring_id,
>                        ^
> drivers/gpu/drm/i915/gvt/execlist.c: In function 'get_next_execlist_slot':
> drivers/gpu/drm/i915/gvt/execlist.c:42:33: error: request for member 'mmio_base' in something not a structure or union
>   (gvt->dev_priv->engine[ring_id].mmio_base + (offset))
>                                  ^  
> drivers/gpu/drm/i915/gvt/execlist.c:248:19: note: in expansion of macro 'execlist_ring_mmio'
>   u32 status_reg = execlist_ring_mmio(vgpu->gvt, ring_id,
>                    ^
> drivers/gpu/drm/i915/gvt/execlist.c: In function 'release_shadow_batch_buffer':
> drivers/gpu/drm/i915/gvt/execlist.c:501:4: warning: 'drm_gem_object_unreference' is deprecated [-Wdeprecated-declarations]
>     drm_gem_object_unreference(&(entry_obj->obj->base));
>     ^
> In file included from drivers/gpu/drm/i915/gvt/execlist.c:35:0:
> drivers/gpu/drm/i915/i915_drv.h:2344:13: note: declared here
>  extern void drm_gem_object_unreference(struct drm_gem_object *);
>              ^
> drivers/gpu/drm/i915/gvt/execlist.c: In function 'release_shadow_wa_ctx':
> drivers/gpu/drm/i915/gvt/execlist.c:514:2: warning: 'drm_gem_object_unreference' is deprecated [-Wdeprecated-declarations]
>   drm_gem_object_unreference(&(wa_ctx->indirect_ctx.obj->base));
>   ^
> In file included from drivers/gpu/drm/i915/gvt/execlist.c:35:0:
> drivers/gpu/drm/i915/i915_drv.h:2344:13: note: declared here
>  extern void drm_gem_object_unreference(struct drm_gem_object *);
>              ^
> drivers/gpu/drm/i915/gvt/execlist.c: In function 'init_vgpu_execlist':
> drivers/gpu/drm/i915/gvt/execlist.c:42:33: error: request for member 'mmio_base' in something not a structure or union
>   (gvt->dev_priv->engine[ring_id].mmio_base + (offset))
>                                  ^
> drivers/gpu/drm/i915/gvt/execlist.c:248:19: note: in expansi
> on of macro 'execlist_ring_mmio'
>   u32 status_reg = execlist_ring_mmio(vgpu->gvt, ring_id,
>                    ^
> drivers/gpu/drm/i915/gvt/execlist.c: In function 'release_sh
> adow_batch_buffer':
> drivers/gpu/drm/i915/gvt/execlist.c:501:4: warning: 'drm_gem
> _object_unreference' is deprecated [-Wdeprecated-declarations]
>     drm_gem_object_unreference(&(entry_obj->obj->base));
>     ^
> In file included from drivers/gpu/drm/i915/gvt/execlist.c:35
> :0:
> drivers/gpu/drm/i915/i915_drv.h:2344:13: note: declared here
>  extern void drm_gem_object_unreference(struct drm_gem_object *);
>              ^
> drivers/gpu/drm/i915/gvt/execlist.c: In function 'release_sh
> adow_wa_ctx':
> drivers/gpu/drm/i915/gvt/execlist.c:514:2: warning: 'drm_gem_object_unreference' is deprecated [-Wdeprecated-declarations]
>   drm_gem_object_unreference(&(wa_ctx->indirect_ctx.obj->base));
>   ^
> In file included from drivers/gpu/drm/i915/gvt/execlist.c:35:0:
> drivers/gpu/drm/i915/i915_drv.h:2344:13: note: declared here
>  extern void drm_gem_object_unreference(struct drm_gem_object *);
>              ^
> drivers/gpu/drm/i915/gvt/execlist.c: In function 'init_vgpu_execlist':
> drivers/gpu/drm/i915/gvt/execlist.c:42:33: error: request for member 'mmio_base' in something not a structure or union
>   (gvt->dev_priv->engine[ring_id].mmio_base + (offset))
>                                  ^
> drivers/gpu/drm/i915/gvt/execlist.c:798:23: note: in expansion of macro 'execlist_ring_mmio'
>   ctx_status_ptr_reg = execlist_ring_mmio(vgpu->gvt, ring_id,
>                        ^
> 
> Caused by commit
> 
>   3b3f1650b1ca ("drm/i915: Allocate intel_engine_cs structure only for the enabled engines")
> 
> interacting with some other commits that were merged without fixes being applied (probably) in commit
> 
>   06a75ace46e2 ("Merge tag 'gvt-next-2016-10-14' of https://github.com/01org/gvt-linux into drm-intel-next-queued")
> 
> I have used the version of the drm-intel tree from next-20161017 for
> today.

Yeah, I forgot to compile test with DRM_I915_GVT=y after the merge :(
Pushing out the fixup patch right now. Sorry for the mess.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2016-10-18  6:26 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-17 23:59 linux-next: build failure after merge of the drm-intel tree Stephen Rothwell
2016-10-18  6:26 ` Daniel Vetter [this message]
  -- strict thread matches above, loose matches on Subject: below --
2026-03-16 17:15 Mark Brown
2026-03-17  9:00 ` Jani Nikula
2024-08-29  3:46 Stephen Rothwell
2024-06-12  4:12 Stephen Rothwell
2024-06-17 19:19 ` Rodrigo Vivi
2024-06-27 21:48   ` Dave Airlie
2024-06-28  9:29     ` Jani Nikula
2022-08-26  2:36 Stephen Rothwell
2022-08-26  7:50 ` Jani Nikula
2022-04-26  2:08 Stephen Rothwell
2022-04-26  7:36 ` Jani Nikula
2022-02-15  1:10 Stephen Rothwell
2022-02-15 12:11 ` Jani Nikula
2021-05-21  1:58 Stephen Rothwell
2021-01-19 23:57 Stephen Rothwell
2021-01-21  1:11 ` Stephen Rothwell
2019-12-09 22:39 Stephen Rothwell
2019-12-09 22:42 ` Stephen Rothwell
2019-12-11  9:31   ` Jani Nikula
2019-03-27  2:59 Stephen Rothwell
2019-03-27 11:07 ` Joonas Lahtinen
2019-03-27 11:31   ` Stephen Rothwell
2018-07-13  1:54 Stephen Rothwell
2018-07-23  2:29 ` Stephen Rothwell
2018-05-08  1:07 Stephen Rothwell
2018-05-08  7:08 ` Oleksandr Andrushchenko
2018-05-09  5:02 ` Stephen Rothwell
2018-05-09  5:08   ` Christoph Hellwig
2018-05-09  5:18     ` Stephen Rothwell
2016-08-23  1:56 Stephen Rothwell
2016-08-23 16:33 ` Daniel Vetter
2016-05-23  2:24 Stephen Rothwell
2014-04-03  1:24 Stephen Rothwell
2014-03-19  0:53 Stephen Rothwell
2014-03-19  1:18 ` Steven Rostedt
2013-02-15  2:37 Stephen Rothwell
2013-02-15  9:30 ` Daniel Vetter
2013-02-15 16:16   ` Jesse Barnes
2013-02-19  2:01     ` Stephen Rothwell
2013-02-19 10:28       ` Daniel Vetter

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=20161018062600.GS20761@phenom.ffwll.local \
    --to=daniel@ffwll.ch \
    --cc=akash.goel@intel.com \
    --cc=daniel.vetter@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=sfr@canb.auug.org.au \
    --cc=tvrtko.ursulin@intel.com \
    --cc=zhi.a.wang@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox