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
next 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).