Re: [PATCH 1/9] x86: change most X86_32 pt_regs members to unsigned long

From: Alexey Dobriyan
Date: Fri Feb 08 2008 - 16:14:13 EST


On Fri, Feb 08, 2008 at 12:09:56PM -0800, Harvey Harrison wrote:
> All but ax and orig_ax can move with no changes.

> --- a/include/asm-x86/ptrace.h
> +++ b/include/asm-x86/ptrace.h
> @@ -36,23 +36,23 @@ struct pt_regs {
> #else /* __KERNEL__ */
>
> struct pt_regs {
> - long bx;
> - long cx;
> - long dx;
> - long si;
> - long di;
> - long bp;
> + unsigned long bx;
> + unsigned long cx;
> + unsigned long dx;
> + unsigned long si;
> + unsigned long di;
> + unsigned long bp;
> long ax;
> - int ds;
> - int es;
> - int fs;
> + unsigned long ds;
> + unsigned long es;
> + unsigned long fs;
> /* int gs; */
> long orig_ax;
> - long ip;
> - int cs;
> - long flags;
> - long sp;
> - int ss;
> + unsigned long ip;
> + unsigned long cs;
> + unsigned long flags;
> + unsigned long sp;
> + unsigned long ss;

Why?! You don't have even minimal chances to unify pt_regs for i386 and
x86_64.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/