On 08/30/2012 06:55 PM, Eric Blake wrote: > On 08/30/2012 11:47 AM, Jeff Cody wrote: >> Signed-off-by: Jeff Cody <jc...@redhat.com> >> --- >> qerror.h | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/qerror.h b/qerror.h >> index d0a76a4..7396184 100644 >> --- a/qerror.h >> +++ b/qerror.h >> @@ -219,6 +219,9 @@ void assert_no_error(Error *err); >> #define QERR_TOO_MANY_FILES \ >> ERROR_CLASS_GENERIC_ERROR, "Too many open files" >> >> +#define QERR_TOP_NOT_FOUND \ >> + ERROR_CLASS_GENERIC_ERROR, "Top image file %s not found" >> + > > Are you sure this patch is needed, or should you be using error_setg()? > > http://lists.gnu.org/archive/html/qemu-devel/2012-08/msg04980.html >
I think you are right, error_setg() would be the best course - although I based these patches only on what was in qemu/master currently.
