Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: oza Pawandeep <oza.pawandeep@gmail.com>
To: Tom Tromey <tromey@redhat.com>
Cc: gdb@sourceware.org, gdb-patches@sourceware.org
Subject: Re: [design change] record-replay linux ABI level
Date: Sun, 13 May 2012 07:19:00 -0000	[thread overview]
Message-ID: <CAK1A=4wzBfbc1OpbP2BNMGVw0iiWMG81fy1+peq_r5rFkenyhA@mail.gmail.com> (raw)
In-Reply-To: <87aa1gqhnq.fsf@fleche.redhat.com>

currently on i386 following is the function:

static enum gdb_syscall
i386_canonicalize_syscall (int syscall)
{
  enum { i386_syscall_max = 499 };

  if (syscall <= i386_syscall_max)
    return syscall;
  else
    return -1;
}

which is just straight mapping.

If we use generic enum defination, we will end up adding some
additional syscalls for ARM and

arm_canonicalize_syscall(int syscall)
end up having switch {case} and having one-to one mapping for some
syscalls and rest syscalls would be shift by 'n' position.
which looks clumsy to me.


I am trying to see if there is more generic way which would take care
of all archor move the defination to arch files.
will try to see what best could be done.

Regards,
Oza.



On Thu, May 10, 2012 at 7:09 PM, Tom Tromey <tromey@redhat.com> wrote:
> Oza> The definition of system call record maps fine to x86.  but arm
> Oza> syscall numbers are different. [partially] for e.g. on x86 sycall
> Oza> number for sys_epoll_create = 254 while on ARM it is 250.  the more
> Oza> we go down on defined system calls the more the numbers are
> Oza> differing on ARM and we loose one to one trivial mapping.
>
> My understanding of the current design is that the ARM code would see
> the syscall 250, and have a mapping to turn that into
> gdb_sys_epoll_create (== 254).  This can be done bidirectionally with
> two lookup tables.
>
> I suppose this could still not work in some scenarios.  One question is
> whether these occur in practice or are merely theoretical.
>
> I don't really care about this API either way.
> With a solid justification it is fine to change it.
>
> Tom


  reply	other threads:[~2012-05-13  7:19 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CAK1A=4xtgYd8hQEwHxjLQiv4eqhCu0cSRDmmbFJvBDJwDxUM+Q__46748.0269181125$1336555010$gmane$org@mail.gmail.com>
2012-05-09 20:39 ` Tom Tromey
2012-05-10  8:49   ` oza Pawandeep
     [not found]   ` <CAK1A=4xbh0M=yfc2MQpZdDCJEPnL3_z8=TA0VSE7qVCoO0Dn-Q__42617.423789534$1336639800$gmane$org@mail.gmail.com>
2012-05-10 13:39     ` Tom Tromey
2012-05-13  7:19       ` oza Pawandeep [this message]
2012-05-13  7:33         ` oza Pawandeep
2012-05-13  9:47           ` oza Pawandeep
2012-05-14 14:57             ` Joel Brobecker
2012-05-15  5:20               ` oza Pawandeep
2012-05-15  5:34                 ` Joel Brobecker
     [not found]               ` <CAK1A=4yervLeVDQ-r49n95ftrB27u8K+R1hfstz1oFwTNX=t7Q__24807.0006179207$1337059251$gmane$org@mail.gmail.com>
2012-05-15 16:42                 ` Tom Tromey
2012-05-16 10:48                   ` oza Pawandeep
2012-05-16 14:57                     ` Tom Tromey
2012-06-05  9:16                       ` oza Pawandeep
2012-06-06 18:17                         ` Tom Tromey
2012-06-05  9:19                       ` oza Pawandeep
2012-05-09  9:16 oza Pawandeep

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='CAK1A=4wzBfbc1OpbP2BNMGVw0iiWMG81fy1+peq_r5rFkenyhA@mail.gmail.com' \
    --to=oza.pawandeep@gmail.com \
    --cc=gdb-patches@sourceware.org \
    --cc=gdb@sourceware.org \
    --cc=tromey@redhat.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