From: j.glisse@gmail.com
To: dri-devel@lists.freedesktop.org
Cc: thellstrom@vmware.com
Subject: ttm: merge ttm_backend & ttm_tt, introduce ttm dma allocator
Date: Thu, 3 Nov 2011 19:39:04 -0400 [thread overview]
Message-ID: <1320363555-28438-1-git-send-email-j.glisse@gmail.com> (raw)
Hi,
So updated patchset, only patch 5 seen change since last set.
Last 3 patch are from your patchset, modified on top of mine.
Konrad so i added you dma pool allocator on top of that
and added support for it to radeon. All in all it's slightly
smaller than your patchset.
Biggest change is use of a list_head in ttm_tt to keep the
dma_page list inside the ttm_tt object allowing faster and
lot simpler deallocation of page.
I only briefly test this code, it seems ok so far. Did you
tested booting kernel with swiotlb=force and with your patchset ?
Because here it doesn't work. I still don't understand why
swiotlb want to create a bounce page when the page supplied
fit the constraint. Need to dig into kernel history to see if
there is any good reasons for that.
Otherwise i believe this whole patchset make things cleaner
and simpler for ttm.
Cheers,
Jerome Glisse
next reply other threads:[~2011-11-03 23:35 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-03 23:39 j.glisse [this message]
2011-11-03 23:39 ` [PATCH 01/11] drm/ttm: remove userspace backed ttm object support j.glisse
2011-11-03 23:39 ` [PATCH 02/11] drm/ttm: remove split btw highmen and lowmem page j.glisse
2011-11-03 23:39 ` [PATCH 03/11] drm/ttm: remove unused backend flags field j.glisse
2011-11-03 23:39 ` [PATCH 04/11] drm/ttm: use ttm put pages function to properly restore cache attribute j.glisse
2011-11-03 23:39 ` [PATCH 05/11] drm/ttm: convert page allocation to use page ptr array instead of list V3 j.glisse
2011-11-03 23:39 ` [PATCH 06/11] drm/ttm: test for dma_address array allocation failure j.glisse
2011-11-03 23:39 ` [PATCH 07/11] drm/ttm: merge ttm_backend and ttm_tt j.glisse
2011-11-07 14:39 ` Konrad Rzeszutek Wilk
2011-11-03 23:39 ` [PATCH 08/11] drm/ttm: introduce callback for ttm_tt populate & unpopulate j.glisse
2011-11-07 14:40 ` Konrad Rzeszutek Wilk
2011-11-07 17:35 ` Rob Clark
2011-11-07 19:39 ` Jerome Glisse
2011-11-07 20:17 ` Alex Deucher
2011-11-03 23:39 ` [PATCH 09/11] ttm: Provide DMA aware TTM page pool code j.glisse
2011-11-03 23:39 ` [PATCH 10/11] swiotlb: Expose swiotlb_nr_tlb function to modules j.glisse
2011-11-03 23:39 ` [PATCH 11/11] drm/radeon/kms: Enable the TTM DMA pool if swiotlb is on j.glisse
2011-11-07 17:50 ` ttm: merge ttm_backend & ttm_tt, introduce ttm dma allocator Konrad Rzeszutek Wilk
-- strict thread matches above, loose matches on Subject: below --
2011-11-09 20:22 j.glisse
2011-11-09 20:25 ` Thomas Hellstrom
2011-11-09 21:03 ` Jerome Glisse
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=1320363555-28438-1-git-send-email-j.glisse@gmail.com \
--to=j.glisse@gmail.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=thellstrom@vmware.com \
/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