Bug 100011

The function load_i915_kernel_module() in file src/intel_device.c
of xf86-video-intel reverses the meaning of the return value of
xorg server's xf86LoadKernelModule().  xf86LoadKernelModule()
actually returns 0 if the loading fails, non-zero if success.

For example, from xorg server 1.19.1's source code,
file hw/xfree86/os-support/linux/lnx_kmod.c:

 * Input:
 *    modName - name of the kernel module (Ex: "tdfx")
 * Return:
 *    0 for failure, 1 for success
 */
int
xf86LoadKernelModule(const char *modName)

        if (WIFEXITED(status) && WEXITSTATUS(status) == 0) {
            return 1;           /* success! */
        }
        else {
            return 0;
        }

And from file hw/xfree86/os-support/bsd/bsd_kmod.c

 * Return:
 *    0 for failure, 1 for success
 */
int
xf86LoadKernelModule(const char *modName)
{
    if (kldload(modName) != -1)
        return 1;
    else
        return 0;
}

On FreeBSD there is an additional complication that if
"i915kms" is added to kernel_module_names, the inverted use
of the return value causes both i915kms.ko and i915.ko to
be loaded.

Therefore I believe a patch similar to the following needs to be made
to src/intel_device.c:

@@ -233,9 +242,8 @@ static int load_i915_kernel_module(void)
 	const char **kn;
 
 	for (kn = kernel_module_names; *kn; kn++)
-		if (xf86LoadKernelModule(*kn) == 0)
+		if (xf86LoadKernelModule(*kn))
 			return 0;
-
 	return -1;
 }