Re: [Qemu-devel] [PATCH 3/3] block: add 'node-name' field to BLOCK_IMAGE_CORRUPTED

[Alberto Garcia]

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
compatibility.

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.

Berto