drm/i915/guc: Rename i915_guc_submission.c|h to intel_guc_submission.c|h
authorSagar Arun Kamble <sagar.a.kamble@intel.com>
Thu, 16 Nov 2017 13:32:41 +0000 (19:02 +0530)
committerChris Wilson <chris@chris-wilson.co.uk>
Thu, 16 Nov 2017 15:06:18 +0000 (15:06 +0000)
With all component structures and functions named appropriately, change
the names of GuC submission source files. There were bunch of style issues
in guc_submission.c that are highlighted now by checkpatch. Fix those.
Update name in Documentation/gpu. (Joonas)

v2: Rebase.

v3: Rebase.

Signed-off-by: Sagar Arun Kamble <sagar.a.kamble@intel.com>
Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Michal Winiarski <michal.winiarski@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Acked-by: Chris Wilson <chris@chris-wilson.co.uk>
Acked-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Acked-by: Oscar Mateo <oscar.mateo@intel.com>
Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/1510839162-25197-6-git-send-email-sagar.a.kamble@intel.com
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Documentation/gpu/i915.rst
drivers/gpu/drm/i915/Makefile
drivers/gpu/drm/i915/i915_debugfs.c
drivers/gpu/drm/i915/i915_guc_submission.c [deleted file]
drivers/gpu/drm/i915/i915_guc_submission.h [deleted file]
drivers/gpu/drm/i915/intel_guc.c
drivers/gpu/drm/i915/intel_guc_submission.c [new file with mode: 0644]
drivers/gpu/drm/i915/intel_guc_submission.h [new file with mode: 0644]
drivers/gpu/drm/i915/intel_uc.c

index 2e7ee0313c1cd6c0377d2b4a3f4948641f9a84a2..21577eabaf78586fb35b3408d2a34cfa820c700f 100644 (file)
@@ -350,10 +350,10 @@ GuC-specific firmware loader
 GuC-based command submission
 ----------------------------
 
-.. kernel-doc:: drivers/gpu/drm/i915/i915_guc_submission.c
+.. kernel-doc:: drivers/gpu/drm/i915/intel_guc_submission.c
    :doc: GuC-based command submission
 
-.. kernel-doc:: drivers/gpu/drm/i915/i915_guc_submission.c
+.. kernel-doc:: drivers/gpu/drm/i915/intel_guc_submission.c
    :internal:
 
 GuC Firmware Layout
index f7afd44214b5120e973da9feb70c2fae6afd56cb..c3649ec5b041cb28e94b5e415dad73e4fa9ece8b 100644 (file)
@@ -82,10 +82,10 @@ i915-y += intel_uc.o \
          intel_uc_fw.o \
          intel_guc.o \
          intel_guc_ct.o \
-         intel_guc_log.o \
          intel_guc_fw.o \
-         intel_huc.o \
-         i915_guc_submission.o
+         intel_guc_log.o \
+         intel_guc_submission.o \
+         intel_huc.o
 
 # autogenerated null render state
 i915-y += intel_renderstate_gen6.o \
index ce023587dd30c4b6822fd533d22e38f4a37a5569..ff8f508a7661485edf3e4ee101258c2558714f39 100644 (file)
@@ -30,7 +30,7 @@
 #include <linux/sort.h>
 #include <linux/sched/mm.h>
 #include "intel_drv.h"
