On 22 March 2018 at 21:58, Laurent Vivier <laur...@vivier.eu> wrote: > 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. > Moerover, this allows to compare easier functions > between architectures (it helps to debug problems). > Adding new functions for a new architecture will > be facilitated too. > > As we are doing that for signal.c, we can also > do that for main.c, for the cpu loop part > and the cpu loop prologue too.
Yeah, this is something I've kind of had on my backlog of "it would be nice to do this" for years. Thanks for putting in the work! > 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. Yeah, that's reasonable. (Since the move will reduce the effectiveness of git blame anyway we might want to do a fix-coding-style pass afterwards, but we can do that as a separate patchset.) thanks -- PMM
