On Fri, 14 Jan 2022 at 06:38, Warner Losh <i...@bsdimp.com> wrote:
>
>
>
> 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).
(Maybe you had in mind the similar comment that used to be a few lines
above this one and which you removed in patch 5?)
With a fixed commit message:
Reviewed-by: Peter Maydell <peter.mayd...@linaro.org>
thanks
-- PMM
