* [radeon agp] add blacklist for thinkpad T40p
@ 2015-11-28 21:01 ` Pavel Machek
0 siblings, 0 replies; 10+ messages in thread
From: Pavel Machek @ 2015-11-28 21:01 UTC (permalink / raw)
To: Trivial patch monkey, alexander.deucher, christian.koenig,
airlied, dri-devel, linux-kernel
Thinkpad T40p needs agpmode 1.
Signed-off-by: Pavel Machek <pavel@ucw.cz>
diff --git a/drivers/gpu/drm/radeon/radeon_agp.c b/drivers/gpu/drm/radeon/radeon_agp.c
index fe994aa..c77d349 100644
--- a/drivers/gpu/drm/radeon/radeon_agp.c
+++ b/drivers/gpu/drm/radeon/radeon_agp.c
@@ -54,6 +54,9 @@ static struct radeon_agpmode_quirk radeon_agpmode_quirk_list[] = {
/* Intel 82855PM host bridge / Mobility 9600 M10 RV350 Needs AGPMode 1 (lp #195051) */
{ PCI_VENDOR_ID_INTEL, 0x3340, PCI_VENDOR_ID_ATI, 0x4e50,
PCI_VENDOR_ID_IBM, 0x0550, 1},
+ /* Intel 82855PM host bridge / RV250/M9 GL [Mobility FireGL 9000/Radeon 9000] needs AGPMode 1 (Thinkpad T40p) */
+ { PCI_VENDOR_ID_INTEL, 0x3340, PCI_VENDOR_ID_ATI, 0x4c66,
+ PCI_VENDOR_ID_IBM, 0x054d, 1},
/* Intel 82855PM host bridge / Mobility M7 needs AGPMode 1 */
{ PCI_VENDOR_ID_INTEL, 0x3340, PCI_VENDOR_ID_ATI, 0x4c57,
PCI_VENDOR_ID_IBM, 0x0530, 1},
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 10+ messages in thread* [radeon agp] add blacklist for thinkpad T40p
@ 2015-11-28 21:01 ` Pavel Machek
0 siblings, 0 replies; 10+ messages in thread
From: Pavel Machek @ 2015-11-28 21:01 UTC (permalink / raw)
To: Trivial patch monkey, alexander.deucher, christian.koenig,
airlied, dri-devel, linux-kernel
Thinkpad T40p needs agpmode 1.
Signed-off-by: Pavel Machek <pavel@ucw.cz>
diff --git a/drivers/gpu/drm/radeon/radeon_agp.c b/drivers/gpu/drm/radeon/radeon_agp.c
index fe994aa..c77d349 100644
--- a/drivers/gpu/drm/radeon/radeon_agp.c
+++ b/drivers/gpu/drm/radeon/radeon_agp.c
@@ -54,6 +54,9 @@ static struct radeon_agpmode_quirk radeon_agpmode_quirk_list[] = {
/* Intel 82855PM host bridge / Mobility 9600 M10 RV350 Needs AGPMode 1 (lp #195051) */
{ PCI_VENDOR_ID_INTEL, 0x3340, PCI_VENDOR_ID_ATI, 0x4e50,
PCI_VENDOR_ID_IBM, 0x0550, 1},
+ /* Intel 82855PM host bridge / RV250/M9 GL [Mobility FireGL 9000/Radeon 9000] needs AGPMode 1 (Thinkpad T40p) */
+ { PCI_VENDOR_ID_INTEL, 0x3340, PCI_VENDOR_ID_ATI, 0x4c66,
+ PCI_VENDOR_ID_IBM, 0x054d, 1},
/* Intel 82855PM host bridge / Mobility M7 needs AGPMode 1 */
{ PCI_VENDOR_ID_INTEL, 0x3340, PCI_VENDOR_ID_ATI, 0x4c57,
PCI_VENDOR_ID_IBM, 0x0530, 1},
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [radeon agp] add blacklist for thinkpad T40p
2015-11-28 21:01 ` Pavel Machek
@ 2015-11-30 19:47 ` Alex Deucher
-1 siblings, 0 replies; 10+ messages in thread
From: Alex Deucher @ 2015-11-30 19:47 UTC (permalink / raw)
To: Pavel Machek
Cc: Trivial patch monkey, LKML, Maling list - DRI developers,
Deucher, Alexander, Christian Koenig
On Sat, Nov 28, 2015 at 4:01 PM, Pavel Machek <pavel@ucw.cz> wrote:
>
> Thinkpad T40p needs agpmode 1.
>
> Signed-off-by: Pavel Machek <pavel@ucw.cz>
Seems odd that this wouldn't have been found earlier given how popular
thinkpads are. Applied. Thanks,
Alex
>
> diff --git a/drivers/gpu/drm/radeon/radeon_agp.c b/drivers/gpu/drm/radeon/radeon_agp.c
> index fe994aa..c77d349 100644
> --- a/drivers/gpu/drm/radeon/radeon_agp.c
> +++ b/drivers/gpu/drm/radeon/radeon_agp.c
> @@ -54,6 +54,9 @@ static struct radeon_agpmode_quirk radeon_agpmode_quirk_list[] = {
> /* Intel 82855PM host bridge / Mobility 9600 M10 RV350 Needs AGPMode 1 (lp #195051) */
> { PCI_VENDOR_ID_INTEL, 0x3340, PCI_VENDOR_ID_ATI, 0x4e50,
> PCI_VENDOR_ID_IBM, 0x0550, 1},
> + /* Intel 82855PM host bridge / RV250/M9 GL [Mobility FireGL 9000/Radeon 9000] needs AGPMode 1 (Thinkpad T40p) */
> + { PCI_VENDOR_ID_INTEL, 0x3340, PCI_VENDOR_ID_ATI, 0x4c66,
> + PCI_VENDOR_ID_IBM, 0x054d, 1},
> /* Intel 82855PM host bridge / Mobility M7 needs AGPMode 1 */
> { PCI_VENDOR_ID_INTEL, 0x3340, PCI_VENDOR_ID_ATI, 0x4c57,
> PCI_VENDOR_ID_IBM, 0x0530, 1},
>
> --
> (english) http://www.livejournal.com/~pavelmachek
> (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [radeon agp] add blacklist for thinkpad T40p
@ 2015-11-30 19:47 ` Alex Deucher
0 siblings, 0 replies; 10+ messages in thread
From: Alex Deucher @ 2015-11-30 19:47 UTC (permalink / raw)
To: Pavel Machek
Cc: Trivial patch monkey, Deucher, Alexander, Christian Koenig,
Dave Airlie, Maling list - DRI developers, LKML
On Sat, Nov 28, 2015 at 4:01 PM, Pavel Machek <pavel@ucw.cz> wrote:
>
> Thinkpad T40p needs agpmode 1.
>
> Signed-off-by: Pavel Machek <pavel@ucw.cz>
Seems odd that this wouldn't have been found earlier given how popular
thinkpads are. Applied. Thanks,
Alex
>
> diff --git a/drivers/gpu/drm/radeon/radeon_agp.c b/drivers/gpu/drm/radeon/radeon_agp.c
> index fe994aa..c77d349 100644
> --- a/drivers/gpu/drm/radeon/radeon_agp.c
> +++ b/drivers/gpu/drm/radeon/radeon_agp.c
> @@ -54,6 +54,9 @@ static struct radeon_agpmode_quirk radeon_agpmode_quirk_list[] = {
> /* Intel 82855PM host bridge / Mobility 9600 M10 RV350 Needs AGPMode 1 (lp #195051) */
> { PCI_VENDOR_ID_INTEL, 0x3340, PCI_VENDOR_ID_ATI, 0x4e50,
> PCI_VENDOR_ID_IBM, 0x0550, 1},
> + /* Intel 82855PM host bridge / RV250/M9 GL [Mobility FireGL 9000/Radeon 9000] needs AGPMode 1 (Thinkpad T40p) */
> + { PCI_VENDOR_ID_INTEL, 0x3340, PCI_VENDOR_ID_ATI, 0x4c66,
> + PCI_VENDOR_ID_IBM, 0x054d, 1},
> /* Intel 82855PM host bridge / Mobility M7 needs AGPMode 1 */
> { PCI_VENDOR_ID_INTEL, 0x3340, PCI_VENDOR_ID_ATI, 0x4c57,
> PCI_VENDOR_ID_IBM, 0x0530, 1},
>
> --
> (english) http://www.livejournal.com/~pavelmachek
> (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [radeon agp] add blacklist for thinkpad T40p
2015-11-30 19:47 ` Alex Deucher
@ 2015-12-02 11:15 ` Emil Velikov
-1 siblings, 0 replies; 10+ messages in thread
From: Emil Velikov @ 2015-12-02 11:15 UTC (permalink / raw)
To: Alex Deucher
Cc: Trivial patch monkey, LKML, Maling list - DRI developers,
Pavel Machek, Deucher, Alexander, Christian Koenig
On 30 November 2015 at 19:47, Alex Deucher <alexdeucher@gmail.com> wrote:
> On Sat, Nov 28, 2015 at 4:01 PM, Pavel Machek <pavel@ucw.cz> wrote:
>>
>> Thinkpad T40p needs agpmode 1.
>>
>> Signed-off-by: Pavel Machek <pavel@ucw.cz>
>
> Seems odd that this wouldn't have been found earlier given how popular
> thinkpads are. Applied. Thanks,
>
Wondering if it wouldn't be better to apply these restrictions within
the AGP driver. I have a vague recollection that (at least?) some of
these are chipset limitations/bugs ?
Thanks
Emil
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [radeon agp] add blacklist for thinkpad T40p
@ 2015-12-02 11:15 ` Emil Velikov
0 siblings, 0 replies; 10+ messages in thread
From: Emil Velikov @ 2015-12-02 11:15 UTC (permalink / raw)
To: Alex Deucher
Cc: Pavel Machek, Trivial patch monkey, LKML,
Maling list - DRI developers, Deucher, Alexander,
Christian Koenig
On 30 November 2015 at 19:47, Alex Deucher <alexdeucher@gmail.com> wrote:
> On Sat, Nov 28, 2015 at 4:01 PM, Pavel Machek <pavel@ucw.cz> wrote:
>>
>> Thinkpad T40p needs agpmode 1.
>>
>> Signed-off-by: Pavel Machek <pavel@ucw.cz>
>
> Seems odd that this wouldn't have been found earlier given how popular
> thinkpads are. Applied. Thanks,
>
Wondering if it wouldn't be better to apply these restrictions within
the AGP driver. I have a vague recollection that (at least?) some of
these are chipset limitations/bugs ?
Thanks
Emil
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [radeon agp] add blacklist for thinkpad T40p
2015-12-02 11:15 ` Emil Velikov
@ 2015-12-02 16:33 ` Alex Deucher
-1 siblings, 0 replies; 10+ messages in thread
From: Alex Deucher @ 2015-12-02 16:33 UTC (permalink / raw)
To: Emil Velikov
Cc: Trivial patch monkey, LKML, Maling list - DRI developers,
Pavel Machek, Deucher, Alexander, Christian Koenig
On Wed, Dec 2, 2015 at 6:15 AM, Emil Velikov <emil.l.velikov@gmail.com> wrote:
> On 30 November 2015 at 19:47, Alex Deucher <alexdeucher@gmail.com> wrote:
>> On Sat, Nov 28, 2015 at 4:01 PM, Pavel Machek <pavel@ucw.cz> wrote:
>>>
>>> Thinkpad T40p needs agpmode 1.
>>>
>>> Signed-off-by: Pavel Machek <pavel@ucw.cz>
>>
>> Seems odd that this wouldn't have been found earlier given how popular
>> thinkpads are. Applied. Thanks,
>>
> Wondering if it wouldn't be better to apply these restrictions within
> the AGP driver. I have a vague recollection that (at least?) some of
> these are chipset limitations/bugs ?
I think it's probably better in the GPU drivers. AGP was always
problematic. There are a number of quirks that were necessary for
some users but not others on the same hardware. Additionally, the
combination of vendors or even driver versions/behaviors were
problematic. E.g., with UMS, the drivers didn't use much gart since
there was just a single static allocation. Once we switched to KMS,
lots of new bugs surfaced. Chipset A plus GPU vendor B worked fined,
but chipset A with GPU vendor C was problematic.
Alex
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [radeon agp] add blacklist for thinkpad T40p
@ 2015-12-02 16:33 ` Alex Deucher
0 siblings, 0 replies; 10+ messages in thread
From: Alex Deucher @ 2015-12-02 16:33 UTC (permalink / raw)
To: Emil Velikov
Cc: Pavel Machek, Trivial patch monkey, LKML,
Maling list - DRI developers, Deucher, Alexander,
Christian Koenig
On Wed, Dec 2, 2015 at 6:15 AM, Emil Velikov <emil.l.velikov@gmail.com> wrote:
> On 30 November 2015 at 19:47, Alex Deucher <alexdeucher@gmail.com> wrote:
>> On Sat, Nov 28, 2015 at 4:01 PM, Pavel Machek <pavel@ucw.cz> wrote:
>>>
>>> Thinkpad T40p needs agpmode 1.
>>>
>>> Signed-off-by: Pavel Machek <pavel@ucw.cz>
>>
>> Seems odd that this wouldn't have been found earlier given how popular
>> thinkpads are. Applied. Thanks,
>>
> Wondering if it wouldn't be better to apply these restrictions within
> the AGP driver. I have a vague recollection that (at least?) some of
> these are chipset limitations/bugs ?
I think it's probably better in the GPU drivers. AGP was always
problematic. There are a number of quirks that were necessary for
some users but not others on the same hardware. Additionally, the
combination of vendors or even driver versions/behaviors were
problematic. E.g., with UMS, the drivers didn't use much gart since
there was just a single static allocation. Once we switched to KMS,
lots of new bugs surfaced. Chipset A plus GPU vendor B worked fined,
but chipset A with GPU vendor C was problematic.
Alex
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [radeon agp] add blacklist for thinkpad T40p
2015-12-02 16:33 ` Alex Deucher
@ 2015-12-02 17:04 ` Emil Velikov
-1 siblings, 0 replies; 10+ messages in thread
From: Emil Velikov @ 2015-12-02 17:04 UTC (permalink / raw)
To: Alex Deucher
Cc: Trivial patch monkey, LKML, Maling list - DRI developers,
Pavel Machek, Deucher, Alexander, Christian Koenig
On 2 December 2015 at 16:33, Alex Deucher <alexdeucher@gmail.com> wrote:
> On Wed, Dec 2, 2015 at 6:15 AM, Emil Velikov <emil.l.velikov@gmail.com> wrote:
>> On 30 November 2015 at 19:47, Alex Deucher <alexdeucher@gmail.com> wrote:
>>> On Sat, Nov 28, 2015 at 4:01 PM, Pavel Machek <pavel@ucw.cz> wrote:
>>>>
>>>> Thinkpad T40p needs agpmode 1.
>>>>
>>>> Signed-off-by: Pavel Machek <pavel@ucw.cz>
>>>
>>> Seems odd that this wouldn't have been found earlier given how popular
>>> thinkpads are. Applied. Thanks,
>>>
>> Wondering if it wouldn't be better to apply these restrictions within
>> the AGP driver. I have a vague recollection that (at least?) some of
>> these are chipset limitations/bugs ?
>
> I think it's probably better in the GPU drivers. AGP was always
> problematic. There are a number of quirks that were necessary for
> some users but not others on the same hardware. Additionally, the
> combination of vendors or even driver versions/behaviors were
> problematic. E.g., with UMS, the drivers didn't use much gart since
> there was just a single static allocation. Once we switched to KMS,
> lots of new bugs surfaced. Chipset A plus GPU vendor B worked fined,
> but chipset A with GPU vendor C was problematic.
>
Looking at how many cases with identical vendor/device (yet differing
sub vendor/device) are there, plus some of these also available in
nouveau I'm wondering if it's truly a case of things "working" for
some as opposed to different test(ing procedure) being applied. That
assumption comes along nicely with how 'delayed' this report is.
That said I'm not pushing on anything, just pointing out the pattern I see.
Emil
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [radeon agp] add blacklist for thinkpad T40p
@ 2015-12-02 17:04 ` Emil Velikov
0 siblings, 0 replies; 10+ messages in thread
From: Emil Velikov @ 2015-12-02 17:04 UTC (permalink / raw)
To: Alex Deucher
Cc: Pavel Machek, Trivial patch monkey, LKML,
Maling list - DRI developers, Deucher, Alexander,
Christian Koenig
On 2 December 2015 at 16:33, Alex Deucher <alexdeucher@gmail.com> wrote:
> On Wed, Dec 2, 2015 at 6:15 AM, Emil Velikov <emil.l.velikov@gmail.com> wrote:
>> On 30 November 2015 at 19:47, Alex Deucher <alexdeucher@gmail.com> wrote:
>>> On Sat, Nov 28, 2015 at 4:01 PM, Pavel Machek <pavel@ucw.cz> wrote:
>>>>
>>>> Thinkpad T40p needs agpmode 1.
>>>>
>>>> Signed-off-by: Pavel Machek <pavel@ucw.cz>
>>>
>>> Seems odd that this wouldn't have been found earlier given how popular
>>> thinkpads are. Applied. Thanks,
>>>
>> Wondering if it wouldn't be better to apply these restrictions within
>> the AGP driver. I have a vague recollection that (at least?) some of
>> these are chipset limitations/bugs ?
>
> I think it's probably better in the GPU drivers. AGP was always
> problematic. There are a number of quirks that were necessary for
> some users but not others on the same hardware. Additionally, the
> combination of vendors or even driver versions/behaviors were
> problematic. E.g., with UMS, the drivers didn't use much gart since
> there was just a single static allocation. Once we switched to KMS,
> lots of new bugs surfaced. Chipset A plus GPU vendor B worked fined,
> but chipset A with GPU vendor C was problematic.
>
Looking at how many cases with identical vendor/device (yet differing
sub vendor/device) are there, plus some of these also available in
nouveau I'm wondering if it's truly a case of things "working" for
some as opposed to different test(ing procedure) being applied. That
assumption comes along nicely with how 'delayed' this report is.
That said I'm not pushing on anything, just pointing out the pattern I see.
Emil
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2015-12-02 17:04 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-11-28 21:01 [radeon agp] add blacklist for thinkpad T40p Pavel Machek
2015-11-28 21:01 ` Pavel Machek
2015-11-30 19:47 ` Alex Deucher
2015-11-30 19:47 ` Alex Deucher
2015-12-02 11:15 ` Emil Velikov
2015-12-02 11:15 ` Emil Velikov
2015-12-02 16:33 ` Alex Deucher
2015-12-02 16:33 ` Alex Deucher
2015-12-02 17:04 ` Emil Velikov
2015-12-02 17:04 ` Emil Velikov
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.