From: "Christian König" <ckoenig.leichtzumerken@gmail.com>
To: Al Dunsmuir <al.dunsmuir@sympatico.ca>,
Dave Airlie <airlied@gmail.com>,
Alex Deucher <alexdeucher@gmail.com>
Cc: nouveau <nouveau@lists.freedesktop.org>,
Maling list - DRI developers <dri-devel@lists.freedesktop.org>,
amd-gfx list <amd-gfx@lists.freedesktop.org>
Subject: Re: [RFC] Remove AGP support from Radeon/Nouveau/TTM
Date: Tue, 12 May 2020 10:11:42 +0200 [thread overview]
Message-ID: <7a506bb2-1bde-ebde-9bf1-bc115b99e873@gmail.com> (raw)
In-Reply-To: <1266714942.20200511165648@sympatico.ca>
Am 11.05.20 um 22:56 schrieb Al Dunsmuir:
> Hello Dave,
>
> On Monday, May 11, 2020, 4:43:01 PM, Dave Airlie wrote:
>> On Tue, 12 May 2020 at 06:28, Alex Deucher <alexdeucher@gmail.com> wrote:
>>> [SNIP]
>> Maybe we can find some way to compartmentalise AGP further?
>> Dave.
> Significantly reduced caching on memory accesses definitely sounds
> like something that would be noticeable and objectionable.
>
> I would speculate that this would also vary a lot across chipsets,
> depending on the capabilities of the PCI MMU vs the AGP MMU.
>
> In the end, it may be best to leave things as is, or as Dave suggested
> try to keep AGP in the picture.
The problem is that AGP was never really supported/implemented that well
in the first place.
The fact that the core linux kernel and the DMA API doesn't support
uncached memory and we had to change the caching attributes of pages
under the hood has resulted in a huge number of problems over the years.
Keeping it as it is is also not a really doable option because TTM
already has major problems keeping up with the requirements for modern
hardware, see my presentation here as well:
https://fosdem.org/2020/schedule/event/ttm/
Redesigning the old AGP support into something which isn't so ugly and
doesn't blocks the new requirements has the huge risk of breaking things
even harder, e.g. black screen instead of just reduced performance.
So removing/disabling AGP by default still sounds like the best option
to me for end users.
> Nothing is ever simple, is it?
At least not with AGP, no :) It has been a total beast to support and
keep working.
Do I get this right that I can ping you to test things?
Thanks for the feedback,
Christian.
> Al
>
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2020-05-12 8:11 UTC|newest]
Thread overview: 65+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-11 17:17 [RFC] Remove AGP support from Radeon/Nouveau/TTM Christian König
2020-05-11 17:17 ` [PATCH 1/3] drm/radeon: remove AGP support Christian König
2020-05-11 20:12 ` Alex Deucher
2020-05-12 8:16 ` Michel Dänzer
2020-05-12 13:36 ` Alex Deucher
2020-05-12 20:52 ` [Nouveau] " Roy Spliet
2020-05-12 21:12 ` Alex Deucher
2020-05-13 7:46 ` Christian König
2020-05-13 7:52 ` Michel Dänzer
2020-05-12 18:20 ` Thomas Zimmermann
2020-05-12 18:29 ` Alex Deucher
2020-05-11 17:17 ` [PATCH 2/3] drm/nouveau: " Christian König
2020-05-11 17:17 ` [PATCH 3/3] drm/ttm: " Christian König
2020-05-11 20:14 ` [RFC] Remove AGP support from Radeon/Nouveau/TTM Al Dunsmuir
2020-05-11 20:27 ` Alex Deucher
2020-05-11 20:43 ` Dave Airlie
2020-05-11 20:56 ` Al Dunsmuir
2020-05-12 8:11 ` Christian König [this message]
2020-05-11 20:59 ` Emil Velikov
2020-05-11 20:46 ` Al Dunsmuir
2020-05-12 18:29 ` [Nouveau] " Thomas Zimmermann
2020-05-12 18:32 ` Alex Deucher
2020-05-12 19:10 ` Thomas Zimmermann
2020-05-12 19:47 ` Alex Deucher
2020-05-13 9:27 ` Emil Velikov
2020-05-13 11:03 ` Thomas Zimmermann
2020-05-13 11:00 ` Thomas Zimmermann
2020-05-11 19:55 John Paul Adrian Glaubitz
2020-05-11 20:05 ` Alex Deucher
2020-05-11 20:25 ` Rui Salvaterra
2020-05-11 20:51 ` Alex Deucher
2020-05-12 7:57 ` Michel Dänzer
2020-05-12 9:21 ` Rui Salvaterra
2020-05-12 9:40 ` Karoly Balogh (Charlie/SGR)
2020-05-12 13:22 ` Alex Deucher
2020-05-12 16:37 ` Daniel Vetter
2020-05-12 17:02 ` Rui Salvaterra
2020-05-12 18:21 ` Alex Deucher
2020-05-12 20:12 ` Dave Airlie
2020-05-13 7:26 ` Christian König
2020-05-12 18:22 ` Alex Deucher
2020-05-13 7:19 ` Daniel Vetter
2020-05-13 7:55 ` Christian König
2020-05-13 10:25 ` Daniel Vetter
2020-05-13 9:28 ` Rui Salvaterra
2020-05-13 10:26 ` Michel Dänzer
2020-05-13 10:29 ` Daniel Vetter
2020-05-13 10:32 ` Michel Dänzer
2020-05-13 10:39 ` Rui Salvaterra
2020-05-13 10:57 ` Michel Dänzer
2020-05-13 11:07 ` Rui Salvaterra
2020-05-13 13:44 ` Christian Zigotzky
2020-05-13 14:01 ` John Paul Adrian Glaubitz
2020-05-13 14:01 ` Rui Salvaterra
2020-05-13 20:19 ` Bertrand Dekoninck
2020-05-13 20:36 ` Rui Salvaterra
2020-05-11 20:41 ` John Paul Adrian Glaubitz
2020-05-11 20:46 ` Alex Deucher
2020-05-11 21:50 ` John Paul Adrian Glaubitz
2020-05-12 7:43 ` Christian König
2020-05-11 20:12 ` Christian König
2020-05-11 20:24 ` John Paul Adrian Glaubitz
2020-05-11 20:48 ` Gerhard Pircher
2020-05-12 5:04 ` David VANTYGHEM
2020-05-12 9:20 ` John Paul Adrian Glaubitz
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=7a506bb2-1bde-ebde-9bf1-bc115b99e873@gmail.com \
--to=ckoenig.leichtzumerken@gmail.com \
--cc=airlied@gmail.com \
--cc=al.dunsmuir@sympatico.ca \
--cc=alexdeucher@gmail.com \
--cc=amd-gfx@lists.freedesktop.org \
--cc=christian.koenig@amd.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=nouveau@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).