dri-devel.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
From: David Herrmann <dh.herrmann@gmail.com>
To: dri-devel@lists.freedesktop.org
Cc: Dave Airlie <airlied@redhat.com>, Daniel Vetter <daniel.vetter@ffwll.ch>
Subject: [PATCH 0/4] DRM: Remove MM "Pre-Alloc"
Date: Thu, 25 Jul 2013 15:55:58 +0200	[thread overview]
Message-ID: <1374760562-6096-1-git-send-email-dh.herrmann@gmail.com> (raw)

Hi

See patch 4/4 for a more detailed explanation of this series. I basically kill
off the whole drm_mm pre-alloc code as it really doesn't make any sense with
todays infrastructure. No drm_mm user runs in atomic context. We use pre-alloc
only to allow allocation while holding a spin-lock. But we can easily
kzalloc() the node before taking the spinlock and use the drm_mm_insert_*()
helpers directly.

This series converts the last pre-alloc users (ttm and i915-gem-stolen) to use
the already established kzalloc()+drm_mm_insert_*() helpers.

The last patch removes a bunch of old drm_mm code, so Daniel can tackle his
"drm_mm documentation" TODO list.

Cheers
David

David Herrmann (4):
  drm/mm: add "best_match" to drm_mm_insert_node()
  drm/ttm: replace drm_mm_pre_get() by direct alloc
  drm/i915: pre-alloc instead of drm_mm search/get_block
  drm/mm: remove unused API

 drivers/gpu/drm/drm_mm.c               | 183 +++++----------------------------
 drivers/gpu/drm/drm_vma_manager.c      |   4 +-
 drivers/gpu/drm/i915/i915_gem.c        |   3 +-
 drivers/gpu/drm/i915/i915_gem_stolen.c |  72 ++++++++-----
 drivers/gpu/drm/sis/sis_mm.c           |   4 +-
 drivers/gpu/drm/ttm/ttm_bo_manager.c   |  40 ++++---
 drivers/gpu/drm/via/via_mm.c           |   4 +-
 include/drm/drm_mm.h                   | 131 +++++------------------
 8 files changed, 123 insertions(+), 318 deletions(-)

-- 
1.8.3.3

             reply	other threads:[~2013-07-25 13:56 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-25 13:55 David Herrmann [this message]
2013-07-25 13:55 ` [PATCH 1/4] drm/mm: add "best_match" to drm_mm_insert_node() David Herrmann
2013-07-25 14:15   ` Daniel Vetter
2013-07-27 11:36   ` [PATCH 1/4] drm/mm: add "best_match" flag " David Herrmann
2013-08-05  7:46     ` Daniel Vetter
2013-08-06  9:39       ` David Herrmann
2013-08-06 11:03         ` Daniel Vetter
2013-07-25 13:56 ` [PATCH 2/4] drm/ttm: replace drm_mm_pre_get() by direct alloc David Herrmann
2013-07-25 14:25   ` Daniel Vetter
2013-07-27 11:37   ` [PATCH v2 " David Herrmann
2013-07-25 13:56 ` [PATCH 3/4] drm/i915: pre-alloc instead of drm_mm search/get_block David Herrmann
2013-07-25 14:14   ` Chris Wilson
2013-07-27 11:38   ` [PATCH v2 " David Herrmann
2013-07-27 13:06     ` Chris Wilson
2013-07-27 13:09       ` David Herrmann
2013-07-27 13:15         ` Chris Wilson
2013-07-27 14:21     ` [PATCH v3 " David Herrmann
2013-08-05  7:49       ` Daniel Vetter
2013-07-25 13:56 ` [PATCH 4/4] drm/mm: remove unused API David Herrmann
2013-07-25 14:27   ` Daniel Vetter
2013-07-27 11:39   ` [PATCH v2 " David Herrmann

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=1374760562-6096-1-git-send-email-dh.herrmann@gmail.com \
    --to=dh.herrmann@gmail.com \
    --cc=airlied@redhat.com \
    --cc=daniel.vetter@ffwll.ch \
    --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;
as well as URLs for NNTP newsgroup(s).