Re: [Qemu-devel] [PATCH 0/4] block: Fix error report for wrong file format
Am 17.01.2013 09:33, schrieb Stefan Hajnoczi: > On Wed, Jan 16, 2013 at 07:53:35PM +0100, Stefan Weil wrote: >> Am 15.12.2012 15:09, schrieb Stefan Weil: >>> These patches improve the error report if the file format was >>> specified explicitly (example: -drive file=myfile,format=qcow2) >>> and the given format does not match the real format. >>> >>> This fixes those bugs: >>> >>> https://bugzilla.redhat.com/show_bug.cgi?id=556482 >>> https://bugs.launchpad.net/qemu/+bug/1090600 >>> >>> [PATCH 1/4] block: Add special error code for wrong format >>> [PATCH 2/4] block: Improve error report for wrong format >>> [PATCH 3/4] block: Use new error code for wrong format in selected >>> [PATCH 4/4] block/vdi: Improved return values from vdi_open and >> >> Hi Stefan und Kevin, >> >> these patches are still in my local queue. >> >> Do you plan to add them to the block queue, or would >> you prefer another solution for the open bug reports? > > Looks okay to me. I'm not thrilled about introducing a non-system error > code, would have rather have used EINVAL or ENOTTY. But that's not a > killer and I see the reason you chose to do that. > > Kevin: Any comments before I merge this? Yes, I commented on the patches. I think the minimum that should change is moving the error code definition to where other error codes are defined in order to avoid future collisions. Ideally we'd convert bdrv_open to Error and avoid all this error code stuff, but I'm not requesting this now. Kevin
Re: [Qemu-devel] [PATCH 0/4] block: Fix error report for wrong file format
On Wed, Jan 16, 2013 at 07:53:35PM +0100, Stefan Weil wrote: > Am 15.12.2012 15:09, schrieb Stefan Weil: > >These patches improve the error report if the file format was > >specified explicitly (example: -drive file=myfile,format=qcow2) > >and the given format does not match the real format. > > > >This fixes those bugs: > > > >https://bugzilla.redhat.com/show_bug.cgi?id=556482 > >https://bugs.launchpad.net/qemu/+bug/1090600 > > > >[PATCH 1/4] block: Add special error code for wrong format > >[PATCH 2/4] block: Improve error report for wrong format > >[PATCH 3/4] block: Use new error code for wrong format in selected > >[PATCH 4/4] block/vdi: Improved return values from vdi_open and > > Hi Stefan und Kevin, > > these patches are still in my local queue. > > Do you plan to add them to the block queue, or would > you prefer another solution for the open bug reports? Looks okay to me. I'm not thrilled about introducing a non-system error code, would have rather have used EINVAL or ENOTTY. But that's not a killer and I see the reason you chose to do that. Kevin: Any comments before I merge this? Stefan
Re: [Qemu-devel] [PATCH 0/4] block: Fix error report for wrong file format
Am 15.12.2012 15:09, schrieb Stefan Weil: These patches improve the error report if the file format was specified explicitly (example: -drive file=myfile,format=qcow2) and the given format does not match the real format. This fixes those bugs: https://bugzilla.redhat.com/show_bug.cgi?id=556482 https://bugs.launchpad.net/qemu/+bug/1090600 [PATCH 1/4] block: Add special error code for wrong format [PATCH 2/4] block: Improve error report for wrong format [PATCH 3/4] block: Use new error code for wrong format in selected [PATCH 4/4] block/vdi: Improved return values from vdi_open and Hi Stefan und Kevin, these patches are still in my local queue. Do you plan to add them to the block queue, or would you prefer another solution for the open bug reports? Regards Stefan
[Qemu-devel] [PATCH 0/4] block: Fix error report for wrong file format
These patches improve the error report if the file format was specified explicitly (example: -drive file=myfile,format=qcow2) and the given format does not match the real format. This fixes those bugs: https://bugzilla.redhat.com/show_bug.cgi?id=556482 https://bugs.launchpad.net/qemu/+bug/1090600 [PATCH 1/4] block: Add special error code for wrong format [PATCH 2/4] block: Improve error report for wrong format [PATCH 3/4] block: Use new error code for wrong format in selected [PATCH 4/4] block/vdi: Improved return values from vdi_open and