| Message ID | 1317674600-19083-2-git-send-email-sw@weilnetz.de |
|---|---|
| State | Accepted |
| Headers | show |
Am 03.10.2011 22:43, schrieb Stefan Weil: > The code is unused since 8 years, so remove it. > > Signed-off-by: Stefan Weil <sw@weilnetz.de> > --- > linux-user/signal.c | 5 +---- > 1 files changed, 1 insertions(+), 4 deletions(-) > > diff --git a/linux-user/signal.c b/linux-user/signal.c > index 89276eb..40c5eb1 100644 > --- a/linux-user/signal.c > +++ b/linux-user/signal.c > @@ -1274,10 +1274,7 @@ setup_return(CPUState *env, struct > target_sigaction *ka, > > if (__put_user(retcodes[idx], rc)) > return 1; > -#if 0 > - flush_icache_range((abi_ulong)rc, > - (abi_ulong)(rc + 1)); > -#endif > + > retcode = rc_addr + thumb; > } > This is a trivial patch, so I send it to qemu-trivial, too. The patch is independent of the rest of the series. Cheers, Stefan Weil
On 3 October 2011 21:43, Stefan Weil <sw@weilnetz.de> wrote: > The code is unused since 8 years, so remove it. > > Signed-off-by: Stefan Weil <sw@weilnetz.de> > --- > linux-user/signal.c | 5 +---- > 1 files changed, 1 insertions(+), 4 deletions(-) > > diff --git a/linux-user/signal.c b/linux-user/signal.c > index 89276eb..40c5eb1 100644 > --- a/linux-user/signal.c > +++ b/linux-user/signal.c > @@ -1274,10 +1274,7 @@ setup_return(CPUState *env, struct target_sigaction *ka, > > if (__put_user(retcodes[idx], rc)) > return 1; > -#if 0 > - flush_icache_range((abi_ulong)rc, > - (abi_ulong)(rc + 1)); > -#endif > + > retcode = rc_addr + thumb; > } (since this is in the ARM section of this file) Since this is writing guest code instructions, flushing the host icache is clearly not the right thing. I'm just wondering how this ensures that we don't have stale translated code for this address, though... -- PMM
On Mon, Oct 03, 2011 at 10:43:19PM +0200, Stefan Weil wrote: > The code is unused since 8 years, so remove it. > > Signed-off-by: Stefan Weil <sw@weilnetz.de> > --- > linux-user/signal.c | 5 +---- > 1 files changed, 1 insertions(+), 4 deletions(-) Thanks, applied to the trivial patches tree: http://repo.or.cz/w/qemu/stefanha.git/shortlog/refs/heads/trivial-patches Stefan
diff --git a/linux-user/signal.c b/linux-user/signal.c index 89276eb..40c5eb1 100644 --- a/linux-user/signal.c +++ b/linux-user/signal.c @@ -1274,10 +1274,7 @@ setup_return(CPUState *env, struct target_sigaction *ka, if (__put_user(retcodes[idx], rc)) return 1; -#if 0 - flush_icache_range((abi_ulong)rc, - (abi_ulong)(rc + 1)); -#endif + retcode = rc_addr + thumb; }
The code is unused since 8 years, so remove it. Signed-off-by: Stefan Weil <sw@weilnetz.de> --- linux-user/signal.c | 5 +---- 1 files changed, 1 insertions(+), 4 deletions(-)