dri-devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: alexdeucher@gmail.com
To: dri-devel@lists.freedesktop.org
Cc: Alex Deucher <alexander.deucher@amd.com>
Subject: [PATCH 0/2] Properly suspend/resume sub-allocator
Date: Mon, 11 Mar 2013 17:06:22 -0400	[thread overview]
Message-ID: <1363035984-4861-1-git-send-email-alexdeucher@gmail.com> (raw)

From: Alex Deucher <alexander.deucher@amd.com>

This patch set is a bit of a heavy hammer, but I'm
not sure it's worth the effort to create separate
suspend functions for all sub-allocator
users that basically just calls radeon_sa_bo_manager_suspend()
and fix up all the sub allocator init functions to
call radeon_sa_bo_manager_start() since all we are really
avoiding is the allocation of sub-allocator bo.  Additionally
in the VM case, we have to tear down all the VMs on suspend
since the physical addresses of various buffers may have changed
on resume.

The first patch fixes suspend and resume on cards with VM in use.

The second patches doesn't fix any bugs that I know of, but
the current code seems wrong to me unless I'm missing something.

Alex Deucher (2):
  drm/radeon: fix S/R on VM systems (cayman/TN/SI)
  drm/radeon: tear down the IB pool on suspend

 drivers/gpu/drm/radeon/evergreen.c |    1 +
 drivers/gpu/drm/radeon/ni.c        |    2 ++
 drivers/gpu/drm/radeon/r100.c      |    1 +
 drivers/gpu/drm/radeon/r300.c      |    1 +
 drivers/gpu/drm/radeon/r420.c      |    1 +
 drivers/gpu/drm/radeon/r600.c      |    1 +
 drivers/gpu/drm/radeon/rs400.c     |    1 +
 drivers/gpu/drm/radeon/rs600.c     |    1 +
 drivers/gpu/drm/radeon/rs690.c     |    1 +
 drivers/gpu/drm/radeon/rv515.c     |    1 +
 drivers/gpu/drm/radeon/rv770.c     |    1 +
 drivers/gpu/drm/radeon/si.c        |    2 ++
 12 files changed, 14 insertions(+), 0 deletions(-)

-- 
1.7.7.5

             reply	other threads:[~2013-03-11 21:06 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-11 21:06 alexdeucher [this message]
2013-03-11 21:06 ` [PATCH 1/2] drm/radeon: fix S/R on VM systems (cayman/TN/SI) alexdeucher
2013-03-12  8:59   ` Christian König
2013-03-11 21:06 ` [PATCH 2/2] drm/radeon: tear down the IB pool on suspend alexdeucher
2013-03-12  8:52   ` Christian König

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=1363035984-4861-1-git-send-email-alexdeucher@gmail.com \
    --to=alexdeucher@gmail.com \
    --cc=alexander.deucher@amd.com \
    --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