<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>Priority</th>
<td>medium
</td>
</tr>
<tr>
<th>Bug ID</th>
<td><a class="bz_bug_link
bz_status_NEW "
title="NEW --- - first argument to drmOpen() is a no-op"
href="https://bugs.freedesktop.org/show_bug.cgi?id=72849">72849</a>
</td>
</tr>
<tr>
<th>Assignee</th>
<td>xorg-driver-ati@lists.x.org
</td>
</tr>
<tr>
<th>Summary</th>
<td>first argument to drmOpen() is a no-op
</td>
</tr>
<tr>
<th>QA Contact</th>
<td>xorg-team@lists.x.org
</td>
</tr>
<tr>
<th>Severity</th>
<td>normal
</td>
</tr>
<tr>
<th>Classification</th>
<td>Unclassified
</td>
</tr>
<tr>
<th>OS</th>
<td>FreeBSD
</td>
</tr>
<tr>
<th>Reporter</th>
<td>rmh@freebsd.org
</td>
</tr>
<tr>
<th>Hardware</th>
<td>Other
</td>
</tr>
<tr>
<th>Status</th>
<td>NEW
</td>
</tr>
<tr>
<th>Version</th>
<td>unspecified
</td>
</tr>
<tr>
<th>Component</th>
<td>Driver/Radeon
</td>
</tr>
<tr>
<th>Product</th>
<td>xorg
</td>
</tr></table>
<p>
<div>
<pre>Created <span class=""><a href="attachment.cgi?id=90951" name="attach_90951" title="patch">attachment 90951</a> <a href="attachment.cgi?id=90951&action=edit" title="patch">[details]</a></span>
patch
Since the introduction of a call to drmCheckModesettingSupported() in
radeon_kernel_mode_enabled(), with abort condition if such call fails, the
first argument to drmOpen() call in radeon_open_drm_master() has become a no-op
red herring.
Such argument (a kernel module name) is supposed to result in load of specified
kernel module. However, this will never happen. The problem is that if the code
containing drmOpen() call is reached, it means that
drmCheckModesettingSupported() check has previously succeeded, which implies
the module is already loaded.
So, drmOpen() will never load a kernel module. But it gives the impression that
it will.
In order to avoid this confusion, I suggest replacing it with NULL, like
xf86-video-intel driver does:
src/intel_device.c: fd = drmOpen(NULL, id);
and then taking care of module load elsewhere (I'll submit a separate bug for
that).</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>