All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Christian König" <ckoenig.leichtzumerken@gmail.com>
To: Alex Deucher <alexdeucher@gmail.com>, Roy Spliet <nouveau@spliet.org>
Cc: nouveau <nouveau@lists.freedesktop.org>,
	"Michel Dänzer" <michel@daenzer.net>,
	"amd-gfx list" <amd-gfx@lists.freedesktop.org>,
	"Maling list - DRI developers" <dri-devel@lists.freedesktop.org>
Subject: Re: [Nouveau] [PATCH 1/3] drm/radeon: remove AGP support
Date: Wed, 13 May 2020 09:46:19 +0200	[thread overview]
Message-ID: <5c3da9d3-8b77-e97f-28a2-e1083a53a1d7@gmail.com> (raw)
In-Reply-To: <CADnq5_NK6ziShvkDug5xGHm7oZTf9qEu41gOxvNANQHFD2a-qQ@mail.gmail.com>

Am 12.05.20 um 23:12 schrieb Alex Deucher:
> On Tue, May 12, 2020 at 4:52 PM Roy Spliet <nouveau@spliet.org> wrote:
>> Op 12-05-2020 om 14:36 schreef Alex Deucher:
>>> On Tue, May 12, 2020 at 4:16 AM Michel Dänzer <michel@daenzer.net> wrote:
>>>> On 2020-05-11 10:12 p.m., Alex Deucher wrote:
>>>>> On Mon, May 11, 2020 at 1:17 PM Christian König
>>>>> <ckoenig.leichtzumerken@gmail.com> wrote:
>>>>>> AGP is deprecated for 10+ years now and not used any more on modern hardware.
>>>>>>
>>>>>> Old hardware should continue to work in PCI mode.
>>>>> Might want to clarify that there is no loss of functionality here.
>>>>> Something like:
>>>>>
>>>>> "There is no loss of functionality here.  GPUs will continue to
>>>>> function.  This just drops the use of the AGP MMU in the chipset in
>>>>> favor of the MMU on the device which has proven to be much more
>>>>> reliable.  Due to its unreliability, AGP support has been disabled on
>>>>> PowerPC for years already so there is no change on PowerPC."
>>>> There's a difference between something being disabled by default or not
>>>> being available at all. We may decide it's worth it anyway, but let's do
>>>> it based on facts.
>>>>
>>> I didn't mean to imply that AGP GART support was already removed.  But
>>> for the vast majority of users the end result is the same.  If you
>>> knew enough re-enable AGP GART, you probably wouldn't have been as
>>> confused about what this patch set does either.  To reiterate, this
>>> patch set does not remove support for AGP cards, it only removes the
>>> support for AGP GART.  The cards will still be functional using the
>>> device GART.  There may be performance tradeoffs there in some cases.
>> I'll volunteer to be the one asking: how big is this performance
>> difference? Have any benchmarks been run before and after removal of AGP
>> GART code on affected nouveau/radeon systems? Or is this code being
>> dropped _just_ because it's cumbersome, with no regard for metrics that
>> determine the value of AGP GART support?
>>
> I don't think anyone has any solid numbers, just anecdotal from
> memory.  I certainly don't have any functional AGP systems at this
> point.  It's mostly just cumbersome and would allow us to clean ttm
> and probably improve stability at the same time.  At least on the
> radeon side, the only native AGP cards were r1xx, r2xx, and some of
> the early r3xx boards.  Once we switched to pcie mid-way through r3xx,
> everything was native pcie and the AGP cards used a pcie to AGP bridge
> chip so they had a decent on chip MMU.  Those older cards topped out
> at maybe 32 or 64 MB of vram, so they are going to be hard pressed to
> deal with modern desktops anyway.  No idea what sort of GART
> capabilities NV AGP hardware at this time had.

I could only test with an old x86 Mac and an r3xx generation hw and in 
this case making the switch didn't had any noticeable effect at all.

But I didn't do more than playing around with the desktop effects and 
playing a video.

I do have a PC x86 AGP board lying around somewhere here, going top give 
that one a try a well.

Christian.

>
> Alex
>
>> Roy
>>
>>> Alex
>>> _______________________________________________
>>> Nouveau mailing list
>>> Nouveau@lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/nouveau
>>>

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

WARNING: multiple messages have this Message-ID (diff)
From: "Christian König" <ckoenig.leichtzumerken-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Alex Deucher
	<alexdeucher-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Roy Spliet <nouveau-NQbd8FSOZ1kdnm+yROfE0A@public.gmane.org>
