On Tue, Oct 05, 2021 at 12:45:52PM -0400, Jonah Palmer wrote: > From: Laurent Vivier <lviv...@redhat.com> > > This new command shows the information of a VirtQueue element. > > Signed-off-by: Jonah Palmer <jonah.pal...@oracle.com> > --- > +++ b/qapi/virtio.json > +## > +# @VirtioRingAvail: > +# > +# @flags: VRingAvail flags > +# > +# @idx: VRingAvail idx
Is it worth being consistent... > +## > +# @VirtioQueueElement: > +# > +# @device-name: VirtIODevice name (for reference) > +# > +# @index: index of the element in the queue ...and spelling things 'index' everywhere instead of sometimes abbreviating? But overall, it looks like you did a nice job of making the command machine-parseable, while still leaving the flexibility to alter it as needed since it is only for developers under the x-debug- namespace. -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org