Re: [Qemu-devel] [PATCH v2 0/9] QMP/HMP: add error reason to open failures

2013-06-11 Thread Luiz Capitulino
On Tue, 11 Jun 2013 09:44:08 +0200 Stefan Hajnoczi wrote: > On Mon, Jun 10, 2013 at 01:02:19PM -0400, Luiz Capitulino wrote: > I should have qualified by Reviewed-by: with "reviewed block layer changes". I've added it only to the block layer patches.

Re: [Qemu-devel] [PATCH v2 0/9] QMP/HMP: add error reason to open failures

2013-06-11 Thread Kevin Wolf
Am 10.06.2013 um 19:02 hat Luiz Capitulino geschrieben: > I was surprised to find out that we still have old users of > QERR_OPEN_FILE_FAILED that print errors like: > > (qemu) dump-guest-memory -p /lkmads/foo > Could not open '/lkmads/foo' > (qemu) > > This series converts all those users to a n

Re: [Qemu-devel] [PATCH v2 0/9] QMP/HMP: add error reason to open failures

2013-06-11 Thread Stefan Hajnoczi
On Mon, Jun 10, 2013 at 01:02:19PM -0400, Luiz Capitulino wrote: I should have qualified by Reviewed-by: with "reviewed block layer changes". Stefan

Re: [Qemu-devel] [PATCH v2 0/9] QMP/HMP: add error reason to open failures

2013-06-11 Thread Stefan Hajnoczi
On Mon, Jun 10, 2013 at 01:02:19PM -0400, Luiz Capitulino wrote: > I was surprised to find out that we still have old users of > QERR_OPEN_FILE_FAILED that print errors like: > > (qemu) dump-guest-memory -p /lkmads/foo > Could not open '/lkmads/foo' > (qemu) > > This series converts all those use

[Qemu-devel] [PATCH v2 0/9] QMP/HMP: add error reason to open failures

2013-06-10 Thread Luiz Capitulino
I was surprised to find out that we still have old users of QERR_OPEN_FILE_FAILED that print errors like: (qemu) dump-guest-memory -p /lkmads/foo Could not open '/lkmads/foo' (qemu) This series converts all those users to a new helper called error_setg_file_open(), which adds error reason to open