From patchwork Fri Mar 23 22:57:19 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Laurent Vivier X-Patchwork-Id: 890299 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (mailfrom) smtp.mailfrom=nongnu.org (client-ip=2001:4830:134:3::11; helo=lists.gnu.org; envelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=vivier.eu Received: from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 407Jr46RN9z9s0v for ; Sat, 24 Mar 2018 09:58:59 +1100 (AEDT) Received: from localhost ([::1]:40144 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ezVeA-0004ig-Te for incoming@patchwork.ozlabs.org; Fri, 23 Mar 2018 18:58:54 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45279) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ezVdU-0004iL-Nv for qemu-devel@nongnu.org; Fri, 23 Mar 2018 18:58:14 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ezVdR-0000WL-HN for qemu-devel@nongnu.org; Fri, 23 Mar 2018 18:58:12 -0400 Received: from mout.kundenserver.de ([212.227.17.24]:40881) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1ezVdR-0000Vd-6c; Fri, 23 Mar 2018 18:58:09 -0400 Received: from localhost.localdomain ([78.238.229.36]) by mrelayeu.kundenserver.de (mreue103 [212.227.15.183]) with ESMTPSA (Nemesis) id 0MWySC-1f2vNl2vRM-00W19X; Fri, 23 Mar 2018 23:57:53 +0100 From: Laurent Vivier To: qemu-devel@nongnu.org Date: Fri, 23 Mar 2018 23:57:19 +0100 Message-Id: <20180323225739.17329-1-laurent@vivier.eu> X-Mailer: git-send-email 2.14.3 X-Provags-ID: V03:K0:qXyMSt4jbu7TPAzsZ5aImxUzgQI8uhp2AlNhCyClob1aPqphLrF 6pwHN+t/XtyH/ZCv7bJJUEuxarRcYcg7JxKj9E+tH0AUlv7mw53LWQZWscpbOpscZrIrxqI Zw6+TFn6qnr5Z/PzX/VRAgSyW++mzs1pq/Pzd5MyY6Jdq8cTr06B7AkjGREta3xQe4e9APB r6jA8oKnWTIvxREqTBD4w== X-UI-Out-Filterresults: notjunk:1; V01:K0:nbjQxCCb7Oo=:XtHiol0z0HyN8LIQwyBn72 ivUxihSdWYIfSzMslYDFL/xP6oLQ5J2hb0MU3bG4EU6C1WPdmygM9tktz4AJMD/Bj/X9gIaGQ AewSnylVAFprC4ujK3hpZGmwSDKXn/Cp2s+Cl5acbnJlOC42ctrQNvWWcoWu9E/wxszwFCS87 CXYv3UVIfvWhsTmH0ZSbhsE7KnWOKzssn29C7B0YBw9CR/vFd/pllu3XT2VhcTwylJHAsDCUy yRsYC1QqbZ+zfwuwe2+c+FMgwmAu17YaINPqXfzJV6MQF+5Cxlfq1O2ZENGetR690FngAunn8 D7N2TmKXSQaDMfmS7kuDRcQ9G1AaMA6ppptd+yiECmTBAZQmYQynCoQWBRhxIO6GNy7b5E/kr 9rDFY1PxJ5rmCsT7KbUvby5bTbBgZO8M1aDnay1KDjwLiw+Tu8xWTaJrx/pzM0gMzrcSDQc+N 0G/wufSUzZycjL4ipiwaxUBaMoYgN7IwQB6KkRdWf3HlLnxtlGt7cY5PsRbkz9PvjuOnp0SAk 8o44QTg97hDJMWdyCxaUG5Flht/Mbre74y80ZUw/0j5b0htrsQb+ickehSNSGXCywsISU4P09 +5Oyi2bmPgmKqe9xdZHYSY65xfLNWginZP72YTxGw2nLXxSX/nFjA+HANSZ6ziRr16Si1tcbx sW1zdJKhq2HXlk4ziORXhEOOYZUltLcXh5hUGPizDjDS5A78pYhWwEsUzlvwm6UIweLT761b+ T9Bv6McLNsQj7iLRJZ6RsUTaRcxaajOAUttU6g== X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 212.227.17.24 Subject: [Qemu-devel] [PATCH for 2.13 v2 00/20] linux-user: move arch specific parts to arch directories X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Cornelia Huck , Riku Voipio , Laurent Vivier , =?utf-8?q?Philippe_Mathieu-Daud?= =?utf-8?b?w6k=?= , qemu-s390x@nongnu.org Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: "Qemu-devel" 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. v2: - only move parts from signal.c - link them instead of including them - one patch by architecture - add a first patch to prepare the change. The first patch adds signal-common.h to define what is needed by the signal.c of the architectures. It adds a "do-nothing" signal.c in each arch directory and the rule needed to build them in Makefile.objs. Then the process is simple... for each architecture: - copy the arch specific code from signal.c to /signal.c - add includes (including signal-common.h) - export setup_rt_frame() and setup_frame() (remove static in /signal.c, add the declaration in /target_signal.h) When the arch has 32bit and 64bit architectures, it's a little bit more complicated: - ppc/ppc64: nothing special to do, all is in ppc/, there is no ppc64 directory, - arm/aarch64: one file for arm, one file for aarch64 - i386/x86_64, sparc/sparc64, mips/mips64: update each target_signal.h, include the 32bit signal.c file into the 64bit signal.c file to avoid to duplicate code (and add a guard to not include the 32bit target_signal.h) Laurent Vivier (20): linux-user: create a dummy per arch signal.c linux-user: move aarch64 signal.c parts to aarch64 directory linux-user: move arm signal.c parts to arm directory linux-user: move sh4 signal.c parts to sh4 directory linux-user: move microblaze signal.c parts to microblaze directory linux-user: move cris signal.c parts to cris directory linux-user: move nios2 signal.c parts to nios2 directory linux-user: move openrisc signal.c parts to openrisc directory linux-user: move s390x signal.c parts to s390x directory linux-user: move m68k signal.c parts to m68k directory linux-user: move alpha signal.c parts to alpha directory linux-user: move tilegx signal.c parts to tilegx directory linux-user: move riscv signal.c parts to riscv directory linux-user: move hppa signal.c parts to hppa directory linux-user: move xtensa signal.c parts to xtensa directory linux-user: move i386/x86_64 signal.c parts to i386 directory linux-user: move sparc/sparc64 signal.c parts to sparc directory linux-user: move mips/mips64 signal.c parts to mips directory linux-user: move ppc/ppc64 signal.c parts to ppc directory linux-user: define TARGET_ARCH_HAS_SETUP_FRAME linux-user/Makefile.objs | 2 +- linux-user/aarch64/signal.c | 579 +++ linux-user/aarch64/target_signal.h | 6 + linux-user/alpha/signal.c | 280 ++ linux-user/alpha/target_signal.h | 6 + linux-user/arm/signal.c | 772 ++++ linux-user/arm/target_signal.h | 7 +- linux-user/cris/signal.c | 189 + linux-user/cris/target_signal.h | 7 +- linux-user/hppa/signal.c | 210 ++ linux-user/hppa/target_signal.h | 3 + linux-user/i386/signal.c | 602 +++ linux-user/i386/target_signal.h | 6 + linux-user/m68k/signal.c | 428 +++ linux-user/m68k/target_signal.h | 7 +- linux-user/microblaze/signal.c | 248 ++ linux-user/microblaze/target_signal.h | 7 +- linux-user/mips/signal.c | 400 ++ linux-user/mips/target_signal.h | 10 +- linux-user/mips64/signal.c | 20 + linux-user/mips64/target_signal.h | 4 +- linux-user/nios2/signal.c | 254 ++ linux-user/nios2/target_signal.h | 4 + linux-user/openrisc/signal.c | 231 ++ linux-user/openrisc/target_signal.h | 4 +- linux-user/ppc/signal.c | 689 ++++ linux-user/ppc/target_signal.h | 9 +- linux-user/riscv/signal.c | 218 ++ linux-user/riscv/target_signal.h | 3 + linux-user/s390x/signal.c | 327 ++ linux-user/s390x/target_signal.h | 7 +- linux-user/sh4/signal.c | 350 ++ linux-user/sh4/target_signal.h | 6 + linux-user/signal-common.h | 50 + linux-user/signal.c | 6654 +-------------------------------- linux-user/sparc/signal.c | 624 ++++ linux-user/sparc/target_signal.h | 7 +- linux-user/sparc64/signal.c | 20 + linux-user/sparc64/target_signal.h | 7 +- linux-user/tilegx/signal.c | 186 + linux-user/tilegx/target_signal.h | 4 +- linux-user/x86_64/signal.c | 20 + linux-user/x86_64/target_signal.h | 3 + linux-user/xtensa/signal.c | 275 ++ linux-user/xtensa/target_signal.h | 3 + 45 files changed, 7155 insertions(+), 6593 deletions(-) create mode 100644 linux-user/aarch64/signal.c create mode 100644 linux-user/alpha/signal.c create mode 100644 linux-user/arm/signal.c create mode 100644 linux-user/cris/signal.c create mode 100644 linux-user/hppa/signal.c create mode 100644 linux-user/i386/signal.c create mode 100644 linux-user/m68k/signal.c create mode 100644 linux-user/microblaze/signal.c create mode 100644 linux-user/mips/signal.c create mode 100644 linux-user/mips64/signal.c create mode 100644 linux-user/nios2/signal.c create mode 100644 linux-user/openrisc/signal.c create mode 100644 linux-user/ppc/signal.c create mode 100644 linux-user/riscv/signal.c create mode 100644 linux-user/s390x/signal.c create mode 100644 linux-user/sh4/signal.c create mode 100644 linux-user/signal-common.h create mode 100644 linux-user/sparc/signal.c create mode 100644 linux-user/sparc64/signal.c create mode 100644 linux-user/tilegx/signal.c create mode 100644 linux-user/x86_64/signal.c create mode 100644 linux-user/xtensa/signal.c Reviewed-by: Richard Henderson Reviewed-by: Alex Bennée