From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50146) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b3LFY-000229-G7 for qemu-devel@nongnu.org; Thu, 19 May 2016 06:32:17 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1b3LFU-00074B-9O for qemu-devel@nongnu.org; Thu, 19 May 2016 06:32:15 -0400 Received: from mx1.redhat.com ([209.132.183.28]:54720) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b3LFU-000747-3q for qemu-devel@nongnu.org; Thu, 19 May 2016 06:32:12 -0400 References: <1463412982-6681-1-git-send-email-pbonzini@redhat.com> <1463412982-6681-42-git-send-email-pbonzini@redhat.com> <20160519121733.4ebe3c07.cornelia.huck@de.ibm.com> From: Paolo Bonzini Message-ID: Date: Thu, 19 May 2016 12:32:05 +0200 MIME-Version: 1.0 In-Reply-To: <20160519121733.4ebe3c07.cornelia.huck@de.ibm.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 41/50] s390x: move stuff out of cpu.h List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Cornelia Huck Cc: qemu-devel@nongnu.org On 19/05/2016 12:17, Cornelia Huck wrote: > On Mon, 16 May 2016 17:36:13 +0200 > Paolo Bonzini wrote: >=20 > Maybe mention which stuff was moved why ;) Right: ---- Move cpu_inject_* to the only C file where they are used. Move ioinst.h declarations that need S390CPU to cpu.h, to make ioinst.h independent of cpu.h. Move channel declarations that only need SubchDev from cpu.h to css.h, to make more channel users independent of cpu.h. ---- >> -#include "css.h" >> +#include >=20 > Wonder why we didn't need that one before? It's not needed, hw/s390x/css.h includes it. But this file uses AdapterRoutes, so I added it explicitly. >> +#include >> >> #define VIRTUAL_CSSID 0xfe >> >> diff --git a/hw/s390x/css.h b/include/hw/s390x/css.h >> similarity index 76% >> rename from hw/s390x/css.h >> rename to include/hw/s390x/css.h >> index a320eea..98b2e2c 100644 >> --- a/hw/s390x/css.h >> +++ b/include/hw/s390x/css.h >> @@ -14,7 +14,7 @@ >> >> #include "hw/s390x/adapter.h" >> #include "hw/s390x/s390_flic.h" >> -#include "ioinst.h" >> +#include "hw/s390x/ioinst.h" >> >> /* Channel subsystem constants. */ >> #define MAX_SCHID 65535 >> @@ -67,6 +67,7 @@ typedef struct CMBE { >> uint32_t reserved[7]; >> } QEMU_PACKED CMBE; >> >> +typedef struct SubchDev SubchDev; >> struct SubchDev { >=20 > Maybe do typedef and declaration all in one, to match the other > structures in this file? Ok, will do. Paolo >> /* channel-subsystem related things: */ >> uint8_t cssid; >=20 > In any case: >=20 > Acked-by: Cornelia Huck >=20