Path: csiph.com!aioe.org!bofh.it!news.nic.it!robomod
From: Kathryn Tolsen <katybethtolsen@gmail.com>
Newsgroups: linux.debian.bugs.dist,linux.debian.kernel
Subject: Bug#931901: Ping to Ben Hutchings, identifying possible source of problem
Date: Mon, 15 Jul 2019 15:00:01 +0200
Message-ID: <yk8Kd-2XP-1@gated-at.bofh.it>
References: <yiY0x-fz-1@gated-at.bofh.it>
Old-Return-Path: <debbugs@buxtehude.debian.org>
Reply-To: Kathryn Tolsen <katybethtolsen@gmail.com>, 931901@bugs.debian.org
Resent-To: debian-bugs-dist@lists.debian.org
Resent-Cc: Debian Kernel Team <debian-kernel@lists.debian.org>
MIME-Version: 1.0
Content-Type: multipart/alternative; boundary="0000000000003c0ab9058db7af5e"
Approved: robomod@news.nic.it
Lines: 41
Organization: linux.* mail to news gateway
Sender: robomod@news.nic.it
Xref: csiph.com linux.debian.bugs.dist:964672 linux.debian.kernel:64437

--0000000000003c0ab9058db7af5e
Content-Type: text/plain; charset="UTF-8"

It would appear to my untrained eye, that this commit on May 15, 2018 by
Ben Hutchings is responsible for this issue:

https://salsa.debian.org/kernel-team/linux/commit/e7bdd24f189b1b8775665d56a1230f7b12915ffe

This appears to be debian specific and not an upstream issue as the
upstream code doesn't contain any such checks:

https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/drivers/gpu/drm/radeon/radeon_drv.c?h=v4.19.59#n340

It would appear this is an incorrect assumption to make, that all cards
newer than the R600 require firmware. I can't make out what this code is
doing without digging way deeper but I think this is what is causing this
issue. There would need to be an exception here or the patch would need to
be rolled back or came at from a different angle entirely.

--0000000000003c0ab9058db7af5e
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr">It would appear to my untrained eye, that this commit on M=
ay 15, 2018 by Ben Hutchings is responsible for this issue:=C2=A0<div><br><=
div><a href=3D"https://salsa.debian.org/kernel-team/linux/commit/e7bdd24f18=
9b1b8775665d56a1230f7b12915ffe">https://salsa.debian.org/kernel-team/linux/=
commit/e7bdd24f189b1b8775665d56a1230f7b12915ffe</a><br></div><div><br></div=
><div>This appears to be debian specific and not an upstream issue as the u=
pstream code doesn&#39;t contain any such checks:</div></div><div><br></div=
><div><a href=3D"https://git.kernel.org/pub/scm/linux/kernel/git/stable/lin=
ux.git/tree/drivers/gpu/drm/radeon/radeon_drv.c?h=3Dv4.19.59#n340">https://=
git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/drivers/gpu/d=
rm/radeon/radeon_drv.c?h=3Dv4.19.59#n340</a><br></div><div><br></div><div>I=
t would appear this is an incorrect assumption to make, that all cards newe=
r than the R600 require firmware. I can&#39;t make out what this code is do=
ing without digging way deeper but I think this is what is causing this iss=
ue. There would need to be an exception here or the patch would need to be =
rolled back or came at from a different angle entirely.</div></div>

--0000000000003c0ab9058db7af5e--