[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: Compile files only once: some planning
From: |
Paul Brook |
Subject: |
Re: [Qemu-devel] Re: Compile files only once: some planning |
Date: |
Wed, 24 Mar 2010 17:27:57 +0000 |
User-agent: |
KMail/1.12.4 (Linux/2.6.32-trunk-amd64; KDE/4.3.4; x86_64; ; ) |
> >>>> 1) make CPUState define only common fields. Include CPUState at the
> >>>> beginning of each per-target CPUXYZState.
> >>>
> >>> Irritatingly, the common fields contain quite big TLBs. And the
> >>> offsets from the start of env affect the compactness of the code
> >>> generated from TCG. We really really want the general registers
> >>> to come first to make sure that those offsets fit the host's
> >>> reg+offset addressing mode.
> >>
> >> What about adding a 512-bytes (or more) block or something like that at
> >> the beginning of CPUState with a union, so you can put the per-target
> >> stuff there?
> >
> > Is it really worth the hassle? Anything touching CPUState is probably
> > going to be CPU specific anyway.
>
> qemu-timer.c, hw/dma.c is not and these are the first two files I looked
> at. translate-all.c is the third, and it is except for a trivial cleanup.
The use in hw/dma.c is incorrect. See previous discussion about how
qemu_bh_schedule_idle needs to go away.
I'm also unconvinced by your numbers. My i386-softmmu/ directory contains only
43 object files, most of are device emulation and don't touch CPU state at
all. arm-softmmu/ contains a good number more, but that's mostly board init
(which needs to know which CPU it's creating), and devices that are only used
by one board so noone's bothered to move them into libhw.
Paul
- [Qemu-devel] Re: Compile files only once: some planning, (continued)
Re: [Qemu-devel] Re: Compile files only once: some planning, Jamie Lokier, 2010/03/24
[Qemu-devel] Re: Compile files only once: some planning, Paolo Bonzini, 2010/03/24
Re: [Qemu-devel] Re: Compile files only once: some planning, Richard Henderson, 2010/03/24
Re: [Qemu-devel] Re: Compile files only once: some planning, Richard Henderson, 2010/03/24
Re: [Qemu-devel] Re: Compile files only once: some planning, Blue Swirl, 2010/03/24
[Qemu-devel] Re: Compile files only once: some planning, Michael S. Tsirkin, 2010/03/24