Re: netbsd32_{,u}int64 in sys/types.h for compat/sys/siginfo.h

[Rin Okuyama <rokuyama.rk%gmail.com@localhost>]

Thank you for your comment!

On 2019/11/17 22:42, Kamil Rytarowski wrote:
> Please check it also with picotrace/truss:
>
> http://pkgsrc.se/devel/picotrace

netbsd32_signal.c needed to catch up with kern_sig.c so that syscall
information is provided with SIGTRAP TRAP_SCE/TRAP_SCX. I committed
the fix and picotrace/truss works fine now on COMPAT_NETBSD32.

On 2019/11/17 22:42, Kamil Rytarowski wrote:
> On 17.11.2019 04:34, Rin Okuyama wrote:
> > Hi,
> >
> > In order to distangle circular dependency between
> > <sys/compat/sys/siginfo.h> v.s. <machine/netbsd32_machdep.h>,
> > I propose
> >
> > (1) Move NETBSD32_INT64_ALIGN from <machine/netbsd32_machdep.h> to
> > <machine/types.h>
> >
> > (2) Move netbsd32_{,u}int64 from <sys/compat/netbsd32/netbsd32.h> to
> > <sys/types.h>
> >
> > See attached patch for example on amd64.
>
> What do you think about duplicating the defines of netbsd32_uint64
> inside sys/compat/sys/siginfo.h + adding a comment about keeping it in
> sync with netbsd32.h?
>
> I think that avoiding spaghetti dependencies is a benefit.
>
> We already duplicated there _ptrace_state, removing circular
> dependencies between sys/ptrace.h and sys/siginfo.h.

I don't think this is a good idea in this case. If we want to have
duplicate define of netbsd32_uint64, and to avoid an "#ifdef __x86_64__"
mess in <compat/sys/siginfo.h>, we need to move NETBSD32_INT64_ALIGN to
<machine/somewhere.h> other than <machine/netbsd32_machdep.h>. If so,
why not <machine/types.h>?

Also, in my proposal, spaghetti dependencies are avoided in the end;
everyone depends on <sys/types.h>, and not on each other.

Thanks,
rin