On Sat, Oct 17, 2015 at 2:53 AM, Alex Williamson <alex.williamson@xxxxxxxxxx> wrote: > v3: Incorporate suggestion from Igor to move the string length test > out to the property parsing. A string that's too long will now > get an error like: > > $ qemu-system-x86_64 -cpu qemu64,hv_vendor_id=123456789abcd > qemu-system-x86_64: Property 'host-x86_64-cpu.hv-vendor-id' doesn't take value '123456789abcd' > > v2: Remove abort, but truncate string > > --- > > Alex Williamson (2): > qapi: Create DEFINE_PROP_STRING_LEN > kvm: Allow the Hyper-V vendor ID to be specified Looks good to me. Reviewed-by: Amos Kong <kongjianjun@xxxxxxxxx> > > > hw/core/qdev-properties.c | 7 +++++++ > include/hw/qdev-core.h | 1 + > include/hw/qdev-properties.h | 16 ++++++++++++++-- > target-i386/cpu-qom.h | 1 + > target-i386/cpu.c | 1 + > target-i386/kvm.c | 8 +++++++- > 6 files changed, 31 insertions(+), 3 deletions(-) > -- 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
