On Wed, Oct 11, 2017 at 6:23 AM, Laurent Vivier <[email protected]> wrote: > Le 26/09/2017 à 01:19, Carlo Marcelo Arenas Belón a écrit : >> remove unnecessary sock_type enum and other unused surrounding code to >> allow for per arch sockbits to mirror better linux headers for maintenance >> >> Signed-off-by: Carlo Marcelo Arenas Belón <[email protected]> >> --- >> linux-user/alpha/sockbits.h | 36 ---------------------------- >> linux-user/hppa/sockbits.h | 30 ----------------------- >> linux-user/mips/sockbits.h | 35 --------------------------- >> linux-user/socket.h | 58 >> ++++++++++++++++++++++----------------------- >> linux-user/sparc/sockbits.h | 35 --------------------------- >> 5 files changed, 29 insertions(+), 165 deletions(-) > > I think it's not a good idea to mix again what we have just unmixed.
do you object to the fact it was done in 2 steps (first moving out from socket.h and then added it back as a single block)? it was done that way to reuse as much as possible the code that was already reviewed. the end result is the important part, though; and it is a sockbits.h for those 4 architectures (and a socket.h for the rest) that could be automatically generated or compared with the corresponding linux headers and therefore hopefully be easier to maintain going forward. since we are going to redo the series anyway, could we get some discussion/agreement on the design of it?, have to admit I am a little surprised by all the patches flying by the list but not many conversations, and being new to qemu linux-user I think I could have benefited from it to avoid some of the mistakes made. Carlo
