In article <20151130224719.7d1b...@cvs.netbsd.org>, Paul Goyette <source-changes-d@NetBSD.org> wrote: >-=-=-=-=-=-
>+ const struct sc_auto *auto_list; >+ int code; That should be u_int. >- if (module_autoload(syscalls_autoload[i].al_module, >- MODULE_CLASS_ANY) != 0 || >+ if (module_autoload(auto_list->al_module, >+ MODULE_CLASS_ANY) != 0 || KNF. >+# autoloadprefix the prefix for the autoload table name Why isn't that the emulation name? >+ * Autoloadable syscall definition >+ */ >+struct sc_auto { >+ u_int al_code; >+ const char *al_module; >+}; A better name would be sc_autoload. christos