[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] keyval: accept escaped commas in implied option
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH 1/2] keyval: accept escaped commas in implied option |
Date: |
Wed, 11 Nov 2020 10:53:53 +0000 |
User-agent: |
Mutt/1.14.6 (2020-07-11) |
On Wed, Nov 11, 2020 at 05:45:20AM -0500, Paolo Bonzini wrote:
> This is used with the weirdly-named device "SUNFD,two", so accepting it
> is also a preparatory step towards keyval-ifying -device and the
> device_add monitor command. But in general it is an unexpected wart
> of the keyval syntax and leads to suboptimal errors compared to QemuOpts:
If "SUNFD,two" is the only wierdly named device, can we just rename
it to get rid of the comma, and then put validation in QOM to forbid
commas entirely. eg rename it to "SUNFD-two"
Just have a targetted hack in vl.c to replace any use of "SUNFD,two"
with the new name before parsing in keyval, if we care enough about
back compat for this niche hardware device.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|