<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:daniel@ffwll.ch" title="Daniel Vetter <daniel@ffwll.ch>"> <span class="fn">Daniel Vetter</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEEDINFO "
title="NEEDINFO --- - [GM45]igt/module_reload cause <3>[ 159.350832] [drm:drm_mm_takedown] *ERROR* Memory manager not clean. Delaying takedown"
href="https://bugs.freedesktop.org/show_bug.cgi?id=65953">bug 65953</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">Status</td>
<td>NEW
</td>
<td>NEEDINFO
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEEDINFO "
title="NEEDINFO --- - [GM45]igt/module_reload cause <3>[ 159.350832] [drm:drm_mm_takedown] *ERROR* Memory manager not clean. Delaying takedown"
href="https://bugs.freedesktop.org/show_bug.cgi?id=65953#c8">Comment # 8</a>
on <a class="bz_bug_link
bz_status_NEEDINFO "
title="NEEDINFO --- - [GM45]igt/module_reload cause <3>[ 159.350832] [drm:drm_mm_takedown] *ERROR* Memory manager not clean. Delaying takedown"
href="https://bugs.freedesktop.org/show_bug.cgi?id=65953">bug 65953</a>
from <span class="vcard"><a class="email" href="mailto:daniel@ffwll.ch" title="Daniel Vetter <daniel@ffwll.ch>"> <span class="fn">Daniel Vetter</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=65953#c7">comment #7</a>)
<span class="quote">> (In reply to <a href="show_bug.cgi?id=65953#c6">comment #6</a>)
> > Created <span class=""><a href="attachment.cgi?id=81590" name="attach_81590" title="don't tear down un-initiaizled stolen drm_mm">attachment 81590</a> <a href="attachment.cgi?id=81590&action=edit" title="don't tear down un-initiaizled stolen drm_mm">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=65953&attachment=81590'>[review]</a> [review] [review]
> > don't tear down un-initiaizled stolen drm_mm
> >
> > Please test the attached patch, thanks.
>
> It still exists.</span >
Can you please double-check the patch with the following debug diff applied on
top (and then attach dmesg afterwards)?
diff --git a/drivers/gpu/drm/drm_mm.c b/drivers/gpu/drm/drm_mm.c
index f9d4873..d303e31 100644
--- a/drivers/gpu/drm/drm_mm.c
+++ b/drivers/gpu/drm/drm_mm.c
@@ -699,8 +699,8 @@ void drm_mm_takedown(struct drm_mm * mm)
{
struct drm_mm_node *entry, *next;
- if (!list_empty(&mm->head_node.node_list)) {
- DRM_ERROR("Memory manager not clean. Delaying takedown\n");
+ if (WARN(!list_empty(&mm->head_node.node_list),
+ "Memory manager not clean. Delaying takedown\n")) {
return;
}</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
</ul>
</body>
</html>