Message ID | 018bba1e097552bc4054e6d90e3f03e7c9a632bb.1368702323.git.mst@redhat.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On 5/16/2013 7:15 AM, Michael S. Tsirkin wrote: > The only reason uaccess routines might sleep > is if they fault. Make this explicit. > > Signed-off-by: Michael S. Tsirkin <mst@redhat.com> > --- > arch/tile/include/asm/uaccess.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Acked-by: Chris Metcalf <cmetcalf@tilera.com>
diff --git a/arch/tile/include/asm/uaccess.h b/arch/tile/include/asm/uaccess.h index 8a082bc..e4d44bd 100644 --- a/arch/tile/include/asm/uaccess.h +++ b/arch/tile/include/asm/uaccess.h @@ -442,7 +442,7 @@ extern unsigned long __copy_in_user_inatomic( static inline unsigned long __must_check __copy_in_user(void __user *to, const void __user *from, unsigned long n) { - might_sleep(); + might_fault(); return __copy_in_user_inatomic(to, from, n); }
The only reason uaccess routines might sleep is if they fault. Make this explicit. Signed-off-by: Michael S. Tsirkin <mst@redhat.com> --- arch/tile/include/asm/uaccess.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)