Le 28/03/2018 à 16:41, Alex Bennée a écrit : > > Laurent Vivier <laur...@vivier.eu> writes: > >> Some files like signal.c are really hard to read >> because all architectures are mixed in the same >> file. >> >> This series moves from signal.c these parts to >> the architecture dedicated directories in linux-user. >> Moreover, this allows to compare easier functions >> between architectures (it helps to debug problems). >> Adding new functions for a new architecture will >> be facilitated too. >> >> checkpatch.pl is not happy... but I only want to >> move code from a file to another. I don't want >> to change the content of the parts I move. > > I think de-tabifying is worth it as it's a whitespace fix. Sure you can > leave everything else. > > Anyway for the rest of the series it looks good to me (although I only > tested ARM). > > Reviewed-by: Alex Bennée <alex.ben...@linaro.org>
Thank you, For testing, I have "debootstrap"ed the following architectures: stretch: s390x ppc64le mipsel mips64el mips arm aarch64 jessie: ppc lenny: hppa sid: m68k ppc64 sh4 I've tried sparc32plus, sparc64 and alpha but they are already broken without this patch series. For the other architectures, I rely only on the results of "make check". (microblaze, cris, nios2, openrisc, tilegx, riscv, xtensa). If someone has pointers to debian repo for these archs, please share. Thanks, Laurent
