[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 for-2.11 00/10] target/s390x: cleanup cpu.h
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-devel] [PATCH v1 for-2.11 00/10] target/s390x: cleanup cpu.h |
Date: |
Thu, 17 Aug 2017 15:11:34 +0200 |
On Thu, 17 Aug 2017 11:22:15 +0200
David Hildenbrand <address@hidden> wrote:
> cpu.h is accessed outside of target/s390x. It should only contain
> what is expected to be accessed outside of this folder. Therefore, create
> internal.h and move a lot to that file. In addition, introduce
> kvm-stub.c and kvm_390x.h for kvm specific functions.
>
> Let's see what compilers think about this version.
It seems they like it :)
>
> RFC -> v1:
> - (hopefully) fixed a compile error
> - move some functions from cpu.h to the only c file they are used in
> - move kvm function and stubs to kvm_s390x.h and kvm-stub.c
> - smaller requested style fixes
>
>
> David Hildenbrand (10):
> target/s390x: move cc_name() to cc_helper.c
> target/s390x: move cpu_mmu_idx_to_asc() to excp_helper.c
> target/s390x: move psw_key_valid() to mem_helper.c
> target/s390x: move s390_do_cpu_reset() to diag.c
> target/s390x: move get_per_in_range() to misc_helper.c
> target/s390x: introduce internal.h
> target/s390x: move a couple of functions to cpu.c
> s390x: avoid calling kvm_ functions outside of target/s390x/
> s390x/kvm: move KVM declarations and stubs to separate files
> target/s390x: cleanup cpu.h
Generally, this looks good. I'll probably apply patches 1-8. For patch
9, I'll wait for an update (no need to resend the whole series). Still
undecided about patch 10.
- Re: [Qemu-devel] [PATCH v1 for-2.11 09/10] s390x/kvm: move KVM declarations and stubs to separate files, (continued)
- Re: [Qemu-devel] [PATCH v1 for-2.11 09/10] s390x/kvm: move KVM declarations and stubs to separate files, Cornelia Huck, 2017/08/17
- Re: [Qemu-devel] [PATCH v1 for-2.11 09/10] s390x/kvm: move KVM declarations and stubs to separate files, David Hildenbrand, 2017/08/17
- Re: [Qemu-devel] [PATCH v1 for-2.11 09/10] s390x/kvm: move KVM declarations and stubs to separate files, Thomas Huth, 2017/08/17
- Re: [Qemu-devel] [PATCH v1 for-2.11 09/10] s390x/kvm: move KVM declarations and stubs to separate files, Cornelia Huck, 2017/08/17
- Re: [Qemu-devel] [PATCH v1 for-2.11 09/10] s390x/kvm: move KVM declarations and stubs to separate files, David Hildenbrand, 2017/08/17
- Re: [Qemu-devel] [PATCH v1 for-2.11 09/10] s390x/kvm: move KVM declarations and stubs to separate files, David Hildenbrand, 2017/08/17
[Qemu-devel] [PATCH v1 for-2.11 10/10] target/s390x: cleanup cpu.h, David Hildenbrand, 2017/08/17
Re: [Qemu-devel] [PATCH v1 for-2.11 00/10] target/s390x: cleanup cpu.h, Philippe Mathieu-Daudé, 2017/08/17
Re: [Qemu-devel] [PATCH v1 for-2.11 00/10] target/s390x: cleanup cpu.h,
Cornelia Huck <=