[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for 1.4 0/4] Simplify Makefile.objs some more
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH for 1.4 0/4] Simplify Makefile.objs some more |
Date: |
Sat, 19 Jan 2013 23:06:16 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 |
Il 19/01/2013 19:11, Andreas Färber ha scritto:
>> > The patches are mostly mechanical substitutions, and there is no
>> > user-visible change---neither in total build time, nor in the files that
>> > are linked into the executables.
> Without having tested this yet I want to remind that it is necessary for
> qom/cpu.c to be built twice
Hmm, it's not anymore actually (since libuser was removed). It hasn't
been built twice for a month and apparently nothing broke.
> , to not run into similar issues like
> util/oslib-posix.c.
The only dependency is
#if !defined(CONFIG_USER_ONLY)
int kvm_fd;
bool kvm_vcpu_dirty;
#endif
struct KVMState *kvm_state;
struct kvm_run *kvm_run;
Plenty of other fields are meaningless for !CONFIG_USER_ONLY but are
unconditionally present in struct CPUState. Given this inconsistency,
why is it still useful to build it twice?
Paolo
- [Qemu-devel] [PATCH for 1.4 0/4] Simplify Makefile.objs some more, Paolo Bonzini, 2013/01/19
- [Qemu-devel] [PATCH 2/4] build: use -$(CONFIG_SECCOMP) instead of ifeq, Paolo Bonzini, 2013/01/19
- [Qemu-devel] [PATCH 3/4] build: remove universal-obj-y, Paolo Bonzini, 2013/01/19
- [Qemu-devel] [PATCH 4/4] build: remove extra-obj-y, Paolo Bonzini, 2013/01/19
- Re: [Qemu-devel] [PATCH for 1.4 0/4] Simplify Makefile.objs some more, Andreas Färber, 2013/01/19
- Re: [Qemu-devel] [PATCH for 1.4 0/4] Simplify Makefile.objs some more,
Paolo Bonzini <=
Re: [Qemu-devel] [PATCH for 1.4 0/4] Simplify Makefile.objs some more, Paolo Bonzini, 2013/01/25