"Maciej W. Rozycki" wrote:
> On Wed, 7 Aug 2002, Carsten Langgaard wrote:
>
> > Here is a patch that fixes the ipc syscalls in the o32
> > wrapper/conversion routines.
> > Needed when running a 64-bit kernel on an o32 userland.
>
> Hmm, this looks dubious:
>
> +#define AA(__x) ((unsigned long)((int)__x))
>
> You probably want either:
>
> ((unsigned long)((unsigned int)__x))
>
> or
>
> ((long)((int)__x)).
>
> Since you are using pointers, likely the latter.
>
A(__x) is returning a unsigned long, so I try to stick to that.
I need the (int) typecast to make sure things got sign extended.
So if you want to change it, please go for the latter ((long)(int)).
What about the rest of the patch ?
>
> Sending patches within a mail's body would ease commenting them, BTW. I
> had to copy and paste the line above manually -- with gpm it's not a big
> problem for a single line, but it gets tedious for larger chunks and gpm
> is not everywhere.
Ralf has asked me to send the patches as attachment, because I use Netscape
as my mailer.
>
> --
> + Maciej W. Rozycki, Technical University of Gdansk, Poland +
> +--------------------------------------------------------------+
> + e-mail: macro@ds2.pg.gda.pl, PGP key available +
--
_ _ ____ ___ Carsten Langgaard Mailto:carstenl@mips.com
|\ /|||___)(___ MIPS Denmark Direct: +45 4486 5527
| \/ ||| ____) Lautrupvang 4B Switch: +45 4486 5555
TECHNOLOGIES 2750 Ballerup Fax...: +45 4486 5556
Denmark http://www.mips.com
|