dri-devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: bugzilla-daemon@freedesktop.org
To: dri-devel@lists.freedesktop.org
Subject: [Bug 100941] Improve time to suspend on Radeon HD 6310
Date: Sun, 07 May 2017 08:55:11 +0000	[thread overview]
Message-ID: <bug-100941-502-hvMfk3NKWW@http.bugs.freedesktop.org/> (raw)
In-Reply-To: <bug-100941-502@http.bugs.freedesktop.org/>


[-- Attachment #1.1: Type: text/plain, Size: 1421 bytes --]

https://bugs.freedesktop.org/show_bug.cgi?id=100941

Christian König <deathsimple@vodafone.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |enhancement
         Resolution|NOTABUG                     |---
             Status|RESOLVED                    |REOPENED
           Priority|medium                      |high

--- Comment #4 from Christian König <deathsimple@vodafone.de> ---
Ah, sorry! Yes that is a known issue on APUs.

See the comment in radeon_bo_evict_vram():

        /* late 2.6.33 fix IGP hibernate - we need pm ops to do this correct */
        if (0 && (rdev->flags & RADEON_IS_IGP)) {
                if (rdev->mc.igp_sideport_enabled == false)
                        /* Useless to evict on IGP chips */
                        return 0;
        }
        return ttm_bo_evict_mm(&rdev->mman.bdev, TTM_PL_VRAM);

The problem is the driver doesn't know if we are suspending or hibernating.

For pure suspending (when system memory is still being refreshed) you don't
need to evict VRAM on APUs.

But for hibernating (when system memory is written to disk) you need to evict
VRAM because the normal OS doesn't know about it and won't back it up.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[-- Attachment #1.2: Type: text/html, Size: 3127 bytes --]

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

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  parent reply	other threads:[~2017-05-07  8:55 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-05  7:04 [Bug 100941] Improve time to suspend on bugzilla-daemon
2017-05-05  7:05 ` bugzilla-daemon
2017-05-05  7:06 ` bugzilla-daemon
2017-05-06 18:50 ` [Bug 100941] Improve time to suspend on Radeon HD 6310 bugzilla-daemon
2017-05-06 18:57 ` bugzilla-daemon
2017-05-06 19:44 ` bugzilla-daemon
2017-05-07  8:55 ` bugzilla-daemon [this message]
2018-07-31 16:41 ` bugzilla-daemon
2019-11-19  9:28 ` bugzilla-daemon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bug-100941-502-hvMfk3NKWW@http.bugs.freedesktop.org/ \
    --to=bugzilla-daemon@freedesktop.org \
    --cc=dri-devel@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox