[PATCH] drm/i915/bios: Avoid temporary allocation whilst searching for downclock

Keith Packard keithp at keithp.com
Wed Jul 13 11:13:24 PDT 2011


On Tue,  5 Jul 2011 09:55:34 +0100, Chris Wilson <chris at chris-wilson.co.uk> wrote:

> Alan Cox reported a missing check on the kmalloc return value for the
> allocation of a temporary mode used for searching for the LVDS downlock
> frequency. This allocation is roughly 200 bytes, a little too large to
> friviously place on the stack. However, we can simply use the few bytes
> we need stored within the original DVO timing data, skip the translation
> and do the compare directly between the timing data rather than on a
> mode, thus avoiding the need for any temporary allocations.

Can you clean this one up for -next? I'm merging bug fixes to -next to
send off for post-3.0 integration.

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20110713/2d5de019/attachment.pgp>


More information about the dri-devel mailing list