Cc: nouveau
	<nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>,
	"Michel Dänzer" <michel-otUistvHUpPR7s880joybQ@public.gmane.org>,
	"amd-gfx list"
	<amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>,
	"Maling list - DRI developers"
	<dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>
Subject: Re: [PATCH 1/3] drm/radeon: remove AGP support
Date: Wed, 13 May 2020 09:46:19 +0200	[thread overview]
Message-ID: <5c3da9d3-8b77-e97f-28a2-e1083a53a1d7@gmail.com> (raw)
In-Reply-To: <CADnq5_NK6ziShvkDug5xGHm7oZTf9qEu41gOxvNANQHFD2a-qQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

Am 12.05.20 um 23:12 schrieb Alex Deucher:
> On Tue, May 12, 2020 at 4:52 PM Roy Spliet <nouveau@spliet.org> wrote:
>> Op 12-05-2020 om 14:36 schreef Alex Deucher:
>>> On Tue, May 12, 2020 at 4:16 AM Michel Dänzer <michel@daenzer.net> wrote:
>>>> On 2020-05-11 10:12 p.m., Alex Deucher wrote:
>>>>> On Mon, May 11, 2020 at 1:17 PM Christian König
>>>>> <ckoenig.leichtzumerken@gmail.com> wrote:
>>>>>> AGP is deprecated for 10+ years now and not used any more on modern hardware.
>>>>>>
>>>>>> Old hardware should continue to work in PCI mode.
>>>>> Might want to clarify that there is no loss of functionality here.
>>>>> Something like:
>>>>>
>>>>> "There is no loss of functionality here.  GPUs will continue to
>>>>> function.  This just drops the use of the AGP MMU in the chipset in
>>>>> favor of the MMU on the device which has proven to be much more
>>>>> reliable.  Due to its unreliability, AGP support has been disabled on
>>>>> PowerPC for years already so there is no change on PowerPC."
>>>> There's a difference between something being disabled by default or not
>>>> being available at all. We may decide it's worth it anyway, but let's do
>>>> it based on facts.
>>>>
>>> I didn't mean to imply that AGP GART support was already removed.  But
>>> for the vast majority of users the end result is the same.  If you
>>> knew enough re-enable AGP GART, you probably wouldn't have been as
>>> confused about what this patch set does either.  To reiterate, this
>>> patch set does not remove support for AGP cards, it only removes the
>>> support for AGP GART.  The cards will still be functional using the
>>> device GART.  There may be performance tradeoffs there in some cases.
>> I'll volunteer to be the one asking: how big is this performance
>> difference? Have any benchmarks been run before and after removal of AGP
>> GART code on affected nouveau/radeon systems? Or is this code being
>> dropped _just_ because it's cumbersome, with no regard for metrics that
>> determine the value of AGP GART support?
>>
> I don't think anyone has any solid numbers, just anecdotal from
> memory.  I certainly don't have any functional AGP systems at this
> point.  It's mostly just cumbersome and would allow us to clean ttm
> and probably improve stability at the same time.  At least on the
> radeon side, the only native AGP cards were r1xx, r2xx, and some of
> the early r3xx boards.  Once we switched to pcie mid-way through r3xx,
> everything was native pcie and the AGP cards used a pcie to AGP bridge
> chip so they had a decent on chip MMU.  Those older cards topped out
> at maybe 32 or 64 MB of vram, so they are going to be hard pressed to
> deal with modern desktops anyway.  No idea what sort of GART
> capabilities NV AGP hardware at this time had.

I could only test with an old x86 Mac and an r3xx generation hw and in 
this case making the switch didn't had any noticeable effect at all.

But I didn't do more than playing around with the desktop effects and 
playing a video.

I do have a PC x86 AGP board lying around somewhere here, going top give 
that one a try a well.

Christian.

>
> Alex
>
>> Roy
>>
>>> Alex
>>> _______________________________________________
>>> Nouveau mailing list
>>> Nouveau@lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/nouveau
>>>

_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/nouveau

WARNING: multiple messages have this Message-ID (diff)
From: "Christian König" <ckoenig.leichtzumerken@gmail.com>
To: Alex Deucher <alexdeucher@gmail.com>, Roy Spliet <nouveau@spliet.org>
Cc: nouveau <nouveau@lists.freedesktop.org>,
	"Michel Dänzer" <michel@daenzer.net>,
	"amd-gfx list" <amd-gfx@lists.freedesktop.org>,
	"Maling list - DRI developers" <dri-devel@lists.freedesktop.org>
