[kvmarm] [PATCH v5.1 0/2] KVM: ARM: Rename KVM_SET_DEVICE_ADDRESS

Marc Zyngier marc.zyngier at arm.com
Thu Jan 10 06:53:16 EST 2013


On Thu, 10 Jan 2013 10:17:09 +0000, Peter Maydell
<peter.maydell at linaro.org> wrote:
> On 9 January 2013 22:30, Alexander Graf <agraf at suse.de> wrote:
>> In fact, in this particular case one could merge all of the patches
>> except for the
>> particular ioctl implementation and just give the respective addresses
>> default
>> values until there is an API to set them, similar to how we did things
>> with PVR
>> in the beginning.
> 
> That's what we started with and we got review comments saying "yuck,
> don't hard code this"...

Even worse: hard-coding things breaks kvmtool (we do not emulate a
vexpress and have a rather different memory map) and arm64 (the
requirements on GIC mappings are different).

So hard coding addresses is already unacceptable.

        M.
-- 
Fast, cheap, reliable. Pick two.



More information about the linux-arm-kernel mailing list