<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body><span class="vcard"><a class="email" href="mailto:pmenzel+bugs.freedesktop@molgen.mpg.de" title="Paul Menzel <pmenzel+bugs.freedesktop@molgen.mpg.de>"> <span class="fn">Paul Menzel</span></a>
</span> changed
          <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED INVALID - radeon_gart_table_vram_pin takes 473 ms during ACPI S3 resume"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=107328">bug 107328</a>
          <br>
             <table border="1" cellspacing="0" cellpadding="8">
          <tr>
            <th>What</th>
            <th>Removed</th>
            <th>Added</th>
          </tr>

         <tr>
           <td style="text-align:right;">Resolution</td>
           <td>WONTFIX
           </td>
           <td>INVALID
           </td>
         </tr></table>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED INVALID - radeon_gart_table_vram_pin takes 473 ms during ACPI S3 resume"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=107328#c2">Comment # 2</a>
              on <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED INVALID - radeon_gart_table_vram_pin takes 473 ms during ACPI S3 resume"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=107328">bug 107328</a>
              from <span class="vcard"><a class="email" href="mailto:pmenzel+bugs.freedesktop@molgen.mpg.de" title="Paul Menzel <pmenzel+bugs.freedesktop@molgen.mpg.de>"> <span class="fn">Paul Menzel</span></a>
</span></b>
        <pre>Just as a follow-up, the long time seems to have been caused by ftrace.

With the patch below on top of drm-tip from this morning, it only takes 7 ms. 

diff --git a/drivers/gpu/drm/radeon/radeon_gart.c
b/drivers/gpu/drm/radeon/radeon_gart.c
index 1cef155cc933..f79a2f8b8d8e 100644
--- a/drivers/gpu/drm/radeon/radeon_gart.c
+++ b/drivers/gpu/drm/radeon/radeon_gart.c
@@ -175,10 +175,13 @@ int radeon_gart_table_vram_pin(struct radeon_device
*rdev)
                /* We might have dropped some GART table updates while it
wasn't
                 * mapped, restore all entries
                 */
+               DRM_INFO("GART: Restore entries: num cpu pages %u, num gpu
pages %u\n",
+                               rdev->gart.num_cpu_pages,
rdev->gart.num_gpu_pages);
                for (i = 0; i < rdev->gart.num_gpu_pages; i++)
                        radeon_gart_set_page(rdev, i,
rdev->gart.pages_entry[i]);
                mb();
                radeon_gart_tlb_flush(rdev);
+               DRM_INFO("GART: Done restoring entries\n");
        }

        return r;</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>