Re: ARM - RDI ethernet support

[Fernando Nasser <fnasser at cygnus dot com>]

Hi Dave,

Thank you for your patches.

I will incorporate the Tcl part of it soon, so we will be able to select
RDI over UDP on Insight.
As soon as I find some time, I will try to figure why pressing the STOP
button does not work.  Note that it will be somewhat difficult for me as
I don't have an interruptible target on hands, but I will at least check
if interrupt_target() gets called.

With regards to the interrupt patch to ardi.c, we have already
incorporated a patch from Grant Edwards that does the same thing.  Due
to my vacations and the difficulties I have to test (lack of ICE
target), I had no time to incorporate it in the snapshots, which I was
planning to do this week.

I am attaching the current patch and I would appreciate if you give it a
spin.  If you have any problems please let us know.  We can try to merge
the two patches (I don't think it will be necessary though).

Regards,
Fernando

-- 
Fernando Nasser
Cygnus Solutions - Toronto Office       E-Mail:  fnasser@cygnus.com
2323 Yonge Street, Suite #300           Tel:  416-482-2661 ext. 311
Toronto, Ontario   M4P 2C9              Fax:  416-482-6299


Index: ardi.c
===================================================================
RCS file: /cvs/cvsfiles/devo/gdb/rdi-share/ardi.c,v
retrieving revision 1.5
diff -c -r1.5 ardi.c
*** ardi.c      1999/11/01 15:29:56     1.5
--- ardi.c      1999/12/13 22:31:16
***************
*** 392,397 ****
--- 392,398 ----
  #endif
  
  static bool boot_interrupted = FALSE;
+ static volatile bool interrupt_request = FALSE;
  
  static void ardi_sigint_handler(int sig) {
  #ifdef DEBUG
***************
*** 401,406 ****
--- 402,408 ----
      IGNORE(sig);
  #endif
      boot_interrupted = TRUE;
+     interrupt_request = TRUE;
  #ifndef __unix
      signal(SIGINT, ardi_sigint_handler);
  #endif
***************
*** 1306,1312 ****
    return RDIError_NoError;
  }
  
- static volatile bool interrupt_request = FALSE;
  
  static void interrupt_target( void )
  {
--- 1308,1313 ----
***************
*** 1397,1402 ****
--- 1398,1404 ----
    angel_DebugPrint("Waiting for program to finish...\n");
  #endif
  
+   signal(SIGINT, ardi_sigint_handler);
    while( executing )
    {
        if (interrupt_request)
***************
*** 1406,1411 ****
--- 1408,1415 ----
        }
        Adp_AsynchronousProcessing( async_block_on_nothing );
    }
+   signal(SIGINT, SIG_IGN);
+ 
  
  #ifdef TEST_DC_APPL
    Adp_Install_DC_Appl_Handler( NULL );