Subject: Re: [Nouveau] [PATCH 1/3] drm/radeon: remove AGP support
Date: Wed, 13 May 2020 09:46:19 +0200	[thread overview]
Message-ID: <5c3da9d3-8b77-e97f-28a2-e1083a53a1d7@gmail.com> (raw)
In-Reply-To: <CADnq5_NK6ziShvkDug5xGHm7oZTf9qEu41gOxvNANQHFD2a-qQ@mail.gmail.com>

Am 12.05.20 um 23:12 schrieb Alex Deucher:
> On Tue, May 12, 2020 at 4:52 PM Roy Spliet <nouveau@spliet.org> wrote:
>> Op 12-05-2020 om 14:36 schreef Alex Deucher:
>>> On Tue, May 12, 2020 at 4:16 AM Michel Dänzer <michel@daenzer.net> wrote:
>>>> On 2020-05-11 10:12 p.m., Alex Deucher wrote:
>>>>> On Mon, May 11, 2020 at 1:17 PM Christian König
>>>>> <ckoenig.leichtzumerken@gmail.com> wrote:
>>>>>> AGP is deprecated for 10+ years now and not used any more on modern hardware.
>>>>>>
>>>>>> Old hardware should continue to work in PCI mode.
>>>>> Might want to clarify that there is no loss of functionality here.
>>>>> Something like:
>>>>>
>>>>> "There is no loss of functionality here.  GPUs will continue to
>>>>> function.  This just drops the use of the AGP MMU in the chipset in
>>>>> favor of the MMU on the device which has proven to be much more
>>>>> reliable.  Due to its unreliability, AGP support has been disabled on
>>>>> PowerPC for years already so there is no change on PowerPC."
>>>> There's a difference between something being disabled by default or not
>>>> being available at all. We may decide it's worth it anyway, but let's do
>>>> it based on facts.
>>>>
>>> I didn't mean to imply that AGP GART support was already removed.  But
>>> for the vast majority of users the end result is the same.  If you
>>> knew enough re-enable AGP GART, you probably wouldn't have been as
>>> confused about what this patch set does either.  To reiterate, this
>>> patch set does not remove support for AGP cards, it only removes the
>>> support for AGP GART.  The cards will still be functional using the
>>> device GART.  There may be performance tradeoffs there in some cases.
>> I'll volunteer to be the one asking: how big is this performance
>> difference? Have any benchmarks been run before and after removal of AGP
>> GART code on affected nouveau/radeon systems? Or is this code being
>> dropped _just_ because it's cumbersome, with no regard for metrics that
>> determine the value of AGP GART support?
>>
> I don't think anyone has any solid numbers, just anecdotal from
> memory.  I certainly don't have any functional AGP systems at this
> point.  It's mostly just cumbersome and would allow us to clean ttm
> and probably improve stability at the same time.  At least on the
> radeon side, the only native AGP cards were r1xx, r2xx, and some of
> the early r3xx boards.  Once we switched to pcie mid-way through r3xx,
> everything was native pcie and the AGP cards used a pcie to AGP bridge
> chip so they had a decent on chip MMU.  Those older cards topped out
> at maybe 32 or 64 MB of vram, so they are going to be hard pressed to
> deal with modern desktops anyway.  No idea what sort of GART
> capabilities NV AGP hardware at this time had.

I could only test with an old x86 Mac and an r3xx generation hw and in 
this case making the switch didn't had any noticeable effect at all.

But I didn't do more than playing around with the desktop effects and 
playing a video.

I do have a PC x86 AGP board lying around somewhere here, going top give 
that one a try a well.

Christian.

>
> Alex
>
>> Roy
>>
>>> Alex
>>> _______________________________________________
>>> Nouveau mailing list
>>> Nouveau@lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/nouveau
>>>

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2020-05-13  7:46 UTC|newest]

