On Wed, 18 Nov 2009 13:58:17 -0600 Anthony Liguori <aligu...@linux.vnet.ibm.com> wrote:
> Daniel P. Berrange wrote: > > On Tue, Nov 17, 2009 at 05:43:54PM -0200, Luiz Capitulino wrote: > > > >> QError is a high-level data type which represents an exception > >> in QEMU, it stores the following error information: > >> > >> - class Error class name (eg. "ServiceUnavailable") > >> - description A detailed error description, which can contain > >> references to run-time error data > >> - filename The file name of where the error occurred > >> - line number The exact line number of the error > >> > > > > If we're going to collect these two, then also add in the function > > name, since that's typically more useful than filename/line number > > alone. > > > > I'm not convinced it's a good idea to put that info on the wire. It's > unstable across any build of qemu. However, since it's extra info, it > doesn't bother me that much if people think it's useful for debugging > purposes. It's really for debugging, so that we can have a detailed error description when the error macro has a wrong syntax. That said, we could have a compile time switch to activate extra debugging information on the wire. But that's a brainstorm.