[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [qemu-s390x] [PATCH v1 1/7] s390x/tod: factor out TOD into separate
From: |
Cornelia Huck |
Subject: |
Re: [qemu-s390x] [PATCH v1 1/7] s390x/tod: factor out TOD into separate device |
Date: |
Mon, 4 Jun 2018 14:14:34 +0200 |
On Mon, 4 Jun 2018 14:00:43 +0200
David Hildenbrand <address@hidden> wrote:
> On 04.06.2018 13:38, Cornelia Huck wrote:
> > On Fri, 25 May 2018 13:37:02 +0200
> > David Hildenbrand <address@hidden> wrote:
> >
> >> Let's treat this like a separate device. TCG will have to store the
> >> actual state/time later on.
> >>
> >> Include cpu-qom.h in kvm_s390x.h (due to S390CPU) to compile tod-kvm.c.
> >
> > Can't you include it in tod-kvm.c instead? Why was it not needed before?
>
> Guess I can, kvm_s390x.h looks to me like the right place, as S390CPU is
> used in there.
>
> All other users include "cpu.h" already, so it isn't needed.
Hm, and for kvm_s390_irq, this file is using a forward declaration,
which would likely be more awkward for this structure.
Header dependencies are always fun :)