Peter Maydell <peter.maydell@xxxxxxxxxx> writes: > On 29 August 2012 00:48, Rusty Russell <rusty.russell@xxxxxxxxxx> wrote: >> No structures at all any more. > > I'm not fussed whether we use structs for the core regs or > not; they're not exactly going to change in future so it's > purely a question of whether you think it's aesthetically > prettier to have everything funneled through the one ioctl. Avi wanted to get rid of the structures. But I thought "it will never change" was already proven untrue, for your psuedo-secure mode proposal? >> + /* Coprocessor 0 means we want a core register. */ >> + if ((u32)reg->id >> KVM_REG_ARM_COPROC_START == 0) >> + return set_core_reg(vcpu, reg); > > ...but if we do go this path, you can't use coprocessor 0 > to mean core register -- cp0 could be a valid coprocessor > (the ARM ARM reserves cp0..cp7 for "vendor specific features"). > Use something outside 0..15. OK, changed that too (16). Thanks, Rusty. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html