On Wed, 2018-05-09 at 13:28 -0700, Matthew Garrett wrote: > @@ -565,10 +588,11 @@ static int __init init_evm(void) > */ > static int __init evm_display_config(void) > { > - char **xattrname; > + struct xattr_list *xattr; > + > + list_for_each_entry(xattr, &evm_config_xattrnames, list) > + pr_info("%s\n", xattr->name); > > - for (xattrname = evm_config_xattrnames; *xattrname != NULL; xattrname++) > - pr_info("%s\n", *xattrname); > return 0; > } > The list is only created in init_evm(), which is initialized at late_initcall(), yet here we're displaying the list at pure_initcall(). None of the xattrs are being displayed. Mimi