Re: [Qemu-devel] [PATCH v2 1/4] sdbus: add a QMP command to access a SDBus

2019-03-11 Thread Markus Armbruster
Peter Maydell writes: > On Mon, 11 Mar 2019 at 13:43, Eduardo Habkost wrote: >> >> On Mon, Mar 11, 2019 at 12:49:50PM +0100, Thomas Huth wrote: >> > On 08/03/2019 17.11, Philippe Mathieu-Daudé wrote: >> > > Hi Markus, >> > > >> > > [Asking again from the correct series thread] >> > > >> > > On 1

Re: [Qemu-devel] [PATCH v2 1/4] sdbus: add a QMP command to access a SDBus

2019-03-11 Thread Eduardo Habkost
On Mon, Mar 11, 2019 at 12:49:50PM +0100, Thomas Huth wrote: > On 08/03/2019 17.11, Philippe Mathieu-Daudé wrote: > > Hi Markus, > > > > [Asking again from the correct series thread] > > > > On 1/3/18 10:49 PM, Philippe Mathieu-Daudé wrote: > >> Use Base64 to serialize the binary blobs in JSON. >

Re: [Qemu-devel] [PATCH v2 1/4] sdbus: add a QMP command to access a SDBus

2019-03-11 Thread Peter Maydell
On Mon, 11 Mar 2019 at 13:43, Eduardo Habkost wrote: > > On Mon, Mar 11, 2019 at 12:49:50PM +0100, Thomas Huth wrote: > > On 08/03/2019 17.11, Philippe Mathieu-Daudé wrote: > > > Hi Markus, > > > > > > [Asking again from the correct series thread] > > > > > > On 1/3/18 10:49 PM, Philippe Mathieu-D

Re: [Qemu-devel] [PATCH v2 1/4] sdbus: add a QMP command to access a SDBus

2019-03-11 Thread Thomas Huth
On 08/03/2019 17.11, Philippe Mathieu-Daudé wrote: > Hi Markus, > > [Asking again from the correct series thread] > > On 1/3/18 10:49 PM, Philippe Mathieu-Daudé wrote: >> Use Base64 to serialize the binary blobs in JSON. >> So far at most 512 bytes will be transfered, which result >> in a 684 byt

Re: [Qemu-devel] [PATCH v2 1/4] sdbus: add a QMP command to access a SDBus

2019-03-08 Thread Philippe Mathieu-Daudé
Hi Markus, [Asking again from the correct series thread] On 1/3/18 10:49 PM, Philippe Mathieu-Daudé wrote: > Use Base64 to serialize the binary blobs in JSON. > So far at most 512 bytes will be transfered, which result > in a 684 bytes payload. > Since this command is intented for qtesting, it is

Re: [Qemu-devel] [PATCH v2 1/4] sdbus: add a QMP command to access a SDBus

2018-01-05 Thread Philippe Mathieu-Daudé
Hi Eric, On Fri, Jan 5, 2018 at 12:29 PM, Eric Blake wrote: > On 01/03/2018 03:49 PM, Philippe Mathieu-Daudé wrote: [...] >> +if (ambiguous) { >> +error_setg(errp, "Path '%s' is ambiguous", qom_path); >> +} else { >> +error_set(errp, ERROR_CLASS_DEVICE_NOT_

Re: [Qemu-devel] [PATCH v2 1/4] sdbus: add a QMP command to access a SDBus

2018-01-05 Thread Eric Blake
On 01/05/2018 10:06 AM, Philippe Mathieu-Daudé wrote: > Hi Eric, > > On 01/05/2018 12:29 PM, Eric Blake wrote: >> On 01/03/2018 03:49 PM, Philippe Mathieu-Daudé wrote: >>> Use Base64 to serialize the binary blobs in JSON. >>> So far at most 512 bytes will be transfered, which result >> >> s/transf

Re: [Qemu-devel] [PATCH v2 1/4] sdbus: add a QMP command to access a SDBus

2018-01-05 Thread Philippe Mathieu-Daudé
Hi Eric, On 01/05/2018 12:29 PM, Eric Blake wrote: > On 01/03/2018 03:49 PM, Philippe Mathieu-Daudé wrote: >> Use Base64 to serialize the binary blobs in JSON. >> So far at most 512 bytes will be transfered, which result > > s/transfered/transferred/ > >> in a 684 bytes payload. >> Since this co

Re: [Qemu-devel] [PATCH v2 1/4] sdbus: add a QMP command to access a SDBus

2018-01-05 Thread Eric Blake
On 01/03/2018 03:49 PM, Philippe Mathieu-Daudé wrote: > Use Base64 to serialize the binary blobs in JSON. > So far at most 512 bytes will be transfered, which result s/transfered/transferred/ > in a 684 bytes payload. > Since this command is intented for qtesting, it is acceptable. s/intented/in

Re: [Qemu-devel] [PATCH v2 1/4] sdbus: add a QMP command to access a SDBus

2018-01-05 Thread Stefan Hajnoczi
On Wed, Jan 03, 2018 at 06:49:22PM -0300, Philippe Mathieu-Daudé wrote: > Use Base64 to serialize the binary blobs in JSON. > So far at most 512 bytes will be transfered, which result > in a 684 bytes payload. > Since this command is intented for qtesting, it is acceptable. Not a requirement, but

[Qemu-devel] [PATCH v2 1/4] sdbus: add a QMP command to access a SDBus

2018-01-03 Thread Philippe Mathieu-Daudé
Use Base64 to serialize the binary blobs in JSON. So far at most 512 bytes will be transfered, which result in a 684 bytes payload. Since this command is intented for qtesting, it is acceptable. Signed-off-by: Philippe Mathieu-Daudé --- qapi-schema.json| 41 +