On 26.07.2009, at 14:46, Avi Kivity wrote:
On 07/26/2009 03:26 PM, Alexander Graf wrote:
Cool, next week :-). The current version works fine without. And I
need a weekend for a change :-D.
Having returned from a 10-day weekend, I can only recommend this.
Hope you had fun and didn't read too much mail :-).
Yuck. What do we do? Implement set_bit_u64() and friends?
That's what I'd think we should do, yes.
For qemu I sent a patch that always uses "uint64_t" because that
works fine with the current code (32-bit on 64-bit uses the right
bytes for LE, 64-bit on 64-bit doesn't change). PPC is not
implement yet for anyone either and for now we only have dirty
logging with 64 bit kernels and 32-bit or 64-bit userspace, so
that's fine too.
As soon as someone implement dirty logging on a 32-bit big endian
target, it will break though, so before that day comes we need to
change set_bit to set_bit_64.
Perhaps we can get away with disabling 32-on-64 for the
configurations that break to start with.
32-on-64 only breaks for big endian which nobody except for this
target implements. Since my PPC enabling patches for qemu change the
type to uint64_t always, nobody is broken right now.
Alex
--
To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html