[Bug 98420] Memory leak in drm_atomic.c eventually (few days) consuming all RAM (on at least one system configuration)

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=98420

--- Comment #5 from Chris Wilson <chris at chris-wilson.co.uk> ---
Yeah, whether or not an unref() function is NULL safe depends on whereabouts in
the kernel you are. In kernel/ it is considered bad practice to err on the side
of safety (as it encourages sloppy use, if the caller can't decide on whether
NULL is an error or not, why should the core code be nice). On the other hand,
in drivers/gpu/drm we tend to favour NULL-safe teardown functions as then we
avoid proliferation of if (!NULL) tests. You win some, you lose some.

Reposted to dri-devel@ giving you the kudos for finding and fixing the bug.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/intel-gfx-bugs/attachments/20161025/0f1cdf84/attachment.html>