(+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 <chr....@gmx.net> 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 <jrnie...@gmail.com> Thanks. Jonathan [*] http://driverdev.linuxdriverproject.org/pipermail/devel/2012-June/027377.html -- To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/20120620014605.GB2961@burratino
