2017-12-08 0:40 GMT+08:00 Al Viro [off-list ref]:

On Mon, Nov 27, 2017 at 08:27:53PM +0800, Greentime Hu wrote:

quoted

+void do_page_fault(unsigned long entry, unsigned long addr,
+                unsigned int error_code, struct pt_regs *regs)

[snip]

quoted

+     /*
+      * If we're in an interrupt or have no user
+      * context, we must not take the fault..
+      */
+     if (unlikely(in_atomic() || !mm))

Broken.  in_atomic() is wrong here - it should be faulthandler_disabled().

Thanks.
I will include <linux/uaccess.h> and replace in_atomic() with
faulthandler_disabled()
I will fix it in the next version patch.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html