Commit b4bc8428 authored by Dmitry Torokhov's avatar Dmitry Torokhov Committed by Rusty Russell

module: deal with alignment issues in built-in module versions

On m68k natural alignment is 2-byte boundary but we are trying to
align structures in __modver section on sizeof(void *) boundary.
This causes trouble when we try to access elements in this section
in array-like fashion when create "version" attributes for built-in
modules.

Moreover, as DaveM said, we can't reliably put structures into
independent objects, put them into a special section, and then expect
array access over them (via the section boundaries) after linking the
objects together to just "work" due to variable alignment choices in
different situations. The only solution that seems to work reliably
is to make an array of plain pointers to the objects in question and
put those pointers in the special section.
Reported-by: default avatarGeert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: default avatarDmitry Torokhov <dtor@vmware.com>
Signed-off-by: default avatarRusty Russell <rusty@rustcorp.com.au>
parent 61c4f2c8
...@@ -174,10 +174,7 @@ extern struct module __this_module; ...@@ -174,10 +174,7 @@ extern struct module __this_module;
#define MODULE_VERSION(_version) \ #define MODULE_VERSION(_version) \
extern ssize_t __modver_version_show(struct module_attribute *, \ extern ssize_t __modver_version_show(struct module_attribute *, \
struct module *, char *); \ struct module *, char *); \
static struct module_version_attribute __modver_version_attr \ static struct module_version_attribute ___modver_attr = { \
__used \
__attribute__ ((__section__ ("__modver"),aligned(sizeof(void *)))) \
= { \
.mattr = { \ .mattr = { \
.attr = { \ .attr = { \
.name = "version", \ .name = "version", \
...@@ -187,7 +184,10 @@ extern struct module __this_module; ...@@ -187,7 +184,10 @@ extern struct module __this_module;
}, \ }, \
.module_name = KBUILD_MODNAME, \ .module_name = KBUILD_MODNAME, \
.version = _version, \ .version = _version, \
} }; \
static const struct module_version_attribute \
__used __attribute__ ((__section__ ("__modver"))) \
* __moduleparam_const __modver_attr = &___modver_attr
#endif #endif
/* Optional firmware file (or files) needed by the module /* Optional firmware file (or files) needed by the module
......
...@@ -821,15 +821,18 @@ ssize_t __modver_version_show(struct module_attribute *mattr, ...@@ -821,15 +821,18 @@ ssize_t __modver_version_show(struct module_attribute *mattr,
return sprintf(buf, "%s\n", vattr->version); return sprintf(buf, "%s\n", vattr->version);
} }
extern struct module_version_attribute __start___modver[], __stop___modver[]; extern const struct module_version_attribute *__start___modver[];
extern const struct module_version_attribute *__stop___modver[];
static void __init version_sysfs_builtin(void) static void __init version_sysfs_builtin(void)
{ {
const struct module_version_attribute *vattr; const struct module_version_attribute **p;
struct module_kobject *mk; struct module_kobject *mk;
int err; int err;
for (vattr = __start___modver; vattr < __stop___modver; vattr++) { for (p = __start___modver; p < __stop___modver; p++) {
const struct module_version_attribute *vattr = *p;
mk = locate_module_kobject(vattr->module_name); mk = locate_module_kobject(vattr->module_name);
if (mk) { if (mk) {
err = sysfs_create_file(&mk->kobj, &vattr->mattr.attr); err = sysfs_create_file(&mk->kobj, &vattr->mattr.attr);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment