Re: [Qemu-devel] [RFC 5/6] linux-user: fix 64bit versions of sparc and mips

[Carlo Arenas]

On Fri, Sep 22, 2017 at 10:01 AM, Laurent Vivier <address@hidden> wrote:

> Le 22/09/2017 à 16:02, Carlo Marcelo Arenas Belón a écrit :
> > Signed-off-by: Carlo Marcelo Arenas Belón <address@hidden>
> > ---
> >  linux-user/mips64/sockbits.h  | 1 +
> >  linux-user/socket.h           | 4 ++--
> >  linux-user/sparc64/sockbits.h | 1 +
>
> Who includes the two new files?
>

No one yet, but hopefully will make clearer to anyone looking later for
arch specific values that they were indeed equivalent to their non 64bit
versions.

It also mimics what termbits.h is doing and so might allow in the future to
drop the common socket.h for an arch specific one, which will be easier to
maintain and keep in sync with the linux kernel.


> Moreover TARGET_MIPS/TARGET_SPARC (base arch) are always defined when
> TARGET_MIPS64/TARGET_SPARC64 (target arch) are defined.
>
> So I think this patch is useless.
>

ok, glad to know I was wrong about those architectures being broken, still
suspect s390x is though, but don't want to address this in this series,
like I didn't want to include the bigger refactoring explained above.

I am curious also over users of those architectures and their suggestions,
which is why I sent this series as an RFC; is there a better forum for that
somewhere else?

Carlo