On Wednesday, November 12, 2014 04:25:15 PM Richard Guy Briggs wrote: > On 14/11/12, Paul Moore wrote: > > My apologies, I should have noticed this sooner, but printk_once() is > > probably not a good choice here as only the first invalid netlink message > > will be displayed. This is fine if all the invalid netlink messages > > happen the same, but that isn't likely to be the case. > > This was the same situation with WARN_ONCE(), hence my comment about > difficulty in debugging... Unfortunately I didn't realize your point until now. > > Richard, any objections if I convert the printk_once() to a printk(WARN) > > and update the patch description accordingly? > > Use pr_warn() instead... Normally, yes, but the rest of the SELinux code uses printk(WARN) and I'm a stickler for consistency. -- paul moore security and virtualization @ redhat -- To unsubscribe from this list: send the line "unsubscribe trinity" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html