On Tue, Jun 12, 2018 at 7:50 AM, Zhouyang Jia <jiazhouyan...@gmail.com> wrote: > When try_module_get fails, the lack of error-handling code may > cause unexpected results.
What results? What is recovery approach here? How printing message change a behaviour? > This patch adds error-handling code after calling try_module_get. > + if (!kgdb_connected) { > + if (!try_module_get(THIS_MODULE)) > + printk(KERN_ERR "kgdboc: cannot get module.\n"); So, what's wrong with pr_err() ? > + } -- With Best Regards, Andy Shevchenko ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ Kgdb-bugreport mailing list Kgdb-bugreport@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/kgdb-bugreport