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

Reply via email to