Re: [SCM] master: Add debug register group and alases.

[Phil Muldoon <pmuldoon@redhat.com>]

pmuldoon@sourceware.org wrote:

This patch is a small commit to add a debug register group, and control 
and status register aliases. In the hardware documentation the 
terminology refers to debug control and debug status registers. These 
correspond to D6 and D7 in IA32, and DR6 and DR7 in X8664. These aliases 
are no more than a convenience to allow the programmer to refer to the 
registers by name as they are referred to in the AMD and INTEL 
documentation. The debug groups are just convenient ways to access the 
debug registers as part of a group, distinct from other registers.

Regards

Phil
> The branch, master has been updated
>        via  60e515c2f0c29dda5da86102343827f80b034a8b (commit)
>       from  00345416a6028de2e659b7a01d3b1c54adca694f (commit)
>
> Those revisions listed above that are new to this repository have
> not appeared on any other notification email.
>
> - Log -----------------------------------------------------------------
> commit 60e515c2f0c29dda5da86102343827f80b034a8b
> Author: Phil Muldoon <pmuldoon@redhat.com>
> Date:   Wed Mar 26 23:37:27 2008 +0000
>
>     Add debug register group and alases.
>     
>     2008-03-26  Phil Muldoon  <pmuldoon@redhat.com>
>     
>     	* X8664Registers.java: Add debug register group.
>     	Add debug control and status register aliases.
>     	* IA32Registers.java: Ditto.
>
> -----------------------------------------------------------------------
>
> Summary of changes:
>  frysk-core/frysk/isa/registers/ChangeLog           |    6 ++++++
>  frysk-core/frysk/isa/registers/IA32Registers.java  |   13 ++++++++++++-
>  frysk-core/frysk/isa/registers/X8664Registers.java |   13 ++++++++++++-
>  3 files changed, 30 insertions(+), 2 deletions(-)
>
> First 500 lines of diff:
> diff --git a/frysk-core/frysk/isa/registers/ChangeLog b/frysk-core/frysk/isa/registers/ChangeLog
> index 810212b..5ca7bbd 100644
> --- a/frysk-core/frysk/isa/registers/ChangeLog
> +++ b/frysk-core/frysk/isa/registers/ChangeLog
> @@ -1,3 +1,9 @@
> +2008-03-26  Phil Muldoon  <pmuldoon@redhat.com>
> +
> +	* X8664Registers.java: Add debug register group.
> +	Add debug control and status register aliases.
> +	* IA32Registers.java: Ditto.
> +	
>  2008-01-24  Andrew Cagney  <cagney@redhat.com>
>  
>  	* IA32Registers.java: Move to here from frysk.isa.
> diff --git a/frysk-core/frysk/isa/registers/IA32Registers.java b/frysk-core/frysk/isa/registers/IA32Registers.java
> index 2cfc8ba..ed5bd91 100644
> --- a/frysk-core/frysk/isa/registers/IA32Registers.java
> +++ b/frysk-core/frysk/isa/registers/IA32Registers.java
> @@ -1,6 +1,6 @@
>  // This file is part of the program FRYSK.
>  //
> -// Copyright 2007, Red Hat Inc.
> +// Copyright 2007, 2008, Red Hat Inc.
>  //
>  // FRYSK is free software; you can redistribute it and/or modify it
>  // under the terms of the GNU General Public License as published by
> @@ -116,6 +116,17 @@ public class IA32Registers extends Registers {
>      public static final Register ORIG_EAX
>  	= new Register("orig_eax", StandardTypes.INT32L_T);
>  
> +    // Add hardware manual aliases
> +    public static final Register DEBUG_STATUS = D6;
> +    public static final Register DEBUG_CONTROL = D7;
> +
> +    // Add Debug register group.
> +    public static final RegisterGroup DEBUG_REGS_GROUP
> +	= new RegisterGroup("debugregs",
> +			    new Register[] {
> +				D0, D1, D2, D3 , D4, D5, D6, D7
> +			    });
> +
>      public final static RegisterGroup REGS_GROUP
>  	= new RegisterGroup("regs",
>  			    new Register[] {
> diff --git a/frysk-core/frysk/isa/registers/X8664Registers.java b/frysk-core/frysk/isa/registers/X8664Registers.java
> index b1ab548..8b4efb2 100644
> --- a/frysk-core/frysk/isa/registers/X8664Registers.java
> +++ b/frysk-core/frysk/isa/registers/X8664Registers.java
> @@ -1,6 +1,6 @@
>  // This file is part of the program FRYSK.
>  //
> -// Copyright 2007, Red Hat Inc.
> +// Copyright 2007, 2008, Red Hat Inc.
>  //
>  // FRYSK is free software; you can redistribute it and/or modify it
>  // under the terms of the GNU General Public License as published by
> @@ -127,6 +127,17 @@ public class X8664Registers extends Registers {
>      public static final Register ORIG_RAX
>  	= new Register("orig_rax", StandardTypes.INT64L_T);
>  
> +    // Add hardware manual aliases
> +    public static final Register DEBUG_STATUS = DR6;
> +    public static final Register DEBUG_CONTROL = DR7;
> +
> +    // Add debug register group.
> +    public static final RegisterGroup DEBUG_REGS_GROUP
> +	= new RegisterGroup("debugregs",
> +			    new Register[] {
> +				DR0, DR1, DR2, DR3 , DR4, DR5, DR6, DR7
> +			    });
> +
>      public static final RegisterGroup REGS_GROUP
>  	= new RegisterGroup("regs",
>  			    new Register[] {
>
>
> hooks/post-receive
> --
> frysk system monitor/debugger
>