Thread overview: 81+ 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 ` Christian König
2020-05-11 17:17 ` Christian König
2020-05-11 17:17 ` [PATCH 1/3] drm/radeon: remove AGP support Christian König
2020-05-11 17:17   ` Christian König
2020-05-11 17:17   ` Christian König
2020-05-11 20:12   ` Alex Deucher
2020-05-11 20:12     ` Alex Deucher
2020-05-11 20:12     ` Alex Deucher
2020-05-12  8:16     ` Michel Dänzer
2020-05-12  8:16       ` Michel Dänzer
2020-05-12  8:16       ` Michel Dänzer
2020-05-12 13:36       ` Alex Deucher
2020-05-12 13:36         ` Alex Deucher
2020-05-12 13:36         ` Alex Deucher
2020-05-12 20:52         ` [Nouveau] " Roy Spliet
2020-05-12 20:52           ` Roy Spliet
2020-05-12 20:52           ` Roy Spliet
2020-05-12 21:12           ` [Nouveau] " Alex Deucher
2020-05-12 21:12             ` Alex Deucher
2020-05-12 21:12             ` Alex Deucher
2020-05-13  7:46             ` Christian König [this message]
2020-05-13  7:46               ` Christian König
2020-05-13  7:46               ` Christian König
2020-05-13  7:52               ` [Nouveau] " Michel Dänzer
2020-05-13  7:52                 ` Michel Dänzer
2020-05-13  7:52                 ` Michel Dänzer
2020-05-12 18:20   ` Thomas Zimmermann
2020-05-12 18:20     ` Thomas Zimmermann
2020-05-12 18:20     ` Thomas Zimmermann
2020-05-12 18:29     ` Alex Deucher
2020-05-12 18:29       ` Alex Deucher
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   ` Christian König
2020-05-11 17:17   ` Christian König
2020-05-11 17:17 ` [PATCH 3/3] drm/ttm: " Christian König
2020-05-11 17:17   ` Christian König
2020-05-11 17:17   ` Christian König
2020-05-11 20:14 ` [RFC] Remove AGP support from Radeon/Nouveau/TTM Al Dunsmuir
2020-05-11 20:14   ` Al Dunsmuir
2020-05-11 20:14   ` Al Dunsmuir
2020-05-11 20:27   ` Alex Deucher
2020-05-11 20:27     ` Alex Deucher
2020-05-11 20:27     ` Alex Deucher
2020-05-11 20:43     ` Dave Airlie
2020-05-11 20:43       ` Dave Airlie
2020-05-11 20:43       ` Dave Airlie
2020-05-11 20:56       ` Al Dunsmuir
2020-05-11 20:56         ` Al Dunsmuir
2020-05-11 20:56         ` Al Dunsmuir
2020-05-12  8:11         ` Christian König
2020-05-12  8:11           ` Christian König
2020-05-12  8:11           ` Christian König
2020-05-11 20:59       ` Emil Velikov
2020-05-11 20:59         ` Emil Velikov
2020-05-11 20:59         ` Emil Velikov
2020-05-11 20:46     ` Al Dunsmuir
2020-05-11 20:46       ` Al Dunsmuir
2020-05-11 20:46       ` Al Dunsmuir
2020-05-12 18:29 ` [Nouveau] " Thomas Zimmermann
2020-05-12 18:29   ` Thomas Zimmermann
2020-05-12 18:29   ` Thomas Zimmermann
2020-05-12 18:32   ` Alex Deucher
2020-05-12 18:32     ` Alex Deucher
2020-05-12 18:32     ` Alex Deucher
2020-05-12 19:10     ` Thomas Zimmermann
2020-05-12 19:10       ` Thomas Zimmermann
2020-05-12 19:10       ` Thomas Zimmermann
2020-05-12 19:47       ` Alex Deucher
2020-05-12 19:47         ` Alex Deucher
2020-05-12 19:47         ` Alex Deucher
2020-05-13  9:27         ` Emil Velikov
2020-05-13  9:27           ` Emil Velikov
2020-05-13  9:27           ` Emil Velikov
2020-05-13 11:03           ` [Nouveau] " Thomas Zimmermann
2020-05-13 11:03             ` Thomas Zimmermann
2020-05-13 11:03             ` Thomas Zimmermann
2020-05-13 11:00 ` Thomas Zimmermann
2020-05-13 11:00   ` Thomas Zimmermann
2020-05-13 11:00   ` Thomas Zimmermann

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=5c3da9d3-8b77-e97f-28a2-e1083a53a1d7@gmail.com \
    --to=ckoenig.leichtzumerken@gmail.com \
    --cc=alexdeucher@gmail.com \
    --cc=amd-gfx@lists.freedesktop.org \
    --cc=christian.koenig@amd.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=michel@daenzer.net \
    --cc=nouveau@lists.freedesktop.org \
    --cc=nouveau@spliet.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.