[PATCH] Removal of *user_ret macros

From: SL Baur (steve@kbuxd.necst.nec.co.jp)
Date: Thu Oct 10 2002 - 20:30:23 EST


This is a janitor patch to remove some of the remaining *user_ret
macros that were missed. I am told the ones in parisc have already
been fixed and are waiting a merge. This patch was done against 2.5
bk current, but it will apply to 2.4 as well.

You can import this changeset into BK by piping this whole message to:
'| bk receive [path to repository]' or apply the patch as usual.

===================================================================

ChangeSet@1.747, 2002-10-10 18:00:45+09:00, steve@sofia.bsd2.kbnes.nec.co.jp
  Janitor clean up. Macros containing return statements are evil.
  

 uaccess.h | 24 ------------------------
 1 files changed, 24 deletions(-)

diff -Nru a/include/asm-cris/uaccess.h b/include/asm-cris/uaccess.h
--- a/include/asm-cris/uaccess.h Thu Oct 10 18:03:41 2002
+++ b/include/asm-cris/uaccess.h Thu Oct 10 18:03:41 2002
@@ -147,25 +147,6 @@
 #define __put_user(x,ptr) \
   __put_user_nocheck((__typeof__(*(ptr)))(x),(ptr),sizeof(*(ptr)))
 
-/*
- * The "xxx_ret" versions return constant specified in third argument, if
- * something bad happens. These macros can be optimized for the
- * case of just returning from the function xxx_ret is used.
- */
-
-#define put_user_ret(x,ptr,ret) \
- do { if (put_user(x,ptr)) return ret; } while (0)
-
-#define get_user_ret(x,ptr,ret) \
- do { if (get_user(x,ptr)) return ret; } while (0)
-
-#define __put_user_ret(x,ptr,ret) \
- do { if (__put_user(x,ptr)) return ret; } while (0)
-
-#define __get_user_ret(x,ptr,ret) \
- do { if (__get_user(x,ptr)) return ret; } while (0)
-
-
 extern long __put_user_bad(void);
 
 #define __put_user_nocheck(x,ptr,size) \
@@ -1016,11 +997,6 @@
 (__builtin_constant_p(n) ? \
  __constant_copy_to_user(to, from, n) : \
  __generic_copy_to_user(to, from, n))
-
-#define copy_to_user_ret(to,from,n,retval) \
- do { if (copy_to_user(to,from,n)) return retval; } while (0)
-#define copy_from_user_ret(to,from,n,retval) \
- do { if (copy_from_user(to,from,n)) return retval; } while (0)
 
 /* We let the __ versions of copy_from/to_user inline, because they're often
  * used in fast paths and have only a small space overhead.

===================================================================

This BitKeeper patch contains the following changesets:
1.747
## Wrapped with gzip_uu ##

begin 664 bkpatch5654
M'XL(`&U"I3T``[64;6O;,!#'7U>?XJ#O5BR?9-FN#1G9FK'1;2QD]'51Y$OM
M-I:#I7@4_.&GN-"5LJY[ZEG@!TYW_O_OAX[APE%?'CE/`[%C^-`Y']ZZ3:/Y
MVE62WZPM.6[)<-/QZUU(675=2(GKKJ5XVA:[WL37VC:^ZR/)4Q:2EMJ;&@;J
M77DD>'+_Q=_NJ#Q:O7M_\>G-BK'9#,YJ;:_H*WF8S5BH,.AMY>;:U]O.<M]K
MZUKR.G1OQ_O442+*<*4B3S#-1I&ARD<C*B&T$E2A5*>98@L]--7E.6TVU-_.
M=:5W?M+QN))`+*1$E>`HBBR1;`&"YRH'E+'`L$"<EHBE2D^P"`\PR9X_[1*<
M"(B0O87_J^>,&3B_\QG,EK2%_8X#?-:F[QR8SGK=V,9>04]^W]OPF]I32]8[
MT#T!#<V6AQ+L(X1^0K+E#_-9](?!&&IDKV%W&.O/Y376;/<5Q=JUD>D;%^^U
M,>0<KQ_H33.5C++`%,<$59:EE3!9BNEZO7G6YN<Z"#Q,%D.,2:)$,?'V])X#
M@"\HYQ]K2TRD$F.2YH@3H.HQGHB_C2=")-5+\EG"BMIN('AU&6"$=D(T@'<W
MAR\0]=^F%4!:_F(D?X'E0J3!D"+<413PX#`R-9D;MV]G>1K"5,B^`SNYLJ/]
#!```
`
end

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Tue Oct 15 2002 - 22:00:40 EST