]> asedeno.scripts.mit.edu Git - linux.git/blob - drivers/gpu/drm/i915/intel_guc_log.c
Merge branch 'for-next' of git://git.kernel.org/pub/scm/linux/kernel/git/nab/target...
[linux.git] / drivers / gpu / drm / i915 / intel_guc_log.c
1 /*
2  * Copyright © 2014-2017 Intel Corporation
3  *
4  * Permission is hereby granted, free of charge, to any person obtaining a
5  * copy of this software and associated documentation files (the "Software"),
6  * to deal in the Software without restriction, including without limitation
7  * the rights to use, copy, modify, merge, publish, distribute, sublicense,
8  * and/or sell copies of the Software, and to permit persons to whom the
9  * Software is furnished to do so, subject to the following conditions:
10  *
11  * The above copyright notice and this permission notice (including the next
12  * paragraph) shall be included in all copies or substantial portions of the
13  * Software.
14  *
15  * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
16  * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
17  * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
18  * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
19  * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
20  * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
21  * IN THE SOFTWARE.
22  *
23  */
24 #include <linux/debugfs.h>
25 #include <linux/relay.h>
26 #include "i915_drv.h"
27
28 static void guc_log_capture_logs(struct intel_guc *guc);
29
30 /**
31  * DOC: GuC firmware log
32  *
33  * Firmware log is enabled by setting i915.guc_log_level to non-negative level.
34  * Log data is printed out via reading debugfs i915_guc_log_dump. Reading from
35  * i915_guc_load_status will print out firmware loading status and scratch
36  * registers value.
37  *
38  */
39
40 static int guc_log_flush_complete(struct intel_guc *guc)
41 {
42         u32 action[] = {
43                 INTEL_GUC_ACTION_LOG_BUFFER_FILE_FLUSH_COMPLETE
44         };
45
46         return intel_guc_send(guc, action, ARRAY_SIZE(action));
47 }
48
49 static int guc_log_flush(struct intel_guc *guc)
50 {
51         u32 action[] = {
52                 INTEL_GUC_ACTION_FORCE_LOG_BUFFER_FLUSH,
53                 0
54         };
55
56         return intel_guc_send(guc, action, ARRAY_SIZE(action));
57 }
58
59 static int guc_log_control(struct intel_guc *guc, u32 control_val)
60 {
61         u32 action[] = {
62                 INTEL_GUC_ACTION_UK_LOG_ENABLE_LOGGING,
63                 control_val
64         };
65
66         return intel_guc_send(guc, action, ARRAY_SIZE(action));
67 }
68
69 /*
70  * Sub buffer switch callback. Called whenever relay has to switch to a new
71  * sub buffer, relay stays on the same sub buffer if 0 is returned.
72  */
73 static int subbuf_start_callback(struct rchan_buf *buf,
74                                  void *subbuf,
75                                  void *prev_subbuf,
76                                  size_t prev_padding)
77 {
78         /* Use no-overwrite mode by default, where relay will stop accepting
79          * new data if there are no empty sub buffers left.
80          * There is no strict synchronization enforced by relay between Consumer
81          * and Producer. In overwrite mode, there is a possibility of getting
82          * inconsistent/garbled data, the producer could be writing on to the
83          * same sub buffer from which Consumer is reading. This can't be avoided
84          * unless Consumer is fast enough and can always run in tandem with
85          * Producer.
86          */
87         if (relay_buf_full(buf))
88                 return 0;
89
90         return 1;
91 }
92
93 /*
94  * file_create() callback. Creates relay file in debugfs.
95  */
96 static struct dentry *create_buf_file_callback(const char *filename,
97                                                struct dentry *parent,
98                                                umode_t mode,
99                                                struct rchan_buf *buf,
100                                                int *is_global)
101 {
102         struct dentry *buf_file;
103
104         /* This to enable the use of a single buffer for the relay channel and
105          * correspondingly have a single file exposed to User, through which
106          * it can collect the logs in order without any post-processing.
107          * Need to set 'is_global' even if parent is NULL for early logging.
108          */
109         *is_global = 1;
110
111         if (!parent)
112                 return NULL;
113
114         /* Not using the channel filename passed as an argument, since for each
115          * channel relay appends the corresponding CPU number to the filename
116          * passed in relay_open(). This should be fine as relay just needs a
117          * dentry of the file associated with the channel buffer and that file's
118          * name need not be same as the filename passed as an argument.
119          */
120         buf_file = debugfs_create_file("guc_log", mode,
121                                        parent, buf, &relay_file_operations);
122         return buf_file;
123 }
124
125 /*
126  * file_remove() default callback. Removes relay file in debugfs.
127  */
128 static int remove_buf_file_callback(struct dentry *dentry)
129 {
130         debugfs_remove(dentry);
131         return 0;
132 }
133
134 /* relay channel callbacks */
135 static struct rchan_callbacks relay_callbacks = {
136         .subbuf_start = subbuf_start_callback,
137         .create_buf_file = create_buf_file_callback,
138         .remove_buf_file = remove_buf_file_callback,
139 };
140
141 static int guc_log_relay_file_create(struct intel_guc *guc)
142 {
143         struct drm_i915_private *dev_priv = guc_to_i915(guc);
144         struct dentry *log_dir;
145         int ret;
146
147         if (i915.guc_log_level < 0)
148                 return 0;
149
150         /* For now create the log file in /sys/kernel/debug/dri/0 dir */
151         log_dir = dev_priv->drm.primary->debugfs_root;
152
153         /* If /sys/kernel/debug/dri/0 location do not exist, then debugfs is
154          * not mounted and so can't create the relay file.
155          * The relay API seems to fit well with debugfs only, for availing relay
156          * there are 3 requirements which can be met for debugfs file only in a
157          * straightforward/clean manner :-
158          * i)   Need the associated dentry pointer of the file, while opening the
159          *      relay channel.
160          * ii)  Should be able to use 'relay_file_operations' fops for the file.
161          * iii) Set the 'i_private' field of file's inode to the pointer of
162          *      relay channel buffer.
163          */
164         if (!log_dir) {
165                 DRM_ERROR("Debugfs dir not available yet for GuC log file\n");
166                 return -ENODEV;
167         }
168
169         ret = relay_late_setup_files(guc->log.runtime.relay_chan, "guc_log", log_dir);
170         if (ret < 0 && ret != -EEXIST) {
171                 DRM_ERROR("Couldn't associate relay chan with file %d\n", ret);
172                 return ret;
173         }
174
175         return 0;
176 }
177
178 static void guc_move_to_next_buf(struct intel_guc *guc)
179 {
180         /* Make sure the updates made in the sub buffer are visible when
181          * Consumer sees the following update to offset inside the sub buffer.
182          */
183         smp_wmb();
184
185         /* All data has been written, so now move the offset of sub buffer. */
186         relay_reserve(guc->log.runtime.relay_chan, guc->log.vma->obj->base.size);
187
188         /* Switch to the next sub buffer */
189         relay_flush(guc->log.runtime.relay_chan);
190 }
191
192 static void *guc_get_write_buffer(struct intel_guc *guc)
193 {
194         if (!guc->log.runtime.relay_chan)
195                 return NULL;
196
197         /* Just get the base address of a new sub buffer and copy data into it
198          * ourselves. NULL will be returned in no-overwrite mode, if all sub
199          * buffers are full. Could have used the relay_write() to indirectly
200          * copy the data, but that would have been bit convoluted, as we need to
201          * write to only certain locations inside a sub buffer which cannot be
202          * done without using relay_reserve() along with relay_write(). So its
203          * better to use relay_reserve() alone.
204          */
205         return relay_reserve(guc->log.runtime.relay_chan, 0);
206 }
207
208 static bool guc_check_log_buf_overflow(struct intel_guc *guc,
209                                        enum guc_log_buffer_type type,
210                                        unsigned int full_cnt)
211 {
212         unsigned int prev_full_cnt = guc->log.prev_overflow_count[type];
213         bool overflow = false;
214
215         if (full_cnt != prev_full_cnt) {
216                 overflow = true;
217
218                 guc->log.prev_overflow_count[type] = full_cnt;
219                 guc->log.total_overflow_count[type] += full_cnt - prev_full_cnt;
220
221                 if (full_cnt < prev_full_cnt) {
222                         /* buffer_full_cnt is a 4 bit counter */
223                         guc->log.total_overflow_count[type] += 16;
224                 }
225                 DRM_ERROR_RATELIMITED("GuC log buffer overflow\n");
226         }
227
228         return overflow;
229 }
230
231 static unsigned int guc_get_log_buffer_size(enum guc_log_buffer_type type)
232 {
233         switch (type) {
234         case GUC_ISR_LOG_BUFFER:
235                 return (GUC_LOG_ISR_PAGES + 1) * PAGE_SIZE;
236         case GUC_DPC_LOG_BUFFER:
237                 return (GUC_LOG_DPC_PAGES + 1) * PAGE_SIZE;
238         case GUC_CRASH_DUMP_LOG_BUFFER:
239                 return (GUC_LOG_CRASH_PAGES + 1) * PAGE_SIZE;
240         default:
241                 MISSING_CASE(type);
242         }
243
244         return 0;
245 }
246
247 static void guc_read_update_log_buffer(struct intel_guc *guc)
248 {
249         unsigned int buffer_size, read_offset, write_offset, bytes_to_copy, full_cnt;
250         struct guc_log_buffer_state *log_buf_state, *log_buf_snapshot_state;
251         struct guc_log_buffer_state log_buf_state_local;
252         enum guc_log_buffer_type type;
253         void *src_data, *dst_data;
254         bool new_overflow;
255
256         if (WARN_ON(!guc->log.runtime.buf_addr))
257                 return;
258
259         /* Get the pointer to shared GuC log buffer */
260         log_buf_state = src_data = guc->log.runtime.buf_addr;
261
262         /* Get the pointer to local buffer to store the logs */
263         log_buf_snapshot_state = dst_data = guc_get_write_buffer(guc);
264
265         /* Actual logs are present from the 2nd page */
266         src_data += PAGE_SIZE;
267         dst_data += PAGE_SIZE;
268
269         for (type = GUC_ISR_LOG_BUFFER; type < GUC_MAX_LOG_BUFFER; type++) {
270                 /* Make a copy of the state structure, inside GuC log buffer
271                  * (which is uncached mapped), on the stack to avoid reading
272                  * from it multiple times.
273                  */
274                 memcpy(&log_buf_state_local, log_buf_state,
275                        sizeof(struct guc_log_buffer_state));
276                 buffer_size = guc_get_log_buffer_size(type);
277                 read_offset = log_buf_state_local.read_ptr;
278                 write_offset = log_buf_state_local.sampled_write_ptr;
279                 full_cnt = log_buf_state_local.buffer_full_cnt;
280
281                 /* Bookkeeping stuff */
282                 guc->log.flush_count[type] += log_buf_state_local.flush_to_file;
283                 new_overflow = guc_check_log_buf_overflow(guc, type, full_cnt);
284
285                 /* Update the state of shared log buffer */
286                 log_buf_state->read_ptr = write_offset;
287                 log_buf_state->flush_to_file = 0;
288                 log_buf_state++;
289
290                 if (unlikely(!log_buf_snapshot_state))
291                         continue;
292
293                 /* First copy the state structure in snapshot buffer */
294                 memcpy(log_buf_snapshot_state, &log_buf_state_local,
295                        sizeof(struct guc_log_buffer_state));
296
297                 /* The write pointer could have been updated by GuC firmware,
298                  * after sending the flush interrupt to Host, for consistency
299                  * set write pointer value to same value of sampled_write_ptr
300                  * in the snapshot buffer.
301                  */
302                 log_buf_snapshot_state->write_ptr = write_offset;
303                 log_buf_snapshot_state++;
304
305                 /* Now copy the actual logs. */
306                 if (unlikely(new_overflow)) {
307                         /* copy the whole buffer in case of overflow */
308                         read_offset = 0;
309                         write_offset = buffer_size;
310                 } else if (unlikely((read_offset > buffer_size) ||
311                                     (write_offset > buffer_size))) {
312                         DRM_ERROR("invalid log buffer state\n");
313                         /* copy whole buffer as offsets are unreliable */
314                         read_offset = 0;
315                         write_offset = buffer_size;
316                 }
317
318                 /* Just copy the newly written data */
319                 if (read_offset > write_offset) {
320                         i915_memcpy_from_wc(dst_data, src_data, write_offset);
321                         bytes_to_copy = buffer_size - read_offset;
322                 } else {
323                         bytes_to_copy = write_offset - read_offset;
324                 }
325                 i915_memcpy_from_wc(dst_data + read_offset,
326                                     src_data + read_offset, bytes_to_copy);
327
328                 src_data += buffer_size;
329                 dst_data += buffer_size;
330         }
331
332         if (log_buf_snapshot_state)
333                 guc_move_to_next_buf(guc);
334         else {
335                 /* Used rate limited to avoid deluge of messages, logs might be
336                  * getting consumed by User at a slow rate.
337                  */
338                 DRM_ERROR_RATELIMITED("no sub-buffer to capture logs\n");
339                 guc->log.capture_miss_count++;
340         }
341 }
342
343 static void capture_logs_work(struct work_struct *work)
344 {
345         struct intel_guc *guc =
346                 container_of(work, struct intel_guc, log.runtime.flush_work);
347
348         guc_log_capture_logs(guc);
349 }
350
351 static bool guc_log_has_runtime(struct intel_guc *guc)
352 {
353         return guc->log.runtime.buf_addr != NULL;
354 }
355
356 static int guc_log_runtime_create(struct intel_guc *guc)
357 {
358         struct drm_i915_private *dev_priv = guc_to_i915(guc);
359         void *vaddr;
360         struct rchan *guc_log_relay_chan;
361         size_t n_subbufs, subbuf_size;
362         int ret = 0;
363
364         lockdep_assert_held(&dev_priv->drm.struct_mutex);
365
366         GEM_BUG_ON(guc_log_has_runtime(guc));
367
368         /* Create a WC (Uncached for read) vmalloc mapping of log
369          * buffer pages, so that we can directly get the data
370          * (up-to-date) from memory.
371          */
372         vaddr = i915_gem_object_pin_map(guc->log.vma->obj, I915_MAP_WC);
373         if (IS_ERR(vaddr)) {
374                 DRM_ERROR("Couldn't map log buffer pages %d\n", ret);
375                 return PTR_ERR(vaddr);
376         }
377
378         guc->log.runtime.buf_addr = vaddr;
379
380          /* Keep the size of sub buffers same as shared log buffer */
381         subbuf_size = guc->log.vma->obj->base.size;
382
383         /* Store up to 8 snapshots, which is large enough to buffer sufficient
384          * boot time logs and provides enough leeway to User, in terms of
385          * latency, for consuming the logs from relay. Also doesn't take
386          * up too much memory.
387          */
388         n_subbufs = 8;
389
390         /* Create a relay channel, so that we have buffers for storing
391          * the GuC firmware logs, the channel will be linked with a file
392          * later on when debugfs is registered.
393          */
394         guc_log_relay_chan = relay_open(NULL, NULL, subbuf_size,
395                                         n_subbufs, &relay_callbacks, dev_priv);
396         if (!guc_log_relay_chan) {
397                 DRM_ERROR("Couldn't create relay chan for GuC logging\n");
398
399                 ret = -ENOMEM;
400                 goto err_vaddr;
401         }
402
403         GEM_BUG_ON(guc_log_relay_chan->subbuf_size < subbuf_size);
404         guc->log.runtime.relay_chan = guc_log_relay_chan;
405
406         INIT_WORK(&guc->log.runtime.flush_work, capture_logs_work);
407
408         /*
409          * GuC log buffer flush work item has to do register access to
410          * send the ack to GuC and this work item, if not synced before
411          * suspend, can potentially get executed after the GFX device is
412          * suspended.
413          * By marking the WQ as freezable, we don't have to bother about
414          * flushing of this work item from the suspend hooks, the pending
415          * work item if any will be either executed before the suspend
416          * or scheduled later on resume. This way the handling of work
417          * item can be kept same between system suspend & rpm suspend.
418          */
419         guc->log.runtime.flush_wq = alloc_ordered_workqueue("i915-guc_log",
420                                                 WQ_HIGHPRI | WQ_FREEZABLE);
421         if (!guc->log.runtime.flush_wq) {
422                 DRM_ERROR("Couldn't allocate the wq for GuC logging\n");
423                 ret = -ENOMEM;
424                 goto err_relaychan;
425         }
426
427         return 0;
428
429 err_relaychan:
430         relay_close(guc->log.runtime.relay_chan);
431 err_vaddr:
432         i915_gem_object_unpin_map(guc->log.vma->obj);
433         guc->log.runtime.buf_addr = NULL;
434         return ret;
435 }
436
437 static void guc_log_runtime_destroy(struct intel_guc *guc)
438 {
439         /*
440          * It's possible that the runtime stuff was never allocated because
441          * guc_log_level was < 0 at the time
442          **/
443         if (!guc_log_has_runtime(guc))
444                 return;
445
446         destroy_workqueue(guc->log.runtime.flush_wq);
447         relay_close(guc->log.runtime.relay_chan);
448         i915_gem_object_unpin_map(guc->log.vma->obj);
449         guc->log.runtime.buf_addr = NULL;
450 }
451
452 static int guc_log_late_setup(struct intel_guc *guc)
453 {
454         struct drm_i915_private *dev_priv = guc_to_i915(guc);
455         int ret;
456
457         lockdep_assert_held(&dev_priv->drm.struct_mutex);
458
459         if (!guc_log_has_runtime(guc)) {
460                 /* If log_level was set as -1 at boot time, then setup needed to
461                  * handle log buffer flush interrupts would not have been done yet,
462                  * so do that now.
463                  */
464                 ret = guc_log_runtime_create(guc);
465                 if (ret)
466                         goto err;
467         }
468
469         ret = guc_log_relay_file_create(guc);
470         if (ret)
471                 goto err_runtime;
472
473         return 0;
474
475 err_runtime:
476         guc_log_runtime_destroy(guc);
477 err:
478         /* logging will remain off */
479         i915.guc_log_level = -1;
480         return ret;
481 }
482
483 static void guc_log_capture_logs(struct intel_guc *guc)
484 {
485         struct drm_i915_private *dev_priv = guc_to_i915(guc);
486
487         guc_read_update_log_buffer(guc);
488
489         /* Generally device is expected to be active only at this
490          * time, so get/put should be really quick.
491          */
492         intel_runtime_pm_get(dev_priv);
493         guc_log_flush_complete(guc);
494         intel_runtime_pm_put(dev_priv);
495 }
496
497 static void guc_flush_logs(struct intel_guc *guc)
498 {
499         struct drm_i915_private *dev_priv = guc_to_i915(guc);
500
501         if (!i915.enable_guc_submission || (i915.guc_log_level < 0))
502                 return;
503
504         /* First disable the interrupts, will be renabled afterwards */
505         gen9_disable_guc_interrupts(dev_priv);
506
507         /* Before initiating the forceful flush, wait for any pending/ongoing
508          * flush to complete otherwise forceful flush may not actually happen.
509          */
510         flush_work(&guc->log.runtime.flush_work);
511
512         /* Ask GuC to update the log buffer state */
513         guc_log_flush(guc);
514
515         /* GuC would have updated log buffer by now, so capture it */
516         guc_log_capture_logs(guc);
517 }
518
519 int intel_guc_log_create(struct intel_guc *guc)
520 {
521         struct i915_vma *vma;
522         unsigned long offset;
523         uint32_t size, flags;
524         int ret;
525
526         GEM_BUG_ON(guc->log.vma);
527
528         if (i915.guc_log_level > GUC_LOG_VERBOSITY_MAX)
529                 i915.guc_log_level = GUC_LOG_VERBOSITY_MAX;
530
531         /* The first page is to save log buffer state. Allocate one
532          * extra page for others in case for overlap */
533         size = (1 + GUC_LOG_DPC_PAGES + 1 +
534                 GUC_LOG_ISR_PAGES + 1 +
535                 GUC_LOG_CRASH_PAGES + 1) << PAGE_SHIFT;
536
537         /* We require SSE 4.1 for fast reads from the GuC log buffer and
538          * it should be present on the chipsets supporting GuC based
539          * submisssions.
540          */
541         if (WARN_ON(!i915_has_memcpy_from_wc())) {
542                 ret = -EINVAL;
543                 goto err;
544         }
545
546         vma = intel_guc_allocate_vma(guc, size);
547         if (IS_ERR(vma)) {
548                 ret = PTR_ERR(vma);
549                 goto err;
550         }
551
552         guc->log.vma = vma;
553
554         if (i915.guc_log_level >= 0) {
555                 ret = guc_log_runtime_create(guc);
556                 if (ret < 0)
557                         goto err_vma;
558         }
559
560         /* each allocated unit is a page */
561         flags = GUC_LOG_VALID | GUC_LOG_NOTIFY_ON_HALF_FULL |
562                 (GUC_LOG_DPC_PAGES << GUC_LOG_DPC_SHIFT) |
563                 (GUC_LOG_ISR_PAGES << GUC_LOG_ISR_SHIFT) |
564                 (GUC_LOG_CRASH_PAGES << GUC_LOG_CRASH_SHIFT);
565
566         offset = guc_ggtt_offset(vma) >> PAGE_SHIFT; /* in pages */
567         guc->log.flags = (offset << GUC_LOG_BUF_ADDR_SHIFT) | flags;
568
569         return 0;
570
571 err_vma:
572         i915_vma_unpin_and_release(&guc->log.vma);
573 err:
574         /* logging will be off */
575         i915.guc_log_level = -1;
576         return ret;
577 }
578
579 void intel_guc_log_destroy(struct intel_guc *guc)
580 {
581         guc_log_runtime_destroy(guc);
582         i915_vma_unpin_and_release(&guc->log.vma);
583 }
584
585 int i915_guc_log_control(struct drm_i915_private *dev_priv, u64 control_val)
586 {
587         struct intel_guc *guc = &dev_priv->guc;
588
589         union guc_log_control log_param;
590         int ret;
591
592         log_param.value = control_val;
593
594         if (log_param.verbosity < GUC_LOG_VERBOSITY_MIN ||
595             log_param.verbosity > GUC_LOG_VERBOSITY_MAX)
596                 return -EINVAL;
597
598         /* This combination doesn't make sense & won't have any effect */
599         if (!log_param.logging_enabled && (i915.guc_log_level < 0))
600                 return 0;
601
602         ret = guc_log_control(guc, log_param.value);
603         if (ret < 0) {
604                 DRM_DEBUG_DRIVER("guc_logging_control action failed %d\n", ret);
605                 return ret;
606         }
607
608         if (log_param.logging_enabled) {
609                 i915.guc_log_level = log_param.verbosity;
610
611                 /* If log_level was set as -1 at boot time, then the relay channel file
612                  * wouldn't have been created by now and interrupts also would not have
613                  * been enabled. Try again now, just in case.
614                  */
615                 ret = guc_log_late_setup(guc);
616                 if (ret < 0) {
617                         DRM_DEBUG_DRIVER("GuC log late setup failed %d\n", ret);
618                         return ret;
619                 }
620
621                 /* GuC logging is currently the only user of Guc2Host interrupts */
622                 gen9_enable_guc_interrupts(dev_priv);
623         } else {
624                 /* Once logging is disabled, GuC won't generate logs & send an
625                  * interrupt. But there could be some data in the log buffer
626                  * which is yet to be captured. So request GuC to update the log
627                  * buffer state and then collect the left over logs.
628                  */
629                 guc_flush_logs(guc);
630
631                 /* As logging is disabled, update log level to reflect that */
632                 i915.guc_log_level = -1;
633         }
634
635         return ret;
636 }
637
638 void i915_guc_log_register(struct drm_i915_private *dev_priv)
639 {
640         if (!i915.enable_guc_submission || i915.guc_log_level < 0)
641                 return;
642
643         mutex_lock(&dev_priv->drm.struct_mutex);
644         guc_log_late_setup(&dev_priv->guc);
645         mutex_unlock(&dev_priv->drm.struct_mutex);
646 }
647
648 void i915_guc_log_unregister(struct drm_i915_private *dev_priv)
649 {
650         if (!i915.enable_guc_submission)
651                 return;
652
653         mutex_lock(&dev_priv->drm.struct_mutex);
654         /* GuC logging is currently the only user of Guc2Host interrupts */
655         gen9_disable_guc_interrupts(dev_priv);
656         guc_log_runtime_destroy(&dev_priv->guc);
657         mutex_unlock(&dev_priv->drm.struct_mutex);
658 }