Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Hui Zhu <teawater@gmail.com>
Cc: gdb@sourceware.org
Subject: System call support in process record and replay
Date: Sat, 30 May 2009 10:11:00 -0000	[thread overview]
Message-ID: <83iqji529q.fsf@gnu.org> (raw)

I have a question about general design of the system call support for
the record/replay target, for systems whose system calls are entered
through software interrupts.

The following excerpt from i386-tdep.c shows the currently-only
implementation, for Linux system calls entered via INT 80h:

    case 0xcd:
      {
	int ret;
	if (target_read_memory (ir.addr, &tmpu8, 1))
	  {
	    if (record_debug)
	      printf_unfiltered (_("Process record: error reading memory "
				   "at addr 0x%s len = 1.\n"),
				 paddr_nz (ir.addr));
	    return -1;
	  }
	ir.addr++;
	if (tmpu8 != 0x80
	    || gdbarch_tdep (gdbarch)->i386_intx80_record == NULL)
	  {
	    printf_unfiltered (_("Process record doesn't support "
				 "instruction int 0x%02x.\n"),
			       tmpu8);
	    ir.addr -= 2;
	    goto no_support;
	  }
	ret = gdbarch_tdep (gdbarch)->i386_intx80_record (ir.regcache);
	if (ret)
	  return ret;
      }
      break;

Now, suppose there is another x86 target whose system calls are
entered through 3 software interrupts: 0x10, 0x21, and 0x31.  Does
this mean that to support such a target, we will need to define 3
additional members of `struct gdbarch_tdep', one each for every one of
the above interrupt numbers, and then tweak the above code to call
each member whenever the corresponding interrupt number is seen in the
instruction stream?  And adding support for Windows syscalls means
that yet another member, for INT 2Eh, should be added?  That seems
rather inelegant and wasteful to me (since these members will go
unused on every x86 target that does not use those interrupts), but if
that's the design we want to follow, I'm okay with it.


             reply	other threads:[~2009-05-30 10:11 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-30 10:11 Eli Zaretskii [this message]
2009-05-30 21:13 ` Doug Evans
2009-05-31  5:58   ` Hui Zhu
2009-05-31 17:47 ` Mark Kettenis
2009-06-01 15:09   ` Eli Zaretskii
2009-06-01 19:29     ` Michael Snyder
2009-06-01  7:14 ` Andi Kleen
2009-06-01  7:19   ` Hui Zhu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=83iqji529q.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=gdb@sourceware.org \
    --cc=teawater@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox