[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] block: add 'node-name' field to BLOCK_IMAGE
Re: [Qemu-devel] [PATCH 3/3] block: add 'node-name' field to BLOCK_IMAGE_CORRUPTED
Mon, 23 Mar 2015 10:53:11 +0100
On Mon, Mar 23, 2015 at 10:20:31AM +0100, Markus Armbruster wrote:
> > +- "device": Device name (json-string)
> > +- "node-name": Node name, if it's present (json-string)
> By convention, we mark optional members like this:
> - "node-name": Node name (json-string, optional)
You're right, thanks for pointing that out!
This can be fixed when committing the patch, otherwise I can resend
the series again.
> Ignorant question: can "device" be ""? If yes, we should document
> what that means, possibly in a separate patch.
Yes it can be "", the purpose of the patch is precisely to expand the
event with a new field that can identify the node in such cases.
The only reason why it is not optional is to keep backward
But it's true that it can be clarified in the documentation, I can
send a follow-up patch as soon as this one is merged.