[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 33/34] linux-user: Support for restarting system
From: |
Edgar E. Iglesias |
Subject: |
Re: [Qemu-devel] [PATCH 33/34] linux-user: Support for restarting system calls for CRIS targets |
Date: |
Fri, 11 Sep 2015 16:18:45 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, Sep 10, 2015 at 08:12:21PM +0100, Peter Maydell wrote:
> On 6 September 2015 at 00:57, Timothy E Baldwin
> <address@hidden> wrote:
> > Signed-off-by: Timothy Edward Baldwin <address@hidden>
> > ---
> >
> > WARNING - NOT TESTED
> >
> > linux-user/cris/syscall.h | 2 ++
> > linux-user/main.c | 6 +++++-
> > linux-user/signal.c | 2 +-
> > 3 files changed, 8 insertions(+), 2 deletions(-)
> >
> > diff --git a/linux-user/cris/syscall.h b/linux-user/cris/syscall.h
> > index 2957b0d..29218e2 100644
> > --- a/linux-user/cris/syscall.h
> > +++ b/linux-user/cris/syscall.h
> > @@ -44,3 +44,5 @@ struct target_pt_regs {
> > #define TARGET_MLOCKALL_MCL_FUTURE 2
> >
> > #endif
> > +
> > +#define TARGET_USE_ERESTARTSYS 1
> > diff --git a/linux-user/main.c b/linux-user/main.c
> > index a59907e..a9eb15c 100644
> > --- a/linux-user/main.c
> > +++ b/linux-user/main.c
> > @@ -2878,7 +2878,11 @@ void cpu_loop(CPUCRISState *env)
> > env->pregs[7],
> > env->pregs[11],
> > 0, 0);
> > - env->regs[10] = ret;
> > + if (ret == -TARGET_ERESTARTSYS) {
> > + env->pc -= 2;
> > + } else if (ret != -TARGET_QEMU_ESIGRETURN) {
> > + env->regs[10] = ret;
> > + }
> > break;
> > case EXCP_DEBUG:
> > {
> > diff --git a/linux-user/signal.c b/linux-user/signal.c
> > index e17514e..3741517 100644
> > --- a/linux-user/signal.c
> > +++ b/linux-user/signal.c
> > @@ -3761,7 +3761,7 @@ long do_sigreturn(CPUCRISState *env)
> >
> > restore_sigcontext(&frame->sc, env);
> > unlock_user_struct(frame, frame_addr, 0);
> > - return env->regs[10];
> > + return -TARGET_QEMU_ESIGRETURN;
> > badframe:
> > force_sig(TARGET_SIGSEGV);
> > }
>
> Looks OK, but this one I'm not sure enough about the CRIS instruction
> set and the surrounding code doesn't clarify. Edgar -- is the CRIS
> instruction for "take linux system call" always exactly 2 bytes long?
>
Yes, those are always 2 bytes.
Cheers,
Edgar
- [Qemu-devel] [PATCH 12/34] linux-user: Support for restarting system calls for APLHA targets, (continued)
- [Qemu-devel] [PATCH 12/34] linux-user: Support for restarting system calls for APLHA targets, Timothy E Baldwin, 2015/09/05
- [Qemu-devel] [PATCH 28/34] linux-user: Restart sigaction() if signal pending, Timothy E Baldwin, 2015/09/05
- [Qemu-devel] [PATCH 23/34] linux-user: Queue synchronous signals separately, Timothy E Baldwin, 2015/09/05
- [Qemu-devel] [PATCH 32/34] linux-user: Support for restarting system calls for S390 targets, Timothy E Baldwin, 2015/09/05
- [Qemu-devel] [PATCH 22/34] linux-user: Remove real-time signal queuing, Timothy E Baldwin, 2015/09/05
- [Qemu-devel] [PATCH 33/34] linux-user: Support for restarting system calls for CRIS targets, Timothy E Baldwin, 2015/09/05
[Qemu-devel] [PATCH 18/34] linux-user: Fix race between multiple signals, Timothy E Baldwin, 2015/09/05
[Qemu-devel] [PATCH 24/34] linux-user: Restart execve() if signal pending, Timothy E Baldwin, 2015/09/05
[Qemu-devel] [PATCH 34/34] linux-user: Remove TARGET_USE_ERESTARTSYS, Timothy E Baldwin, 2015/09/05
[Qemu-devel] [PATCH 31/34] linux-user: Support for restarting system calls for M68K targets, Timothy E Baldwin, 2015/09/05