On Thu, Jan 13, 2022 at 10:15 AM Peter Maydell <peter.mayd...@linaro.org>
wrote:
> On Sun, 9 Jan 2022 at 16:26, Warner Losh <i...@bsdimp.com> wrote:
> >
> > The code has moved in FreeBSD since the emulator was started, update the
> > comment to reflect that change. Remove now-redundant comment saying the
> > same thing (but incorrectly).
> >
> > Signed-off-by: Warner Losh <i...@bsdimp.com>
> > ---
> > bsd-user/arm/target_arch_cpu.h | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/bsd-user/arm/target_arch_cpu.h
> b/bsd-user/arm/target_arch_cpu.h
> > index 05b19ce6119..905f13aa1b9 100644
> > --- a/bsd-user/arm/target_arch_cpu.h
> > +++ b/bsd-user/arm/target_arch_cpu.h
> > @@ -73,7 +73,7 @@ static inline void target_cpu_loop(CPUARMState *env)
> > int32_t syscall_nr = n;
> > int32_t arg1, arg2, arg3, arg4, arg5, arg6, arg7,
> arg8;
> >
> > - /* See arm/arm/trap.c cpu_fetch_syscall_args() */
> > + /* See arm/arm/syscall.c cpu_fetch_syscall_args() */
> > if (syscall_nr == TARGET_FREEBSD_NR_syscall) {
> > syscall_nr = env->regs[0];
> > arg1 = env->regs[1];
>
> Commit message says we're updating one comment and deleting a
> second one; code only does an update, no delete ?
>
Commit is right, commit message is wrong. I'll fix the commit message. I got
this confused with part 8 where I kinda sorta did something similar (but not
that similar).
Warner
