patches/gdb/6.3/750-debian_sparc-singlestep.patch
author "Yann E. MORIN" <yann.morin.1998@anciens.enib.fr>
Sun May 20 13:48:26 2007 +0000 (2007-05-20)
changeset 112 ea15433daba0
permissions -rw-r--r--
Ah! I finally have a progress bar that doesn't stall the build!
- pipe size in Linux is only 8*512=4096 bytes
- pipe size is not setable
- when the feeding process spits out data faster than the eating
process can read it, then the feeding process stalls after 4KiB
of data sent to the pipe
- for us, the progress bar would spawn a sub-shell every line,
and the sub-shell would in turn spawn a 'date' command.
Which was sloooww as hell, and would cause some kind of a
starvation: the pipe was full most of the time, and the
feeding process was stalled all this time.

Now, we use internal variables and a little hack based onan offset
to determine the elapsed time. Much faster this way, but still
CPU-intensive.
     1 Status: submitted for comments
     2 
     3 2004-11-13  Daniel Jacobowitz  <dan@debian.org>
     4 
     5 	* sparc-tdep.c (sparc_software_single_step): Handle stepping to NULL.
     6 
     7 Index: gdb-6.3/gdb/sparc-tdep.c
     8 ===================================================================
     9 --- gdb-6.3.orig/gdb/sparc-tdep.c	2004-06-06 22:02:55.000000000 -0400
    10 +++ gdb-6.3/gdb/sparc-tdep.c	2004-11-13 17:06:05.000000000 -0500
    11 @@ -1026,10 +1026,10 @@
    12  
    13    if (insert_breakpoints_p)
    14      {
    15 -      CORE_ADDR pc;
    16 +      CORE_ADDR pc, orig_npc;
    17  
    18        pc = sparc_address_from_register (tdep->pc_regnum);
    19 -      npc = sparc_address_from_register (tdep->npc_regnum);
    20 +      orig_npc = npc = sparc_address_from_register (tdep->npc_regnum);
    21  
    22        /* Analyze the instruction at PC.  */
    23        nnpc = sparc_analyze_control_transfer (pc, &npc);
    24 @@ -1039,9 +1039,10 @@
    25  	target_insert_breakpoint (nnpc, nnpc_save);
    26  
    27        /* Assert that we have set at least one breakpoint, and that
    28 -         they're not set at the same spot.  */
    29 -      gdb_assert (npc != 0 || nnpc != 0);
    30 -      gdb_assert (nnpc != npc);
    31 +         they're not set at the same spot - unless we're going
    32 +	 from here straight to NULL, i.e. a call or jump to 0.  */
    33 +      gdb_assert (npc != 0 || nnpc != 0 || orig_npc == 0);
    34 +      gdb_assert (nnpc != npc || orig_npc == 0);
    35      }
    36    else
    37      {