-#include "i915_guc_submission.h"
+#include "intel_guc_submission.h"
 
 static inline struct drm_i915_private *node_to_i915(struct drm_info_node *node)
 {
diff --git a/drivers/gpu/drm/i915/i915_guc_submission.c b/drivers/gpu/drm/i915/i915_guc_submission.c
deleted file mode 100644 (file)
index 22dbc25..0000000
+++ /dev/null
@@ -1,1463 +0,0 @@
-/*
- * Copyright © 2014 Intel Corporation
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the "Software"),
- * to deal in the Software without restriction, including without limitation
- * the rights to use, copy, modify, merge, publish, distribute, sublicense,
- * and/or sell copies of the Software, and to permit persons to whom the
- * Software is furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice (including the next
- * paragraph) shall be included in all copies or substantial portions of the
- * Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
- * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
- * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
- * IN THE SOFTWARE.
- *
- */
-
-#include <linux/circ_buf.h>
-#include <trace/events/dma_fence.h>
-
-#include "i915_guc_submission.h"
-#include "i915_drv.h"
-
-/**
- * DOC: GuC-based command submission
- *
- * GuC client:
- * A intel_guc_client refers to a submission path through GuC. Currently, there
- * are two clients. One of them (the execbuf_client) is charged with all
- * submissions to the GuC, the other one (preempt_client) is responsible for
- * preempting the execbuf_client. This struct is the owner of a doorbell, a
- * process descriptor and a workqueue (all of them inside a single gem object
- * that contains all required pages for these elements).
- *
- * GuC stage descriptor:
- * During initialization, the driver allocates a static pool of 1024 such
- * descriptors, and shares them with the GuC.
- * Currently, there exists a 1:1 mapping between a intel_guc_client and a
- * guc_stage_desc (via the client's stage_id), so effectively only one
- * gets used. This stage descriptor lets the GuC know about the doorbell,
- * workqueue and process descriptor. Theoretically, it also lets the GuC
- * know about our HW contexts (context ID, etc...), but we actually
- * employ a kind of submission where the GuC uses the LRCA sent via the work
- * item instead (the single guc_stage_desc associated to execbuf client
- * contains information about the default kernel context only, but this is
- * essentially unused). This is called a "proxy" submission.
- *
- * The Scratch registers:
- * There are 16 MMIO-based registers start from 0xC180. The kernel driver writes
- * a value to the action register (SOFT_SCRATCH_0) along with any data. It then
- * triggers an interrupt on the GuC via another register write (0xC4C8).
- * Firmware writes a success/fail code back to the action register after
- * processes the request. The kernel driver polls waiting for this update and
- * then proceeds.
- * See intel_guc_send()
- *
- * Doorbells:
- * Doorbells are interrupts to uKernel. A doorbell is a single cache line (QW)
- * mapped into process space.
- *
- * Work Items:
- * There are several types of work items that the host may place into a
- * workqueue, each with its own requirements and limitations. Currently only
- * WQ_TYPE_INORDER is needed to support legacy submission via GuC, which
- * represents in-order queue. The kernel driver packs ring tail pointer and an
- * ELSP context descriptor dword into Work Item.
- * See guc_add_request()
- *
- * ADS:
- * The Additional Data Struct (ADS) has pointers for different buffers used by
- * the GuC. One single gem object contains the ADS struct itself (guc_ads), the
- * scheduling policies (guc_policies), a structure describing a collection of
- * register sets (guc_mmio_reg_state) and some extra pages for the GuC to save
- * its internal state for sleep.
- *
- */
-
-static inline bool is_high_priority(struct intel_guc_client *client)
-{
-       return (client->priority == GUC_CLIENT_PRIORITY_KMD_HIGH ||
-               client->priority == GUC_CLIENT_PRIORITY_HIGH);
-}
-
-static int __reserve_doorbell(struct intel_guc_client *client)
-{
-       unsigned long offset;
-       unsigned long end;
-       u16 id;
-
-       GEM_BUG_ON(client->doorbell_id != GUC_DOORBELL_INVALID);
-
-       /*
-        * The bitmap tracks which doorbell registers are currently in use.
-        * It is split into two halves; the first half is used for normal
-        * priority contexts, the second half for high-priority ones.
-        */
-       offset = 0;
-       end = GUC_NUM_DOORBELLS/2;
-       if (is_high_priority(client)) {
-               offset = end;
-               end += offset;
-       }
-
-       id = find_next_zero_bit(client->guc->doorbell_bitmap, end, offset);
-       if (id == end)
-               return -ENOSPC;
-
-       __set_bit(id, client->guc->doorbell_bitmap);
-       client->doorbell_id = id;
-       DRM_DEBUG_DRIVER("client %u (high prio=%s) reserved doorbell: %d\n",
-                        client->stage_id, yesno(is_high_priority(client)),
-                        id);
-       return 0;
-}
-
-static void __unreserve_doorbell(struct intel_guc_client *client)
-{
-       GEM_BUG_ON(client->doorbell_id == GUC_DOORBELL_INVALID);
-
-       __clear_bit(client->doorbell_id, client->guc->doorbell_bitmap);
-       client->doorbell_id = GUC_DOORBELL_INVALID;
-}
-
-/*
- * Tell the GuC to allocate or deallocate a specific doorbell
- */
-
-static int __guc_allocate_doorbell(struct intel_guc *guc, u32 stage_id)
-{
-       u32 action[] = {
-               INTEL_GUC_ACTION_ALLOCATE_DOORBELL,
-               stage_id
-       };
-
-       return intel_guc_send(guc, action, ARRAY_SIZE(action));
-}
-
-static int __guc_deallocate_doorbell(struct intel_guc *guc, u32 stage_id)
-{
-       u32 action[] = {
-               INTEL_GUC_ACTION_DEALLOCATE_DOORBELL,
-               stage_id
-       };
-
-       return intel_guc_send(guc, action, ARRAY_SIZE(action));
-}
-
-static struct guc_stage_desc *__get_stage_desc(struct intel_guc_client *client)
-{
-       struct guc_stage_desc *base = client->guc->stage_desc_pool_vaddr;
-
-       return &base[client->stage_id];
-}
-
-/*
- * Initialise, update, or clear doorbell data shared with the GuC
- *
- * These functions modify shared data and so need access to the mapped
- * client object which contains the page being used for the doorbell
- */
-
-static void __update_doorbell_desc(struct intel_guc_client *client, u16 new_id)
-{
-       struct guc_stage_desc *desc;
-
-       /* Update the GuC's idea of the doorbell ID */
-       desc = __get_stage_desc(client);
-       desc->db_id = new_id;
-}
-
-static struct guc_doorbell_info *__get_doorbell(struct intel_guc_client *client)
-{
-       return client->vaddr + client->doorbell_offset;
-}
-
-static bool has_doorbell(struct intel_guc_client *client)
-{
-       if (client->doorbell_id == GUC_DOORBELL_INVALID)
-               return false;
-
-       return test_bit(client->doorbell_id, client->guc->doorbell_bitmap);
-}
-
-static int __create_doorbell(struct intel_guc_client *client)
-{
-       struct guc_doorbell_info *doorbell;
-       int err;
-
-       doorbell = __get_doorbell(client);
-       doorbell->db_status = GUC_DOORBELL_ENABLED;
-       doorbell->cookie = 0;
-
-       err = __guc_allocate_doorbell(client->guc, client->stage_id);
-       if (err) {
-               doorbell->db_status = GUC_DOORBELL_DISABLED;
-               DRM_ERROR("Couldn't create client %u doorbell: %d\n",
-                         client->stage_id, err);
-       }
-
-       return err;
-}
-
-static int __destroy_doorbell(struct intel_guc_client *client)
-{
-       struct drm_i915_private *dev_priv = guc_to_i915(client->guc);
-       struct guc_doorbell_info *doorbell;
-       u16 db_id = client->doorbell_id;
-
-       GEM_BUG_ON(db_id >= GUC_DOORBELL_INVALID);
-
-       doorbell = __get_doorbell(client);
-       doorbell->db_status = GUC_DOORBELL_DISABLED;
-       doorbell->cookie = 0;
-
-       /* Doorbell release flow requires that we wait for GEN8_DRB_VALID bit
-        * to go to zero after updating db_status before we call the GuC to
-        * release the doorbell */
-       if (wait_for_us(!(I915_READ(GEN8_DRBREGL(db_id)) & GEN8_DRB_VALID), 10))
-               WARN_ONCE(true, "Doorbell never became invalid after disable\n");
-
-       return __guc_deallocate_doorbell(client->guc, client->stage_id);
-}
-
-static int create_doorbell(struct intel_guc_client *client)
-{
-       int ret;
-
-       ret = __reserve_doorbell(client);
-       if (ret)
-               return ret;
-
-       __update_doorbell_desc(client, client->doorbell_id);
-
-       ret = __create_doorbell(client);
-       if (ret)
-               goto err;
-
-       return 0;
-
-err:
-       __update_doorbell_desc(client, GUC_DOORBELL_INVALID);
-       __unreserve_doorbell(client);
-       return ret;
-}
-
-static int destroy_doorbell(struct intel_guc_client *client)
-{
-       int err;
-
-       GEM_BUG_ON(!has_doorbell(client));
-
-       /* XXX: wait for any interrupts */
-       /* XXX: wait for workqueue to drain */
-
-       err = __destroy_doorbell(client);
-       if (err)
-               return err;
-
-       __update_doorbell_desc(client, GUC_DOORBELL_INVALID);
-
-       __unreserve_doorbell(client);
-
-       return 0;
-}
-
-static unsigned long __select_cacheline(struct intel_guc* guc)
-{
-       unsigned long offset;
-
-       /* Doorbell uses a single cache line within a page */
-       offset = offset_in_page(guc->db_cacheline);
-
-       /* Moving to next cache line to reduce contention */
-       guc->db_cacheline += cache_line_size();
-
-       DRM_DEBUG_DRIVER("reserved cacheline 0x%lx, next 0x%x, linesize %u\n",
-                       offset, guc->db_cacheline, cache_line_size());
-       return offset;
-}
-
-static inline struct guc_process_desc *
-__get_process_desc(struct intel_guc_client *client)
-{
-       return client->vaddr + client->proc_desc_offset;
-}
-
-/*
- * Initialise the process descriptor shared with the GuC firmware.
- */
-static void guc_proc_desc_init(struct intel_guc *guc,
-                              struct intel_guc_client *client)
-{
-       struct guc_process_desc *desc;
-
-       desc = memset(__get_process_desc(client), 0, sizeof(*desc));
-
-       /*
-        * XXX: pDoorbell and WQVBaseAddress are pointers in process address
-        * space for ring3 clients (set them as in mmap_ioctl) or kernel
-        * space for kernel clients (map on demand instead? May make debug
-        * easier to have it mapped).
-        */
-       desc->wq_base_addr = 0;
-       desc->db_base_addr = 0;
-
-       desc->stage_id = client->stage_id;
-       desc->wq_size_bytes = GUC_WQ_SIZE;
-       desc->wq_status = WQ_STATUS_ACTIVE;
-       desc->priority = client->priority;
-}
-
-static int guc_stage_desc_pool_create(struct intel_guc *guc)
-{
-       struct i915_vma *vma;
-       void *vaddr;
-
-       vma = intel_guc_allocate_vma(guc,
-                                    PAGE_ALIGN(sizeof(struct guc_stage_desc) *
-                                    GUC_MAX_STAGE_DESCRIPTORS));
-       if (IS_ERR(vma))
-               return PTR_ERR(vma);
-
-       vaddr = i915_gem_object_pin_map(vma->obj, I915_MAP_WB);
-       if (IS_ERR(vaddr)) {
-               i915_vma_unpin_and_release(&vma);
-               return PTR_ERR(vaddr);
-       }
-
-       guc->stage_desc_pool = vma;
-       guc->stage_desc_pool_vaddr = vaddr;
-       ida_init(&guc->stage_ids);
-
-       return 0;
-}
-
-static void guc_stage_desc_pool_destroy(struct intel_guc *guc)
-{
-       ida_destroy(&guc->stage_ids);
-       i915_gem_object_unpin_map(guc->stage_desc_pool->obj);
-       i915_vma_unpin_and_release(&guc->stage_desc_pool);
-}
-
-/*
- * Initialise/clear the stage descriptor shared with the GuC firmware.
- *
- * This descriptor tells the GuC where (in GGTT space) to find the important
- * data structures relating to this client (doorbell, process descriptor,
- * write queue, etc).
- */
-static void guc_stage_desc_init(struct intel_guc *guc,
-                               struct intel_guc_client *client)
-{
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
-       struct intel_engine_cs *engine;
-       struct i915_gem_context *ctx = client->owner;
-       struct guc_stage_desc *desc;
-       unsigned int tmp;
-       u32 gfx_addr;
-
-       desc = __get_stage_desc(client);
-       memset(desc, 0, sizeof(*desc));
-
-       desc->attribute = GUC_STAGE_DESC_ATTR_ACTIVE | GUC_STAGE_DESC_ATTR_KERNEL;
-       if (is_high_priority(client))
-               desc->attribute |= GUC_STAGE_DESC_ATTR_PREEMPT;
-       desc->stage_id = client->stage_id;
-       desc->priority = client->priority;
-       desc->db_id = client->doorbell_id;
-
-       for_each_engine_masked(engine, dev_priv, client->engines, tmp) {
-               struct intel_context *ce = &ctx->engine[engine->id];
-               u32 guc_engine_id = engine->guc_id;
-               struct guc_execlist_context *lrc = &desc->lrc[guc_engine_id];
-
-               /* TODO: We have a design issue to be solved here. Only when we
-                * receive the first batch, we know which engine is used by the
-                * user. But here GuC expects the lrc and ring to be pinned. It
-                * is not an issue for default context, which is the only one
-                * for now who owns a GuC client. But for future owner of GuC
-                * client, need to make sure lrc is pinned prior to enter here.
-                */
-               if (!ce->state)
-                       break;  /* XXX: continue? */
-
-               /*
-                * XXX: When this is a GUC_STAGE_DESC_ATTR_KERNEL client (proxy
-                * submission or, in other words, not using a direct submission
-                * model) the KMD's LRCA is not used for any work submission.
-                * Instead, the GuC uses the LRCA of the user mode context (see
-                * guc_add_request below).
-                */
-               lrc->context_desc = lower_32_bits(ce->lrc_desc);
-
-               /* The state page is after PPHWSP */
-               lrc->ring_lrca =
-                       guc_ggtt_offset(ce->state) + LRC_STATE_PN * PAGE_SIZE;
-
-               /* XXX: In direct submission, the GuC wants the HW context id
-                * here. In proxy submission, it wants the stage id */
-               lrc->context_id = (client->stage_id << GUC_ELC_CTXID_OFFSET) |
-                               (guc_engine_id << GUC_ELC_ENGINE_OFFSET);
-
-               lrc->ring_begin = guc_ggtt_offset(ce->ring->vma);
-               lrc->ring_end = lrc->ring_begin + ce->ring->size - 1;
-               lrc->ring_next_free_location = lrc->ring_begin;
-               lrc->ring_current_tail_pointer_value = 0;
-
-               desc->engines_used |= (1 << guc_engine_id);
-       }
-
-       DRM_DEBUG_DRIVER("Host engines 0x%x => GuC engines used 0x%x\n",
-                       client->engines, desc->engines_used);
-       WARN_ON(desc->engines_used == 0);
-
-       /*
-        * The doorbell, process descriptor, and workqueue are all parts
-        * of the client object, which the GuC will reference via the GGTT
-        */
-       gfx_addr = guc_ggtt_offset(client->vma);
-       desc->db_trigger_phy = sg_dma_address(client->vma->pages->sgl) +
-                               client->doorbell_offset;
-       desc->db_trigger_cpu = ptr_to_u64(__get_doorbell(client));
-       desc->db_trigger_uk = gfx_addr + client->doorbell_offset;
-       desc->process_desc = gfx_addr + client->proc_desc_offset;
-       desc->wq_addr = gfx_addr + GUC_DB_SIZE;
-       desc->wq_size = GUC_WQ_SIZE;
-
-       desc->desc_private = ptr_to_u64(client);
-}
-
-static void guc_stage_desc_fini(struct intel_guc *guc,
-                               struct intel_guc_client *client)
-{
-       struct guc_stage_desc *desc;
-
-       desc = __get_stage_desc(client);
-       memset(desc, 0, sizeof(*desc));
-}
-
-static int guc_shared_data_create(struct intel_guc *guc)
-{
-       struct i915_vma *vma;
-       void *vaddr;
-
-       vma = intel_guc_allocate_vma(guc, PAGE_SIZE);
-       if (IS_ERR(vma))
-               return PTR_ERR(vma);
-
-       vaddr = i915_gem_object_pin_map(vma->obj, I915_MAP_WB);
-       if (IS_ERR(vaddr)) {
-               i915_vma_unpin_and_release(&vma);
-               return PTR_ERR(vaddr);
-       }
-
-       guc->shared_data = vma;
-       guc->shared_data_vaddr = vaddr;
-
-       return 0;
-}
-
-static void guc_shared_data_destroy(struct intel_guc *guc)
-{
-       i915_gem_object_unpin_map(guc->shared_data->obj);
-       i915_vma_unpin_and_release(&guc->shared_data);
-}
-
-/* Construct a Work Item and append it to the GuC's Work Queue */
-static void guc_wq_item_append(struct intel_guc_client *client,
-                              u32 target_engine, u32 context_desc,
-                              u32 ring_tail, u32 fence_id)
-{
-       /* wqi_len is in DWords, and does not include the one-word header */
-       const size_t wqi_size = sizeof(struct guc_wq_item);
-       const u32 wqi_len = wqi_size / sizeof(u32) - 1;
-       struct guc_process_desc *desc = __get_process_desc(client);
-       struct guc_wq_item *wqi;
-       u32 wq_off;
-
-       lockdep_assert_held(&client->wq_lock);
-
-       /* For now workqueue item is 4 DWs; workqueue buffer is 2 pages. So we
-        * should not have the case where structure wqi is across page, neither
-        * wrapped to the beginning. This simplifies the implementation below.
-        *
-        * XXX: if not the case, we need save data to a temp wqi and copy it to
-        * workqueue buffer dw by dw.
-        */
-       BUILD_BUG_ON(wqi_size != 16);
-
-       /* Free space is guaranteed. */
-       wq_off = READ_ONCE(desc->tail);
-       GEM_BUG_ON(CIRC_SPACE(wq_off, READ_ONCE(desc->head),
-                             GUC_WQ_SIZE) < wqi_size);
-       GEM_BUG_ON(wq_off & (wqi_size - 1));
-
-       /* WQ starts from the page after doorbell / process_desc */
-       wqi = client->vaddr + wq_off + GUC_DB_SIZE;
-
-       /* Now fill in the 4-word work queue item */
-       wqi->header = WQ_TYPE_INORDER |
-                     (wqi_len << WQ_LEN_SHIFT) |
-                     (target_engine << WQ_TARGET_SHIFT) |
-                     WQ_NO_WCFLUSH_WAIT;
-       wqi->context_desc = context_desc;
-       wqi->submit_element_info = ring_tail << WQ_RING_TAIL_SHIFT;
-       GEM_BUG_ON(ring_tail > WQ_RING_TAIL_MAX);
-       wqi->fence_id = fence_id;
-
-       /* Make the update visible to GuC */
-       WRITE_ONCE(desc->tail, (wq_off + wqi_size) & (GUC_WQ_SIZE - 1));
-}
-
-static void guc_reset_wq(struct intel_guc_client *client)
-{
-       struct guc_process_desc *desc = __get_process_desc(client);
-
-       desc->head = 0;
-       desc->tail = 0;
-}
-
-static void guc_ring_doorbell(struct intel_guc_client *client)
-{
-       struct guc_doorbell_info *db;
-       u32 cookie;
-
-       lockdep_assert_held(&client->wq_lock);
-
-       /* pointer of current doorbell cacheline */
-       db = __get_doorbell(client);
-
-       /*
-        * We're not expecting the doorbell cookie to change behind our back,
-        * we also need to treat 0 as a reserved value.
-        */
-       cookie = READ_ONCE(db->cookie);
-       WARN_ON_ONCE(xchg(&db->cookie, cookie + 1 ?: cookie + 2) != cookie);
-
-       /* XXX: doorbell was lost and need to acquire it again */
-       GEM_BUG_ON(db->db_status != GUC_DOORBELL_ENABLED);
-}
-
-static void guc_add_request(struct intel_guc *guc,
-                           struct drm_i915_gem_request *rq)
-{
-       struct intel_guc_client *client = guc->execbuf_client;
-       struct intel_engine_cs *engine = rq->engine;
-       u32 ctx_desc = lower_32_bits(intel_lr_context_descriptor(rq->ctx, engine));
-       u32 ring_tail = intel_ring_set_tail(rq->ring, rq->tail) / sizeof(u64);
-
-       spin_lock(&client->wq_lock);
-
-       guc_wq_item_append(client, engine->guc_id, ctx_desc,
-                          ring_tail, rq->global_seqno);
-       guc_ring_doorbell(client);
-
-       client->submissions[engine->id] += 1;
-
-       spin_unlock(&client->wq_lock);
-}
-
-/*
- * When we're doing submissions using regular execlists backend, writing to
- * ELSP from CPU side is enough to make sure that writes to ringbuffer pages
- * pinned in mappable aperture portion of GGTT are visible to command streamer.
- * Writes done by GuC on our behalf are not guaranteeing such ordering,
- * therefore, to ensure the flush, we're issuing a POSTING READ.
- */
-static void flush_ggtt_writes(struct i915_vma *vma)
-{
-       struct drm_i915_private *dev_priv = to_i915(vma->obj->base.dev);
-
-       if (i915_vma_is_map_and_fenceable(vma))
-               POSTING_READ_FW(GUC_STATUS);
-}
-
-#define GUC_PREEMPT_FINISHED 0x1
-#define GUC_PREEMPT_BREADCRUMB_DWORDS 0x8
-static void inject_preempt_context(struct work_struct *work)
-{
-       struct guc_preempt_work *preempt_work =
-               container_of(work, typeof(*preempt_work), work);
-       struct intel_engine_cs *engine = preempt_work->engine;
-       struct intel_guc *guc = container_of(preempt_work, typeof(*guc),
-                                            preempt_work[engine->id]);
-       struct intel_guc_client *client = guc->preempt_client;
-       struct guc_stage_desc *stage_desc = __get_stage_desc(client);
-       struct intel_ring *ring = client->owner->engine[engine->id].ring;
-       u32 ctx_desc = lower_32_bits(intel_lr_context_descriptor(client->owner,
-                                                                engine));
-       u32 *cs = ring->vaddr + ring->tail;
-       u32 data[7];
-
-       if (engine->id == RCS) {
-               cs = gen8_emit_ggtt_write_rcs(cs, GUC_PREEMPT_FINISHED,
-                               intel_hws_preempt_done_address(engine));
-       } else {
-               cs = gen8_emit_ggtt_write(cs, GUC_PREEMPT_FINISHED,
-                               intel_hws_preempt_done_address(engine));
-               *cs++ = MI_NOOP;
-               *cs++ = MI_NOOP;
-       }
-       *cs++ = MI_USER_INTERRUPT;
-       *cs++ = MI_NOOP;
-
-       GEM_BUG_ON(!IS_ALIGNED(ring->size,
-                              GUC_PREEMPT_BREADCRUMB_DWORDS * sizeof(u32)));
-       GEM_BUG_ON((void *)cs - (ring->vaddr + ring->tail) !=
-                  GUC_PREEMPT_BREADCRUMB_DWORDS * sizeof(u32));
-
-       ring->tail += GUC_PREEMPT_BREADCRUMB_DWORDS * sizeof(u32);
-       ring->tail &= (ring->size - 1);
-
-       flush_ggtt_writes(ring->vma);
-
-       spin_lock_irq(&client->wq_lock);
-       guc_wq_item_append(client, engine->guc_id, ctx_desc,
-                          ring->tail / sizeof(u64), 0);
-       spin_unlock_irq(&client->wq_lock);
-
-       /*
-        * If GuC firmware performs an engine reset while that engine had
-        * a preemption pending, it will set the terminated attribute bit
-        * on our preemption stage descriptor. GuC firmware retains all
-        * pending work items for a high-priority GuC client, unlike the
-        * normal-priority GuC client where work items are dropped. It
-        * wants to make sure the preempt-to-idle work doesn't run when
-        * scheduling resumes, and uses this bit to inform its scheduler
-        * and presumably us as well. Our job is to clear it for the next
-        * preemption after reset, otherwise that and future preemptions
-        * will never complete. We'll just clear it every time.
-        */
-       stage_desc->attribute &= ~GUC_STAGE_DESC_ATTR_TERMINATED;
-
-       data[0] = INTEL_GUC_ACTION_REQUEST_PREEMPTION;
-       data[1] = client->stage_id;
-       data[2] = INTEL_GUC_PREEMPT_OPTION_DROP_WORK_Q |
-                 INTEL_GUC_PREEMPT_OPTION_DROP_SUBMIT_Q;
-       data[3] = engine->guc_id;
-       data[4] = guc->execbuf_client->priority;
-       data[5] = guc->execbuf_client->stage_id;
-       data[6] = guc_ggtt_offset(guc->shared_data);
-
-       if (WARN_ON(intel_guc_send(guc, data, ARRAY_SIZE(data)))) {
-               execlists_clear_active(&engine->execlists,
-                                      EXECLISTS_ACTIVE_PREEMPT);
-               tasklet_schedule(&engine->execlists.tasklet);
-       }
-}
-
-/*
- * We're using user interrupt and HWSP value to mark that preemption has
- * finished and GPU is idle. Normally, we could unwind and continue similar to
- * execlists submission path. Unfortunately, with GuC we also need to wait for
- * it to finish its own postprocessing, before attempting to submit. Otherwise
- * GuC may silently ignore our submissions, and thus we risk losing request at
- * best, executing out-of-order and causing kernel panic at worst.
- */
-#define GUC_PREEMPT_POSTPROCESS_DELAY_MS 10
-static void wait_for_guc_preempt_report(struct intel_engine_cs *engine)
-{
-       struct intel_guc *guc = &engine->i915->guc;
-       struct guc_shared_ctx_data *data = guc->shared_data_vaddr;
-       struct guc_ctx_report *report =
-               &data->preempt_ctx_report[engine->guc_id];
-
-       WARN_ON(wait_for_atomic(report->report_return_status ==
-                               INTEL_GUC_REPORT_STATUS_COMPLETE,
-                               GUC_PREEMPT_POSTPROCESS_DELAY_MS));
-       /*
-        * GuC is expecting that we're also going to clear the affected context
-        * counter, let's also reset the return status to not depend on GuC
-        * resetting it after recieving another preempt action
-        */
-       report->affected_count = 0;
-       report->report_return_status = INTEL_GUC_REPORT_STATUS_UNKNOWN;
-}
-
-/**
- * guc_submit() - Submit commands through GuC
- * @engine: engine associated with the commands
- *
- * The only error here arises if the doorbell hardware isn't functioning
- * as expected, which really shouln't happen.
- */
-static void guc_submit(struct intel_engine_cs *engine)
-{
-       struct intel_guc *guc = &engine->i915->guc;
-       struct intel_engine_execlists * const execlists = &engine->execlists;
-       struct execlist_port *port = execlists->port;
-       unsigned int n;
-
-       for (n = 0; n < execlists_num_ports(execlists); n++) {
-               struct drm_i915_gem_request *rq;
-               unsigned int count;
-
-               rq = port_unpack(&port[n], &count);
-               if (rq && count == 0) {
-                       port_set(&port[n], port_pack(rq, ++count));
-
-                       flush_ggtt_writes(rq->ring->vma);
-
-                       guc_add_request(guc, rq);
-               }
-       }
-}
-
-static void port_assign(struct execlist_port *port,
-                       struct drm_i915_gem_request *rq)
-{
-       GEM_BUG_ON(rq == port_request(port));
-
-       if (port_isset(port))
-               i915_gem_request_put(port_request(port));
-
-       port_set(port, port_pack(i915_gem_request_get(rq), port_count(port)));
-}
-
-static void guc_dequeue(struct intel_engine_cs *engine)
-{
-       struct intel_engine_execlists * const execlists = &engine->execlists;
-       struct execlist_port *port = execlists->port;
-       struct drm_i915_gem_request *last = NULL;
-       const struct execlist_port * const last_port =
-               &execlists->port[execlists->port_mask];
-       bool submit = false;
-       struct rb_node *rb;
-
-       spin_lock_irq(&engine->timeline->lock);
-       rb = execlists->first;
-       GEM_BUG_ON(rb_first(&execlists->queue) != rb);
-
-       if (!rb)
-               goto unlock;
-
-       if (HAS_LOGICAL_RING_PREEMPTION(engine->i915) && port_isset(port)) {
-               struct guc_preempt_work *preempt_work =
-                       &engine->i915->guc.preempt_work[engine->id];
-
-               if (rb_entry(rb, struct i915_priolist, node)->priority >
-                   max(port_request(port)->priotree.priority, 0)) {
-                       execlists_set_active(execlists,
-                                            EXECLISTS_ACTIVE_PREEMPT);
-                       queue_work(engine->i915->guc.preempt_wq,
-                                  &preempt_work->work);
-                       goto unlock;
-               } else if (port_isset(last_port)) {
-                       goto unlock;
-               }
-
-               port++;
-       }
-
-       do {
-               struct i915_priolist *p = rb_entry(rb, typeof(*p), node);
-               struct drm_i915_gem_request *rq, *rn;
-
-               list_for_each_entry_safe(rq, rn, &p->requests, priotree.link) {
-                       if (last && rq->ctx != last->ctx) {
-                               if (port == last_port) {
-                                       __list_del_many(&p->requests,
-                                                       &rq->priotree.link);
-                                       goto done;
-                               }
-
-                               if (submit)
-                                       port_assign(port, last);
-                               port++;
-                       }
-
-                       INIT_LIST_HEAD(&rq->priotree.link);
-
-                       __i915_gem_request_submit(rq);
-                       trace_i915_gem_request_in(rq, port_index(port, execlists));
-                       last = rq;
-                       submit = true;
-               }
-
-               rb = rb_next(rb);
-               rb_erase(&p->node, &execlists->queue);
-               INIT_LIST_HEAD(&p->requests);
-               if (p->priority != I915_PRIORITY_NORMAL)
-                       kmem_cache_free(engine->i915->priorities, p);
-       } while (rb);
-done:
-       execlists->first = rb;
-       if (submit) {
-               port_assign(port, last);
-               execlists_set_active(execlists, EXECLISTS_ACTIVE_USER);
-               guc_submit(engine);
-       }
-unlock:
-       spin_unlock_irq(&engine->timeline->lock);
-}
-
-static void guc_submission_tasklet(unsigned long data)
-{
-       struct intel_engine_cs * const engine = (struct intel_engine_cs *)data;
-       struct intel_engine_execlists * const execlists = &engine->execlists;
-       struct execlist_port *port = execlists->port;
-       struct drm_i915_gem_request *rq;
-
-       rq = port_request(&port[0]);
-       while (rq && i915_gem_request_completed(rq)) {
-               trace_i915_gem_request_out(rq);
-               i915_gem_request_put(rq);
-
-               execlists_port_complete(execlists, port);
-
-               rq = port_request(&port[0]);
-       }
-       if (!rq)
-               execlists_clear_active(execlists, EXECLISTS_ACTIVE_USER);
-
-       if (execlists_is_active(execlists, EXECLISTS_ACTIVE_PREEMPT) &&
-           intel_read_status_page(engine, I915_GEM_HWS_PREEMPT_INDEX) ==
-           GUC_PREEMPT_FINISHED) {
-               execlists_cancel_port_requests(&engine->execlists);
-               execlists_unwind_incomplete_requests(execlists);
-
-               wait_for_guc_preempt_report(engine);
-
-               execlists_clear_active(execlists, EXECLISTS_ACTIVE_PREEMPT);
-               intel_write_status_page(engine, I915_GEM_HWS_PREEMPT_INDEX, 0);
-       }
-
-       if (!execlists_is_active(execlists, EXECLISTS_ACTIVE_PREEMPT))
-               guc_dequeue(engine);
-}
-
-/*
- * Everything below here is concerned with setup & teardown, and is
- * therefore not part of the somewhat time-critical batch-submission
- * path of guc_submit() above.
- */
-
-/* Check that a doorbell register is in the expected state */
-static bool doorbell_ok(struct intel_guc *guc, u16 db_id)
-{
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
-       u32 drbregl;
-       bool valid;
-
-       GEM_BUG_ON(db_id >= GUC_DOORBELL_INVALID);
-
-       drbregl = I915_READ(GEN8_DRBREGL(db_id));
-       valid = drbregl & GEN8_DRB_VALID;
-
-       if (test_bit(db_id, guc->doorbell_bitmap) == valid)
-               return true;
-
-       DRM_DEBUG_DRIVER("Doorbell %d has unexpected state (0x%x): valid=%s\n",
-                        db_id, drbregl, yesno(valid));
-
-       return false;
-}
-
-/*
- * If the GuC thinks that the doorbell is unassigned (e.g. because we reset and
- * reloaded the GuC FW) we can use this function to tell the GuC to reassign the
- * doorbell to the rightful owner.
- */
-static int __reset_doorbell(struct intel_guc_client *client, u16 db_id)
-{
-       int err;
-
-       __update_doorbell_desc(client, db_id);
-       err = __create_doorbell(client);
-       if (!err)
-               err = __destroy_doorbell(client);
-
-       return err;
-}
-
-/*
- * Set up & tear down each unused doorbell in turn, to ensure that all doorbell
- * HW is (re)initialised. For that end, we might have to borrow the first
- * client. Also, tell GuC about all the doorbells in use by all clients.
- * We do this because the KMD, the GuC and the doorbell HW can easily go out of
- * sync (e.g. we can reset the GuC, but not the doorbel HW).
- */
-static int guc_init_doorbell_hw(struct intel_guc *guc)
-{
-       struct intel_guc_client *client = guc->execbuf_client;
-       bool recreate_first_client = false;
-       u16 db_id;
-       int ret;
-
-       /* For unused doorbells, make sure they are disabled */
-       for_each_clear_bit(db_id, guc->doorbell_bitmap, GUC_NUM_DOORBELLS) {
-               if (doorbell_ok(guc, db_id))
-                       continue;
-
-               if (has_doorbell(client)) {
-                       /* Borrow execbuf_client (we will recreate it later) */
-                       destroy_doorbell(client);
-                       recreate_first_client = true;
-               }
-
-               ret = __reset_doorbell(client, db_id);
-               WARN(ret, "Doorbell %u reset failed, err %d\n", db_id, ret);
-       }
-
-       if (recreate_first_client) {
-               ret = __reserve_doorbell(client);
-               if (unlikely(ret)) {
-                       DRM_ERROR("Couldn't re-reserve first client db: %d\n", ret);
-                       return ret;
-               }
-
-               __update_doorbell_desc(client, client->doorbell_id);
-       }
-
-       /* Now for every client (and not only execbuf_client) make sure their
-        * doorbells are known by the GuC */
-       ret = __create_doorbell(guc->execbuf_client);
-       if (ret)
-               return ret;
-
-       ret = __create_doorbell(guc->preempt_client);
-       if (ret) {
-               __destroy_doorbell(guc->execbuf_client);
-               return ret;
-       }
-
-       /* Read back & verify all (used & unused) doorbell registers */
-       for (db_id = 0; db_id < GUC_NUM_DOORBELLS; ++db_id)
-               WARN_ON(!doorbell_ok(guc, db_id));
-
-       return 0;
-}
-
-/**
- * guc_client_alloc() - Allocate an intel_guc_client
- * @dev_priv:  driver private data structure
- * @engines:   The set of engines to enable for this client
- * @priority:  four levels priority _CRITICAL, _HIGH, _NORMAL and _LOW
- *             The kernel client to replace ExecList submission is created with
- *             NORMAL priority. Priority of a client for scheduler can be HIGH,
- *             while a preemption context can use CRITICAL.
- * @ctx:       the context that owns the client (we use the default render
- *             context)
- *
- * Return:     An intel_guc_client object if success, else NULL.
- */
-static struct intel_guc_client *
-guc_client_alloc(struct drm_i915_private *dev_priv,
-                u32 engines,
-                u32 priority,
-                struct i915_gem_context *ctx)
-{
-       struct intel_guc_client *client;
-       struct intel_guc *guc = &dev_priv->guc;
-       struct i915_vma *vma;
-       void *vaddr;
-       int ret;
-
-       client = kzalloc(sizeof(*client), GFP_KERNEL);
-       if (!client)
-               return ERR_PTR(-ENOMEM);
-
-       client->guc = guc;
-       client->owner = ctx;
-       client->engines = engines;
-       client->priority = priority;
-       client->doorbell_id = GUC_DOORBELL_INVALID;
-       spin_lock_init(&client->wq_lock);
-
-       ret = ida_simple_get(&guc->stage_ids, 0, GUC_MAX_STAGE_DESCRIPTORS,
-                               GFP_KERNEL);
-       if (ret < 0)
-               goto err_client;
-
-       client->stage_id = ret;
-
-       /* The first page is doorbell/proc_desc. Two followed pages are wq. */
-       vma = intel_guc_allocate_vma(guc, GUC_DB_SIZE + GUC_WQ_SIZE);
-       if (IS_ERR(vma)) {
-               ret = PTR_ERR(vma);
-               goto err_id;
-       }
-
-       /* We'll keep just the first (doorbell/proc) page permanently kmap'd. */
-       client->vma = vma;
-
-       vaddr = i915_gem_object_pin_map(vma->obj, I915_MAP_WB);
-       if (IS_ERR(vaddr)) {
-               ret = PTR_ERR(vaddr);
-               goto err_vma;
-       }
-       client->vaddr = vaddr;
-
-       client->doorbell_offset = __select_cacheline(guc);
-
-       /*
-        * Since the doorbell only requires a single cacheline, we can save
-        * space by putting the application process descriptor in the same
-        * page. Use the half of the page that doesn't include the doorbell.
-        */
-       if (client->doorbell_offset >= (GUC_DB_SIZE / 2))
-               client->proc_desc_offset = 0;
-       else
-               client->proc_desc_offset = (GUC_DB_SIZE / 2);
-
-       guc_proc_desc_init(guc, client);
-       guc_stage_desc_init(guc, client);
-
-       ret = create_doorbell(client);
-       if (ret)
-               goto err_vaddr;
-
-       DRM_DEBUG_DRIVER("new priority %u client %p for engine(s) 0x%x: stage_id %u\n",
-                        priority, client, client->engines, client->stage_id);
-       DRM_DEBUG_DRIVER("doorbell id %u, cacheline offset 0x%lx\n",
-                        client->doorbell_id, client->doorbell_offset);
-
-       return client;
-
-err_vaddr:
-       i915_gem_object_unpin_map(client->vma->obj);
-err_vma:
-       i915_vma_unpin_and_release(&client->vma);
-err_id:
-       ida_simple_remove(&guc->stage_ids, client->stage_id);
-err_client:
-       kfree(client);
-       return ERR_PTR(ret);
-}
-
-static void guc_client_free(struct intel_guc_client *client)
-{
-       /*
-        * XXX: wait for any outstanding submissions before freeing memory.
-        * Be sure to drop any locks
-        */
-
-       /* FIXME: in many cases, by the time we get here the GuC has been
-        * reset, so we cannot destroy the doorbell properly. Ignore the
-        * error message for now */
-       destroy_doorbell(client);
-       guc_stage_desc_fini(client->guc, client);
-       i915_gem_object_unpin_map(client->vma->obj);
-       i915_vma_unpin_and_release(&client->vma);
-       ida_simple_remove(&client->guc->stage_ids, client->stage_id);
-       kfree(client);
-}
-
-static int guc_clients_create(struct intel_guc *guc)
-{
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
-       struct intel_guc_client *client;
-
-       GEM_BUG_ON(guc->execbuf_client);
-       GEM_BUG_ON(guc->preempt_client);
-
-       client = guc_client_alloc(dev_priv,
-                                 INTEL_INFO(dev_priv)->ring_mask,
-                                 GUC_CLIENT_PRIORITY_KMD_NORMAL,
-                                 dev_priv->kernel_context);
-       if (IS_ERR(client)) {
-               DRM_ERROR("Failed to create GuC client for submission!\n");
-               return PTR_ERR(client);
-       }
-       guc->execbuf_client = client;
-
-       client = guc_client_alloc(dev_priv,
-                                 INTEL_INFO(dev_priv)->ring_mask,
-                                 GUC_CLIENT_PRIORITY_KMD_HIGH,
-                                 dev_priv->preempt_context);
-       if (IS_ERR(client)) {
-               DRM_ERROR("Failed to create GuC client for preemption!\n");
-               guc_client_free(guc->execbuf_client);
-               guc->execbuf_client = NULL;
-               return PTR_ERR(client);
-       }
-       guc->preempt_client = client;
-
-       return 0;
-}
-
-static void guc_clients_destroy(struct intel_guc *guc)
-{
-       struct intel_guc_client *client;
-
-       client = fetch_and_zero(&guc->execbuf_client);
-       guc_client_free(client);
-
-       client = fetch_and_zero(&guc->preempt_client);
-       guc_client_free(client);
-}
-
-static void guc_policy_init(struct guc_policy *policy)
-{
-       policy->execution_quantum = POLICY_DEFAULT_EXECUTION_QUANTUM_US;
-       policy->preemption_time = POLICY_DEFAULT_PREEMPTION_TIME_US;
-       policy->fault_time = POLICY_DEFAULT_FAULT_TIME_US;
-       policy->policy_flags = 0;
-}
-
-static void guc_policies_init(struct guc_policies *policies)
-{
-       struct guc_policy *policy;
-       u32 p, i;
-
-       policies->dpc_promote_time = POLICY_DEFAULT_DPC_PROMOTE_TIME_US;
-       policies->max_num_work_items = POLICY_MAX_NUM_WI;
-
-       for (p = 0; p < GUC_CLIENT_PRIORITY_NUM; p++) {
-               for (i = GUC_RENDER_ENGINE; i < GUC_MAX_ENGINES_NUM; i++) {
-                       policy = &policies->policy[p][i];
-
-                       guc_policy_init(policy);
-               }
-       }
-
-       policies->is_valid = 1;
-}
-
-/*
- * The first 80 dwords of the register state context, containing the
- * execlists and ppgtt registers.
- */
-#define LR_HW_CONTEXT_SIZE     (80 * sizeof(u32))
-
-static int guc_ads_create(struct intel_guc *guc)
-{
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
-       struct i915_vma *vma;
-       struct page *page;
-       /* The ads obj includes the struct itself and buffers passed to GuC */
-       struct {
-               struct guc_ads ads;
-               struct guc_policies policies;
-               struct guc_mmio_reg_state reg_state;
-               u8 reg_state_buffer[GUC_S3_SAVE_SPACE_PAGES * PAGE_SIZE];
-       } __packed *blob;
-       struct intel_engine_cs *engine;
-       enum intel_engine_id id;
-       const u32 skipped_offset = LRC_HEADER_PAGES * PAGE_SIZE;
-       const u32 skipped_size = LRC_PPHWSP_SZ * PAGE_SIZE + LR_HW_CONTEXT_SIZE;
-       u32 base;
-
-       GEM_BUG_ON(guc->ads_vma);
-
-       vma = intel_guc_allocate_vma(guc, PAGE_ALIGN(sizeof(*blob)));
-       if (IS_ERR(vma))
-               return PTR_ERR(vma);
-
-       guc->ads_vma = vma;
-
-       page = i915_vma_first_page(vma);
-       blob = kmap(page);
-
-       /* GuC scheduling policies */
-       guc_policies_init(&blob->policies);
-
-       /* MMIO reg state */
-       for_each_engine(engine, dev_priv, id) {
-               blob->reg_state.white_list[engine->guc_id].mmio_start =
-                       engine->mmio_base + GUC_MMIO_WHITE_LIST_START;
-
-               /* Nothing to be saved or restored for now. */
-               blob->reg_state.white_list[engine->guc_id].count = 0;
-       }
-
-       /*
-        * The GuC requires a "Golden Context" when it reinitialises
-        * engines after a reset. Here we use the Render ring default
-        * context, which must already exist and be pinned in the GGTT,
-        * so its address won't change after we've told the GuC where
-        * to find it. Note that we have to skip our header (1 page),
-        * because our GuC shared data is there.
-        */
-       blob->ads.golden_context_lrca =
-               guc_ggtt_offset(dev_priv->kernel_context->engine[RCS].state) + skipped_offset;
-
-       /*
-        * The GuC expects us to exclude the portion of the context image that
-        * it skips from the size it is to read. It starts reading from after
-        * the execlist context (so skipping the first page [PPHWSP] and 80
-        * dwords). Weird guc is weird.
-        */
-       for_each_engine(engine, dev_priv, id)
-               blob->ads.eng_state_size[engine->guc_id] = engine->context_size - skipped_size;
-
-       base = guc_ggtt_offset(vma);
-       blob->ads.scheduler_policies = base + ptr_offset(blob, policies);
-       blob->ads.reg_state_buffer = base + ptr_offset(blob, reg_state_buffer);
-       blob->ads.reg_state_addr = base + ptr_offset(blob, reg_state);
-
-       kunmap(page);
-
-       return 0;
-}
-
-static void guc_ads_destroy(struct intel_guc *guc)
-{
-       i915_vma_unpin_and_release(&guc->ads_vma);
-}
-
-static int guc_preempt_work_create(struct intel_guc *guc)
-{
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
-       struct intel_engine_cs *engine;
-       enum intel_engine_id id;
-
-       /*
-        * Even though both sending GuC action, and adding a new workitem to
-        * GuC workqueue are serialized (each with its own locking), since
-        * we're using mutliple engines, it's possible that we're going to
-        * issue a preempt request with two (or more - each for different
-        * engine) workitems in GuC queue. In this situation, GuC may submit
-        * all of them, which will make us very confused.
-        * Our preemption contexts may even already be complete - before we
-        * even had the chance to sent the preempt action to GuC!. Rather
-        * than introducing yet another lock, we can just use ordered workqueue
-        * to make sure we're always sending a single preemption request with a
-        * single workitem.
-        */
-       guc->preempt_wq = alloc_ordered_workqueue("i915-guc_preempt",
-                                                 WQ_HIGHPRI);
-       if (!guc->preempt_wq)
-               return -ENOMEM;
-
-       for_each_engine(engine, dev_priv, id) {
-               guc->preempt_work[id].engine = engine;
-               INIT_WORK(&guc->preempt_work[id].work, inject_preempt_context);
-       }
-
-       return 0;
-}
-
-static void guc_preempt_work_destroy(struct intel_guc *guc)
-{
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
-       struct intel_engine_cs *engine;
-       enum intel_engine_id id;
-
-       for_each_engine(engine, dev_priv, id)
-               cancel_work_sync(&guc->preempt_work[id].work);
-
-       destroy_workqueue(guc->preempt_wq);
-       guc->preempt_wq = NULL;
-}
-
-/*
- * Set up the memory resources to be shared with the GuC (via the GGTT)
- * at firmware loading time.
- */
-int intel_guc_submission_init(struct intel_guc *guc)
-{
-       int ret;
-
-       if (guc->stage_desc_pool)
-               return 0;
-
-       ret = guc_stage_desc_pool_create(guc);
-       if (ret)
-               return ret;
-       /*
-        * Keep static analysers happy, let them know that we allocated the
-        * vma after testing that it didn't exist earlier.
-        */
-       GEM_BUG_ON(!guc->stage_desc_pool);
-
-       ret = guc_shared_data_create(guc);
-       if (ret)
-               goto err_stage_desc_pool;
-       GEM_BUG_ON(!guc->shared_data);
-
-       ret = intel_guc_log_create(guc);
-       if (ret < 0)
-               goto err_shared_data;
-
-       ret = guc_preempt_work_create(guc);
-       if (ret)
-               goto err_log;
-       GEM_BUG_ON(!guc->preempt_wq);
-
-       ret = guc_ads_create(guc);
-       if (ret < 0)
-               goto err_wq;
-       GEM_BUG_ON(!guc->ads_vma);
-
-       return 0;
-
-err_wq:
-       guc_preempt_work_destroy(guc);
-err_log:
-       intel_guc_log_destroy(guc);
-err_shared_data:
-       guc_shared_data_destroy(guc);
-err_stage_desc_pool:
-       guc_stage_desc_pool_destroy(guc);
-       return ret;
-}
-
-void intel_guc_submission_fini(struct intel_guc *guc)
-{
-       guc_ads_destroy(guc);
-       guc_preempt_work_destroy(guc);
-       intel_guc_log_destroy(guc);
-       guc_shared_data_destroy(guc);
-       guc_stage_desc_pool_destroy(guc);
-}
-
-static void guc_interrupts_capture(struct drm_i915_private *dev_priv)
-{
-       struct intel_rps *rps = &dev_priv->gt_pm.rps;
-       struct intel_engine_cs *engine;
-       enum intel_engine_id id;
-       int irqs;
-
-       /* tell all command streamers to forward interrupts (but not vblank) to GuC */
-       irqs = _MASKED_BIT_ENABLE(GFX_INTERRUPT_STEERING);
-       for_each_engine(engine, dev_priv, id)
-               I915_WRITE(RING_MODE_GEN7(engine), irqs);
-
-       /* route USER_INTERRUPT to Host, all others are sent to GuC. */
-       irqs = GT_RENDER_USER_INTERRUPT << GEN8_RCS_IRQ_SHIFT |
-              GT_RENDER_USER_INTERRUPT << GEN8_BCS_IRQ_SHIFT;
-       /* These three registers have the same bit definitions */
-       I915_WRITE(GUC_BCS_RCS_IER, ~irqs);
-       I915_WRITE(GUC_VCS2_VCS1_IER, ~irqs);
-       I915_WRITE(GUC_WD_VECS_IER, ~irqs);
-
-       /*
-        * The REDIRECT_TO_GUC bit of the PMINTRMSK register directs all
-        * (unmasked) PM interrupts to the GuC. All other bits of this
-        * register *disable* generation of a specific interrupt.
-        *
-        * 'pm_intrmsk_mbz' indicates bits that are NOT to be set when
-        * writing to the PM interrupt mask register, i.e. interrupts
-        * that must not be disabled.
-        *
-        * If the GuC is handling these interrupts, then we must not let
-        * the PM code disable ANY interrupt that the GuC is expecting.
-        * So for each ENABLED (0) bit in this register, we must SET the
-        * bit in pm_intrmsk_mbz so that it's left enabled for the GuC.
-        * GuC needs ARAT expired interrupt unmasked hence it is set in
-        * pm_intrmsk_mbz.
-        *
-        * Here we CLEAR REDIRECT_TO_GUC bit in pm_intrmsk_mbz, which will
-        * result in the register bit being left SET!
-        */
-       rps->pm_intrmsk_mbz |= ARAT_EXPIRED_INTRMSK;
-       rps->pm_intrmsk_mbz &= ~GEN8_PMINTR_DISABLE_REDIRECT_TO_GUC;
-}
-
-static void guc_interrupts_release(struct drm_i915_private *dev_priv)
-{
-       struct intel_rps *rps = &dev_priv->gt_pm.rps;
-       struct intel_engine_cs *engine;
-       enum intel_engine_id id;
-       int irqs;
-
-       /*
-        * tell all command streamers NOT to forward interrupts or vblank
-        * to GuC.
-        */
-       irqs = _MASKED_FIELD(GFX_FORWARD_VBLANK_MASK, GFX_FORWARD_VBLANK_NEVER);
-       irqs |= _MASKED_BIT_DISABLE(GFX_INTERRUPT_STEERING);
-       for_each_engine(engine, dev_priv, id)
-               I915_WRITE(RING_MODE_GEN7(engine), irqs);
-
-       /* route all GT interrupts to the host */
-       I915_WRITE(GUC_BCS_RCS_IER, 0);
-       I915_WRITE(GUC_VCS2_VCS1_IER, 0);
-       I915_WRITE(GUC_WD_VECS_IER, 0);
-
-       rps->pm_intrmsk_mbz |= GEN8_PMINTR_DISABLE_REDIRECT_TO_GUC;
-       rps->pm_intrmsk_mbz &= ~ARAT_EXPIRED_INTRMSK;
-}
-
-static void guc_submission_park(struct intel_engine_cs *engine)
-{
-       intel_engine_unpin_breadcrumbs_irq(engine);
-}
-
-static void guc_submission_unpark(struct intel_engine_cs *engine)
-{
-       intel_engine_pin_breadcrumbs_irq(engine);
-}
-
-int intel_guc_submission_enable(struct intel_guc *guc)
-{
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
-       struct intel_engine_cs *engine;
-       enum intel_engine_id id;
-       int err;
-
-       /*
-        * We're using GuC work items for submitting work through GuC. Since
-        * we're coalescing multiple requests from a single context into a
-        * single work item prior to assigning it to execlist_port, we can
-        * never have more work items than the total number of ports (for all
-        * engines). The GuC firmware is controlling the HEAD of work queue,
-        * and it is guaranteed that it will remove the work item from the
-        * queue before our request is completed.
-        */
-       BUILD_BUG_ON(ARRAY_SIZE(engine->execlists.port) *
-                    sizeof(struct guc_wq_item) *
-                    I915_NUM_ENGINES > GUC_WQ_SIZE);
-
-       /*
-        * We're being called on both module initialization and on reset,
-        * until this flow is changed, we're using regular client presence to
-        * determine which case are we in, and whether we should allocate new
-        * clients or just reset their workqueues.
-        */
-       if (!guc->execbuf_client) {
-               err = guc_clients_create(guc);
-               if (err)
-                       return err;
-       } else {
-               guc_reset_wq(guc->execbuf_client);
-               guc_reset_wq(guc->preempt_client);
-       }
-
-       err = intel_guc_sample_forcewake(guc);
-       if (err)
-               goto err_free_clients;
-
-       err = guc_init_doorbell_hw(guc);
-       if (err)
-               goto err_free_clients;
-
-       /* Take over from manual control of ELSP (execlists) */
-       guc_interrupts_capture(dev_priv);
-
-       for_each_engine(engine, dev_priv, id) {
-               struct intel_engine_execlists * const execlists = &engine->execlists;
-               execlists->tasklet.func = guc_submission_tasklet;
-               engine->park = guc_submission_park;
-               engine->unpark = guc_submission_unpark;
-       }
-
-       return 0;
-
-err_free_clients:
-       guc_clients_destroy(guc);
-       return err;
-}
-
-void intel_guc_submission_disable(struct intel_guc *guc)
-{
-       struct drm_i915_private *dev_priv = guc_to_i915(guc);
-
-       GEM_BUG_ON(dev_priv->gt.awake); /* GT should be parked first */
-
-       guc_interrupts_release(dev_priv);
-
-       /* Revert back to manual ELSP submission */
-       intel_engines_reset_default_submission(dev_priv);
-
-       guc_clients_destroy(guc);
-}
diff --git a/drivers/gpu/drm/i915/i915_guc_submission.h b/drivers/gpu/drm/i915/i915_guc_submission.h
deleted file mode 100644 (file)
index f8ae258..0000000
+++ /dev/null
@@ -1,80 +0,0 @@
-/*
- * Copyright © 2014-2017 Intel Corporation
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the "Software"),
- * to deal in the Software without restriction, including without limitation
- * the rights to use, copy, modify, merge, publish, distribute, sublicense,
- * and/or sell copies of the Software, and to permit persons to whom the
- * Software is furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice (including the next
- * paragraph) shall be included in all copies or substantial portions of the
- * Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
- * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
- * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
- * IN THE SOFTWARE.
- *
- */
-
-#ifndef _I915_GUC_SUBMISSION_H_
-#define _I915_GUC_SUBMISSION_H_
-
-#include <linux/spinlock.h>
-
-#include "i915_gem.h"
-
-struct drm_i915_private;
-
-/*
- * This structure primarily describes the GEM object shared with the GuC.
- * The specs sometimes refer to this object as a "GuC context", but we use
- * the term "client" to avoid confusion with hardware contexts. This
- * GEM object is held for the entire lifetime of our interaction with
- * the GuC, being allocated before the GuC is loaded with its firmware.
- * Because there's no way to update the address used by the GuC after
- * initialisation, the shared object must stay pinned into the GGTT as
- * long as the GuC is in use. We also keep the first page (only) mapped
- * into kernel address space, as it includes shared data that must be
- * updated on every request submission.
- *
- * The single GEM object described here is actually made up of several
- * separate areas, as far as the GuC is concerned. The first page (kept
- * kmap'd) includes the "process descriptor" which holds sequence data for
- * the doorbell, and one cacheline which actually *is* the doorbell; a
- * write to this will "ring the doorbell" (i.e. send an interrupt to the
- * GuC). The subsequent  pages of the client object constitute the work
- * queue (a circular array of work items), again described in the process
- * descriptor. Work queue pages are mapped momentarily as required.
- */
-struct intel_guc_client {
-       struct i915_vma *vma;
-       void *vaddr;
-       struct i915_gem_context *owner;
-       struct intel_guc *guc;
-
-       /* bitmap of (host) engine ids */
-       u32 engines;
-       u32 priority;
-       u32 stage_id;
-       u32 proc_desc_offset;
-
-       u16 doorbell_id;
-       unsigned long doorbell_offset;
-
-       spinlock_t wq_lock;
-       /* Per-engine counts of GuC submissions */
-       u64 submissions[I915_NUM_ENGINES];
-};
-
-int intel_guc_submission_init(struct intel_guc *guc);
-int intel_guc_submission_enable(struct intel_guc *guc);
-void intel_guc_submission_disable(struct intel_guc *guc);
-void intel_guc_submission_fini(struct intel_guc *guc);
-
-#endif
index 9678630a1c70585d2cbe8d677f649a9b71c16ee4..823d0c2e9ad2f2027e498d386cebad5eb78309ae 100644 (file)
@@ -23,8 +23,8 @@
  */
 
 #include "intel_guc.h"
+#include "intel_guc_submission.h"
 #include "i915_drv.h"
-#include "i915_guc_submission.h"
 
 static void gen8_guc_raise_irq(struct intel_guc *guc)
 {
diff --git a/drivers/gpu/drm/i915/intel_guc_submission.c b/drivers/gpu/drm/i915/intel_guc_submission.c
new file mode 100644 (file)
index 0000000..9c9e6ed
--- /dev/null
@@ -0,0 +1,1477 @@
+/*
+ * Copyright © 2014 Intel Corporation
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ *
+ */
+
+#include <linux/circ_buf.h>
+#include <trace/events/dma_fence.h>
+
+#include "intel_guc_submission.h"
+#include "i915_drv.h"
+
+/**
+ * DOC: GuC-based command submission
+ *
+ * GuC client:
+ * A intel_guc_client refers to a submission path through GuC. Currently, there
+ * are two clients. One of them (the execbuf_client) is charged with all
+ * submissions to the GuC, the other one (preempt_client) is responsible for
+ * preempting the execbuf_client. This struct is the owner of a doorbell, a
+ * process descriptor and a workqueue (all of them inside a single gem object
+ * that contains all required pages for these elements).
+ *
+ * GuC stage descriptor:
+ * During initialization, the driver allocates a static pool of 1024 such
+ * descriptors, and shares them with the GuC.
+ * Currently, there exists a 1:1 mapping between a intel_guc_client and a
+ * guc_stage_desc (via the client's stage_id), so effectively only one
+ * gets used. This stage descriptor lets the GuC know about the doorbell,
+ * workqueue and process descriptor. Theoretically, it also lets the GuC
+ * know about our HW contexts (context ID, etc...), but we actually
+ * employ a kind of submission where the GuC uses the LRCA sent via the work
+ * item instead (the single guc_stage_desc associated to execbuf client
+ * contains information about the default kernel context only, but this is
+ * essentially unused). This is called a "proxy" submission.
+ *
+ * The Scratch registers:
+ * There are 16 MMIO-based registers start from 0xC180. The kernel driver writes
+ * a value to the action register (SOFT_SCRATCH_0) along with any data. It then
+ * triggers an interrupt on the GuC via another register write (0xC4C8).
+ * Firmware writes a success/fail code back to the action register after
+ * processes the request. The kernel driver polls waiting for this update and
+ * then proceeds.
+ * See intel_guc_send()
+ *
+ * Doorbells:
+ * Doorbells are interrupts to uKernel. A doorbell is a single cache line (QW)
+ * mapped into process space.
+ *
+ * Work Items:
+ * There are several types of work items that the host may place into a
+ * workqueue, each with its own requirements and limitations. Currently only
+ * WQ_TYPE_INORDER is needed to support legacy submission via GuC, which
+ * represents in-order queue. The kernel driver packs ring tail pointer and an
+ * ELSP context descriptor dword into Work Item.
+ * See guc_add_request()
+ *
+ * ADS:
+ * The Additional Data Struct (ADS) has pointers for different buffers used by
+ * the GuC. One single gem object contains the ADS struct itself (guc_ads), the
+ * scheduling policies (guc_policies), a structure describing a collection of
+ * register sets (guc_mmio_reg_state) and some extra pages for the GuC to save
+ * its internal state for sleep.
+ *
+ */
+
+static inline bool is_high_priority(struct intel_guc_client *client)
+{
+       return (client->priority == GUC_CLIENT_PRIORITY_KMD_HIGH ||
+               client->priority == GUC_CLIENT_PRIORITY_HIGH);
+}
+
+static int __reserve_doorbell(struct intel_guc_client *client)
+{
+       unsigned long offset;
+       unsigned long end;
+       u16 id;
+
+       GEM_BUG_ON(client->doorbell_id != GUC_DOORBELL_INVALID);
+
+       /*
+        * The bitmap tracks which doorbell registers are currently in use.
+        * It is split into two halves; the first half is used for normal
+        * priority contexts, the second half for high-priority ones.
+        */
+       offset = 0;
+       end = GUC_NUM_DOORBELLS / 2;
+       if (is_high_priority(client)) {
+               offset = end;
+               end += offset;
+       }
+
+       id = find_next_zero_bit(client->guc->doorbell_bitmap, end, offset);
+       if (id == end)
+               return -ENOSPC;
+
+       __set_bit(id, client->guc->doorbell_bitmap);
+       client->doorbell_id = id;
+       DRM_DEBUG_DRIVER("client %u (high prio=%s) reserved doorbell: %d\n",
+                        client->stage_id, yesno(is_high_priority(client)),
+                        id);
+       return 0;
+}
+
+static void __unreserve_doorbell(struct intel_guc_client *client)
+{
+       GEM_BUG_ON(client->doorbell_id == GUC_DOORBELL_INVALID);
+
+       __clear_bit(client->doorbell_id, client->guc->doorbell_bitmap);
+       client->doorbell_id = GUC_DOORBELL_INVALID;
+}
+
+/*
+ * Tell the GuC to allocate or deallocate a specific doorbell
+ */
+
+static int __guc_allocate_doorbell(struct intel_guc *guc, u32 stage_id)
+{
+       u32 action[] = {
+               INTEL_GUC_ACTION_ALLOCATE_DOORBELL,
+               stage_id
+       };
+
+       return intel_guc_send(guc, action, ARRAY_SIZE(action));
+}
+
+static int __guc_deallocate_doorbell(struct intel_guc *guc, u32 stage_id)
+{
+       u32 action[] = {
+               INTEL_GUC_ACTION_DEALLOCATE_DOORBELL,
+               stage_id
+       };
+
+       return intel_guc_send(guc, action, ARRAY_SIZE(action));
+}
+
+static struct guc_stage_desc *__get_stage_desc(struct intel_guc_client *client)
+{
+       struct guc_stage_desc *base = client->guc->stage_desc_pool_vaddr;
+
+       return &base[client->stage_id];
+}
+
+/*
+ * Initialise, update, or clear doorbell data shared with the GuC
+ *
+ * These functions modify shared data and so need access to the mapped
+ * client object which contains the page being used for the doorbell
+ */
+
+static void __update_doorbell_desc(struct intel_guc_client *client, u16 new_id)
+{
+       struct guc_stage_desc *desc;
+
+       /* Update the GuC's idea of the doorbell ID */
+       desc = __get_stage_desc(client);
+       desc->db_id = new_id;
+}
+
+static struct guc_doorbell_info *__get_doorbell(struct intel_guc_client *client)
+{
+       return client->vaddr + client->doorbell_offset;
+}
+
+static bool has_doorbell(struct intel_guc_client *client)
+{
+       if (client->doorbell_id == GUC_DOORBELL_INVALID)
+               return false;
+
+       return test_bit(client->doorbell_id, client->guc->doorbell_bitmap);
+}
+
+static int __create_doorbell(struct intel_guc_client *client)
+{
+       struct guc_doorbell_info *doorbell;
+       int err;
+
+       doorbell = __get_doorbell(client);
+       doorbell->db_status = GUC_DOORBELL_ENABLED;
+       doorbell->cookie = 0;
+
+       err = __guc_allocate_doorbell(client->guc, client->stage_id);
+       if (err) {
+               doorbell->db_status = GUC_DOORBELL_DISABLED;
+               DRM_ERROR("Couldn't create client %u doorbell: %d\n",
+                         client->stage_id, err);
+       }
+
+       return err;
+}
+
+static int __destroy_doorbell(struct intel_guc_client *client)
+{
+       struct drm_i915_private *dev_priv = guc_to_i915(client->guc);
+       struct guc_doorbell_info *doorbell;
+       u16 db_id = client->doorbell_id;
+
+       GEM_BUG_ON(db_id >= GUC_DOORBELL_INVALID);
+
+       doorbell = __get_doorbell(client);
+       doorbell->db_status = GUC_DOORBELL_DISABLED;
+       doorbell->cookie = 0;
+
+       /* Doorbell release flow requires that we wait for GEN8_DRB_VALID bit
+        * to go to zero after updating db_status before we call the GuC to
+        * release the doorbell
+        */
+       if (wait_for_us(!(I915_READ(GEN8_DRBREGL(db_id)) & GEN8_DRB_VALID), 10))
+               WARN_ONCE(true, "Doorbell never became invalid after disable\n");
+
+       return __guc_deallocate_doorbell(client->guc, client->stage_id);
+}
+
+static int create_doorbell(struct intel_guc_client *client)
+{
+       int ret;
+
+       ret = __reserve_doorbell(client);
+       if (ret)
+               return ret;
+
+       __update_doorbell_desc(client, client->doorbell_id);
+
+       ret = __create_doorbell(client);
+       if (ret)
+               goto err;
+
+       return 0;
+
+err:
+       __update_doorbell_desc(client, GUC_DOORBELL_INVALID);
+       __unreserve_doorbell(client);
+       return ret;
+}
+
+static int destroy_doorbell(struct intel_guc_client *client)
+{
+       int err;
+
+       GEM_BUG_ON(!has_doorbell(client));
+
+       /* XXX: wait for any interrupts */
+       /* XXX: wait for workqueue to drain */
+
+       err = __destroy_doorbell(client);
+       if (err)
+               return err;
+
+       __update_doorbell_desc(client, GUC_DOORBELL_INVALID);
+
+       __unreserve_doorbell(client);
+
+       return 0;
+}
+
+static unsigned long __select_cacheline(struct intel_guc *guc)
+{
+       unsigned long offset;
+
+       /* Doorbell uses a single cache line within a page */
+       offset = offset_in_page(guc->db_cacheline);
+
+       /* Moving to next cache line to reduce contention */
+       guc->db_cacheline += cache_line_size();
+
+       DRM_DEBUG_DRIVER("reserved cacheline 0x%lx, next 0x%x, linesize %u\n",
+                        offset, guc->db_cacheline, cache_line_size());
+       return offset;
+}
+
+static inline struct guc_process_desc *
+__get_process_desc(struct intel_guc_client *client)
+{
+       return client->vaddr + client->proc_desc_offset;
+}
+
+/*
+ * Initialise the process descriptor shared with the GuC firmware.
+ */
+static void guc_proc_desc_init(struct intel_guc *guc,
+                              struct intel_guc_client *client)
+{
+       struct guc_process_desc *desc;
+
+       desc = memset(__get_process_desc(client), 0, sizeof(*desc));
+
+       /*
+        * XXX: pDoorbell and WQVBaseAddress are pointers in process address
+        * space for ring3 clients (set them as in mmap_ioctl) or kernel
+        * space for kernel clients (map on demand instead? May make debug
+        * easier to have it mapped).
+        */
+       desc->wq_base_addr = 0;
+       desc->db_base_addr = 0;
+
+       desc->stage_id = client->stage_id;
+       desc->wq_size_bytes = GUC_WQ_SIZE;
+       desc->wq_status = WQ_STATUS_ACTIVE;
+       desc->priority = client->priority;
+}
+
+static int guc_stage_desc_pool_create(struct intel_guc *guc)
+{
+       struct i915_vma *vma;
+       void *vaddr;
+
+       vma = intel_guc_allocate_vma(guc,
+                                    PAGE_ALIGN(sizeof(struct guc_stage_desc) *
+                                    GUC_MAX_STAGE_DESCRIPTORS));
+       if (IS_ERR(vma))
+               return PTR_ERR(vma);
+
+       vaddr = i915_gem_object_pin_map(vma->obj, I915_MAP_WB);
+       if (IS_ERR(vaddr)) {
+               i915_vma_unpin_and_release(&vma);
+               return PTR_ERR(vaddr);
+       }
+
+       guc->stage_desc_pool = vma;
+       guc->stage_desc_pool_vaddr = vaddr;
+       ida_init(&guc->stage_ids);
+
+       return 0;
+}
+
+static void guc_stage_desc_pool_destroy(struct intel_guc *guc)
+{
+       ida_destroy(&guc->stage_ids);
+       i915_gem_object_unpin_map(guc->stage_desc_pool->obj);
+       i915_vma_unpin_and_release(&guc->stage_desc_pool);
+}
+
+/*
+ * Initialise/clear the stage descriptor shared with the GuC firmware.
+ *
+ * This descriptor tells the GuC where (in GGTT space) to find the important
+ * data structures relating to this client (doorbell, process descriptor,
+ * write queue, etc).
+ */
+static void guc_stage_desc_init(struct intel_guc *guc,
+                               struct intel_guc_client *client)
+{
+       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_engine_cs *engine;
+       struct i915_gem_context *ctx = client->owner;
+       struct guc_stage_desc *desc;
+       unsigned int tmp;
+       u32 gfx_addr;
+
+       desc = __get_stage_desc(client);
+       memset(desc, 0, sizeof(*desc));
+
+       desc->attribute = GUC_STAGE_DESC_ATTR_ACTIVE |
+                         GUC_STAGE_DESC_ATTR_KERNEL;
+       if (is_high_priority(client))
+               desc->attribute |= GUC_STAGE_DESC_ATTR_PREEMPT;
+       desc->stage_id = client->stage_id;
+       desc->priority = client->priority;
+       desc->db_id = client->doorbell_id;
+
+       for_each_engine_masked(engine, dev_priv, client->engines, tmp) {
+               struct intel_context *ce = &ctx->engine[engine->id];
+               u32 guc_engine_id = engine->guc_id;
+               struct guc_execlist_context *lrc = &desc->lrc[guc_engine_id];
+
+               /* TODO: We have a design issue to be solved here. Only when we
+                * receive the first batch, we know which engine is used by the
+                * user. But here GuC expects the lrc and ring to be pinned. It
+                * is not an issue for default context, which is the only one
+                * for now who owns a GuC client. But for future owner of GuC
+                * client, need to make sure lrc is pinned prior to enter here.
+                */
+               if (!ce->state)
+                       break;  /* XXX: continue? */
+
+               /*
+                * XXX: When this is a GUC_STAGE_DESC_ATTR_KERNEL client (proxy
+                * submission or, in other words, not using a direct submission
+                * model) the KMD's LRCA is not used for any work submission.
+                * Instead, the GuC uses the LRCA of the user mode context (see
+                * guc_add_request below).
+                */
+               lrc->context_desc = lower_32_bits(ce->lrc_desc);
+
+               /* The state page is after PPHWSP */
+               lrc->ring_lrca =
+                       guc_ggtt_offset(ce->state) + LRC_STATE_PN * PAGE_SIZE;
+
+               /* XXX: In direct submission, the GuC wants the HW context id
+                * here. In proxy submission, it wants the stage id
+                */
+               lrc->context_id = (client->stage_id << GUC_ELC_CTXID_OFFSET) |
+                               (guc_engine_id << GUC_ELC_ENGINE_OFFSET);
+
+               lrc->ring_begin = guc_ggtt_offset(ce->ring->vma);
+               lrc->ring_end = lrc->ring_begin + ce->ring->size - 1;
+               lrc->ring_next_free_location = lrc->ring_begin;
+               lrc->ring_current_tail_pointer_value = 0;
+
+               desc->engines_used |= (1 << guc_engine_id);
+       }
+
+       DRM_DEBUG_DRIVER("Host engines 0x%x => GuC engines used 0x%x\n",
+                        client->engines, desc->engines_used);
+       WARN_ON(desc->engines_used == 0);
+
+       /*
+        * The doorbell, process descriptor, and workqueue are all parts
+        * of the client object, which the GuC will reference via the GGTT
+        */
+       gfx_addr = guc_ggtt_offset(client->vma);
+       desc->db_trigger_phy = sg_dma_address(client->vma->pages->sgl) +
+                               client->doorbell_offset;
+       desc->db_trigger_cpu = ptr_to_u64(__get_doorbell(client));
+       desc->db_trigger_uk = gfx_addr + client->doorbell_offset;
+       desc->process_desc = gfx_addr + client->proc_desc_offset;
+       desc->wq_addr = gfx_addr + GUC_DB_SIZE;
+       desc->wq_size = GUC_WQ_SIZE;
+
+       desc->desc_private = ptr_to_u64(client);
+}
+
+static void guc_stage_desc_fini(struct intel_guc *guc,
+                               struct intel_guc_client *client)
+{
+       struct guc_stage_desc *desc;
+
+       desc = __get_stage_desc(client);
+       memset(desc, 0, sizeof(*desc));
+}
+
+static int guc_shared_data_create(struct intel_guc *guc)
+{
+       struct i915_vma *vma;
+       void *vaddr;
+
+       vma = intel_guc_allocate_vma(guc, PAGE_SIZE);
+       if (IS_ERR(vma))
+               return PTR_ERR(vma);
+
+       vaddr = i915_gem_object_pin_map(vma->obj, I915_MAP_WB);
+       if (IS_ERR(vaddr)) {
+               i915_vma_unpin_and_release(&vma);
+               return PTR_ERR(vaddr);
+       }
+
+       guc->shared_data = vma;
+       guc->shared_data_vaddr = vaddr;
+
+       return 0;
+}
+
+static void guc_shared_data_destroy(struct intel_guc *guc)
+{
+       i915_gem_object_unpin_map(guc->shared_data->obj);
+       i915_vma_unpin_and_release(&guc->shared_data);
+}
+
+/* Construct a Work Item and append it to the GuC's Work Queue */
+static void guc_wq_item_append(struct intel_guc_client *client,
+                              u32 target_engine, u32 context_desc,
+                              u32 ring_tail, u32 fence_id)
+{
+       /* wqi_len is in DWords, and does not include the one-word header */
+       const size_t wqi_size = sizeof(struct guc_wq_item);
+       const u32 wqi_len = wqi_size / sizeof(u32) - 1;
+       struct guc_process_desc *desc = __get_process_desc(client);
+       struct guc_wq_item *wqi;
+       u32 wq_off;
+
+       lockdep_assert_held(&client->wq_lock);
+
+       /* For now workqueue item is 4 DWs; workqueue buffer is 2 pages. So we
+        * should not have the case where structure wqi is across page, neither
+        * wrapped to the beginning. This simplifies the implementation below.
+        *
+        * XXX: if not the case, we need save data to a temp wqi and copy it to
+        * workqueue buffer dw by dw.
+        */
+       BUILD_BUG_ON(wqi_size != 16);
+
+       /* Free space is guaranteed. */
+       wq_off = READ_ONCE(desc->tail);
+       GEM_BUG_ON(CIRC_SPACE(wq_off, READ_ONCE(desc->head),
+                             GUC_WQ_SIZE) < wqi_size);
+       GEM_BUG_ON(wq_off & (wqi_size - 1));
+
+       /* WQ starts from the page after doorbell / process_desc */
+       wqi = client->vaddr + wq_off + GUC_DB_SIZE;
+
+       /* Now fill in the 4-word work queue item */
+       wqi->header = WQ_TYPE_INORDER |
+                     (wqi_len << WQ_LEN_SHIFT) |
+                     (target_engine << WQ_TARGET_SHIFT) |
+                     WQ_NO_WCFLUSH_WAIT;
+       wqi->context_desc = context_desc;
+       wqi->submit_element_info = ring_tail << WQ_RING_TAIL_SHIFT;
+       GEM_BUG_ON(ring_tail > WQ_RING_TAIL_MAX);
+       wqi->fence_id = fence_id;
+
+       /* Make the update visible to GuC */
+       WRITE_ONCE(desc->tail, (wq_off + wqi_size) & (GUC_WQ_SIZE - 1));
+}
+
+static void guc_reset_wq(struct intel_guc_client *client)
+{
+       struct guc_process_desc *desc = __get_process_desc(client);
+
+       desc->head = 0;
+       desc->tail = 0;
+}
+
+static void guc_ring_doorbell(struct intel_guc_client *client)
+{
+       struct guc_doorbell_info *db;
+       u32 cookie;
+
+       lockdep_assert_held(&client->wq_lock);
+
+       /* pointer of current doorbell cacheline */
+       db = __get_doorbell(client);
+
+       /*
+        * We're not expecting the doorbell cookie to change behind our back,
+        * we also need to treat 0 as a reserved value.
+        */
+       cookie = READ_ONCE(db->cookie);
+       WARN_ON_ONCE(xchg(&db->cookie, cookie + 1 ?: cookie + 2) != cookie);
+
+       /* XXX: doorbell was lost and need to acquire it again */
+       GEM_BUG_ON(db->db_status != GUC_DOORBELL_ENABLED);
+}
+
+static void guc_add_request(struct intel_guc *guc,
+                           struct drm_i915_gem_request *rq)
+{
+       struct intel_guc_client *client = guc->execbuf_client;
+       struct intel_engine_cs *engine = rq->engine;
+       u32 ctx_desc = lower_32_bits(intel_lr_context_descriptor(rq->ctx,
+                                                                engine));
+       u32 ring_tail = intel_ring_set_tail(rq->ring, rq->tail) / sizeof(u64);
+
+       spin_lock(&client->wq_lock);
+
+       guc_wq_item_append(client, engine->guc_id, ctx_desc,
+                          ring_tail, rq->global_seqno);
+       guc_ring_doorbell(client);
+
+       client->submissions[engine->id] += 1;
+
+       spin_unlock(&client->wq_lock);
+}
+
+/*
+ * When we're doing submissions using regular execlists backend, writing to
+ * ELSP from CPU side is enough to make sure that writes to ringbuffer pages
+ * pinned in mappable aperture portion of GGTT are visible to command streamer.
+ * Writes done by GuC on our behalf are not guaranteeing such ordering,
+ * therefore, to ensure the flush, we're issuing a POSTING READ.
+ */
+static void flush_ggtt_writes(struct i915_vma *vma)
+{
+       struct drm_i915_private *dev_priv = to_i915(vma->obj->base.dev);
+
+       if (i915_vma_is_map_and_fenceable(vma))
+               POSTING_READ_FW(GUC_STATUS);
+}
+
+#define GUC_PREEMPT_FINISHED 0x1
+#define GUC_PREEMPT_BREADCRUMB_DWORDS 0x8
+static void inject_preempt_context(struct work_struct *work)
+{
+       struct guc_preempt_work *preempt_work =
+               container_of(work, typeof(*preempt_work), work);
+       struct intel_engine_cs *engine = preempt_work->engine;
+       struct intel_guc *guc = container_of(preempt_work, typeof(*guc),
+                                            preempt_work[engine->id]);
+       struct intel_guc_client *client = guc->preempt_client;
+       struct guc_stage_desc *stage_desc = __get_stage_desc(client);
+       struct intel_ring *ring = client->owner->engine[engine->id].ring;
+       u32 ctx_desc = lower_32_bits(intel_lr_context_descriptor(client->owner,
+                                                                engine));
+       u32 *cs = ring->vaddr + ring->tail;
+       u32 data[7];
+
+       if (engine->id == RCS) {
+               cs = gen8_emit_ggtt_write_rcs(cs, GUC_PREEMPT_FINISHED,
+                               intel_hws_preempt_done_address(engine));
+       } else {
+               cs = gen8_emit_ggtt_write(cs, GUC_PREEMPT_FINISHED,
+                               intel_hws_preempt_done_address(engine));
+               *cs++ = MI_NOOP;
+               *cs++ = MI_NOOP;
+       }
+       *cs++ = MI_USER_INTERRUPT;
+       *cs++ = MI_NOOP;
+
+       GEM_BUG_ON(!IS_ALIGNED(ring->size,
+                              GUC_PREEMPT_BREADCRUMB_DWORDS * sizeof(u32)));
+       GEM_BUG_ON((void *)cs - (ring->vaddr + ring->tail) !=
+                  GUC_PREEMPT_BREADCRUMB_DWORDS * sizeof(u32));
+
+       ring->tail += GUC_PREEMPT_BREADCRUMB_DWORDS * sizeof(u32);
+       ring->tail &= (ring->size - 1);
+
+       flush_ggtt_writes(ring->vma);
+
+       spin_lock_irq(&client->wq_lock);
+       guc_wq_item_append(client, engine->guc_id, ctx_desc,
+                          ring->tail / sizeof(u64), 0);
+       spin_unlock_irq(&client->wq_lock);
+
+       /*
+        * If GuC firmware performs an engine reset while that engine had
+        * a preemption pending, it will set the terminated attribute bit
+        * on our preemption stage descriptor. GuC firmware retains all
+        * pending work items for a high-priority GuC client, unlike the
+        * normal-priority GuC client where work items are dropped. It
+        * wants to make sure the preempt-to-idle work doesn't run when
+        * scheduling resumes, and uses this bit to inform its scheduler
+        * and presumably us as well. Our job is to clear it for the next
+        * preemption after reset, otherwise that and future preemptions
+        * will never complete. We'll just clear it every time.
+        */
+       stage_desc->attribute &= ~GUC_STAGE_DESC_ATTR_TERMINATED;
+
+       data[0] = INTEL_GUC_ACTION_REQUEST_PREEMPTION;
+       data[1] = client->stage_id;
+       data[2] = INTEL_GUC_PREEMPT_OPTION_DROP_WORK_Q |
+                 INTEL_GUC_PREEMPT_OPTION_DROP_SUBMIT_Q;
+       data[3] = engine->guc_id;
+       data[4] = guc->execbuf_client->priority;
+       data[5] = guc->execbuf_client->stage_id;
+       data[6] = guc_ggtt_offset(guc->shared_data);
+
+       if (WARN_ON(intel_guc_send(guc, data, ARRAY_SIZE(data)))) {
+               execlists_clear_active(&engine->execlists,
+                                      EXECLISTS_ACTIVE_PREEMPT);
+               tasklet_schedule(&engine->execlists.tasklet);
+       }
+}
+
+/*
+ * We're using user interrupt and HWSP value to mark that preemption has
+ * finished and GPU is idle. Normally, we could unwind and continue similar to
+ * execlists submission path. Unfortunately, with GuC we also need to wait for
+ * it to finish its own postprocessing, before attempting to submit. Otherwise
+ * GuC may silently ignore our submissions, and thus we risk losing request at
+ * best, executing out-of-order and causing kernel panic at worst.
+ */
+#define GUC_PREEMPT_POSTPROCESS_DELAY_MS 10
+static void wait_for_guc_preempt_report(struct intel_engine_cs *engine)
+{
+       struct intel_guc *guc = &engine->i915->guc;
+       struct guc_shared_ctx_data *data = guc->shared_data_vaddr;
+       struct guc_ctx_report *report =
+               &data->preempt_ctx_report[engine->guc_id];
+
+       WARN_ON(wait_for_atomic(report->report_return_status ==
+                               INTEL_GUC_REPORT_STATUS_COMPLETE,
+                               GUC_PREEMPT_POSTPROCESS_DELAY_MS));
+       /*
+        * GuC is expecting that we're also going to clear the affected context
+        * counter, let's also reset the return status to not depend on GuC
+        * resetting it after recieving another preempt action
+        */
+       report->affected_count = 0;
+       report->report_return_status = INTEL_GUC_REPORT_STATUS_UNKNOWN;
+}
+
+/**
+ * guc_submit() - Submit commands through GuC
+ * @engine: engine associated with the commands
+ *
+ * The only error here arises if the doorbell hardware isn't functioning
+ * as expected, which really shouln't happen.
+ */
+static void guc_submit(struct intel_engine_cs *engine)
+{
+       struct intel_guc *guc = &engine->i915->guc;
+       struct intel_engine_execlists * const execlists = &engine->execlists;
+       struct execlist_port *port = execlists->port;
+       unsigned int n;
+
+       for (n = 0; n < execlists_num_ports(execlists); n++) {
+               struct drm_i915_gem_request *rq;
+               unsigned int count;
+
+               rq = port_unpack(&port[n], &count);
+               if (rq && count == 0) {
+                       port_set(&port[n], port_pack(rq, ++count));
+
+                       flush_ggtt_writes(rq->ring->vma);
+
+                       guc_add_request(guc, rq);
+               }
+       }
+}
+
+static void port_assign(struct execlist_port *port,
+                       struct drm_i915_gem_request *rq)
+{
+       GEM_BUG_ON(rq == port_request(port));
+
+       if (port_isset(port))
+               i915_gem_request_put(port_request(port));
+
+       port_set(port, port_pack(i915_gem_request_get(rq), port_count(port)));
+}
+
+static void guc_dequeue(struct intel_engine_cs *engine)
+{
+       struct intel_engine_execlists * const execlists = &engine->execlists;
+       struct execlist_port *port = execlists->port;
+       struct drm_i915_gem_request *last = NULL;
+       const struct execlist_port * const last_port =
+               &execlists->port[execlists->port_mask];
+       bool submit = false;
+       struct rb_node *rb;
+
+       spin_lock_irq(&engine->timeline->lock);
+       rb = execlists->first;
+       GEM_BUG_ON(rb_first(&execlists->queue) != rb);
+
+       if (!rb)
+               goto unlock;
+
+       if (HAS_LOGICAL_RING_PREEMPTION(engine->i915) && port_isset(port)) {
+               struct guc_preempt_work *preempt_work =
+                       &engine->i915->guc.preempt_work[engine->id];
+
+               if (rb_entry(rb, struct i915_priolist, node)->priority >
+                   max(port_request(port)->priotree.priority, 0)) {
+                       execlists_set_active(execlists,
+                                            EXECLISTS_ACTIVE_PREEMPT);
+                       queue_work(engine->i915->guc.preempt_wq,
+                                  &preempt_work->work);
+                       goto unlock;
+               } else if (port_isset(last_port)) {
+                       goto unlock;
+               }
+
+               port++;
+       }
+
+       do {
+               struct i915_priolist *p = rb_entry(rb, typeof(*p), node);
+               struct drm_i915_gem_request *rq, *rn;
+
+               list_for_each_entry_safe(rq, rn, &p->requests, priotree.link) {
+                       if (last && rq->ctx != last->ctx) {
+                               if (port == last_port) {
+                                       __list_del_many(&p->requests,
+                                                       &rq->priotree.link);
+                                       goto done;
+                               }
+
+                               if (submit)
+                                       port_assign(port, last);
+                               port++;
+                       }
+
+                       INIT_LIST_HEAD(&rq->priotree.link);
+
+                       __i915_gem_request_submit(rq);
+                       trace_i915_gem_request_in(rq,
+                                                 port_index(port, execlists));
+                       last = rq;
+                       submit = true;
+               }
+
+               rb = rb_next(rb);
+               rb_erase(&p->node, &execlists->queue);
+               INIT_LIST_HEAD(&p->requests);
+               if (p->priority != I915_PRIORITY_NORMAL)
+                       kmem_cache_free(engine->i915->priorities, p);
+       } while (rb);
+done:
+       execlists->first = rb;
+       if (submit) {
+               port_assign(port, last);
+               execlists_set_active(execlists, EXECLISTS_ACTIVE_USER);
+               guc_submit(engine);
+       }
+unlock:
+       spin_unlock_irq(&engine->timeline->lock);
+}
+
+static void guc_submission_tasklet(unsigned long data)
+{
+       struct intel_engine_cs * const engine = (struct intel_engine_cs *)data;
+       struct intel_engine_execlists * const execlists = &engine->execlists;
+       struct execlist_port *port = execlists->port;
+       struct drm_i915_gem_request *rq;
+
+       rq = port_request(&port[0]);
+       while (rq && i915_gem_request_completed(rq)) {
+               trace_i915_gem_request_out(rq);
+               i915_gem_request_put(rq);
+
+               execlists_port_complete(execlists, port);
+
+               rq = port_request(&port[0]);
+       }
+       if (!rq)
+               execlists_clear_active(execlists, EXECLISTS_ACTIVE_USER);
+
+       if (execlists_is_active(execlists, EXECLISTS_ACTIVE_PREEMPT) &&
+           intel_read_status_page(engine, I915_GEM_HWS_PREEMPT_INDEX) ==
+           GUC_PREEMPT_FINISHED) {
+               execlists_cancel_port_requests(&engine->execlists);
+               execlists_unwind_incomplete_requests(execlists);
+
+               wait_for_guc_preempt_report(engine);
+
+               execlists_clear_active(execlists, EXECLISTS_ACTIVE_PREEMPT);
+               intel_write_status_page(engine, I915_GEM_HWS_PREEMPT_INDEX, 0);
+       }
+
+       if (!execlists_is_active(execlists, EXECLISTS_ACTIVE_PREEMPT))
+               guc_dequeue(engine);
+}
+
+/*
+ * Everything below here is concerned with setup & teardown, and is
+ * therefore not part of the somewhat time-critical batch-submission
+ * path of guc_submit() above.
+ */
+
+/* Check that a doorbell register is in the expected state */
+static bool doorbell_ok(struct intel_guc *guc, u16 db_id)
+{
+       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       u32 drbregl;
+       bool valid;
+
+       GEM_BUG_ON(db_id >= GUC_DOORBELL_INVALID);
+
+       drbregl = I915_READ(GEN8_DRBREGL(db_id));
+       valid = drbregl & GEN8_DRB_VALID;
+
+       if (test_bit(db_id, guc->doorbell_bitmap) == valid)
+               return true;
+
+       DRM_DEBUG_DRIVER("Doorbell %d has unexpected state (0x%x): valid=%s\n",
+                        db_id, drbregl, yesno(valid));
+
+       return false;
+}
+
+/*
+ * If the GuC thinks that the doorbell is unassigned (e.g. because we reset and
+ * reloaded the GuC FW) we can use this function to tell the GuC to reassign the
+ * doorbell to the rightful owner.
+ */
+static int __reset_doorbell(struct intel_guc_client *client, u16 db_id)
+{
+       int err;
+
+       __update_doorbell_desc(client, db_id);
+       err = __create_doorbell(client);
+       if (!err)
+               err = __destroy_doorbell(client);
+
+       return err;
+}
+
+/*
+ * Set up & tear down each unused doorbell in turn, to ensure that all doorbell
+ * HW is (re)initialised. For that end, we might have to borrow the first
+ * client. Also, tell GuC about all the doorbells in use by all clients.
+ * We do this because the KMD, the GuC and the doorbell HW can easily go out of
+ * sync (e.g. we can reset the GuC, but not the doorbel HW).
+ */
+static int guc_init_doorbell_hw(struct intel_guc *guc)
+{
+       struct intel_guc_client *client = guc->execbuf_client;
+       bool recreate_first_client = false;
+       u16 db_id;
+       int ret;
+
+       /* For unused doorbells, make sure they are disabled */
+       for_each_clear_bit(db_id, guc->doorbell_bitmap, GUC_NUM_DOORBELLS) {
+               if (doorbell_ok(guc, db_id))
+                       continue;
+
+               if (has_doorbell(client)) {
+                       /* Borrow execbuf_client (we will recreate it later) */
+                       destroy_doorbell(client);
+                       recreate_first_client = true;
+               }
+
+               ret = __reset_doorbell(client, db_id);
+               WARN(ret, "Doorbell %u reset failed, err %d\n", db_id, ret);
+       }
+
+       if (recreate_first_client) {
+               ret = __reserve_doorbell(client);
+               if (unlikely(ret)) {
+                       DRM_ERROR("Couldn't re-reserve first client db: %d\n",
+                                 ret);
+                       return ret;
+               }
+
+               __update_doorbell_desc(client, client->doorbell_id);
+       }
+
+       /* Now for every client (and not only execbuf_client) make sure their
+        * doorbells are known by the GuC
+        */
+       ret = __create_doorbell(guc->execbuf_client);
+       if (ret)
+               return ret;
+
+       ret = __create_doorbell(guc->preempt_client);
+       if (ret) {
+               __destroy_doorbell(guc->execbuf_client);
+               return ret;
+       }
+
+       /* Read back & verify all (used & unused) doorbell registers */
+       for (db_id = 0; db_id < GUC_NUM_DOORBELLS; ++db_id)
+               WARN_ON(!doorbell_ok(guc, db_id));
+
+       return 0;
+}
+
+/**
+ * guc_client_alloc() - Allocate an intel_guc_client
+ * @dev_priv:  driver private data structure
+ * @engines:   The set of engines to enable for this client
+ * @priority:  four levels priority _CRITICAL, _HIGH, _NORMAL and _LOW
+ *             The kernel client to replace ExecList submission is created with
+ *             NORMAL priority. Priority of a client for scheduler can be HIGH,
+ *             while a preemption context can use CRITICAL.
+ * @ctx:       the context that owns the client (we use the default render
+ *             context)
+ *
+ * Return:     An intel_guc_client object if success, else NULL.
+ */
+static struct intel_guc_client *
+guc_client_alloc(struct drm_i915_private *dev_priv,
+                u32 engines,
+                u32 priority,
+                struct i915_gem_context *ctx)
+{
+       struct intel_guc_client *client;
+       struct intel_guc *guc = &dev_priv->guc;
+       struct i915_vma *vma;
+       void *vaddr;
+       int ret;
+
+       client = kzalloc(sizeof(*client), GFP_KERNEL);
+       if (!client)
+               return ERR_PTR(-ENOMEM);
+
+       client->guc = guc;
+       client->owner = ctx;
+       client->engines = engines;
+       client->priority = priority;
+       client->doorbell_id = GUC_DOORBELL_INVALID;
+       spin_lock_init(&client->wq_lock);
+
+       ret = ida_simple_get(&guc->stage_ids, 0, GUC_MAX_STAGE_DESCRIPTORS,
+                            GFP_KERNEL);
+       if (ret < 0)
+               goto err_client;
+
+       client->stage_id = ret;
+
+       /* The first page is doorbell/proc_desc. Two followed pages are wq. */
+       vma = intel_guc_allocate_vma(guc, GUC_DB_SIZE + GUC_WQ_SIZE);
+       if (IS_ERR(vma)) {
+               ret = PTR_ERR(vma);
+               goto err_id;
+       }
+
+       /* We'll keep just the first (doorbell/proc) page permanently kmap'd. */
+       client->vma = vma;
+
+       vaddr = i915_gem_object_pin_map(vma->obj, I915_MAP_WB);
+       if (IS_ERR(vaddr)) {
+               ret = PTR_ERR(vaddr);
+               goto err_vma;
+       }
+       client->vaddr = vaddr;
+
+       client->doorbell_offset = __select_cacheline(guc);
+
+       /*
+        * Since the doorbell only requires a single cacheline, we can save
+        * space by putting the application process descriptor in the same
+        * page. Use the half of the page that doesn't include the doorbell.
+        */
+       if (client->doorbell_offset >= (GUC_DB_SIZE / 2))
+               client->proc_desc_offset = 0;
+       else
+               client->proc_desc_offset = (GUC_DB_SIZE / 2);
+
+       guc_proc_desc_init(guc, client);
+       guc_stage_desc_init(guc, client);
+
+       ret = create_doorbell(client);
+       if (ret)
+               goto err_vaddr;
+
+       DRM_DEBUG_DRIVER("new priority %u client %p for engine(s) 0x%x: stage_id %u\n",
+                        priority, client, client->engines, client->stage_id);
+       DRM_DEBUG_DRIVER("doorbell id %u, cacheline offset 0x%lx\n",
+                        client->doorbell_id, client->doorbell_offset);
+
+       return client;
+
+err_vaddr:
+       i915_gem_object_unpin_map(client->vma->obj);
+err_vma:
+       i915_vma_unpin_and_release(&client->vma);
+err_id:
+       ida_simple_remove(&guc->stage_ids, client->stage_id);
+err_client:
+       kfree(client);
+       return ERR_PTR(ret);
+}
+
+static void guc_client_free(struct intel_guc_client *client)
+{
+       /*
+        * XXX: wait for any outstanding submissions before freeing memory.
+        * Be sure to drop any locks
+        */
+
+       /* FIXME: in many cases, by the time we get here the GuC has been
+        * reset, so we cannot destroy the doorbell properly. Ignore the
+        * error message for now
+        */
+       destroy_doorbell(client);
+       guc_stage_desc_fini(client->guc, client);
+       i915_gem_object_unpin_map(client->vma->obj);
+       i915_vma_unpin_and_release(&client->vma);
+       ida_simple_remove(&client->guc->stage_ids, client->stage_id);
+       kfree(client);
+}
+
+static int guc_clients_create(struct intel_guc *guc)
+{
+       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_guc_client *client;
+
+       GEM_BUG_ON(guc->execbuf_client);
+       GEM_BUG_ON(guc->preempt_client);
+
+       client = guc_client_alloc(dev_priv,
+                                 INTEL_INFO(dev_priv)->ring_mask,
+                                 GUC_CLIENT_PRIORITY_KMD_NORMAL,
+                                 dev_priv->kernel_context);
+       if (IS_ERR(client)) {
+               DRM_ERROR("Failed to create GuC client for submission!\n");
+               return PTR_ERR(client);
+       }
+       guc->execbuf_client = client;
+
+       client = guc_client_alloc(dev_priv,
+                                 INTEL_INFO(dev_priv)->ring_mask,
+                                 GUC_CLIENT_PRIORITY_KMD_HIGH,
+                                 dev_priv->preempt_context);
+       if (IS_ERR(client)) {
+               DRM_ERROR("Failed to create GuC client for preemption!\n");
+               guc_client_free(guc->execbuf_client);
+               guc->execbuf_client = NULL;
+               return PTR_ERR(client);
+       }
+       guc->preempt_client = client;
+
+       return 0;
+}
+
+static void guc_clients_destroy(struct intel_guc *guc)
+{
+       struct intel_guc_client *client;
+
+       client = fetch_and_zero(&guc->execbuf_client);
+       guc_client_free(client);
+
+       client = fetch_and_zero(&guc->preempt_client);
+       guc_client_free(client);
+}
+
+static void guc_policy_init(struct guc_policy *policy)
+{
+       policy->execution_quantum = POLICY_DEFAULT_EXECUTION_QUANTUM_US;
+       policy->preemption_time = POLICY_DEFAULT_PREEMPTION_TIME_US;
+       policy->fault_time = POLICY_DEFAULT_FAULT_TIME_US;
+       policy->policy_flags = 0;
+}
+
+static void guc_policies_init(struct guc_policies *policies)
+{
+       struct guc_policy *policy;
+       u32 p, i;
+
+       policies->dpc_promote_time = POLICY_DEFAULT_DPC_PROMOTE_TIME_US;
+       policies->max_num_work_items = POLICY_MAX_NUM_WI;
+
+       for (p = 0; p < GUC_CLIENT_PRIORITY_NUM; p++) {
+               for (i = GUC_RENDER_ENGINE; i < GUC_MAX_ENGINES_NUM; i++) {
+                       policy = &policies->policy[p][i];
+
+                       guc_policy_init(policy);
+               }
+       }
+
+       policies->is_valid = 1;
+}
+
+/*
+ * The first 80 dwords of the register state context, containing the
+ * execlists and ppgtt registers.
+ */
+#define LR_HW_CONTEXT_SIZE     (80 * sizeof(u32))
+
+static int guc_ads_create(struct intel_guc *guc)
+{
+       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct i915_vma *vma;
+       struct page *page;
+       /* The ads obj includes the struct itself and buffers passed to GuC */
+       struct {
+               struct guc_ads ads;
+               struct guc_policies policies;
+               struct guc_mmio_reg_state reg_state;
+               u8 reg_state_buffer[GUC_S3_SAVE_SPACE_PAGES * PAGE_SIZE];
+       } __packed *blob;
+       struct intel_engine_cs *engine;
+       enum intel_engine_id id;
+       const u32 skipped_offset = LRC_HEADER_PAGES * PAGE_SIZE;
+       const u32 skipped_size = LRC_PPHWSP_SZ * PAGE_SIZE + LR_HW_CONTEXT_SIZE;
+       u32 base;
+
+       GEM_BUG_ON(guc->ads_vma);
+
+       vma = intel_guc_allocate_vma(guc, PAGE_ALIGN(sizeof(*blob)));
+       if (IS_ERR(vma))
+               return PTR_ERR(vma);
+
+       guc->ads_vma = vma;
+
+       page = i915_vma_first_page(vma);
+       blob = kmap(page);
+
+       /* GuC scheduling policies */
+       guc_policies_init(&blob->policies);
+
+       /* MMIO reg state */
+       for_each_engine(engine, dev_priv, id) {
+               blob->reg_state.white_list[engine->guc_id].mmio_start =
+                       engine->mmio_base + GUC_MMIO_WHITE_LIST_START;
+
+               /* Nothing to be saved or restored for now. */
+               blob->reg_state.white_list[engine->guc_id].count = 0;
+       }
+
+       /*
+        * The GuC requires a "Golden Context" when it reinitialises
+        * engines after a reset. Here we use the Render ring default
+        * context, which must already exist and be pinned in the GGTT,
+        * so its address won't change after we've told the GuC where
+        * to find it. Note that we have to skip our header (1 page),
+        * because our GuC shared data is there.
+        */
+       blob->ads.golden_context_lrca =
+               guc_ggtt_offset(dev_priv->kernel_context->engine[RCS].state) +
+               skipped_offset;
+
+       /*
+        * The GuC expects us to exclude the portion of the context image that
+        * it skips from the size it is to read. It starts reading from after
+        * the execlist context (so skipping the first page [PPHWSP] and 80
+        * dwords). Weird guc is weird.
+        */
+       for_each_engine(engine, dev_priv, id)
+               blob->ads.eng_state_size[engine->guc_id] =
+                       engine->context_size - skipped_size;
+
+       base = guc_ggtt_offset(vma);
+       blob->ads.scheduler_policies = base + ptr_offset(blob, policies);
+       blob->ads.reg_state_buffer = base + ptr_offset(blob, reg_state_buffer);
+       blob->ads.reg_state_addr = base + ptr_offset(blob, reg_state);
+
+       kunmap(page);
+
+       return 0;
+}
+
+static void guc_ads_destroy(struct intel_guc *guc)
+{
+       i915_vma_unpin_and_release(&guc->ads_vma);
+}
+
+static int guc_preempt_work_create(struct intel_guc *guc)
+{
+       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_engine_cs *engine;
+       enum intel_engine_id id;
+
+       /*
+        * Even though both sending GuC action, and adding a new workitem to
+        * GuC workqueue are serialized (each with its own locking), since
+        * we're using mutliple engines, it's possible that we're going to
+        * issue a preempt request with two (or more - each for different
+        * engine) workitems in GuC queue. In this situation, GuC may submit
+        * all of them, which will make us very confused.
+        * Our preemption contexts may even already be complete - before we
+        * even had the chance to sent the preempt action to GuC!. Rather
+        * than introducing yet another lock, we can just use ordered workqueue
+        * to make sure we're always sending a single preemption request with a
+        * single workitem.
+        */
+       guc->preempt_wq = alloc_ordered_workqueue("i915-guc_preempt",
+                                                 WQ_HIGHPRI);
+       if (!guc->preempt_wq)
+               return -ENOMEM;
+
+       for_each_engine(engine, dev_priv, id) {
+               guc->preempt_work[id].engine = engine;
+               INIT_WORK(&guc->preempt_work[id].work, inject_preempt_context);
+       }
+
+       return 0;
+}
+
+static void guc_preempt_work_destroy(struct intel_guc *guc)
+{
+       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_engine_cs *engine;
+       enum intel_engine_id id;
+
+       for_each_engine(engine, dev_priv, id)
+               cancel_work_sync(&guc->preempt_work[id].work);
+
+       destroy_workqueue(guc->preempt_wq);
+       guc->preempt_wq = NULL;
+}
+
+/*
+ * Set up the memory resources to be shared with the GuC (via the GGTT)
+ * at firmware loading time.
+ */
+int intel_guc_submission_init(struct intel_guc *guc)
+{
+       int ret;
+
+       if (guc->stage_desc_pool)
+               return 0;
+
+       ret = guc_stage_desc_pool_create(guc);
+       if (ret)
+               return ret;
+       /*
+        * Keep static analysers happy, let them know that we allocated the
+        * vma after testing that it didn't exist earlier.
+        */
+       GEM_BUG_ON(!guc->stage_desc_pool);
+
+       ret = guc_shared_data_create(guc);
+       if (ret)
+               goto err_stage_desc_pool;
+       GEM_BUG_ON(!guc->shared_data);
+
+       ret = intel_guc_log_create(guc);
+       if (ret < 0)
+               goto err_shared_data;
+
+       ret = guc_preempt_work_create(guc);
+       if (ret)
+               goto err_log;
+       GEM_BUG_ON(!guc->preempt_wq);
+
+       ret = guc_ads_create(guc);
+       if (ret < 0)
+               goto err_wq;
+       GEM_BUG_ON(!guc->ads_vma);
+
+       return 0;
+
+err_wq:
+       guc_preempt_work_destroy(guc);
+err_log:
+       intel_guc_log_destroy(guc);
+err_shared_data:
+       guc_shared_data_destroy(guc);
+err_stage_desc_pool:
+       guc_stage_desc_pool_destroy(guc);
+       return ret;
+}
+
+void intel_guc_submission_fini(struct intel_guc *guc)
+{
+       guc_ads_destroy(guc);
+       guc_preempt_work_destroy(guc);
+       intel_guc_log_destroy(guc);
+       guc_shared_data_destroy(guc);
+       guc_stage_desc_pool_destroy(guc);
+}
+
+static void guc_interrupts_capture(struct drm_i915_private *dev_priv)
+{
+       struct intel_rps *rps = &dev_priv->gt_pm.rps;
+       struct intel_engine_cs *engine;
+       enum intel_engine_id id;
+       int irqs;
+
+       /* tell all command streamers to forward interrupts (but not vblank)
+        * to GuC
+        */
+       irqs = _MASKED_BIT_ENABLE(GFX_INTERRUPT_STEERING);
+       for_each_engine(engine, dev_priv, id)
+               I915_WRITE(RING_MODE_GEN7(engine), irqs);
+
+       /* route USER_INTERRUPT to Host, all others are sent to GuC. */
+       irqs = GT_RENDER_USER_INTERRUPT << GEN8_RCS_IRQ_SHIFT |
+              GT_RENDER_USER_INTERRUPT << GEN8_BCS_IRQ_SHIFT;
+       /* These three registers have the same bit definitions */
+       I915_WRITE(GUC_BCS_RCS_IER, ~irqs);
+       I915_WRITE(GUC_VCS2_VCS1_IER, ~irqs);
+       I915_WRITE(GUC_WD_VECS_IER, ~irqs);
+
+       /*
+        * The REDIRECT_TO_GUC bit of the PMINTRMSK register directs all
+        * (unmasked) PM interrupts to the GuC. All other bits of this
+        * register *disable* generation of a specific interrupt.
+        *
+        * 'pm_intrmsk_mbz' indicates bits that are NOT to be set when
+        * writing to the PM interrupt mask register, i.e. interrupts
+        * that must not be disabled.
+        *
+        * If the GuC is handling these interrupts, then we must not let
+        * the PM code disable ANY interrupt that the GuC is expecting.
+        * So for each ENABLED (0) bit in this register, we must SET the
+        * bit in pm_intrmsk_mbz so that it's left enabled for the GuC.
+        * GuC needs ARAT expired interrupt unmasked hence it is set in
+        * pm_intrmsk_mbz.
+        *
+        * Here we CLEAR REDIRECT_TO_GUC bit in pm_intrmsk_mbz, which will
+        * result in the register bit being left SET!
+        */
+       rps->pm_intrmsk_mbz |= ARAT_EXPIRED_INTRMSK;
+       rps->pm_intrmsk_mbz &= ~GEN8_PMINTR_DISABLE_REDIRECT_TO_GUC;
+}
+
+static void guc_interrupts_release(struct drm_i915_private *dev_priv)
+{
+       struct intel_rps *rps = &dev_priv->gt_pm.rps;
+       struct intel_engine_cs *engine;
+       enum intel_engine_id id;
+       int irqs;
+
+       /*
+        * tell all command streamers NOT to forward interrupts or vblank
+        * to GuC.
+        */
+       irqs = _MASKED_FIELD(GFX_FORWARD_VBLANK_MASK, GFX_FORWARD_VBLANK_NEVER);
+       irqs |= _MASKED_BIT_DISABLE(GFX_INTERRUPT_STEERING);
+       for_each_engine(engine, dev_priv, id)
+               I915_WRITE(RING_MODE_GEN7(engine), irqs);
+
+       /* route all GT interrupts to the host */
+       I915_WRITE(GUC_BCS_RCS_IER, 0);
+       I915_WRITE(GUC_VCS2_VCS1_IER, 0);
+       I915_WRITE(GUC_WD_VECS_IER, 0);
+
+       rps->pm_intrmsk_mbz |= GEN8_PMINTR_DISABLE_REDIRECT_TO_GUC;
+       rps->pm_intrmsk_mbz &= ~ARAT_EXPIRED_INTRMSK;
+}
+
+static void guc_submission_park(struct intel_engine_cs *engine)
+{
+       intel_engine_unpin_breadcrumbs_irq(engine);
+}
+
+static void guc_submission_unpark(struct intel_engine_cs *engine)
+{
+       intel_engine_pin_breadcrumbs_irq(engine);
+}
+
+int intel_guc_submission_enable(struct intel_guc *guc)
+{
+       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+       struct intel_engine_cs *engine;
+       enum intel_engine_id id;
+       int err;
+
+       /*
+        * We're using GuC work items for submitting work through GuC. Since
+        * we're coalescing multiple requests from a single context into a
+        * single work item prior to assigning it to execlist_port, we can
+        * never have more work items than the total number of ports (for all
+        * engines). The GuC firmware is controlling the HEAD of work queue,
+        * and it is guaranteed that it will remove the work item from the
+        * queue before our request is completed.
+        */
+       BUILD_BUG_ON(ARRAY_SIZE(engine->execlists.port) *
+                    sizeof(struct guc_wq_item) *
+                    I915_NUM_ENGINES > GUC_WQ_SIZE);
+
+       /*
+        * We're being called on both module initialization and on reset,
+        * until this flow is changed, we're using regular client presence to
+        * determine which case are we in, and whether we should allocate new
+        * clients or just reset their workqueues.
+        */
+       if (!guc->execbuf_client) {
+               err = guc_clients_create(guc);
+               if (err)
+                       return err;
+       } else {
+               guc_reset_wq(guc->execbuf_client);
+               guc_reset_wq(guc->preempt_client);
+       }
+
+       err = intel_guc_sample_forcewake(guc);
+       if (err)
+               goto err_free_clients;
+
+       err = guc_init_doorbell_hw(guc);
+       if (err)
+               goto err_free_clients;
+
+       /* Take over from manual control of ELSP (execlists) */
+       guc_interrupts_capture(dev_priv);
+
+       for_each_engine(engine, dev_priv, id) {
+               struct intel_engine_execlists * const execlists =
+                       &engine->execlists;
+
+               execlists->tasklet.func = guc_submission_tasklet;
+               engine->park = guc_submission_park;
+               engine->unpark = guc_submission_unpark;
+       }
+
+       return 0;
+
+err_free_clients:
+       guc_clients_destroy(guc);
+       return err;
+}
+
+void intel_guc_submission_disable(struct intel_guc *guc)
+{
+       struct drm_i915_private *dev_priv = guc_to_i915(guc);
+
+       GEM_BUG_ON(dev_priv->gt.awake); /* GT should be parked first */
+
+       guc_interrupts_release(dev_priv);
+
+       /* Revert back to manual ELSP submission */
+       intel_engines_reset_default_submission(dev_priv);
+
+       guc_clients_destroy(guc);
+}
diff --git a/drivers/gpu/drm/i915/intel_guc_submission.h b/drivers/gpu/drm/i915/intel_guc_submission.h
new file mode 100644 (file)
index 0000000..e901192
--- /dev/null
@@ -0,0 +1,81 @@
+/*
+ * Copyright © 2014-2017 Intel Corporation
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ *
+ */
+
+#ifndef _I915_GUC_SUBMISSION_H_
+#define _I915_GUC_SUBMISSION_H_
+
+#include <linux/spinlock.h>
+
+#include "i915_gem.h"
+
+struct drm_i915_private;
+
+/*
+ * This structure primarily describes the GEM object shared with the GuC.
+ * The specs sometimes refer to this object as a "GuC context", but we use
+ * the term "client" to avoid confusion with hardware contexts. This
+ * GEM object is held for the entire lifetime of our interaction with
+ * the GuC, being allocated before the GuC is loaded with its firmware.
+ * Because there's no way to update the address used by the GuC after
+ * initialisation, the shared object must stay pinned into the GGTT as
+ * long as the GuC is in use. We also keep the first page (only) mapped
+ * into kernel address space, as it includes shared data that must be
+ * updated on every request submission.
+ *
+ * The single GEM object described here is actually made up of several
+ * separate areas, as far as the GuC is concerned. The first page (kept
+ * kmap'd) includes the "process descriptor" which holds sequence data for
+ * the doorbell, and one cacheline which actually *is* the doorbell; a
+ * write to this will "ring the doorbell" (i.e. send an interrupt to the
+ * GuC). The subsequent  pages of the client object constitute the work
+ * queue (a circular array of work items), again described in the process
+ * descriptor. Work queue pages are mapped momentarily as required.
+ */
+struct intel_guc_client {
+       struct i915_vma *vma;
+       void *vaddr;
+       struct i915_gem_context *owner;
+       struct intel_guc *guc;
+
+       /* bitmap of (host) engine ids */
+       u32 engines;
+       u32 priority;
+       u32 stage_id;
+       u32 proc_desc_offset;
+
+       u16 doorbell_id;
+       unsigned long doorbell_offset;
+
+       /* Protects GuC client's WQ access */
+       spinlock_t wq_lock;
+       /* Per-engine counts of GuC submissions */
+       u64 submissions[I915_NUM_ENGINES];
+};
+
+int intel_guc_submission_init(struct intel_guc *guc);
+int intel_guc_submission_enable(struct intel_guc *guc);
+void intel_guc_submission_disable(struct intel_guc *guc);
+void intel_guc_submission_fini(struct intel_guc *guc);
+
+#endif
index 2fb65eb7cf7485f7d8d4e62dad390d210216f942..1e2a30a40ede342fc96a081191cf8bef954421f8 100644 (file)
@@ -23,8 +23,8 @@
  */
 
 #include "intel_uc.h"
+#include "intel_guc_submission.h"
 #include "i915_drv.h"
-#include "i915_guc_submission.h"
 
 /* Reset GuC providing us with fresh state for both GuC and HuC.
  */