On Fri, Sep 22, 2017 at 10:01 AM, Laurent Vivier <laur...@vivier.eu> wrote:
> Le 22/09/2017 à 16:02, Carlo Marcelo Arenas Belón a écrit : > > Signed-off-by: Carlo Marcelo Arenas Belón <care...@gmail.com> > > --- > > 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