(+cc: devel@driverdev) Hi Ben, Ben Hutchings wrote:
> ZSMALLOC is tristate, but the code has no MODULE_LICENSE and since it > depends on GPL-only symbols it cannot be loaded as a module. This in > turn breaks zram which now depends on it. I assume it's meant to be > Dual BSD/GPL like the other z-stuff. Yep, the opening comment in the file says so. > There is also no module_exit, which will make it impossible to unload. > Add the appropriate module_init and module_exit declarations suggested > by comments. > > Reported-by: Christian Ohm <[email protected]> I sent a similar patch + fixup last week[*]. Yours has a clearer description. The only other difference aside from reordering lines is that I marked zs_init as __init and zs_exit as __exit. [...] > This is compile-tested only. Someone using zram tested last week's patch (which is almost identical) against 3.4.y and let me know it worked by private email, but I don't have permission to name that person. So for what it's worth, Reviewed-by: Jonathan Nieder <[email protected]> Thanks. Jonathan [*] http://driverdev.linuxdriverproject.org/pipermail/devel/2012-June/027377.html -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
