[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110829172146.GE2025@gere.osrc.amd.com>
Date: Mon, 29 Aug 2011 19:21:46 +0200
From: Borislav Petkov <bp@...en8.de>
To: Kyle Moffett <kyle@...fetthome.net>
CC: Alex Deucher <alexdeucher@...il.com>,
Borislav Petkov <bp@...en8.de>,
Peter Zijlstra <peterz@...radead.org>,
Pavel Ivanov <paivanof@...il.com>,
Michel Dänzer <michel@...nzer.net>,
<dri-devel@...ts.freedesktop.org>,
linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: Kernel almost hangs when CONFIG_DRM_RADEON=y
On Mon, Aug 29, 2011 at 12:28:31PM -0400, Kyle Moffett wrote:
> No, Linus pushed back really hard last time this issue came up with
> something; a network driver if I recall correctly.
r8169 probably.
> The issue is that this happens *EVEN FOR MODULAR DRIVERS* during
> suspend/resume. The firmware simply may not be available yet.
>
> If the driver fundamentally cannot work without the firmware then it should
> bind to the device and wait until the first userspace action before requesting
> firmware.
>
> Furthermore, the trend is generally to push the firmware OUT of the kernel
> binary to avoid any chance of license issues. Even if the code is built-in
> it should not need built-in firmware.
>
> The quickest fix is probably something like this:
>
> config DRM_RADEON_FIRMWARE
> tristate
> default m if STANDALONE
> default y
>
> config DRM_RADEON
> depends DRM_RADEON_FIRMWARE
>
> That should prevent somebody from building the radeon driver into the
> kernel unless they manually indicate that they have the extra firmware.
>
> Long-term, the driver should support modular firmware even when it's
> built-in to the kernel.
Yep, and drivers should be able to select the firmware they need without
users even needing to do anything about it except installing some
firmware-nonfree package or whatever.
Yeah, sounds much better than Kconfig actually aiding and abetting
firmware blobs in the kernel and users needing to do stuff.
Thanks.
--
Regards/Gruss,
Boris.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists