On Wed, Feb 16, 2000 at 06:34:29PM +0900, Hiroyuki Machida wrote:
> I think I found a redundant code in copy_from_user() and
> __copy_from_user() at current CVS asm-mips/uaccess.h.
>
> I think '*'-marked part in the definiton is obsolete and
> redundant. It had to used in the exception fixup routine as
> commented at arch/mips/lib/memcpy.S. (Of course the comment is also
> obsolete, I think.)
As you say $at is being used for the exception handling, so it obviously
isn't redundant as you say. Or do I missunderstand what you were trying
to express?
You however made me stump over another bug, the definition of __MODULE_JAL
in <asm/uaccess.h> is wrong when compiling a module. In that case the
generated code will clobber $at which actually should stay unchanged
for the exception handling.
Ralf
|